12 голосов

Термин, когда в процессе рефакторинга поломали код

рефакторинг в результате которого избавились от некачественного кода но при этом сломали половину функционала Это называется "регресс" или "регрессия", причём неважно что вы делали: добавляли новую ...
A K's user avatar
  • 28.6k
10 голосов
Принятый

if–else или if not – какой код удобнее читать?

А + избавляемся от восклицательных знаков, - получаем лишнюю вложенность кода, - не всегда понятно, что значит «не баланс». if ( balance ){ // pay // and // get // the // food }...
9 голосов
Принятый

Создание переменной ссылочного типа при каждой итерации цикла. Это затратно?

Разницы особой нету. Все переменные ссылочного типа без инициализации автоматически инициализируются null-значением. Что сама по себе переменная ссылочного типа? Это переменная, которая ссылается на ...
iluxa1810's user avatar
  • 24.8k
8 голосов

Исключить дублирование кода в функциях с разной константностью

Можно написать что-то вроде следующего struct C { const B& f() const { if (s) return d1; else return d2; } B& f() { return const_cast<B &>( const_cast<const C *>( this ...
Vlad from Moscow's user avatar
8 голосов
Принятый

Рефакторинг большой функции

Если вы делаете правильный рефакторинг, то следуя принципу Single Responsibility при описании методов, количество аргументов редко превышает значения 1, но не следует упираться в эту рекомендацию ...
Firepro's user avatar
  • 9,332
7 голосов

Как рефакторить метод со многими вложенными конструкциями в C# или Java

Почитайте книгу Роберта Мартина "Чистый код" (Clean Code). Не все мысли автора бесспорны, но почитать точно стоит. По крайней мере после чтения, многие вещи будут восприниматься немного по другому. ...
Barmaley's user avatar
  • 81.2k
7 голосов
Принятый

Помогите уменьшить Python код

for n, color in enumerate(['black','red','green']): df['Length'][df.Protocol == three.Protocol[n]].plot(ax=axes[0, n], title=three.Protocol[n], kind='hist', color=color)
Alexey Sumin's user avatar
6 голосов

Рефакторинг в коде, содержащем тесты

Юнит-тесты — такой же код, как и весь остальной код в проекте. Если не следить за чистотой кода, то код превратится в неподдерживаемую лапшу, и это в такой же мере верно для юнит-тестов. Скажем, ...
Kyubey's user avatar
  • 32k
6 голосов
Принятый

Рефакторинг в коде, содержащем тесты

Большинство юнит-тестов - это код, который пишется 1 раз и больше не изменяется. В идеале, такие все вообще тесты. Код, который никогда не изменяется, рефакторить не нужно. Более того, рефакторинг ...
Pavel Mayorov's user avatar
6 голосов

как правильно устранить "запах" кода "Большой класс"?

Под "большим классом" Фаулер понимает не размер в строках, а размер функциональности. То есть когда в классе много разнородных ответственностей класс считается "большим". ...
Cerbo's user avatar
  • 6,853
6 голосов

if–else или if not – какой код удобнее читать?

По моему мнению, в случае «асимметричных» веток, в особенности когда короткая ветка завершает выполнение, нет смысла тянуть её до точки после длинной ветки. Это ухудшает понятность кода, т. к. иначе ...
VladD's user avatar
  • 206k
6 голосов

if–else или if not – какой код удобнее читать?

Применение if/else / if not зависит от контекста применения. Если есть разница операций, но единая часть завершения кода, то требуется использовать первый вариант. Если при невыполнении какого-то ...
Firepro's user avatar
  • 9,332
6 голосов

Как переписать код, чтобы три функции превратить в одну универсальную?

Ваша проблема в том, что вы не можете передать ссылку на глобальную переменную, чтобы записать туда чтобы то ни было. Значит, надо использовать для хранения что-то другое, ссылку на что вы передать ...
Pavel Mayorov's user avatar
6 голосов
Принятый

Оптимизация и рефакторинг алгоритма

Для сдвига на 1 позицию влево можно использовать простой алгоритм - запомнить первый элемент, сдвинуть все остальные, поставить запомненный на последнее место. Для сдвига на небольшое K можно ...
MBo's user avatar
  • 51.6k
5 голосов
Принятый

Синхронный и асинхронный методы и дублирование кода

Да, дедлоки будут. Допустим, вы вызываете в UI-потоке GetDataAsync().Result. При этом код выполняется синхронно до момента await GetDataAsync(_settings.Url, Converter);, и далее возвращается в ...
VladD's user avatar
  • 206k
5 голосов

Как сделать данный код более компактным?

def rcvdata(cmd, size): data = None s.settimeout(1) for trying in range(6): try: sendCommand(cmd) data = s.recv(size) except: if trying =...
andreymal's user avatar
  • 13k
5 голосов

Исключить дублирование кода в функциях с разной константностью

Придумал вариант с шаблонной дружественной функцией: template<class R, class T> R& g(T* t); struct C { const B& f() const { return g<const B&>(this); } B& f() { ...
αλεχολυτ's user avatar
  • 28.5k
5 голосов

Рефакторинг большой функции

Сошлюсь на авторитета :) Но сначала - вы не просто механически поделили функцию на четыре? Слишком уж на это намекает то, что все параметры нужны во всех функциях... Ну, а теперь - слово Фаулеру. О ...
Harry's user avatar
  • 218k
5 голосов
Принятый

Как выделить шаблонность из функции?

Если единственное использование - это X x = expr; *it++ = x;, то эту операцию можно обернуть в std::function<void(X)> f, передавать не итератор а эту функцию, использовать как f(x). Либо можно ...
Abyx's user avatar
  • 30.9k
5 голосов

Создание переменной ссылочного типа при каждой итерации цикла. Это затратно?

Я что-то отличий не вижу, но может кто чего заметит пусть сигнализирует static void Main(string[] args) { List<Person> people1 = GetWithVarInWhile(); List<Person> people2 = ...
Bulson's user avatar
  • 9,431
5 голосов

Стоит ли так рефакторить? Замена условий на простые математические формулы

Такие сокращения не всегда есть хорошо, первый вариант гораздо быстрее и удобнее прочитать, и потратить меньше времени на то, что бы разобраться в логике. P.S. По этому поводу на харбе недавно ...
KOTJlETA's user avatar
  • 167
5 голосов
Принятый

Рефакторинг на python

import operator OPERATORS = { '+': operator.add, '-': operator.sub, '*': operator.mul, '%': operator.mod, '/': operator.truediv, } OPERATORS_STR = ', '.join(OPERATORS.keys()) def main(): ...
AivanF.'s user avatar
  • 9,808
5 голосов
Принятый

Как преобразовать секционированную таблицу в обычную?

У команды ALTER TABLE, обратного аналога клаузы modify_to_partitioned нет. Одна из возможных алтернатив -- экспорт/импорт с опцией PARTITION_OPTIONS=merge. Пример таблицы: create table t1 (id int ...
0xdb's user avatar
  • 51.6k
5 голосов

Можно ли упростить код

Не создавайте большое кол-во переменных, особенно связанных между собой. Их можно объединить в единый объект. Так все переменные, связанные со схемой, можно занести в объект, например Schema. Для ...
Neverm1ndo's user avatar
  • 1,687
4 голоса
Принятый

Рефакторинг вместо if else

Можно добавить константы и заменить на такой вид: switch (userData.status) { case UserData.STATUS_DISCONNECTED: // isDisconnected code break; case UserData....
Pleshevskiy's user avatar
4 голоса
Принятый

Определение стоимости рефакторинга

Эвристический алгоритм для одного разработчика Пометка: допустим, что 3Mb это примерно 80т строк кода. Начинаем делать рефакторинг и делаем его в течение, скажем, 4х часов. Подсчитываем количество ...
edem's user avatar
  • 5,960
4 голоса
Принятый

Express: завернуть логику аутентификации в middleware

В app.post('/login') получаем в body login + password, проверяем данные по базе, если ок, то создаем сессию для пользователя, иначе отправляем ошибку. let u = ... // полученные данных из модели по ...
Aikon Mogwai's user avatar
4 голоса

if–else или if not – какой код удобнее читать?

В большинстве случаев if с выходом удобнее, поскольку не отправляет весь код на лишний уровень вложенности. С другой стороны, если речь идёт уже о некой вложенной структуре, то я предпочитаю плюс ...
Qwertiy's user avatar
  • 122k
4 голоса

Исключить дублирование кода в функциях с разной константностью

Очевидный вариант: class A { bool flag; int a; int b; public: int& get() { return flag ? a : b; } const int& get() const { return ((A*)this)->get(); } }; Пример: int ...
int3's user avatar
  • 2,455

Допускаются только превышающие минимальную длину ответы с наивысшим рейтингом, не являющиеся общими