D129LameUserБез этого никак в командных проектах. Должен быть выдержан единый стиль кода.
Кому должен?
Код должен работать - и в зависимости от задачи, он может быть разным.
Вот мне написали как можно сократить следующий код:
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
List<Instance>instances = null;
if (IsSpesificMachine(type))
{
instances = _db.Instances.Where(x => x.FarmId == farmId && x.MachineName == type).ToList();
}
else
{
instances = _db.Instances.Where(x => x.FarmId == farmId && x.Type == type).ToList();
}
Нахлобуч - однозначно спец.
:-)
1.
2.
3.
4.
5.
6.
var query = _db.Instances.Where(x => x.FarmId == farmId);
query = IsSpesificMachine(type) ?
query.Where(x => x.MachineName == type) :
query.Where(x => x.Type == type);
var instances = query.ToList();
Блестяще. И не так уж сложно. И клин код.
А вот теперь, ситуация - я сделал паблиш (это код веб сайта).
И один пользователь жалуется, что он все сделал, а не работает.
И надо добавить лог, потому, что раз жалуется один, значит испорченные данные в базе - конкретно у него.
Но какие? (базу писал не я).
Так что лог должен выглядеть примерно так:
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
List<Instance>instances = null;
if (IsSpesificMachine(type))
{
instances = _db.Instances.Where(x => x.FarmId == farmId && x.MachineName == type).ToList();
if (instances.Count > 0)
{
log.DebugFormat("Task will be created for one instance {0}" ,type);
}
else
{
log.WarnFormat("Istanсe {0} not exist!",type);
}
}
else
{
instances = _db.Instances.Where(x => x.FarmId == farmId && x.Type == type).ToList();
}
И предположим, что это случилось через 3 месяца после написания круто закрученного кода.
В какой ситуации, будет менее рисковано добавив строку лога, сразу сделать паблиш (да, да, так тоже бывает!)?
Мы все-таки техники, а не художники или скульпторы. И делаем машины, а не произведения дизайнерской фантазии.
А машина должна быть ремонтопригодна.
Должен быть лючок для смазки, и сменный фильтр, когда откручиваешь одну гаечку - сдвигаешь крышечку, поменял, закрутил - и не надо газорезом разрезать отполированный и заваренный намертво корпус.
Должен заказчику.
Ваш пример некорректен. Стиль кода не должен влиять на конечный результат. В вашем случае это рефакторинг, а не стиль кода.
Самописные поделки на одного прогера можно писать как угодно (если не требуется их дальнейшее развитие), а корпоративные проекты - как правило так не пишутся.
Спорить можно бесконечно, но best practice для меня в командных проектах - это единый стиль кода.
Если я руководитель проекта - я прошу соблюдать участников проекта единый стиль кода. Если кто-то не хочет этого делать - я снимаю этого человека с проекта.
Если я исполнитель - я соблюдаю единый стиль, если этого требует РП.
