@madaraice

Оценка кода, кто на новенького?

Всем привет, я новичок в php и веб разработке в целом, пишу мини проект (файлообменник) для личных целей. Пишу и одновременно читаю «PHP 7 в подлиннике» Котерова (перед этим читал “php 7” Мак Гранта). То есть без каких либо видеогайдов и чисто из того что понял из книги. Так вот, пришел к вопросу, что на сколько плох мой код и прошу Вас, чтобы понять на сколько все плохо, а что-то подсказывает, что далеко не идеально.
Проект на гитхабе - https://github.com/madaraice/file-sharing-service
Приму любую критику, не хочу допускать каких-либо глупых ошибок в дальнейшем : - )
  • Вопрос задан
  • 334 просмотра
Пригласить эксперта
Ответы на вопрос 1
SagePtr
@SagePtr
Еда - это святое
Держите исправленную версию)
https://github.com/madaraice/file-sharing-service/...
Самая критическая ошибка - отсутствие проверок. Что если юзер зальёт туда php-файл?
SQL-инъекции в коде тоже присутствуют, строки перед вставкой в БД не обрабатываются никак.
Это я только критические ошибки назвал, влияющие на безопасность.
А остальное вроде стиля кода, который ужасен, ознакомьтесь для начала с PHP: The Right Way (русский перевод).
Ответ написан
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы