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

    +73

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    int[] indexes = list.getSelectedIndices();
        if (indexes.length == 0) {
            indexes = getAllIndexes();
        }
        if (indexes.length == 0) {
            return false;
        }

    индуссы атакуют

    Запостил: Dexter, 15 Августа 2011

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

    • ava Lure Of Chaos 15.08.2011 15:52 # 0

      > int[] indexes = list.getSelectedIndices();
      так все-таки индекс или индис?
      индисский код.
      Ответить

    • ava tir 15.08.2011 15:53 # +2

      тут в опечатке говнокод?
      Ответить

      • ava Lure Of Chaos 15.08.2011 16:01 # 0

        говно в том, что забыли проверить на null
        Ответить

        • ava tir 15.08.2011 16:04 # −2

          а где написано, что там null может приходить? Я, например, вообще стараюсь null избегать, всегда пустой список возвращаю, где это возможно.
          Ответить

          • ava Lure Of Chaos 15.08.2011 16:07 # 0

            более того, не написано, что не может приходить.
            а это чревато.
            Ответить

            • ava tir 15.08.2011 16:15 # +2

              я нигде этого в данном примере не вижу. вы додумали свой вариант, а я свой. мы оба правы. тот кто постил это на говнокод мог бы указать на то, что методы могут возвращать null, ну или хотя бы выложить их реализации.

              А может человек вообще невнимательно посмотрел на этот пример, увидел его как

              int[] indexes = list.getSelectedIndices();
              if (indexes.length == 0) {
              indexes = list.getSelectedIndices();
              }
              if (indexes.length == 0) {
              return false;
              }

              и поспешил сюда запостить.
              Ответить

        • ava roman-kashitsyn 15.08.2011 16:11 # 0

          Спорно, очень спорно. Может, по семантике никогда null не возвращается. Опечатка + просто некрасиво. Наверняка можно было лучше придумать, что-то типа:
          boolean hasSelection = list.numOfSelectedRows() > 0;
int[] indexes = hasSelection ? list.getSelectedIndexes() : getAllIndexes();
if (indexes.length == 0) {
    return; // nothing to process
}
          Ответить

          • ava Lure Of Chaos 15.08.2011 16:15 # 0

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

            • ava roman-kashitsyn 15.08.2011 16:16 # 0

              Я лично код не с первого раза понял. Говно, конечно, но контекста не хватает.
              Ответить

    • ava Dexter 15.08.2011 16:16 # −1

      Я имел в виду то, что код плохо читабелен. Два одинаковых условия, меня очень сильно выбивают и у меня даже дернулась рука объединить их, что бы привело к изменению логики. :)
      Ответить

    • ava Dexter 15.08.2011 16:17 # −1

      Я имел в виду то, что код плохо читабелен. Два одинаковых условия, меня очень сильно выбивают и у меня даже дернулась рука объединить их, что бы привело к изменению логики. :)
      Ответить

    • ava Dexter 15.08.2011 16:22 # 0

      Извините, инет залагал
      Ответить

    • ava Dexter 15.08.2011 16:42 # −3

      Возможно так было бы лучше
      int[] indexes;
if (list.isSelectionEmpty()) {
	indexes = getAllIndexes();
	if (indexes.length == 0) return false;
} else indexes = list.getSelectedIndices();
      Ответить

    • ava tir 15.08.2011 17:04 # 0

      А можно и так:
      ...
      int[] indexes = getIndexesToProcess(list);
      if (isEmpty(indexes)) return false;
      ...

      private int[] getIndexesToProcess(list) {
      int[] indexes = list.getSelectedIndices();
      return isEmpty(indexes) ? getAllIndexes() : indexes;
      }

      private boolean isEmpty(int[] array) {
      return array.length == 0;
      }
      Ответить

    • ava wvxvw 16.08.2011 11:25 # +1

      Вот, хороший пример почему не надо возвращать значения не в конце функции. По-нормальному, должно было быть:
      return list.getSelectedIndices().length > 0 || getAllIndexes().length > 0;
      Ответить

      • ava tir 16.08.2011 14:29 # +1

        ваш вариант и исходный код дают разные результаты
        Ответить

        • ava wvxvw 16.08.2011 15:35 # 0

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

    • ava Lure Of Chaos 16.08.2011 14:15 # 0

      топик можно считать закрытым, только вот последний вопрос:
      почему индуссы? они что, этоссут делают особенно, по-индусски?
      Ответить

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