|
|
Регистрация Восстановить пароль |
Повторная активизация e-mail |
Регистрация | Задать вопрос |
Заплачу за решение |
Новые сообщения |
Сообщения за день |
Расширенный поиск |
Правила |
Всё прочитано |
|
Опции темы | Поиск в этой теме |
22.08.2011, 23:08 | #1 |
Новичок
Джуниор
Регистрация: 22.08.2011
Сообщений: 1
|
Как можно улучшить код
Есть код
Код:
|
22.08.2011, 23:14 | #2 |
Форумчанин
Регистрация: 21.10.2010
Сообщений: 130
|
я бы не писал begin в отделной строке
|
22.08.2011, 23:29 | #3 | |
C++
Форумчанин
Регистрация: 27.03.2011
Сообщений: 803
|
Цитата:
Код:
Ищете информацию по C++?
cplusplus.com |
|
22.08.2011, 23:30 | #4 |
Участник клуба
Регистрация: 15.07.2008
Сообщений: 1,933
|
Сложно что-либо сказать по такому малому куску кода, но, во-первых, у вас k сравнивается с нулём, а не инициализируется им (видимо опечатка), во-вторых, название для этой переменной можно придумать получше, например JobsNumber или что-то в этом духе. В-третьих, хранить в переменной HoursWorked минуты тоже немного неочевидный шаг, можно завести, к примеру, локальную переменную MinutesWorked, а потом присваивать HoursWorked := MinutesWorked / 60. Да, ещё лучше будет, если заменить Avg на, как минимум, Avarage, так будет яснее
|
23.08.2011, 01:16 | #5 | |
Старожил
Регистрация: 12.01.2011
Сообщений: 19,500
|
Цитата:
Ушел с форума, https://www.programmersforum.rocks, alex.pantec@gmail.com, https://github.com/AlexP11223
ЛС отключены Аларом. |
|
23.08.2011, 04:21 | #6 |
Форумчанин
Регистрация: 18.10.2010
Сообщений: 419
|
Код:
1. не длинные комментарии одной строчки писать в конце этой строчки исключение - комментарий перед множеством строк (например, "Зададим свойства отчета") 2. поменьше пробелов, а то сквозь глаза проваливается! k:=0; 3. k:=k+1; - смотрится нагляднее, чем Inc(); но оправдано только если k, а не HoursWorked 4. выровнять однотипные присвоения - "Зададим свойства отчета" (здесь допустимы пробелы) 5. выровнять комментарии по возможности: ...oursWorked/60; // Переведе... ...ed/k; // Вычислим... p.s. я так как-то вообще без отступов привык все делать: Код:
6. никаких переносов во время вызова функции!!!!! Report.Fields("UserField2") := FloatToStr(HoursWorked) + " (" + FloatToStr(Avg) + " ч./работу)"; ибо БЕСЕД! вызов функции одной строчкой, и только! ну разве что за исключением CreateWindow ))) Последний раз редактировалось xrob; 23.08.2011 в 04:24. |
Похожие темы | ||||
Тема | Автор | Раздел | Ответов | Последнее сообщение |
как можно сократить код | sergio280888 | БД в Delphi | 0 | 17.04.2011 17:09 |
Как улучшить код | kirza | Общие вопросы Delphi | 10 | 19.01.2011 18:14 |
Вопрос по символьным массивам. Как улучшить код. | Aerial | Общие вопросы C/C++ | 4 | 13.04.2010 10:54 |
Как можно улучшить эффективность и качество ПП? | Evgenii | Помощь студентам | 5 | 29.01.2009 16:22 |
Как можно сократить код | jocry | Общие вопросы Delphi | 2 | 20.01.2009 08:35 |