Этикет для рефакторинга чужого исходного кода?

Question

Наша команда разработчиков программного обеспечения состоит из группы опытных программистов с различными стилями программирования и предпочтениями. У нас нет стандартов для всего, только самые необходимые вещи, чтобы предотвратить тотальный хаос.

Недавно я наткнулся на какой-то рефакторинг, выполненный коллегой. Мой код выглядел примерно так:

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 #) для рефакторинга чужого исходного кода (как семантического, так и синтаксического)?

Answer 1

Я бы просто поговорил с вашим коллегой-разработчиком о том, что вы хотите изменить и почему. Рефакторинг кода, только если вы согласились на что-то.

Это приведет к дискуссиям, где, скорее всего, вы оба узнаете что-то друг у друга.

Answer 2

Он создал большую часть кода для рассматриваемого проекта? Если бы это было так, я мог бы понять, что он сделал. При входе в проект, который уже идет полным ходом, я стараюсь соответствовать форматированию, которое уже используется во всем коде.

Конечно, это может не относиться к вашей ситуации. Если какой-либо проект в целом получил равный вклад от всех вас, возможно, прислушайтесь к совету Мнемента и доведите вопрос до головы.