Наша команда разработчиков программного обеспечения состоит из группы опытных программистов с различными стилями программирования и предпочтениями. У нас нет стандартов для всего, только самые необходимые вещи, чтобы предотвратить тотальный хаос.
Недавно я наткнулся на какой-то рефакторинг, выполненный коллегой. Мой код выглядел примерно так:
public Person CreateNewPerson(string firstName, string lastName) {
var person = new Person() {
FirstName = firstName,
LastName = lastName
};
return person;
}
Который был реорганизован в это:
public Person CreateNewPerson (string firstName, string lastName) {
Person person = new Person ();
person.FirstName = firstName;
person.LastName = lastName;
return person;
}
Просто потому, что моему коллеге нужно было обновить какой-то другой метод в одном из классов, которые я написал, он также «рефакторинг» метода выше. Напомним, что он один из тех разработчиков, которые презирают синтаксический сахар и используют другую схему размещения / идентификации скобок, чем остальные.
Мой вопрос: каков этикет программиста (C #) для рефакторинга чужого исходного кода (как семантического, так и синтаксического)?