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

Question

[madaraice]

Всем привет, я новичок в php и веб разработке в целом, пишу мини проект (файлообменник) для личных целей. Пишу и одновременно читаю «PHP 7 в подлиннике» Котерова (перед этим читал “php 7” Мак Гранта). То есть без каких либо видеогайдов и чисто из того что понял из книги. Так вот, пришел к вопросу, что на сколько плох мой код и прошу Вас, чтобы понять на сколько все плохо, а что-то подсказывает, что далеко не идеально.
Проект на гитхабе - https://github.com/madaraice/file-sharing-service
Приму любую критику, не хочу допускать каких-либо глупых ошибок в дальнейшем : - )

Comments

[Сергей]

Ну самое очевидное это php.net/manual/ru/book.pdo.php

Answer 1

[SagePtr]

Держите исправленную версию)
https://github.com/madaraice/file-sharing-service/...
Самая критическая ошибка - отсутствие проверок. Что если юзер зальёт туда php-файл?
SQL-инъекции в коде тоже присутствуют, строки перед вставкой в БД не обрабатываются никак.
Это я только критические ошибки назвал, влияющие на безопасность.
А остальное вроде стиля кода, который ужасен, ознакомьтесь для начала с PHP: The Right Way (русский перевод).

Comments

[Roman Terekhin]

+1
- Загрузка только по белым спискам расширений
- Убедиться что php отключен в папке uploads
- Для проверки существования файла использовать is_file вместо file_exists