Как переписать код

@toxin · Регистрация: 19.03.2012

Студворк — интернет-сервис помощи студентам

Добрый день всем, подскажите пожалуйста как можно переписать примерно такой код:

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
public static void runMethods(Object request, RequestClass api, Callback callback) {
        if (request instanceof Some1Rq) {
            api.some1((Some1Rq) request).execute(callback);
        } else if (request instanceof Some2Rq) {
            api.some1((Some2Rq) request).execute(callback);
        } else if (request instanceof Some3Rq) {
            api.some1((Some3Rq) request).execute(callback);
        } else if (request instanceof Some4Rq) {
            api.some1((Some4Rq) request).execute(callback);
        } else if (request instanceof Some5Rq) {
            api.some1((Some5Rq) request).execute(callback);
        }
        и тут ещё куча таких проверок
}

Статический анализатор ругается на такой код, потому что слишком много условий и код тяжело читать, как можно переписать данный код? Помогите пожалуйста.

HighPredator · 28.12.2017, 09:09

А какой смысл в кастах при выполнении? У вас же и так проверяется, что запрос и так соответствует типу внутри условия. Это первый момент. Второй момент: вам нужно (в идеале) принимать в этот метод запрос не как объект "от" типа объект, а от любой базы в иерархии типов запросов. Т.е. завести тип AbstractRequest (или интерфейс) для которого все остальные запросы будут являться наследниками. Третье: проделать тоже самое для метода some1.

То есть. Вы должны:
1) получить следующее:

Java
1
2
3
4
5
public abstract class AbstractRequest {...}
public class Some1Rq extends AbstractRequest {...}
public class Some2Rq extends AbstractRequest {...}
public class Some2Rq extends AbstractRequest {...}
...

2) переписать на

Java
1
2
3
4
public static void runMethods(AbstractRequest request, RequestClass api, Callback callback) {
  api.some1(request).execute(callback);
  // и все
}

3) переписать some1 (возвращаемый тип от фонаря взял, главное параметры)

Java
1
2
3
4
public RequestClass some1(AbstractRequest request) {
  // ... do stuff
  return this;
}

@xoraxax · 28.12.2017, 09:11

Сообщение от toxin

api.some1((Some5Rq) request).execute(callback);

а что внутри some1()? передавай не разные классы, а какой-то общий Request, например.

@toxin · 28.12.2017, 09:29 **[ТС]**

HighPredator, это исторический код и его нельзя так унифицировать и полностью переписать.
Нужно как-то переписать именно тот кусок кода, что я привел.

Добавлено через 1 минуту

Сообщение от xoraxax

а что внутри some1()? передавай не разные классы, а какой-то общий Request, например.

Все, что угодно, код исторический, я не могу переписать его и передавать туда наследника какого-то общего класса.

@xoraxax · 28.12.2017, 09:36

Сообщение от toxin

это исторический код и его нельзя так унифицировать и полностью переписать

так надо не полностью переписать, а базовый класс добавить.

Добавлено через 54 секунды

Сообщение от toxin

Все, что угодно

от этого может зависеть, как это можно переписать

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! */ #include <iostream> #include <stack> #include <cctype>. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .