 Упрощение кода, написание того или иного кода более понятно и просто |
Здравствуйте, гость ( Вход | Регистрация )
AD опции профиля:
|
 5.8.2008, 16:26
Сообщение
#1
|
|
Профессионал  Группа: Участник Сообщений: 2003 Регистрация: 4.2.2008 Из: S-Petersburg Пользователь №: 84 Спасибо сказали: 70 раз(а) Репутация:  17 
 |
Собственно предлагаю сюда задавать и выкладывать куски кода (с пояснениями, конечно, что этот код делает) для того, чтобы помогли его ну как упростить, сделать более читабельным. Возможно, кому-то это поможет.
Если разрешите, то могу начать. |
 |
  
|
 |
Ответов
|
Tonal опции профиля:
|
11.8.2008, 8:05
Сообщение
#2
|
 Активный участник Группа: Участник Сообщений: 452 Регистрация: 6.12.2007 Из: Новосибирск Пользователь №: 34 Спасибо сказали: 69 раз(а) Репутация: 17
|
Цитата Дай - понимаю, курить - понимаю, дай курить - не понимаю! Какое предложение объяснить? Да, и функции, вместо checkStatus* я бы назвал isStatus* - во первых сразу понятно что они возвращают и зачем, во вторых соответствует стилю Qt. 
|
|
|
|
|
AD опции профиля:
|
11.8.2008, 8:48
Сообщение
#3
|
|
Профессионал Группа: Участник Сообщений: 2003 Регистрация: 4.2.2008 Из: S-Petersburg Пользователь №: 84 Спасибо сказали: 70 раз(а) Репутация: 17
|
Цитата(Tonal @ 11.8.2008, 9:05)  После проверок где isContinue = false и checkStatusBreak сразу выходить. Проверку на init_gaps в начало функции. и тоже по false сразу выходить. Изменение, которое после checkStatusDist тоже в 2 отдельные функции. Ну и из (time_t)fabs(double(next -> time_marker - cur -> time_marker)) тоже функцию. Что сделать надо, что именно и куда вынести? |
|
|
|
Сообщений в этой теме
AD Упрощение кода 5.8.2008, 16:26
void* ну вот у меня такая дилема. Нижеприведенный код по... 5.8.2008, 18:46 ViGOur Ну я дуаю ты сам мог бы догадаться, если бы был вн... 5.8.2008, 21:59 void* ViGOur, насчет entryInfoList я знал, просто мне ка... 5.8.2008, 22:21 Red Devil void ProjectView::AddDirs(const QDir ... 6.8.2008, 11:59 void* упростил однако из двух функций сделал четыре
хо... 6.8.2008, 12:49 ViGOur void createItems(const QDir &dir, QTreeWid... 6.8.2008, 14:22 void* ViGOur, спасибо! примерно такое у меня и верте... 6.8.2008, 17:26 Tonal 2 ViGOur
Я бы всё таки выделил из этого кода функц... 7.8.2008, 8:09 ViGOur Цитата(Tonal @ 7.8.2008, 9:09) Я бы всё т... 7.8.2008, 8:27
AD Выдалась возможность, может сможете и мне упростит... 7.8.2008, 9:10 Tonal Вынеси длинные проверки с next -> status... в о... 7.8.2008, 12:02 AD Цитата(Tonal @ 7.8.2008, 13:02) Вынеси дл... 7.8.2008, 12:55 Tonal Без тега кода всяко только уменьшилась!
По к... 8.8.2008, 15:21 Tonal Чё непонятно-то? 10.8.2008, 17:31 AD Цитата(Tonal @ 10.8.2008, 18:31) Чё непон... 10.8.2008, 22:06 Tonal 1) После проверок где...
Код
if(...) isC... 11.8.2008, 11:07 AD Теперь это выглядит более внятно и легче править. ... 11.8.2008, 14:54 Red Devil Кодclass ScrTrack: public TImageClass
{
privat... 12.8.2008, 10:12 rich Может кому поможет:
"Рефакторинг. Улучшение с... 13.12.2008, 0:03 Admin rich, а ты темой не ошибся? Причем тут упрощение к... 13.12.2008, 0:52 alex977 Цитата(Admin @ 13.12.2008, 0:52) rich, а ... 13.12.2008, 11:28 rich Цитата(Admin @ 13.12.2008, 0:52) rich, а ... 14.12.2008, 18:41  |
Теги
Нет тегов для показа
4 чел. читают эту тему (гостей: 4, скрытых пользователей: 0)
Пользователей: 0
|
|
Текстовая версия | Сейчас: 26.11.2024, 3:51 |