12
голосов
Термин, когда в процессе рефакторинга поломали код
рефакторинг в результате которого избавились от некачественного кода но при этом сломали половину функционала
Это называется "регресс" или "регрессия", причём неважно что вы делали: добавляли новую ...
10
голосов
Принятый
if–else или if not – какой код удобнее читать?
А
+ избавляемся от восклицательных знаков,
- получаем лишнюю вложенность кода,
- не всегда понятно, что значит «не баланс».
if ( balance ){
// pay
// and
// get
// the
// food
}...
Общий
9
голосов
Принятый
Создание переменной ссылочного типа при каждой итерации цикла. Это затратно?
Разницы особой нету.
Все переменные ссылочного типа без инициализации автоматически инициализируются null-значением.
Что сама по себе переменная ссылочного типа? Это переменная, которая ссылается на ...
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 ...
8
голосов
Принятый
Рефакторинг большой функции
Если вы делаете правильный рефакторинг, то следуя принципу Single Responsibility при описании методов, количество аргументов редко превышает значения 1, но не следует упираться в эту рекомендацию ...
7
голосов
Как рефакторить метод со многими вложенными конструкциями в C# или Java
Почитайте книгу Роберта Мартина "Чистый код" (Clean Code).
Не все мысли автора бесспорны, но почитать точно стоит. По крайней мере после чтения, многие вещи будут восприниматься немного по другому.
...
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)
6
голосов
Рефакторинг в коде, содержащем тесты
Юнит-тесты — такой же код, как и весь остальной код в проекте. Если не следить за чистотой кода, то код превратится в неподдерживаемую лапшу, и это в такой же мере верно для юнит-тестов.
Скажем, ...
6
голосов
Принятый
Рефакторинг в коде, содержащем тесты
Большинство юнит-тестов - это код, который пишется 1 раз и больше не изменяется. В идеале, такие все вообще тесты. Код, который никогда не изменяется, рефакторить не нужно.
Более того, рефакторинг ...
6
голосов
как правильно устранить "запах" кода "Большой класс"?
Под "большим классом" Фаулер понимает не размер в строках, а размер функциональности. То есть когда в классе много разнородных ответственностей класс считается "большим". ...
6
голосов
if–else или if not – какой код удобнее читать?
По моему мнению, в случае «асимметричных» веток, в особенности когда короткая ветка завершает выполнение, нет смысла тянуть её до точки после длинной ветки. Это ухудшает понятность кода, т. к. иначе ...
6
голосов
if–else или if not – какой код удобнее читать?
Применение if/else / if not зависит от контекста применения. Если есть разница операций, но единая часть завершения кода, то требуется использовать первый вариант.
Если при невыполнении какого-то ...
6
голосов
Как переписать код, чтобы три функции превратить в одну универсальную?
Ваша проблема в том, что вы не можете передать ссылку на глобальную переменную, чтобы записать туда чтобы то ни было. Значит, надо использовать для хранения что-то другое, ссылку на что вы передать ...
6
голосов
Принятый
Оптимизация и рефакторинг алгоритма
Для сдвига на 1 позицию влево можно использовать простой алгоритм - запомнить первый элемент, сдвинуть все остальные, поставить запомненный на последнее место. Для сдвига на небольшое K можно ...
5
голосов
Принятый
Синхронный и асинхронный методы и дублирование кода
Да, дедлоки будут.
Допустим, вы вызываете в UI-потоке GetDataAsync().Result. При этом код выполняется синхронно до момента await GetDataAsync(_settings.Url, Converter);, и далее возвращается в ...
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 =...
5
голосов
Исключить дублирование кода в функциях с разной константностью
Придумал вариант с шаблонной дружественной функцией:
template<class R, class T>
R& g(T* t);
struct C {
const B& f() const { return g<const B&>(this); }
B& f() { ...
5
голосов
Рефакторинг большой функции
Сошлюсь на авторитета :) Но сначала - вы не просто механически поделили функцию на четыре? Слишком уж на это намекает то, что все параметры нужны во всех функциях...
Ну, а теперь - слово Фаулеру. О ...
5
голосов
Принятый
Как выделить шаблонность из функции?
Если единственное использование - это X x = expr; *it++ = x;, то эту операцию можно обернуть в std::function<void(X)> f, передавать не итератор а эту функцию, использовать как f(x).
Либо можно ...
5
голосов
Создание переменной ссылочного типа при каждой итерации цикла. Это затратно?
Я что-то отличий не вижу, но может кто чего заметит пусть сигнализирует
static void Main(string[] args)
{
List<Person> people1 = GetWithVarInWhile();
List<Person> people2 = ...
5
голосов
Стоит ли так рефакторить? Замена условий на простые математические формулы
Такие сокращения не всегда есть хорошо, первый вариант гораздо быстрее и удобнее прочитать, и потратить меньше времени на то, что бы разобраться в логике.
P.S. По этому поводу на харбе недавно ...
5
голосов
Принятый
Рефакторинг на python
import operator
OPERATORS = {
'+': operator.add, '-': operator.sub,
'*': operator.mul, '%': operator.mod,
'/': operator.truediv,
}
OPERATORS_STR = ', '.join(OPERATORS.keys())
def main():
...
5
голосов
Принятый
Как преобразовать секционированную таблицу в обычную?
У команды ALTER TABLE, обратного аналога клаузы modify_to_partitioned нет.
Одна из возможных алтернатив -- экспорт/импорт с опцией PARTITION_OPTIONS=merge.
Пример таблицы:
create table t1 (id int ...
5
голосов
Можно ли упростить код
Не создавайте большое кол-во переменных, особенно связанных между собой. Их можно объединить в единый объект.
Так все переменные, связанные со схемой, можно занести в объект, например Schema. Для ...
4
голоса
Принятый
Рефакторинг вместо if else
Можно добавить константы и заменить на такой вид:
switch (userData.status) {
case UserData.STATUS_DISCONNECTED:
// isDisconnected code
break;
case UserData....
4
голоса
Принятый
Определение стоимости рефакторинга
Эвристический алгоритм для одного разработчика
Пометка: допустим, что 3Mb это примерно 80т строк кода.
Начинаем делать рефакторинг и делаем его в течение, скажем, 4х часов.
Подсчитываем количество ...
4
голоса
Принятый
Express: завернуть логику аутентификации в middleware
В app.post('/login') получаем в body login + password, проверяем данные по базе, если ок, то создаем сессию для пользователя, иначе отправляем ошибку.
let u = ... // полученные данных из модели по ...
4
голоса
if–else или if not – какой код удобнее читать?
В большинстве случаев if с выходом удобнее, поскольку не отправляет весь код на лишний уровень вложенности. С другой стороны, если речь идёт уже о некой вложенной структуре, то я предпочитаю плюс ...
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 ...
Допускаются только превышающие минимальную длину ответы с наивысшим рейтингом, не являющиеся общими
Связанные метки
рефакторинг × 168c# × 42
python × 25
java × 22
инспекция-кода × 21
javascript × 18
php × 17
ооп × 12
c++ × 11
шаблоны-проектирования × 10
python-3.x × 7
чистый-код × 7
алгоритм × 6
оптимизация × 5
методы × 5
sql × 4
функции × 4
классы × 4
архитектура × 4
switch × 4
android × 3
jquery × 3
.net × 3
юнит-тесты × 3
html × 2