Deutsch
Germany.ruФорумы → Архив Досок→ Программирование

C# - pattern matching - many discards

04.03.24 14:54
Re: C# - pattern matching - many discards
 
MrSanders коренной житель
можно пример кода, который ты не пускаешь в прод из-за нарушений конвенции?

Как уже отметили, совершенно обычный код, без особого криминала. Просто нарушающий принятые соглашения. Если я с криминалом нахожу, мне никакие конвеншены не нужны :)
Например, вот такая фигня (кто на яве пишет, поймёт, остальным поясню, имя класса поменял, мало ли, Datenschutz и всё такое):

/**
 * identisch zu
 * @param adresse AdresseMitLand
 */
public Boolean identischZu(AdresseMitLand adresse) {
    return true;
}

Начинаем с вопроса - а нафига мне метод сравнения, которые ничего не сравнивает? Нет ответа. Почему в документации не написано, мол, временная реализация, в настоящий момент, говорит что все всем равны, но надо использовать там-то и там-то потому что в будущем будет заменено.
Где описание параметра, где описание результата? Метод-то публичный.
Почему Boolean? (объект - оболочка вокруг примитивного типа boolean). Он может быть null. Зачем нам такое счастье? Мы (даже в конвенции написали) "используем примитивные типы если нет причин использовать врэпперы, причина описывается в документации".


До сих пор всё такое счастье тупо коммитилось, потом проходило 3 года и все сидели чесали репу а нафига ж такое сотворили, и чего хотели этим сказать.

 

Перейти на