Вопрос про корректность кода.

← →
Fredericco © (2003-06-03 17:47) [0]

Привет всем мастерам.

Я всерьез занялся оптимизацией своего кода.
Вопрос такой, как корректнее?
Так
procedure MyProc; begin if ... then begin ... Exit; end; if ... then begin ... Exit; end; if ... then begin ... Exit; end; end;
или так
procedure MyProc; begin if ... then begin ... end Else if ... then begin ... end Else if ... then begin ... end; end;

Кому не затруднительно напишите ответ.
С глубочайшим уважением Fredericco.

← →
Mike B. © (2003-06-03 17:51) [1]

Что значит корректнее в даннном случае?

← →
DVM © (2003-06-03 17:54) [2]

а CASE там не подойдет?

← →
kull © (2003-06-03 17:55) [3]

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

← →
Ru © (2003-06-03 17:55) [4]

в первом случае end для каждого begin сохраняется чтоб компилятор не ругался, а exit вообще выглядит лишним, по моему.

← →
Юрий Зотов © (2003-06-03 17:56) [5]

Э-э-э, елы-палы! Да разве в этом оптимизация-то? Это же копейки, не с них надо начинать.

← →
Fredericco © (2003-06-03 17:56) [6]

1) Какой код будет выполнятся быстрее.
2) Не влияет ли процедура Exit на читабельность кода.
3) Какой из двух типов ты обычно используешь сам, я доверяю твоему опыту.

← →
Mike B. © (2003-06-03 17:57) [7]

С одной стороны kull © прав, и возможно, вместе с DVM © (03.06.03 17:54) еще больше :) но с другой мы же не знаем, что там внутри этих ифов. Мне кажется, тут общего случая быть не может

← →
Fredericco © (2003-06-03 17:59) [8]

Юрий Зотов © (03.06.03 17:56)
Да вроде как больше уже нечего :-)

← →
Mike B. © (2003-06-03 18:00) [9]

Если так навскидку сказат я бы написал case, но опять-таки см. Mike B. © (03.06.03 17:57) кто его знает, что у тебя там

← →
Fredericco © (2003-06-03 18:03) [10]

2 DVM © (03.06.03 17:54)
CASE - подойдет.
А имеет ли смысл? Я просто не знаю.

2 Ru © (03.06.03 17:55)
Ставлю что бы остальной участок кода не отрабатывался. Нужно ли?

2 Mike B. © (03.06.03 17:57)
Приблизительно следующее.
b:=False; for i:=0 to Txt.Count-1 do begin b:=CheckAddr(Txt.Strings[i]); if b then Break; end; if b then Txt.Add(Chr(5)+Client[GetClientNum(sh)].Driver.Group+":"+Client[GetClientNum(sh)].Driver.Code); if Txt.Count>=4 then begin SendTextToHandle(HandleII,Txt.Text,SType); ExSH:=-1; TranslateCom(Txt.Text,sh,-1,SType); if (Client[GetClientNum(sh)].Driver.Group="10") and (Client[GetClientNum(sh)].Driver.Code="1") then ExSH:=sh; Txt.Insert(1,"1"); Txt.Insert(1,"10"); TranslateCom(Txt.Text,sh,ExSH,SType); end;

← →
Юрий Зотов © (2003-06-03 18:04) [11]

> Fredericco © (03.06.03 17:59)
> Да вроде как больше уже нечего

Если бы я мог сказать такое о своих программах, я был бы счастлив.

← →
Ihor Osov'yak © (2003-06-03 18:05) [12]

в общем случае отдаю предпочтение варианту один.. И дело тут не в оптимизации или отсутствии иной - см. ЮЗ. Имхо, это более читабельно.

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

← →
DVM © (2003-06-03 18:05) [13]

> Fredericco © (03.06.03 18:03)
>CASE - подойдет.
> А имеет ли смысл? Я просто не знаю.

Ну так используй. По моему гораздо читабельнее и короче будет. По скорости тоже самое или такие мизерные отличия, что забудь.

ну вот тебе раз

Лучше варианты в цепочке выстроить в порядке вероятности появления. Тут можно получить выигрыш.

2 Юрий Зотов © (03.06.03 18:04)
Я не так выразился.
Просто я гонял программу в течении суток, на отдельном компьютере, на предмет утечек памяти. Потом решил код оптимизировать. Шел по тексту программы с верху вниз, натыкался на кусок, который можно и по-другому написать. Отдельно проверял все варианты - брал наилучший. И вот дошел до этого кода. Действительно, внизу и верху от этого места еще много всего нехорошего. Просто я пока остановил свой поиск. :-)

Ребят, извините: нужно срочно бежать домой.
Спасибо за ответы!
Надеюсь завтра продолжим!!!

С уважением Fredericco.

Скорость выполнения одинаковая, там где используется exit - больше размер кода (на 2 байта (ret n) на каждый exit (или на 6-8, если там jmp стоит)).

Вопрос про корректность кода. Найти похожие ветки