212 похожих чатов

Привет! Возник вопрос по ооп, поэтому пишу сюда) Есть 2 класса: 1.

СтарыйОтчет
2. НовыйОтчет

В каждом много полей, пусть 10.
НовыйОтчет создается из СтарыйОтчет путем некоторой модификации полей.

Вопрос. Как это сделать? Нормально ли передать старый отчет в конструктор нового отчета и там присвоить поля после соответствующей модицикации?

11 ответов

15 просмотров

Именованный конструктор или перегрузка дкфолтного конструктора если есть в языке

из наводящих вопросов - а из этих 10-ти полей они все одинаковые? какой процент должны отличаться? Что это за поля? Почему они отличаются? Возможно есть набор полей которые по смыслу можно объединить в отдельный тип. К чему вопрос - если это два разных класса, значит что бы "можно было засэтать в конструкторе" у другого класса будет так или иначе какое-то представление о внутренней структуре первого класса. И вопрос почему это разные классы в целом.

Два варианта - либо конструктор, либо фабричный метод (или сервисный метод), создающий новый отчет из старого. Выбор зависит от: - связаны ли эти два отчета по бизнесу? - работают ли с новым отчетом, если соответствующий старый отчет не существует? - будет ли старый отчет в будущем удален из системы?

Rostislav-Teryaev Автор вопроса
Sergey P
из наводящих вопросов - а из этих 10-ти полей они ...

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

Rostislav-Teryaev Автор вопроса
Sergey Alaev
Два варианта - либо конструктор, либо фабричный ме...

По бизнесу связаны С новым работают, если старый не существует Да, старый отчет будет удален

Rostislav Teryaev
По бизнесу связаны С новым работают, если старый н...

тогда NovyOtchetFromStaryOtchetConversionService. тогда модель нового отчета и код, который работает с новым отчетом, не будет знать о старом отчете и тогда удаление старого отчета пройдет максимально безболезненно.

Но у тебя появляется процедура которая знает об обоих, зачем? Удобнее тестить? Нет. Удобнее менять? Нет

Sergey P
Но у тебя появляется процедура которая знает об об...

Удобно херануть разом все штуки завязанные на старый отчёт, и ничего нигде не сломать

Sergey P
Но у тебя появляется процедура которая знает об об...

в идеале, это должно быть единственное место, которое знает об обоих. Лучше изоляция, проще зависимости.

Sergey Alaev
в идеале, это должно быть единственное место, кото...

Хз, как по мне с учётом ограниченной информации о задаче я бы не вводил новых сущностей

Sergey P
Хз, как по мне с учётом ограниченной информации о ...

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

Похожие вопросы

Обсуждают сегодня

Коллеги, я тут для личных нужд пошел ставить MQTT сервер, пощупал mosquitto, но ужаснулся отсутствию такой банальности, как HTTP API для посмотреть список топиков. А тут что,...
Maksim Lapshin
13
Всем привет! Имеется функция: function IsValidChar(ch: UTF8Char): Boolean; var i: Integer; ValidChars: AnsiString; begin ValidChars := 'abcdefghijklmnopqrstuvwxyzABCDE...
Евгений
44
#include <stdio.h> #include <stdlib.h> #include <time.h> void mass_first_generate(int mass[5][7]) {     for (int N = 0; N < 5; N++) {         for (int A = 0; A < 7; A++) {   ...
Чувак
6
Всем привет! Решаю 99 OCaml Problems и столкнулся со следующей проблемой (прошу палками не забивать, я OCaml практически не трогал до этого момента): open OUnit2 let create_...
К|/|pи/\/\ 6е3yглbIи
2
https://www.linkedin.com/posts/ugama-benedicta-kelechi-codergirl-103041300_mobiledevelopment-fluttertraining-handsonlearning-activity-7263445699227254784-IdHB?utm_source=share...
CoderGirl
16
Ну вот просто даже давайте вот как. Какой нибудь конкретный кейс, можете в пример привести, где бч работает и приносит прикладную пользу, а не просто что бы было? Не крипту.
Alexander Andreev
22
возможно ли как-то передать в электрон или таури медиа поток с рендера 2д движка? двиг запускается как dll, а дальше надо как-то отправлять рендер кодировать не подходит, зр...
Kyle Nekto
7
Точно, оно. У тебя там имена потоков выставляются?
Александр (Rouse_) Багель
11
Помогите пожалуйста. Делаю систему плагинов. Проблема сейчас в такая: плагины загружаются в основном потоке. FLibHandle := SafeLoadLibrary(FFileName) Но нужно еще выполнить фу...
Илья 🤣
10
объясните пожалуйста, почему функция не работает должным образом? вроде должно брать активное окно сравнивать его размер с размером экрана, и если есть совпадение = true прове...
JF
12
Карта сайта