gistfile1.md

• MVC - беда беда
• Codestyle checkers off
• Tests off

Привет, коллеги. Вы большие молодцы, что потратили свое время на свое развитие и на улучшение навыков программирования и работы в команде.

Хотя, качество работ, оставляет желать лучшего, вы смогли реализовать базовый функционал на должном уровне.

Хотелось бы вам пожелать уверенности в своих силах, усидчивости и энергии решать сложнейшие задачи в крутых проектах

MVC

MVC самый популярный паттерн проектирования, к сожалению используется без понимания. Большинство работ, подходят к использзованию данного паттерна as-is. Принимая правила фреймворка, который использует данный паттерн, так же надо принимать это паттерн относительно написания кода. То есть - Fat models and skinny controllers, в большинстве работ основная логика, как раз написанна прямо в контроллерах, что не правильно.

Linter

Даже бытрым взглядом замеченно остутсвие включенного линтера у большинства. Чуваки, включенный линтер повышает читабельность кода в разы. И требует всего ничего времени для его включения и настройки. Важно, не забывать, что его все таки надо настраивать. Так как дефолтные настройки все же очень жесткие и не всегда нужны. Но Линтер должен быть включен, хотя бы в минимальном виде. Много конфигов лежит по запросу "linter config [my programming language]"

Tests

Неписать тесты, при использовании современных фреймворков просто стыдно. Это нудная, скучная задача, которая очень сильно облегченна в 2015 году. Хотя бы наличие минимальных тестов повышает стабильность вашего кода в разы и сильно упрашает работу в комманде. Потому, что можно легко вносить изменения в систему и тут же видеть, где что-то отвалилось и куда копать.