Что стоит поправить/подучить?

Question

[Антон Хмыров]

Предистория такая - устал от Java (работаю на ней три года), решил попробовать себя в веб-разработке (PHP). Попросил у компании с вакансиями на джуниоров в этой области тестовое задание, общение с этой компанией не сложилось, поэтому прошу отзывов о коде у экспертов.
Задание (в моем пересказе т.к. не уверен, что имею право выкладывать оригинал): реализовать дерево состоящее из разделов и элементов используя PHP и jQuery.
Ссылка на проект GitHub
Фреймворки на PHP не использовал принципиально, так как считаю, что нужно сначала самому в основах разобраться.
Что верстка никакая сам понимаю.
Реализовал по схеме MVC и добавил ЧПУ (бОльшую часть роутинга взял готовую).
Хотелось бы узнать какие есть ошибки и что стоило бы подучить

Answer 1

[Кирилл Арутюнов]

У вас в контроллерах почему-то модели хранятся. То есть четкого разделения модель/контроллер нет — при тестировании возникнут проблемы.

Читали phptherightway.com?

Также, как правило, в открытом доступе оставляют только точку входа - index.php, остальной код приложения размещают на уровень выше. Например, в папку app. То есть структура примерно такая:

app/
-controllers/
-models/
-...
public_html/
-index.php
-images/
-css
-js/
-...

nbproject в .gitignore следует закинуть. Да и вообще настройки любой IDE следует держать на локалке, чтобы не мусорить в гите.

Comments

[Антон Хмыров]

Похоже, что я запутался в MVC - то что должно быть моделью я записал контроллером. Но тогда я не совсем понимаю - а что в таком случае контроллер? "Правильный путь" начал читать на русском. Про захламленность проекта тоже понял

[Кирилл Арутюнов]

Антон Хмыров: вообще есть 2 концепции тонкий контроллер-толстая модель и наоборот.

В Yii2, например, используется тонкий контроллер. Т.е. контроллер (action в контроллере) принял запрос, подключил нужную модель, отдал данные в модель, модель с данными что-то сделала и отдала ответ в контроллеру. Контроллер передал ответ от модели во View и отправил ответ пользователю.
В модели в это время данные могут пройти валидацию, сохраниться в базу данных, дополнительно какие-то данные из базы подтянуть и т.д.

Посмотрите кусок видео (минут 10-15), в котором рассказывается про тестирование в Yii2, но в данном отрезке автор говорит про разницу моделей в Yii2 и в Symfony2: https://youtu.be/gRmEpUYaS20?t=1h34m55s
Уверен, что некоторое понимание после этого появится.

[Andrzej Wielski]

Антон Хмыров: архитектуру вы выбрали не совсем верную, но я бы не сказал что ваш код ужасен. Скорее наоборот, он в разы выше по качеству, чем у большинства "веб-девевопперов".
Я бы посоветовал на данный момент сразу переходить на разработку с помощью фреймворков. Laravel на мой взгляд идеально вам подойдет.

[Антон Хмыров]

Кирилл Арутюнов: Спасибо за развернутый ответ, обязательно посмотрю.

[Антон Хмыров]

Andrzej Wielski: спасибо за ответ, фреймворки думаю начинать смотреть -
сомневаюсь между Yii2 и laravel

[Марик Цукор]

Антон Хмыров: не сомневайтесь, начинайте изучать bootstrap.

[Марик Цукор]

Антон Хмыров: или skeleton.

[Кирилл Арутюнов]

Антон Хмыров: Можете смело выбирать любой (между Yii2 и Laravel). Оба хороши, оба имеют хорошую документацию и отличное комьюнити.

[Andrzej Wielski]

Антон Хмыров: На мой взгляд у Laravel более низкий порог вхождения, и архитектура получше. Работа с view у Yii реализована не лучшим методом. Laravel же использует свой шаблонизатор Blade, что явно плюс - код шаблона более понятен.