issues
search
kmorenov
/
hillel-mvc_project_1
test mvc structure
0
stars
0
forks
source link
issues
Newest
Newest
Most commented
Recently updated
Oldest
Least commented
Least recently updated
ее тут быть впринципе не должно,странно что во второй модельке без этой строчки.
#15
paladinbbk
opened
6 years ago
1
логики во вьюхе быть не должно. все хелперы должны быть классами
#14
paladinbbk
opened
6 years ago
0
Не безопасные данные идут в запрос. Уязвимость к SQL иньекциям
#13
paladinbbk
opened
6 years ago
0
слишко длинный метод. разделить на несколько. после того как подкличится PDO
#12
paladinbbk
opened
6 years ago
0
мы пока рассматриваем типичную модель в которой все данные берутся из одной базы. подстановка данных должна быть в обьекте конекшен
#11
paladinbbk
opened
6 years ago
0
хранить тут данные не имеет смысла они используются только один раз
#10
paladinbbk
opened
6 years ago
0
устаревшая библиотека удалена в php5.6 нужно использовать PDO
#9
paladinbbk
opened
6 years ago
0
для подобных ситуаций лучше выбрасывать исключение
#8
paladinbbk
opened
6 years ago
0
хорошо бы это в какой то метод обернуть. с передачей данных
#7
paladinbbk
opened
6 years ago
0
не используются немспейсы
#6
paladinbbk
opened
6 years ago
0
в файлах классов не должно быть никакого кода помимо декларации класса и работы с немспейсам
#5
paladinbbk
opened
6 years ago
0
такое писать нельзя. даже если нету автоподгрузки. то нужно хотя бы все инклудить в одном месте
#4
paladinbbk
opened
6 years ago
0
Роутинг работает от ссылки а не от GET параметров. и логика тут неочевидна
#3
paladinbbk
opened
6 years ago
2
не правильная работа с путем. нужно использовать магические константы. __DIR__
#2
paladinbbk
opened
6 years ago
1
не желательно использовать данные от пользователя так. нужно через функцию filter_input
#1
paladinbbk
opened
6 years ago
0