1. C++ / Говнокод #3698

    +160

    1. 1
    2. 2
    3. 3
    4. 4
    m_lActiveTab = GetCurSel();
for (int i = 0; i < GetItemCount(); i++)
	m_cItemSelected[m_lActiveTab] = false;
m_cItemSelected[m_lActiveTab] = true;

    Вот так говнокодят в крупных проектах

    Запостил: Snake2101, 14 Июля 2010

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

    • ava absolut 14.07.2010 11:40 # +2

      Похоже, что копи-паст всему виной.
      Должно быть так :
      m_lActiveTab = GetCurSel();
for (int i = 0; i < GetItemCount(); i++)
	m_cItemSelected[i] = false;
m_cItemSelected[m_lActiveTab] = true;

      Или как-то иначе ?
      Ответить

      • ava Snake2101 14.07.2010 11:41 # 0

        По идеи так и должно было быть
        Ответить

        • ava da4ever 14.07.2010 13:30 # +2

          "по идеи" ?!
          Вон с моего говнокода!
          Ответить

          • ava Webkill 14.07.2010 15:22 # −5

            показать все, что скрытоВ Древней Руси писали "по земли", "по идеи", "в деревни".
            Относительно древних русов "по идее" есть ошибка-аналогия с формами на -а, т.к. мягкие формы (-й-, допустим) имели форму "-и" в данном случаи + смещение падежа с предложного на дательный, тоже гадкая ошибка (что-то вроде "согласно плану" > "согласно плана").
            Граммар-наци 14 века негодуэ.
            Ответить

            • ava Snake2101 14.07.2010 21:31 # 0

              Про правописание можно только пожалуйста не лечить, я и так русский язык не знаю)))
              Ответить

    • ava pincode 14.07.2010 15:07 # 0

      GetItemCount() мульён раз будет вызыватся?
      Ответить

      • ava Webkill 14.07.2010 15:14 # 0

        inline?
        Ответить

      • ava absolut 14.07.2010 15:26 # 0

        GetItemCount() будет вызываться GetItemCount()+1 раз.
        P.S. Если конечно возвращает неотрицательное число и не больше int'a.
        И к тому же, она действительно может быть inline для доступа к целочисленному private полю.
        Ответить

        • ava pincode 14.07.2010 15:45 # 0

          Тогда лучше так:
          int ItemCount = GetItemCount();
for (int i = 0; i < ItemCount; i++)
          Ответить

        • ava Webkill 14.07.2010 15:48 # 0

          > GetItemCount() будет вызываться GetItemCount()+1 раз.

          Функция сама задаёт, сколько раз она будет вызываться, это охуенно
          Ответить

          • ava WebKill_HochetVBan 14.07.2010 15:54 # 0

            >Функция сама задаёт, сколько раз она будет вызываться
            Капитан Очевидность предупреждает, что функция может вызываться и в других кусках кода.

            >охуенно
            В каком месте?
            Ответить

          • ava absolut 14.07.2010 15:59 # 0

            >Функция сама задаёт, сколько раз она будет вызываться
            Вообще-то это задает цикл ;)
            Ответить

            • ava Webkill 14.07.2010 16:06 # −2

              Вам не понять всей глубины сей смысловой рекурсии :(
              Функция сама задаёт, сколько раз она будет вызываться, что тем самым задаст, сколько она будет вызываться, что задаст сколько вызываться.
              Понятно?
              Ответить

        • ava Snake2101 14.07.2010 21:32 # 0

          Могу тебя огорчить, GetItemCount это обложка для API функции SendMessage окнa TabControl
          Ответить

          • ava Snake2101 14.07.2010 21:35 # 0

            Я вообще не понимаю для чего это было сделано т.к в итоге оказалось что эта бойда не где не используется, возможно первоначально когда код писался лет 10 назад это где-то использовалось, а потом забыли убрать
            Ответить

      • ava WebKill_HochetVBan 14.07.2010 15:43 # +1

        Видимо на тот случай, если пользователь успеет за цикл сменить число выделеных элементов. :D
        Ответить

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