Вопросы с тегом «code-quality»

10
В MVC считается ли хорошей практикой иметь частные, не действующие функции в классе контроллера?

Иногда функции действия в классе контроллера могут стать огромными и неприятными, с множеством строк кода, которые просто контролируют поток данных из модели в представление. В какой-то момент эти огромные функции полностью теряют основные принципы хорошего кода, то есть делают только одну вещь,...

10
Как обрабатывать флаг в нескольких if-else

Я, кажется, вижу это достаточно часто в своем коде и других. В этом нет ничего ужасного, но меня это раздражает, поскольку кажется, что это можно сделать лучше. Я предполагаю, что оператор case может иметь немного больше смысла, но часто переменная является типом, который не работает хорошо или...

10
Владение кодом с несколькими командами Scrum

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

10
Отдельные отчеты о покрытии кода для модульных и интеграционных тестов или один отчет для обоих?

Должен ли быть отдельный отчет о покрытии кода для модульных и интеграционных тестов или один отчет о покрытии кода для обоих? Мысль, стоящая за этим, заключается в том, что покрытие кода позволяет нам быть уверенным, что наш код будет покрыт тестами, насколько это возможно (насколько машина может...

10
Интерфейс с только получателями - это запах кода?

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

10
Переработка функции, возвращающей целочисленный код, который представляет множество различных состояний

Я унаследовал некоторый ужасный код, который я включил короткий пример ниже. Есть ли название для этого конкретного анти-паттерна? Какие рекомендации по рефакторингу это? // 0=Need to log in / present username and password // 2=Already logged in // 3=Inactive User found // 4=Valid User...

10
Хороший стиль кода для повсеместной проверки данных?

У меня есть проект, достаточно большой по размеру, и я не могу больше держать в голове каждый аспект. Я имею дело с рядом классов и функций, и я передаю данные. Со временем я заметил, что получаю ошибки, потому что забыл, какую точную форму должны иметь данные, когда я передаю их различным функциям...

9
Как (тактично) сказать моему менеджеру проекта или ведущему разработчику, что кодовая база проекта требует серьезной работы?

Я только что присоединился к (относительно) небольшой команде разработчиков, которая работала над проектом несколько месяцев, если не год. Как и большинство разработчиков, присоединившихся к проекту, я провел первые пару дней, рассматривая кодовую базу проекта. Проект (внутреннее бизнес-приложение...

9
Почему не рекомендуется иметь свойство set-only?

Сегодня на работе один из моих коллег просмотрел мой код и предложил удалить свойство «только для набора» и использовать вместо него метод. Поскольку мы оба были заняты другими делами, он сказал мне взглянуть на Property Designраздел из книги «Руководство по разработке структуры». В книге писатель...

9
В чем разница между статическим анализом кода и проверкой кода?

Я просто хотел узнать, в чем разница между статическим анализом кода и проверкой кода. Как каждый из этих двух сделан? В частности, какие инструменты доступны сегодня для обзора кода / статического анализа PHP? Я также хотел бы знать о хороших инструментах для обзора кода для любого...

9
Вопрос инженера начального уровня относительно управления памятью

Прошло несколько месяцев с тех пор, как я начал свою работу в качестве разработчика программного обеспечения начального уровня. Теперь, когда я прошел некоторые кривые обучения (например, язык, жаргон, синтаксис VB и C #), я начинаю концентрироваться на более эзотерических темах, как написание...

9
Модульное тестирование: «Это запах кода, если вы проводите рефакторинг, а соавторов нет»?

Я читаю «Искусство модульного тестирования» Роя Ошерова. Я нахожусь в разделе 7.2 Написание поддерживаемых тестов, где у автора есть эта заметка о запахе кода: ПРИМЕЧАНИЕ. Когда вы реорганизуете внутреннее состояние, чтобы оно было видимым для внешнего теста, можно ли это считать запахом кода...

9
Является ли «использование» уместным в контексте, где нечего распоряжаться?

В C # usingоператор используется для детерминированного размещения ресурсов без ожидания сборщика мусора. Например, он может быть использован для: Утилизируйте команды или соединения SQL, Закрывайте потоки, освобождая основной источник как файл, Бесплатные элементы GDI +, и т.п. Я заметил, что...

9
Нужно ли проверять использование всего модуля или только аргументы открытых методов?

Я слышал, что рекомендуется проверять аргументы открытых методов: Стоит ли проверять на ноль, если он не ожидает ноль? Должен ли метод проверять свои параметры? MSDN - CA1062: проверка аргументов общедоступных методов (у меня есть .NET фон, но вопрос не специфичен для C #) Мотивация понятна. Если...

9
Как реализовать свойство класса A, которое ссылается на свойство дочернего объекта класса A

У нас есть этот код, который при упрощении выглядит следующим образом: public class Room { public Client Client { get; set; } public long ClientId { get { return Client == null ? 0 : Client.Id; } } } public class Client { public long Id { get; set; } } Теперь у нас есть три точки зрения. 1) Это...