Отправлено Алексей Мусин 19 сентября 2003 г. 10:40 В ответ на: Приведенный Вами в пункте 2 пример - просто шикарный образец нечитаемого кода. отправлено Леонид Иванович 19 сентября 2003 г. 09:00

можно писать так:

SsScrPutNum ( WTime.hour  , SCR_POS_L4 | SCR_POS_P4 | (WTime.hour < 10 ? SCR_NUM_00_FLG : 0));

и можно писать так:

if (WTime.hour < 10)
{
    SsScrPutNum ( WTime.hour  , SCR_POS_L4 | SCR_POS_P4 | SCR_NUM_00_FLG);
}
else
{
    SsScrPutNum ( WTime.hour  , SCR_POS_L4 | SCR_POS_P4);
}

если мне нужно изменить параметр SCR_POS_P4 на SCR_POS_P6, то я должен не забыть сделать это в двух местах.

Везде должна быть мера.

• А вот так не лучше? — smav   (19.09.2003 11:16, 276 байт)
  • нет, потому что — Алексей Мусин   (19.09.2003 12:01, 444 байт)
    • А на мой взгляд, потеря читаемости не компенсируется длиной исходника — smav   (19.09.2003 12:54, пустое)
      • да, но тут кусок читабельный :) — Алексей Мусин   (19.09.2003 13:02, 131 байт)
        • Ну, это уже вопрос вкуса — smav   (19.09.2003 15:56, 273 байт)
          • А что он нарушил? :) — Алексей Мусин   (19.09.2003 16:09, пустое)
            • Здравый смысл :) — smav   (19.09.2003 18:17, 66 байт)
    • А мой-то компилятор похоже дурак :) — Отделение 13   (19.09.2003 12:16, пустое)
  • Объем кода наверное все равно больше, т.к. компилируется в 2 вызова функции — Отделение 13   (19.09.2003 11:42, пустое)