Качество кода

← →
Медвежонок Пятачок © (2009-06-23 14:13) [80]

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

а если придет в голову идея повторного использования этого чуда (собственно для чего и делают пользовательские классы), то его придется переписывать заново.

← →
Student © (2009-06-23 15:13) [81]

Медвежонок Пятачок

Подскажи пожалуйста путь по которому двигаться мне.

← →
Student © (2009-06-23 15:14) [82]

К чему мне прицепиться чтобы выполнять снова и снова код проверки?

← →
Игорь Шевченко © (2009-06-23 16:47) [83]

// constructor / destructor

Убрать. Это мусор, который мешает жить. Лучше отсутствие комментариев, чем такие.

Ini := TIniFile.Create(ExtractFilePath(Paramstr(0)) + "Chas.ini");
try
TimeServ := FileAge(Ini.ReadString("Dir","Trend", "") + "trend.dat");
except
FreeAndNil(Ini);
end;
FreeAndNil(Ini);

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

with TIniFile.Create(ExtractFilePath(Paramstr(0)) + "Chas.ini") do
try
TimeServ := FileAge(ReadString("Dir","Trend", "") + "trend.dat");
finally
Free;
end;

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

посмотрел дальше. Переписать все.

← →
Игорь Шевченко © (2009-06-23 16:56) [84]

Удалено модератором
Примечание: Дубль

← →
Student © (2009-06-23 20:25) [85]

Здравствуйте уважаемый All.
Привёл код к более компактному виду(ссылка на сырцы) http://www.sharemania.ru/0150162
Имя файла: WinChas.rar
Размер файла: 521,70 Kb.

Уважаемый All. Пожалуйста, подскажите мне, каким образом мне избавиться от 22-х таймеров? Принцип? Я думаю отдельный класс строить "по паттерну - посредник", а меня уже накритиковали в [80].

У меня фантазии уже не хватает.

← →
AndreyV © (2009-06-23 20:49) [86]

> [85] Student © (23.06.09 20:25)
> Уважаемый All. Пожалуйста, подскажите мне, каким образом
> мне избавиться от 22-х таймеров? Принцип? Я думаю отдельный
> класс строить "по паттерну - посредник", а меня уже накритиковали
> в [80].
>
> У меня фантазии уже не хватает.

Да ё моё. Я уже начал было отвечать, да удалил. Попробую ещё раз. Не глядя исходников, и не вникая особо:

Как, советовали, оставь один таймер, выбери интервал необходимый для точности. По событию таймера, делай обработку объектов по списку, снова запускай таймер. Список - как тебе удобно, скорее будет массив указателей. Судя по вопросу выше, так.

Ок. Делаю посредника. Пасибо AndreyV

> [87] Student © (23.06.09 21:06)
> Ок. Делаю посредника.

Ты не торопись с посредником: абстрагируйся, представь задачу ещё раз.

> [88] AndreyV © (23.06.09 21:15)
> Ты не торопись с посредником: абстрагируйся, представь задачу
> ещё раз.

Поспи, отвлекись, погуляй, потом вернись.

Качество кода Найти похожие ветки