Наша команда разработчиков программного обеспечения состоит из множества опытных программистов с различными стилями и предпочтениями в программировании. У нас нет стандартов для всего, просто для того, чтобы предотвратить полный хаос.Этикет для рефакторинга исходного кода другого человека?
Недавно я столкнулся с некоторыми рефакторингами, сделанными коллегой. Мой код будет выглядеть примерно так:
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;
}
Просто потому, что мой коллега нужно обновить какой-то другой способ, в одном из классов, которые я написал, он также «переработан» метод выше. Для записи он один из тех разработчиков, который презирает syntactic sugar и использует другую схему размещения/идентификации кронштейнов, чем остальные.
Мой вопрос: Что такое (C#) этикетчик программиста для рефакторинга исходного кода другого человека (как семантического, так и синтаксического)?
Вы должны сделать это сообщество wiki. –
Что касается этикета, лучшим способом, вероятно, было бы противостоять ему и сообщить ему, что вы не цените его рефакторинг вашего кода. Вы также можете сыграть в нее более подэлемент и вернуть свой код и изменить его код, чтобы он соответствовал вашему стилю. В то время как интересно делать сначала, в конце концов, вероятно, вы ничего не получите. –
Я вернул его обратно в свою версию и проверил его в SVN с комментарием «Улучшенная согласованность кода». После этого никаких обновлений не было сделано. Когда я обратился к нему, он более или менее признался, что это действительно не нужно. Поэтому в основном мы согласились не касаться кода друг друга, если это не требуется. – Prutswonder