Скачать: CL | DM;

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

← →
Медвежонок Пятачок ©   (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].
>
> У меня фантазии уже не хватает.

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

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

---

← →
Student ©   (2009-06-23 21:06) [87]

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

---

← →
AndreyV ©   (2009-06-23 21:15) [88]

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

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

---

← →
AndreyV ©   (2009-06-23 21:18) [89]

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

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

---

Страницы: 1 2 3 вся ветка

Скачать: CL | DM;



Память: 0.6 MB
Время: 0.011 c