1. Objective C / Говнокод #12757

    −96

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    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, 16 Марта 2013

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

    • ava zummenix 16.03.2013 18:50 # +2

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

      • ava Dummy00001 16.03.2013 19:18 # +2

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

        • ava zummenix 16.03.2013 19:28 # +1

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

          • ava Dummy00001 16.03.2013 20:10 # +3

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

    • ava guest 16.03.2013 20:08 # +2

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

      • ava NAlexN 16.03.2013 20:14 # 0

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

        • ava guest 22.03.2013 18:00 # 0

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

      • ava Lowezar 17.03.2013 11:31 # +2

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

        • ava 3Doomer 18.03.2013 07:05 # −1

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

        • ava bormand 18.03.2013 08:23 # +3

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

        • ava zummenix 18.03.2013 08:45 # 0

          Мне интересно, так junior'ы пишут или кто "моложе"?
          Ответить

          • ava Lowezar 18.03.2013 09:54 # +1

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

            • ava Lowezar 18.03.2013 15:25 # 0

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

    • ava Govnocoder#0xFF 17.03.2013 21:26 # 0

      М.б. для последующего запиливания фич для каждого значения?
      Ответить

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