1. C# / Говнокод #7328

    +975

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    switch (Game1.mus)
            {
                case false: sb.Draw(disabled, new Vector2(ScrW - 256, 90), Color.White); break;
                case true: sb.Draw(enabled, new Vector2(ScrW - 256, 90), Color.White); break;
            }
            
            switch (Game1.bloooom)
            {
                case false: sb.Draw(disabled, new Vector2(ScrW - 256, 180), Color.White); break;
                case true: sb.Draw(enabled, new Vector2(ScrW - 256, 180), Color.White); break;
            }
            
            switch (Game1.part)
            {
                case false: sb.Draw(disabled, new Vector2(ScrW - 256, 270), Color.White); break; //420
                case true: sb.Draw(enabled, new Vector2(ScrW - 256, 270), Color.White); break;
            }
            
            switch (Game1.eff)
            {
                case false: sb.Draw(disabled, new Vector2(ScrW - 256, 360), Color.White); break; //510
                case true: sb.Draw(enabled, new Vector2(ScrW - 256, 360), Color.White); break;
            }
            switch (Game1.shad)
            {
                case false: sb.Draw(disabled, new Vector2(ScrW - 256, 450), Color.White); break; 
                case true: sb.Draw(enabled, new Vector2(ScrW - 256, 450), Color.White); break;
            }

    Оттуда же, откуда и #5199. Случайно обнаружил...

    Запостил: RaZeR, 24 Июля 2011

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

    • ava Lure Of Chaos 24.07.2011 15:01 # 0

      кейс на булин? я такого еще не видел )))
      Ответить

    • ava inkanus-gray 24.07.2011 19:08 # +4

      Будет весело, если disabled и enabled определены как false и true соответственно.

      И почему-то секцию default я не увидел. Вдруг что-то пойдёт не так?
      Ответить

      • ava SmackMyBitchUp 24.07.2011 20:13 # +2

        у них и так все не так
        Ответить

      • ava sayidandrtfm 24.07.2011 20:30 # 0

        >Будет весело, если disabled и enabled определены как false и true соответственно.
        Тогда выхлоп рефака 4 строки.
        Ответить

      • ava RaZeR 24.07.2011 21:47 # 0

        disabled и enabled - текстуры, но говнокодности фрагмента это не уменьшает :)
        Ответить

    • ava guest 24.07.2011 21:42 # −8

      показать все, что скрытоКонкурс на самый лучший рефакторинг этого кода объявляю открытым.
      Ответить

      • ava inkanus-gray 28.07.2011 14:45 # +1 

        boolean blooms[5] = {Game1.mus, Game1.bloooom, Game1.part, Game1.eff, Game1.shad};
int offsets[5] = {90, 180, 270, 360, 450};
for(i = 0; i < 5; i++) sb.Draw(blooms[i], new Vector2(ScrW - 256, offsets[i]), Color.White);
        Ответить

        • ava absolut 28.07.2011 15:29 # +2

          это не С#. а если C/C++, то размер массива можно не указывать. Ну и булен не знаю куда девать в таком виде.
          Ответить

          • ava inkanus-gray 28.07.2011 19:10 # 0

            Да я понял, когда отправил комментарий. Но это же говнокод, здесь не должно быть всё правильно. А с булином выкручиваться либо тернарным оператором: 
            sb.Draw(blooms[i] ? enabled : disabled, new Vector2(ScrW - 256, offsets[i]), Color.White);

            Либо массивчиком из двух элементов: {disabled, enabled}
            Ответить

        • ava inser 29.07.2011 17:38 # +3

          А ещё сделать:
          for(i = 0; i < 5; i++) sb.Draw(blooms[i], new Vector2(ScrW - 256, 90 * (i + 1)), Color.White);

          и не нужны offsets
          Ответить

    • ava zooz13 24.07.2011 23:55 # −1

      bloooom, bleeeeat'!
      Ответить

    • ava inser 28.07.2011 14:34 # −1

      Я отказываюсь верить в то, что так можно писать на C# ;(
      Ответить

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