Наша команда разработчиков программного обеспечения состоит из множества опытных программистов с различными стилями и предпочтениями в программировании. У нас нет стандартов для всего, просто для того, чтобы предотвратить полный хаос.
Недавно я столкнулся с некоторыми рефакторингами, сделанными коллегой. Мой код выглядел примерно так:
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;
}
Просто потому, что моему коллеге нужно было обновить какой-либо другой метод в одном из классов, которые я написал, он также "реорганизовал" метод выше. Для записи он один из тех разработчиков, который презирает синтаксический сахар и использует другую схему размещения/идентификации кронштейнов, чем остальные из нас.
Мой вопрос: что такое (С#) программный этикет для рефакторинга исходного кода для других людей (как семантического, так и синтаксического)?