Вопросы с меткой [рефакторинг]
Применять для вопросов с просьбой посмотреть и/или улучшить код, или напрямую связанных с самим рефакторингом
129
вопросов
0
голосов
1ответ
60 показов
Помогите улучшить код [закрыт]
У меня есть три множества U - универсальное, A и B - подмножества U, мне необходимо произвести операцию /(/A⋂/B) "/" - отрицание/дополнение. Я написал такой код:
public static Set<Integer&...
0
голосов
1ответ
41 показ
Как организовать Java код и избежать switch конструкций при передаче объекто
Пишу клиент-серверное приложение на ЯП Java. Для передачи данных использую библиотеку KryoNet. суть вопроса - когда передаю некоторые данные (в виде общего Класса) с клиента на сервер или наоборот они ...
1
голос
1ответ
29 показов
Как правильно хранить ссылки, большие строки в коде Python?
Как правильно хранить ссылки, большие строки в коде Python? Правильно ли оставлять ссылки, headers в коде или их нужно куда-то выносить?
0
голосов
0ответов
10 показов
Рефакторинг WebStorm
Нужно сделать большой рефакторинг. Есть файлы TSX в которых имеется использование методы makeStyles из material/ui. В одних случаях использование происходит в начале файла, а в других в конце.
Как, ...
0
голосов
1ответ
23 показа
Рефакторинг кода Laravel
подскажите пожалуйста, код рабочий но хороший ли он с точки зрения написании правильного кода? Можно ли его как нибудь отрефакторить ?
У User есть связь branches
У Branch есть связь kassa_group
$user =...
-2
голоса
1ответ
157 показов
Решение задачи проходит 5/7 тестов [закрыт]
Описание задачи:
На старой заброшенной базе геодезистов вы нашли код, который возвращает секретные послания, зашифрованные среди географических данных. Скрипт содержит несколько синтаксических ошибок ...
1
голос
2ответа
50 показов
Рефакторинг на python
Написал следующий код:
print ("--------------")
print ("--Calculator--")
print ("--------------\n")
print("Enter (Q)uit to exit.\n \n")
while True:
que = ...
3
голоса
1ответ
173 показа
Как правильно сделать рефакторинг кода
Цель рефакторинга - сделать бота более оптимизированным. Бот для одного приложения с play market, в котором есть сообщества и чаты. somelib - это REST API библиотека для этого приложения. Столкнулся с ...
0
голосов
0ответов
56 показов
Нужна помощь с рефакторингом кода. Данный код у меня в 8ми активити. Помогите пжл. Заранее благодарен
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);
initWeb();
initProgressBar();
webView....
2
голоса
2ответа
93 показа
Шаблон “Фабричный метод”. В чём преимущества данного паттерна?
На странице https://refactoring.guru/ru/design-patterns/factory-method написано, что основная задача, которую решает данный паттерн - это избавление кода от привязки к конкретным классам.
Вот пример ...
3
голоса
2ответа
103 показа
Как устранить повторение кода в начале и в конце методов?
Я создаю набор классов, элементов графического интерфейса, для небольшого консольного приложения.
Элементы представляют собой прямоугольники, текстовые поля и списки. Они могут быть показаны и скрыты ...
1
голос
1ответ
39 показов
Организация покрытия проекта тестами
В давнишем проекте еще на symfony 3 покрытие тестами нулевое. Сейчас это очевидно приводит к большому количеству ошибок и портит жизнь.
Как можно достаточно быстро покрыть большой проект тестами и что ...
0
голосов
1ответ
40 показов
Соблюдение PSR-1. Побочные эффекты и знаки. Как соблюсти при Композиции?
Разбираюсь со стандартом PSR-1.
Пункт 3 гласит:
*Файлам СЛЕДУЕТ либо объявлять знаки (классы, функции, константы, и т.д.) или оказывать побочный эффект (например, генерировать вывод, изменять .ini ...
1
голос
2ответа
31 показ
Как имея одну точку входа в приложение (один файл) найти неиспользуемый код в остальных файлах?
Собственно имеется приложение, которое является вынесенной из монолита логикой, микросервис. Главный файл, кроме тестов, один, все сопутствующее это скопированные директориями модули. Но в них куча ...
0
голосов
1ответ
74 показа
Рефакторинг, качество кода [закрыт]
По ссылке на github лежит проект на c# и, частично, на ms sql. Очень нужно улучшить качество кода, но без опыта это сделать очень сложно. Укажите на недочеты, пожалуйста: хочется научиться писать ...
2
голоса
2ответа
102 показа
возможно ли избавиться от try-catch?
подскажите, можно ли этот код, как то упростить, точнее не использовать try catch
Object tParameters = node.get("parameters");
final ObjectMapper mapper = new ObjectMapper();
try {
...
0
голосов
1ответ
34 показа
Рефакторинг поиска в Python
Как правильно будет переписать эту вот часть кода?
roles = [{'Id': 1, 'Name': 'Модератор'}, {'Id': 2, 'Name': 'Пользователь'}]
participant_ = {'Name': 'Elmir', 'Surname': 'I-de', 'RoleId': 1}
...
...
2
голоса
0ответов
116 показов
Как можно улучшить код обработки данных?
Есть набор данных с 3 колонками : REV_NUM_CL, ID_COMMS, ,DT_CONTACT.
Надо выбрать только те данные (значения: REV_NUM_CL, ID_COMMS), которые удовлетворяют любому из условий:
В группировке по ...
1
голос
1ответ
128 показов
Пример применения паттерна “Фабричный метод”. В чём преимущество?
Здесь приведён пример из реальной жизни:
https://refactoring.guru/ru/design-patterns/factory-method/php/example#example-1
У меня два вопроса касательно этого примера:
В чём преимущетсва данных ...
0
голосов
2ответа
87 показов
Как избежать дублирования кода при расширении старой функциональности?
Мне нужен совет, как лучше разделить логику между классами. Или какой паттерн в данной ситуации лучше всего использовать.
У меня есть метод, который умеет отправлять имейл, только одному получателю.
...
1
голос
1ответ
78 показов
Как можно улучшить этот код?
Интересует улучшение реализации интерфейса IValueChangedEventArgsFactory - класса ValueChangedEventArgsFactory. Возможно ли сделать обобщение? Мои попытки не дают результата. Обязательно не ...
0
голосов
1ответ
31 показ
Как лучше построить фрагмент, который будет незначительно отличаться от готового?
Имеется готовый фрагмент. Необходимо создать новый фрагмент, который будет отличаться от готового только тем, что в нем не будет editText'а, и кнопка в action bar'е имеет немного другую логику. Как ...
3
голоса
2ответа
61 показ
Как лучше отрефакторить switch и массу case'ов с кучей кода в них?
Текущий код (PerformTask() уже на 200 строк и типы заданий будут добавляться дальше):
public class Pawn
{
public Task task;
public void PerformTask()
{
switch (task.Type)
...
1
голос
1ответ
78 показов
Пример стратегии на PHP
Здесь приведён пример стратегии на PHP: https://refactoring.guru/ru/design-patterns/strategy/php/example#example-1
Вопрос в следующем: Почему в этом примере нет класса, хранящего ссылку на класс ...
0
голосов
1ответ
51 показ
Си, предотвратить дублирование кода парсера
Разрабатываю парсер одного DSL на Си. Ранее на вход подавались только файлы, но возникла нужда обрабатывать текст в виде char*. Можно ли это сделать просто и адекватно?
Сейчас код упрощённо выглядит ...
0
голосов
1ответ
60 показов
Сгуппировать несколько аннотаций в одну
Заметил у себя в проекте часто повторяющиеся аннотации над одним полем в разных сущностях, можно ли их как то заменить на свою собственную, с сохранением сообщений и лимитов.
Из такого:
@NotBlank(...
2
голоса
1ответ
85 показов
Какой порождающий шаблон проектирования использовать?
Есть два класса с многими по бизнесу общими полями, и с парой специфичных:
class ClassA {
public int Field1 {get;set;} // общее поле 1
//...
public int FieldN {get;set;} // общее поле N
...
1
голос
1ответ
48 показов
Какие ошибки присутствуют в данном “алгоритме”? [закрыт]
Есть рабочий кусок кода нуждающийся в рефакторинге. Он выполняет два ключевых этапа: isDocumentHasChanges - поиск изменений, и если есть то Update.
Я не шибко специалист в плане налаживания кода, ...
7
голосов
1ответ
143 показа
Термин, когда в процессе рефакторинга поломали код
Как называется рефакторинг в результате которого избавились от некачественного кода но при этом сломали половину функционала?
Вроде все компилируется и выглядит идеально, но не работает.
4
голоса
3ответа
130 показов
Должен ли проверяющий метод check/validate возвращать boolean или выбрасывать исключение?
Этот вопрос не обязательно относится к PHP, просто это моя сфера.
При построении методов, которые "узнают" имеет ли объект/переменная что-то, разрешено ли что-то (is, has, can), то возвращается true/...
0
голосов
0ответов
117 показов
Корректность null как дефолтного параметра метода и возвращаемого значения
В книге Роберта Мартина есть главы, в которых он пишет
1) Не возвращайте null
Я видел бесчисленное множество приложений, в которых едва ли не каждая
строка начиналась с проверки null. ...
1
голос
1ответ
69 показов
Приём рефакторинга: извлечение метода. Как применять?
Заранее прошу прощения за то, что вопрос может оказаться холиварным, однако хочется услышать мнение опытных разработчиков.
Итак, есть приём рефакторинга под названием извлечение метода. Он ...
1
голос
0ответов
40 показов
Как вносить изменения с поддержкой старого кода?
Есть проект
В проекте есть заявки, у заявки есть время ее подачи. После подачи заявки идет долгая работа с клиентом.
Частенько приходят задачи такого типа:
Поменять что-то в коде для заявок, которые ...
0
голосов
1ответ
211 показов
Как создать “базовый” компонент в vue?
Есть у меня много страниц со списком элементов. Обращаются к некоторому api и отображают полученное, что-то вроде:
Шаблон:
<template>
<div class="container">
<div class="row">...
0
голосов
0ответов
32 показа
Инструменты для рефакторинга Angular приложения
Подскажите, существуют в природе инструменты, которые могут облегчить рефакторинг приложения?
Например:
переименовываю я класс и везде меняются импорты
переименовал я метод и он переименовывается во ...
0
голосов
2ответа
83 показа
Многократное использование секции с if
В ходе теста, после больших блоков проверяю, не была ли нажата кнопка СТОП.
Есть такая секция с if
if (m_stopTest)
{
manualStopTest(); // Выключаем все
//...
return; // ...
0
голосов
3ответа
91 показ
Регистронезависимый switch
Есть у меня фрагмент кода в котором я хочу заменить if на switch:
if( request.SomeParam.Equals("Something1", StringComparison.InvariantCultureIgnoreCase))
{
Execute1();
}
if (request.SomeParam....
2
голоса
1ответ
43 показа
Пользуетесь ли вы интструментами для code review? [закрыт]
Интересует вопрос, пользуется ли кто-нибудь интструментами для ревью в рамках командной работы? Какими? и почему именно ими? каким образом происходит ревью в вашей компании?
0
голосов
1ответ
87 показов
Как улучшить свой код?
Сейчас читаю Чистый код Мартина и стараюсь улучшить свой код . У меня есть абстрактный класс Filter :
package filters;
import java.util.ArrayList;
import static java.lang.Math.pow;
/**
* This ...
0
голосов
0ответов
35 показов
Рефакторинг Python: задать произвольное число индексов для выборки
Имею две функции:
def matrix_list_company(size, index):
with open(file1) as f1:
data = json.load(f1)
comp = [line[index] for line in data]
return random.sample(comp, int(size))
...
0
голосов
1ответ
29 показов
Code Smells: изменение текущего экземпляра класса
У меня есть класс Person. Данный класс абсурдный и служит только для визуализации проблемы.
public class Person {
private int age;
private String name;
public Person(int age, String name)...
1
голос
1ответ
37 показов
Помогите организовать код
Есть задача получить данные о человеке и высчитать дни до др с помощью классов и конструкторов используя гет, сет.
class Persona {
constructor(first, last, day, month, year) {
this.name = {
...
0
голосов
1ответ
26 показов
Перевести код с использованием классов
Помогите переделать этот код с использованием классов и конструкторов для инициализации объектов классов.
function Persona(first, last, day, month, year) {
this.name = {
first,
...
0
голосов
2ответа
259 показов
Static Factory в Symfony
Всем привет. Мне нужно в домене (в сущности) получать объект из базы данных.
/**
* @return Money
*/
public function getAmountUAH()
{
$itemsUAH = $this->cartItems->filter(function (...
0
голосов
0ответов
48 показов
Помогите найти ошибки в коде javascript? [дубликат]
class dateInput {
constructor() {
this.input = document.querySelector(".input");
this.input.onChange = this.onChange;
}
onChange(event) {
this.inputValue = event.srcElement....
1
голос
2ответа
138 показов
Изменить значение импортированной переменной и передать изменённое значение
1. Кратко
См. заголовок.
2. Цель
Имеется программа, состоящая из модулей. В каждом модуле логируются какие-то действия. При запуске программы показывается имя модуля, выдающего логи.
Это ...
1
голос
0ответов
60 показов
Как писать юнит-тесты на новый код в монолитном методе?
Есть огромный метод F c бизнес логикой покрытой тестами.
public ValidationResult F(...)
{...}
Добавил еще немного:
if(...)
{
...
Repository.UpdateSomeDataInDB(...);
...
}
Задача: Нужно ...
0
голосов
0ответов
52 показа
Есть большой класс, можно ли его разделить на много мелких
Имеется проект в один класс, где есть абсолютно все, что мне было нужно до этого момента, но сейчас нужно расширить функционал в сторону поддержки других осей и исправления косяка с winxp. Пока не ...
0
голосов
2ответа
50 показов
Чистота кода. Как “красиво” работать с функциями которые возвращают списки? Python
Только начинаю изучать python и возникло пару вопросов по "минимизации" кода. Например функция re.findall() возвращает список найденных строк, но мне всегда нужна одна, и для того что бы ее "безопасно"...
0
голосов
1ответ
111 показов
PhpStorm Настройка рефакторинга кода
Есть вопрос про рефакторинг кода в PhpStorm. Проблема заключается в следующием: если ставить табы, то ide считает их как 4 пробела, можно переключатся стрелками, подскажите это не правильная настройка ...