Create с параметрами. Вопрос для эстетов

[Дmmm]

В объекте несколько переменных, инициализируются при создании значениями из некого диалога. Что правильнее с точки зрения ООП, передавать все эти значения как параметры или внутри Create брать их прямо из того диалога?

[Stilet]

Не знаю, как там с точки зрения ООП, а я бы инициализировал их в конструкторе, а вызов диалога опеределил бы в отдельный метод. Хочет проггер вызывать - пусть после создания вызывает.
Или в конструктор внес параметр со значением по умолчанию, установка которого вызывает диалог.

[Дmmm]

А если этот диалог раньше уже вызывался (грубо говоря, это Options) и значения, что нужны, можно брать из его формы? Или важнее, что конструктор без параметров красивше?

[Stilet]

Важна не красота кода а его надежность.
Вызывался заранее? Отлично - присваивай свойствам обьекта после его создания нужные значения.

[Дmmm]

До создания присвоить значений явно не удастся
Я и прошу, чтобы объяснили, чем код типа:
constructor Create;
begin
a1:=dialog.d1;
a2:=dialog.d2;
end;
.....................
Create;
надёжней, чем:
constructor Create(d1, d2);
begin
a1:=d1;
a2:=d2;
end;
.....................
Create(dialog.d1, dialog.d2);
?

[Streletz]

Лично я что-то сомневаюсь, что код типа

Сообщение от Дmmm

constructor Create;
begin
a1:=dialog.d1;
a2:=dialog.d2;
end;

надёжен.
Новый экземпляр Вашего объекта должен знать, что dialog есть ни что иное как экземпляр именно того диалога из которого нужно брать требуемые значения, а не что-то другое.
Благодаря гибкости Delphi это может прокатить(с определёнными оговорками). Однако в целом это далеко не самый лучший вариант.
Это

Сообщение от Дmmm

constructor Create(d1, d2);

и это

Сообщение от Дmmm

Create(dialog.d1, dialog.d2);

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

[Stilet]

надёжней, чем:

Ничем, если этот конструктор находится в том же пространстве что и dialog.
Другое дело если этот класс будет сторонним, у программера на него не будет исходников, и он случайно захочет вместо dialog получать данные из другого источника и жестоко обломится, ибо dialog вызывается непосредственно в конструкторе.

До создания присвоить значений явно не удастся

Ты не понял.

c:=MyClass.create;
c.a1:=dialog.d1;
c.a2:=dialog.d2;
...

Вот что я имел ввиду. Такой вариант наиболее масштабируем, ибо сам класс не завязан с dialog жестко, и может получить свои данные в любое время в любом месте.

[Дmmm]

И в каждом месте, где нужно создать экземпляр класса тянуть за ним хвост с присваиваниями? "Это же неестетично." Да уж, тут "количество данных" имеет значение. А кроме того, как же инкапсуляция?

[Stilet]

А кроме того, как же инкапсуляция?

А зачем она тут нужна?

в каждом месте, где нужно создать экземпляр класса тянуть за ним хвост с присваиваниями?

А что тут такого? Тянуть кучу параметров в конструктор точно так же может показаться неэстетичным.
Делай как тебе удобно, если задача того позволяет.

[Пепел Феникса]

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

А зачем она тут нужна?

А что тут такого? Тянуть кучу параметров в конструктор точно так же может показаться неэстетичным.
Делай как тебе удобно, если задача того позволяет.

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