1. Java / Говнокод #13917

    +70

    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
    // there is class PlayerExt, which extends class Player...
// min >= 0
// max <= players.size()

    List<PlayerExt> players = playerManager.getPlayers(contestId);
    Player[] response = new Player[players.size()];

    for (int i = min; i < max; i++) {
        response[i] = players.get(i);
        if (!players.get(i).isQualified()) {
            response[i].setChipStack(BigDecimal.valueOf(-1));
        }
        response[i].setPosition(i + 1);
        response[i].setCustomerId(players.get(i).getCustomerId());
    }

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

    Запостил: wissenstein, 09 Октября 2013

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

    • ava someone 09.10.2013 16:54 # −1

      Зачем вообще массивы в высокоуровневом коде?
      Ответить

      • ava wissenstein 09.10.2013 17:00 # 0

        Ну, конкретно тут массив требуется выдавать в качестве результата операции. Другое дело — как его сформировать. А когда я сделал рефакторинг этого фрагмента, то последняя сторка цикла стала предпоследней и приняла вид
        player.setCustomerId(player.getCustomerId());

        что и побудило опубликовать фрагмент. :)
        Ответить

        • ava someone 09.10.2013 17:50 # 0

          > Ну, конкретно тут массив требуется выдавать в качестве результата операции.

          Почему бы не отрефакторить в List? Или это legacy API какой-то?
          Ответить

      • ava Lure Of Chaos 11.10.2013 21:59 # +2

        а разве массивы коррелируют с высокоуровневым кодом?
        Ответить

    • ava Qwertiy 09.10.2013 20:49 # 0

      Джава - она и есть джава... Ничего особенного для неё тут не вижу.
      Ну да, пожалуй players.get(i) действительно лучше вынести в переменную. Ну и что?
      Ответить

      • ava anonimb84a2f6fd141 09.10.2013 21:08 # −2 

        List<PlayerExt> players = playerManager.getPlayers(contestId);
List<Player> response = new ArrayList<Player>;

    for (PlayerExt resp: players) {
        if (!resp.isQualified()) {
            resp.setChipStack(BigDecimal.valueOf(-1));
        }
        // ОЛОЛО, ОЛОЛО, ВСЕ ВЫ БЫДЛО И ХУЙЛО!
        resp.setCustomerId(resp.getCustomerId());
        response.add(resp);
    }

        1. foreach
        2. Зачем массив, когда есть List?
        Ответить

        • ava wissenstein 09.10.2013 23:25 # 0

          > 1. foreach
          А куда девать setPosition(i + 1)?

          > 2. Зачем массив, когда есть List?
          См. http://govnokod.ru/13917#comment198991.

          > resp.setCustomerId(resp.getCustomerId()) ;
          Вот тут и суть. Эта строка вообще не нужна. Только лишний раз заргужает процессор. И до рефакторинга не очень сообразишь, что response[i].setCustomerId(players.get(i).getCustome rId()); надо вообще стереть.
          Ответить

          • ava anonimb84a2f6fd141 10.10.2013 02:39 # 0

            1. А на кой он там? Просто интересно.
            2. Ну так сделать toArray(), если уж так нужно. Не вижу ни малейшей причины без нужды возиться с массивами.
            Ответить

            • ava wissenstein 10.10.2013 21:39 # 0

              1. Капитан Очевидность подсказывает, что это позиция игрока (величина временная и в базе данных не хранится) .
              2. Ну, никто, в общем-то не запрещает.
              Ответить

      • ava wissenstein 09.10.2013 23:42 # 0

        Когда вынесешь players.get(i) в переменную player и заменишь обращения response[i] обращениями к ней, вскрывается суть:
        List<PlayerExt> players = playerManager.getPlayers(contestId);
    Player[] response = new Player[players.size()];

    for (int i = min; i < max; i++) {
        Player player = players.get(i);
        if (!player.isQualified()) {
            player.setChipStack(BigDecimal.valueOf(-1));
        }
        player.setPosition(i + 1);
        
        // Unnecessary sh*t
        // player.setCustomerId(player.getCustomerId());
        
        response[i] = player;
    }
        Ответить

    • ava guest 11.10.2013 14:17 # −15

      показать все, что скрытоОТСОСУ У КАВКАЗЦА, ДАМ В ЖОПУ ТАДЖИКУ, ВЫЛИЖУ АНУС УЗБЕКУ vk.com/id29482‎
      Ответить

      • ava guest 11.10.2013 14:30 # −15

        показать все, что скрыто
        ~Клуб знакомств для геев~
Познакомлюсь с парнем
             Сделаю миньет, подставлю попку парню кавказской внешности.
             Прут наглые хуястые самцы, желательно с большим членом, так как  
             очко сильно раздолблено. Могу принять сразу до 3-х парней.
            Пишите [email protected]  /Тарас/
        Ответить

        • ava guest 11.10.2013 14:31 # −15

          показать все, что скрыто
          ~Клуб знакомств для геев~
Познакомлюсь с парнем
             Сделаю миньет, подставлю попку парню кавказской внешности.
             Прут наглые хуястые самцы, желательно с большим членом, так как  
             очко сильно раздолблено. Могу принять сразу до 3-х парней.
            Пишите [email protected]  /Тарас/
          Ответить

          • ava guest 11.10.2013 14:34 # −15

            показать все, что скрыто
            ~Клуб знакомств для геев~
        Познакомлюсь с парнем
          Сделаю миньет, подставлю попку парню кавказской внешности.
          Прут наглые хуястые самцы, желательно с большим членом, так как  
          очко сильно раздолблено. Могу принять сразу до 3-х парней.
            Пишите [email protected]  /Тарас/
            Ответить

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