VA
нахуа? любите клавиатуру изнашивать?
в чем конкретно претензия?
Size: a a a
2020 November 27
z
в чем конкретно претензия?
зачем тут эти проперти? просто чтоб было, потому что гдето сказали так надо
VA
зачем тут эти проперти? просто чтоб было, потому что гдето сказали так надо
а куда их?
RS
зачем тут эти проперти? просто чтоб было, потому что гдето сказали так надо
Да ладно. Что плохого? Он даже геттеры и сеттеры убрал, чтоюы не нагромождать
z
вполне хватило бы
public
PowerTrans: Integer;
SteelType: String;
Frequency: Integer;
LegDiameter: Integer;
DiameterDimension: Integer;
StackNumber: Integer;
PlateStepNumber: Integer;
StepNumber: Integer;
HoleLegDiameter: Integer;
HoleYokeDiameter: Integer;
HoleLegNumber: Integer;
HoleYokeNumber: Integer;
SigmaLeg: Integer;
SigmaYoke: Integer;
HalfOverlap: Integer;
PlateGap: Extended;
WindowHeight: Integer;
LegDimension: Integer;
VoltagePh: Extended;
TurnNumber: Integer;
B1: Integer;СП
вполне хватило бы
public
PowerTrans: Integer;
SteelType: String;
Frequency: Integer;
LegDiameter: Integer;
DiameterDimension: Integer;
StackNumber: Integer;
PlateStepNumber: Integer;
StepNumber: Integer;
HoleLegDiameter: Integer;
HoleYokeDiameter: Integer;
HoleLegNumber: Integer;
HoleYokeNumber: Integer;
SigmaLeg: Integer;
SigmaYoke: Integer;
HalfOverlap: Integer;
PlateGap: Extended;
WindowHeight: Integer;
LegDimension: Integer;
VoltagePh: Extended;
TurnNumber: Integer;
B1: Integer;Те если мне не нужны действия над полями внутри класса то можно просто сделать публичными поля?
RS
https://github.com/SergeyPyatygin/Lazarus/blob/799f765062e0f243dbf7da134d99da5687a0a402/NoLoad/u_main.pas#L131 Сергей я особо в код не углублялся. А то что цикла нет это осознанно?
z
Сергей Пятыгин
Те если мне не нужны действия над полями внутри класса то можно просто сделать публичными поля?
имхо
СП
имхо
Спасибо, учту.
RS
Сергей Пятыгин
Те если мне не нужны действия над полями внутри класса то можно просто сделать публичными поля?
Но многие посчитают это некрасивым. В т.ч. я
z
Но многие посчитают это некрасивым. В т.ч. я
вынужден несогласиться
VA
Сергей Пятыгин
Те если мне не нужны действия над полями внутри класса то можно просто сделать публичными поля?
это плохая практика
СП
https://github.com/SergeyPyatygin/Lazarus/blob/799f765062e0f243dbf7da134d99da5687a0a402/NoLoad/u_main.pas#L131 Сергей я особо в код не углублялся. А то что цикла нет это осознанно?
Идея такая-считать данные в объекты, запустить последовательно два расчетных класса. Не понимаю про цикл?
RS
Сергей Пятыгин
Идея такая-считать данные в объекты, запустить последовательно два расчетных класса. Не понимаю про цикл?
Ну, если такой расчет долго выполняется, то тогда поток имеет смысл, ok
СП
А как по классике ООП нужно делать: инкапсулировать поля даже если они не используются и использовать только свойства или делать поля открытыми?
RS
Ну, если такой расчет долго выполняется, то тогда поток имеет смысл, ok
Все равно не ok. Синхронайз бессмысленен в таком случае
СП
Ну, если такой расчет долго выполняется, то тогда поток имеет смысл, ok
Это так сказать кошка, нановерсия. На ней хочу обкатать подход-архитектуру.
RS
Синхронайз периодически должен быть установлен в таком месте, где бы он периодически ызывался. А у тебя он вызывается в конце расчета. Зачем? Если и так потоки скоро закроются и ты сможешь все обновить
z
проперти нужны когда могут появиться сеттеры\геттеры (наврятли светит в простом обменном интерфейсе), публишед надо чтобы инфа была доступна в ртти для например автосериализации (это в принципе может произойти с данным форматом, если потребуется загрузить\сохранить). нет?
RS
Сергей Пятыгин
Это так сказать кошка, нановерсия. На ней хочу обкатать подход-архитектуру.
У тебя в каком месте кода происходят задержки? Вот там и нужно вставлять синхронайз, а не в конце расчетов перед освобождением потока