Показать сообщение отдельно
  #6  
Старый 07.07.2013, 22:28
Аватар для Страдалецъ
Страдалецъ Страдалецъ вне форума
Гуру
 
Регистрация: 09.03.2009
Адрес: На курорте, из окна вижу теплое Баренцево море. Бррр.
Сообщения: 4,721
Репутация: 52347
По умолчанию

Не, ну конечно вам виднее, как оформлять. Просто несколько широковато выходит. Я например конструкцию if then else оформляю немножко покомпактнее, но кому-то тоже так не нравится. Вот скажем ваш код с моим вариантом форматирования:
Код:
procedure TForm1.SpeedButtonClick(Sender: TObject);
var f: Integer;
    name: string;
begin
 cler;
 if work
 then begin
      try
       Form1.TIdPOP.Connect();
       if Form1.TIdPOP.Connected
       then begin
            all:= Form1.TIdPOP.CheckMessages;
            Form1.Label1.Caption:= IntToStr(all);
            form1.Label3.Font.Color:= clLime;
            form1.Label3.Caption:= '> Последний просмотр был в '+timeToStr(Now);
            Form1.lst1.Clear;
            for f:= 1 to all
            do begin
               Form1.IdMessage2.Clear;
               Form1.TIdPOP.Retrieve(f,Form1.IdMessage2);
               Form1.IdMessage2.CharSet:='koi8-r';
               if Form1.IdMessage2.Subject = ''
               then begin
                    name:= 'Нет заголовка.';
                    dat:= 'not';
                    tim:= 'not';
                    end
               else begin
                    dat:= DateToStr(Form1.IdMessage2.Date);
                    name:= form1.IdMessage2.Subject;
                    tim:= TimeToStr(Form1.IdMessage2.Date);
                    end;
               Form1.lst1.Items.Add('');
               Form1.lst1.Items[f-1]:= IntToStr(f)+ ' > '+ name + ' : '+dat+ ' / '+tim;
               end;
            if allMemory > all
            then allMemory:= all;
            if (allMemory <> all) and (first = False) and (Form2.CheckBox1.Checked = True)
            then theMail;
            if first = True
            then begin
                 first:= False;
                 allMemory:= all;
                 end;
            end;
      except
       Form1.Label3.Font.Color:= clRed;
       form1.Label3.Caption:= '> Превышен лимит времени TIME OUT !';
      end;
      Form1.TIdPOP.Disconnect;
      end
 else form1.Label3.Caption:= '> Не установлены данные адреса! Войдите в НАСТРОЙКИ';
end;
__________________
Жизнь такова какова она есть и больше никакова.
Помогаю за спасибо.
Ответить с цитированием