- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
- 31
- 32
- 33
- 34
- 35
- 36
- 37
- 38
- 39
require_once ('./main/config.php');
class user
{
/*
private login;
private password;
private username;
private about;
*/
function registerUser($login, $password, $username, $about)
{
$db = MysqliDb::getInstance();
$data = Array ("login" => $login,
"password" => $password,
"username" => $username,
"about" => $about,
);
$id = $db->insert ('users', $data);
if($id)
echo 'user was created. Id=' . $id;
}
function authorizeUser($login, $password)
{
$db->where ("id", 1);
$user = $db->getOne ("users");
echo $user['id'];
/*
$logins = $db->getValue ("users", "login", null);
// select login from users
// select login from users limit 5
foreach ($logins as $login)
echo $login;
*/
}
}
Ну как вам? Класс юзера для регистрации и авторизации (недоделан)
guesto 18.05.2016 19:47 # +1
Мы видим что класс User не содержит полей пользователя, зато содержит два метода, один из которых не имеет отношения к объекту User (и должен быть статическим как минимум) да еще и полтно завязан на знание о БД.
И что такое $db? Поле класса? Тоесть у Юзера внутри хранится знание о БД,а больше ничего?
Тот редкий случай, когда хочется сказать "не используйте ООП, пишите в процедурном стиле, Вам ООП не идет'
Folsky 18.05.2016 19:49 # 0
guesto 18.05.2016 19:53 # 0
Зачем вообще люди пишут сами авторизацию и работу с БД?
Например в питонячем джанге все это уже есть)
bormand 18.05.2016 21:12 # +2
1024-- 18.05.2016 21:42 # +1
bormand 18.05.2016 21:45 # +5
Нецелесообразно™, т.к. я не так часто пишу эту строку.
> храните где-то в буфере обмена
Нецелесообразно™, т.к. тогда буфер будет занят.
> вводите вручную
Нецелесообразно™, т.к. долго и нудно.
Я её копипащу из текста и добавляю по краям [i][/i]
inkanus-gray 18.05.2016 21:51 # +1
bormand 18.05.2016 21:55 # +2
Оказывается, что я юзал это слово, когда оно ещё не было мэйнстримом.
1024-- 18.05.2016 21:56 # +2
Какая нецелесообразность™ )))
Где здесь нецелесообразно™?
Помни, 1024--, за тобой нецелесообразно™ следить!
нецелесообрадушный™ питушок
нецелесообразный™ лев
3.14159265 18.05.2016 22:04 # 0
где здесь нецелесообразность™ )))
где здесь нецелесообразный™ питушок
нецелесообразность™ )))
где здесь нецелесообразно™ следить!
нецелесообрадушный™ лев
bormand 18.05.2016 22:08 # +3
inkanus-gray 18.05.2016 21:57 # 0
bormand 18.05.2016 21:59 # +1
inkanus-gray 18.05.2016 22:00 # 0
bormand 18.05.2016 22:02 # 0
inkanus-gray 18.05.2016 22:06 # +5
Отсыпаю по 10 штук:
[ ] Ideographic space
[ ] Em quad
[ ] Em space
[ ] Figure space
[ ] En quad
[ ] En space
[ ] Three-Per-Em Space
[ ] Four-Per-Em Space
[ ] Narrow no-break space
[ ] Punctuation space
[ ] Six-Per-Em Space
[ ] Thin space
[ ] Hair space
bormand 18.05.2016 22:12 # 0
3.14159265 18.05.2016 22:12 # 0
inkanus-gray 18.05.2016 22:13 # +1
bormand 18.05.2016 22:15 # +2
inkanus-gray 18.05.2016 22:16 # +2
bormand 18.05.2016 22:21 # 0
inkanus-gray 18.05.2016 22:26 # 0
bormand 18.05.2016 22:27 # 0
Прикольная задачка, надо будет завтра подумать.
inkanus-gray 16.07.2016 23:06 # +2
PE3uHOBbIu_nemyx 19.10.2020 18:27 # 0
PE3uHOBbIu_nemyx 19.10.2020 18:25 # 0
3.14159265 18.05.2016 22:31 # +1
1024-- 18.05.2016 22:21 # 0
Для бормандовых волосатых sql-запросов?
bormand 18.05.2016 22:23 # 0
3.14159265 18.05.2016 22:24 # 0
bormand 18.05.2016 22:24 # 0
CHayT 18.05.2016 22:54 # +2
inkanus-gray 18.05.2016 21:15 # 0
Локальные переменные не нужны же.
bormand 18.05.2016 21:19 # +2
Это локальная переменная же, в которую положили инстанс синглтона MysqliDb::getInstance().
З.Ы. Из ООП и паттернов автор осилил только самую бесполезную хуйню - синглтоны. Слава богу, что до наследования не дочитал...
inkanus-gray 18.05.2016 21:20 # 0
bormand 18.05.2016 21:22 # 0
inkanus-gray 18.05.2016 21:24 # 0
Упадёт. В PHP нет статических типов, тип определяется значением, а у null не может быть полей/методов.
bormand 18.05.2016 21:27 # +1
MAKAKA 20.10.2020 01:07 # 0
guest 20.05.2016 00:44 # +2
Нецелесообразно использовать ООП в небольшом проекте
Нецелесообразно использовать ООП, ведь оно все равно не решает всех проблем
bormand 18.05.2016 21:32 # +1
Тогда было бы global... Класс же (недоделан).
Folsky 18.05.2016 23:01 # 0
bormand 18.05.2016 21:20 # +3
Вот же они:
bormand 18.05.2016 20:03 # −1
> echo $user['id'];
Юзер будет очень рад видеть всё это посреди странички.
guestinho 19.05.2016 00:03 # +2