конструкция if else

[Emperator12]

if checkbox1.Checked = true then begin
work:= true;
doChek1(log,pass, work, forum1);
end else if checkbox1.Checked = false then  begin
work:= false;
doChek1(log,pass, work, forum1);

if work = true and checkbox5.checked = false then begin
   showmessage('Записать в файл без тегов');
   end else if work = true and checkbox5.checked = true then begin
   showmessage('Записать в файл с тегами');
   end else if work = false and checkbox5.checked  = false then begin
   showmessage('Ничего не делать');
   end;

ДАЖЕ Когда оба чек боксы не отмечены, пишет, что мол записать в файл с тегами. Почему ? где ошибка ?

[BDA]

Попробуйте расставить скобки в логических выражениях, по-моему "and" старше "=".
(Профит http://informatics.mccme.ru/moodle/m...&chapterid=255)

[Emperator12]

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

Попробуйте расставить скобки в логических выражениях, по-моему "and" старше "=".
(Профит http://informatics.mccme.ru/moodle/m...&chapterid=255)

Чтоо ? "=" - оператор присвоения. "and" - логический оператор.
Как "and" может быть старше "=" ?

[Emperator12]

Скобки ( ) помогли, кстати.

[Voody]

Причем тут присвоение.
= - это операция отношения
И как видно по ссылке, операция and имеет приоритет перед =, а значит и выполняется сначала and

[Serge_Bliznykov]

надо логические выражения обязательно брать в скобки
вот так:

if (work = true) and (checkbox5.checked = false) then begin
   end else if (work = true) and (checkbox5.checked = true) then begin
   showmessage('Записать в файл с тегами');
   end else if (work = false) and (checkbox5.checked  = false) then begin ...

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

if work and Not checkbox5.checked then begin
   showmessage('Записать в файл без тегов');
   end 
   else if work and checkbox5.checked then begin
   showmessage('Записать в файл с тегами');
   end 
   else if Not work and Not checkbox5.checked then begin
   showmessage('Ничего не делать');
   end;

ну и ещё. обратите внимание, что, очевидно, один из вариантов в вашем примере вообще не обрабатывается (это когда work - false и checked - true)

[Emperator12]

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

ну и ещё. обратите внимание, что, очевидно, один из вариантов в вашем примере вообще не обрабатывается (это когда work - false и checked - true)

Такого варианта у меня не может быть. Ну то есть если work - false и checked - true, тогда ничего не нужно делать.
Всем спасибо.

[Serge_Bliznykov]

Ну то есть если work - false и checked - true, тогда ничего не нужно делать.

ну, судя по этому коду:

if Not work and Not checkbox5.checked then begin
   showmessage('Ничего не делать');

в случае, когда work - false и checked - false - тоже ничего не надо делать!
поэтому Ваш код легко вырождается в такой:

if work then begin
   if checkbox5.checked then
     showmessage('Записать в файл с тегами')
   else
     showmessage('Записать в файл без тегов');
end
else
    showmessage('Ничего не делать');

это и проще, и нагляднее и надёжнее. Рекомендую!