В общем-то, согласен - см. мой первый пост в этой теме. Замечание было локальное.

Разбиваю функцию Action() на много маленьких функций, но им придется передавать кучу параметров, что делать?

Подумать, может ты неправ ;-)

Bsa, не, это точно, мне же самому удобнее будет, еще можно как я понял улучшить код, делаю вот такие штуки, вместо старых ифов:

Только не понимаю какой символ за что отвечает, если можно, поясните.
Вот по типу этих ифов, наверняка есть какие другие хитрости, я знаю их множество, очень они мне нужны, посветите, пожалуйста.

Это сообщение отредактировал(а) m1kle - 30.5.2007, 17:25

Удобнее передавать кучу параметров?


И чем это лучше if-а? На две буквы короче?

ИМХО код должен быть прежде всего легок для чтения и понимания людьми, удобен в использовании (12 параметров этому не способствуют), а потом уже эффективен и.т.д Причем если код прост, понятен и удобен, то он скорее всего уже эффективен (я не имею ввиду использование плохих алгоритмов итд).

Если действительно хочешь улучшить код, то это

очень хороший совет  
Тут дело не в хиртостях "по типу этих ифов", а в всём дизайне, подумай как можно упростить код так, что бы не было необходимости передавать много параметров и писать длинные вложенные циклы с кучей if-ов (это называется "спагетти-код").
Например передавай ссылку на структуру, меняй в специализированных функциях только нужные переменные, а не все сразу.

nickless
1) Разбить большую функцию на много маленьких, помойму это очень удобно и разумно.
2) "Хитрости" тоже для удобства, и уменьшения длинны кода ( если он не длинный, с ним легче имхо работать ).
3) А насчет структур. А что плохого в передаче многих парметров. Стрктуры это уменьшение эффективности программы(время выполнения кода), и куча лишних строчей.

Можешь раскритиковать, буду только благодарен ( я новечок в этом деле )

а вот кстати проверил и получил такие результаты




разница налицо  

Добавлено через 2 минуты и 53 секунды

будешь когда-нибудь на WinAPI писать (  ) - поймешь, что куча параметров - это не сильно здорово, мягко говоря

Неудачный пример.
Лучше попробовать такой:

Так как в твоем случае компилятор серьезную оптимизацию сделал. Замена 0 на 1 по флагу через:Думаю, что заменить 1234 на 4321 таким же образом у него вряд ли получится.   

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

1. нарисовать блок схему (в крупных проектах она просто необходима, чтобы не сойти с ума)
2. перейти к Объектно Ориентированному Программированию (ООП)

по блок схеме легко находить уязвимые места и это позволит абстрагироваться от кода, позволит сосредоточится на идеи.. это важняк!! не ленись делать блок схемы.. это так же важно как и комментарии..
первое приемущество ООП над структурированным программированием заключается в ИНКАПСУЛЯЦИИ данных в один объект, что повысит как читабельность так и управляемость.. и к тому же ты сможешь размножать эти объекты в любом количестве.. потом тебе на помощь придет НАСЛЕДОВАНИЕ и ПОЛИМОРФИЗМ, может, тоже придет..

учи ООП.. важняк!! ;)

Это сообщение отредактировал(а) fish9370 - 31.5.2007, 20:54

bsa, 
это все понятно, я не спорю

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

Это сообщение отредактировал(а) MAKCim - 31.5.2007, 09:39

Я тоже решил проверить, только со включенной оптимизацией   



И никакой разницы  

  
И ещё СТЛ.