Кресты / Говнокод #4428 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
int X,Y,Z;
X=StrToInt(Edit1->Text);
Y=StrToInt(Edit2->Text);
Z=X+Y;
Edit3->Text=IntToStr(Z),

Запостил: Мартин Мартин, (Updated )

Комментарии (10) RSS

  • ava Kornew # Ссылка на оригинал 0

    жуть какая, это же надо так!! фииии... (табличка::сарказм)

    по большей степени ГК тут в именах классов...
    Ответить

    • ava absolut # Ссылка на оригинал 0 ⇈

      имен классов тут нет вообще 🙂 есть имена переменных
      дефолтовые названия для RAD систем типа Delphi.
      Ответить

      • ava Мартин # Ссылка на оригинал 0 ⇈

        Писался код в спп билдере. Кроме имен компонентов, еще мне непонятно зачем вообще нужно было вводить переменные... если 1 строки хватило бы на сложение.
        Ответить

        • ava absolut # Ссылка на оригинал 0 ⇈

          > если 1 строки хватило бы на сложение
          Типа так: 
          Edit3->Text = IntToStr( StrToInt(Edit1->Text) + StrToInt(Edit2->Text) );
          ?
          Первый вариант зато более очевидный и наглядный. Ну может быть Z следовало убрать. А X,Y я бы оставил.
          Ответить

          • ava Мартин # Ссылка на оригинал 0 ⇈

            >> Типа так:
            Edit3->Text = IntToStr( StrToInt(Edit1->Text) + StrToInt(Edit2->Text) );
            Да.

            >> Первый вариант зато более очевидный и наглядный.
            Когда программа небольшая и строк мало - соглашусь. Иначе зачем попусту тратить ресурсы? И разбираться в коде сложнее.
            Ответить

            • ava bugmenot # Ссылка на оригинал 0 ⇈

              трехадресный код же. на сваливании в кучу ничего не сэкономится
              Ответить

            • ava absolut # Ссылка на оригинал 0 ⇈

              А с чего вы взяли что компилятор не сможет при оптимизации убрать переменную?
              Писать надо так чтобы понятнее было человеку, а не компьютеру. Сложнее разбираться в одной длинной строке, чем в нескольких коротких.
              Ответить

            • ava TheCalligrapher # Ссылка на оригинал 0 ⇈

              Никакой траты ресурсов там нет и в помине. Разбираться в коде с переменными - именно как в оригинале - существенно легче. Более того, множественные вызовы функций в одном statement - за это сразу в сад. Программа не должны содержать более одного вызова функции на statement, на случай если потом понадобится ее пошагово отлаживать. И именно для облегчения отладки промежуточные результаты лучше сохранать в переменных.
              Ответить

Добавить комментарий

Семь раз отмерь — один отрежь, guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓