evgeshonok
Новичок
1
Нужна критика
- Автор темы evgeshonok
- Дата начала
evgeshonok
Новичок
Чем плох яндекс.хранилище?
Просьба прокомментировать плохие стороны
evgeshonok
Новичок
Попрошу у Вас совета по оптимизации и объяснения, если не трудно, ибо для меня это важно.
Просто вынести переменную $connect из методов и объявить как свойство класса, а в конструкторе объявлять, как объект PDO(чушь, наверное, сморозил), или объявить ее как глобальную в файле конфигурации и пользоваться везде?
Gas
может по одной?
У меня в директории с контролерами пусто.
В целом, конечно, очень слабенько, так писали лет 5+ назад, сейчас с таким кодом найти работу будет очень сложно.
Почитай про стандарты кодирования, лучше код оформлять в таком ключе:
http://symfony.com/doc/2.0/contributing/code/standards.html
http://unofficial-zf2.readthedocs.org/en/wrap/ref/coding.standard.html
PSR 1,2 - https://github.com/php-fig/fig-standards/tree/master/accepted
Сейчас везде принято использовать autoload PSR-0, а не вереницы require, подход по работе с базой никуда не годиться, сделай хотя бы через singleton, хоть это и антипатрн, но будет в 100 раз лучше чем нынешний подход. От мешанины php-html надо избавляться (check.php), в шаблонах мне кажется xss'ы возможны, это что сразу в глаза бросилось по просмотру нескольких файлов.
С одной стороны хорошо для опыта самому всё делать, но если вопрос как быстрее прокачаться, чтоб взяли на работу, то лучше этот пример переписать на каком-то фреймворке, типа laravel и выложить таки на github.
C фреймворком код всё-таки станет выглядеть приличней и небольшой опыт станет менее заметен для ревьюверов )
В целом, конечно, очень слабенько, так писали лет 5+ назад, сейчас с таким кодом найти работу будет очень сложно.
Почитай про стандарты кодирования, лучше код оформлять в таком ключе:
http://symfony.com/doc/2.0/contributing/code/standards.html
http://unofficial-zf2.readthedocs.org/en/wrap/ref/coding.standard.html
PSR 1,2 - https://github.com/php-fig/fig-standards/tree/master/accepted
Сейчас везде принято использовать autoload PSR-0, а не вереницы require, подход по работе с базой никуда не годиться, сделай хотя бы через singleton, хоть это и антипатрн, но будет в 100 раз лучше чем нынешний подход. От мешанины php-html надо избавляться (check.php), в шаблонах мне кажется xss'ы возможны, это что сразу в глаза бросилось по просмотру нескольких файлов.
С одной стороны хорошо для опыта самому всё делать, но если вопрос как быстрее прокачаться, чтоб взяли на работу, то лучше этот пример переписать на каком-то фреймворке, типа laravel и выложить таки на github.
C фреймворком код всё-таки станет выглядеть приличней и небольшой опыт станет менее заметен для ревьюверов )
