Субъективный Си / Говнокод #12757 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
int arrayCount = [itemArray count];
if( currentHeight < 10000 ) {
    currentItemType = random() % arrayCount;
} else if( currentHeight < 50000 ) {
    currentItemType = random() % arrayCount;
} else if( currentHeight < 100000 ) {
    currentItemType = random() % arrayCount;
} else {
    currentItemType = random() % arrayCount;
}

Чем выше горы - тем рандомней сборы.

Запостил: NAlexN NAlexN, (Updated )

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

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

    Что натолкнуло человека, который писал этот код, на такое сверхчитабельное решение? 🙂
    Ответить

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

      либо гк, либо для отладки. другого объяснения не может быть. я склоняюсь к первому.
      Ответить

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

        Да, скорее гк. А что можно отлаживать таким способом?
        Ответить

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

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

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

    Для отладки или для заглушки. Наверное, логика должна быть разной для разных значений "currentHeight". А сейчас просто поставили заглушку, потому что пока точно неизвестно, какая именно должна быть формула или константа для соответствующих случаев.
    Ответить

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

      Я вас наверное разочарую, но это было в релиз версии игры. Товарищь не знал про srand(time(0)); и arc4random, а хотелось динамической генерации уровней, выкрутился как мог, видимо. И ведь его совсем не смутило что рандом каждый раз совсем не рандом...
      Ответить

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

        >Я вас наверное разочарую, но это было в релиз версии игры
        тогда забыл написать todo, не успел исправить к релизу и т.д. я отказываюсь верить, что это нарочно
        Ответить

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

      Я-б тоже так подумал, если бы не наткнулся недавно на код коллеги...
      if (prov_tag == PROVIDER_YAHOO) {
  initCampGrid();
  initAdsGrid();
} elseif (prov_tag == PROVIDER_GOOGLE) {
  initCampGrid();
  initAdsGrid();
} //...так ещё десяток идентичных, чтобы ура, найти различие:
... elseif (prov_tag = PROVIDER_APPNEXUS) {
  initCampGrid();
  initAdsGrid();
  initCreativeGrid();
}
      Ответить

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

        А надо так, да?
        initCampGrid();
initAdsGrid();
if (prov_tag = PROVIDER_APPNEXUS)
  initCreativeGrid();
        Ответить

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

        > prov_tag = PROVIDER_APPNEXUS
        Это всегда true. Си ошибок не прощает.
        Ответить

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

          В том и беда, что человек - уверенный середнячок. Но когда видет подобные "начала" - не пытается это переписать. Там сначала нормально было, 5 провайдеров, все одинакоыве кроме несчастного Яху, для которого первый иф и добавился...
          Ответить

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

            Видит*.
            Блин, в хром надо услугу наподобие перевода: автокоррекцию текста на страницах. 🙂 А то так начетаисся фсяких, потом руки сами повторяют...
            Ответить

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

Где здесь C++, guest?!

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

    ↑
    8
    ↓