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

    +76

    1. 1
    2. 2
    3. 3
    if (true == args[argInx++].equals("request")) {
  // ...
}

    Бульшит Йода-стайл

    Запостил: roman-kashitsyn, 06 Ноября 2012

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

    • ava guest 06.11.2012 20:35 # +3

      оп, оп, оп, оп оп
      бульщит йода-стайл.
      Ответить

    • ava Lure Of Chaos 06.11.2012 22:31 # 0

      может, человек правда не знает
      http://docs.oracle.com/javase/1.4.2/docs/api/java/util/Arrays.html#binarySearch(java.lang.Object[], java.lang.Object)
      Ответить

      • ava 3.14159265 06.11.2012 22:33 # +2

        Оно бажное.
        Ответить

      • ava bormand 06.11.2012 23:18 # +1

        А кто сказал, что он ищет "request" в массиве? Может быть он просто проверяет написано ли в argInx аргументе слово "request":
        ./program --action request
./program --action wait
        Ответить

    • ava TarasB 06.11.2012 22:39 # −5

      показать все, что скрытоЭто из-за уёбищного сишного дегенеративного наследия, в котором одиночное равно означало присваивание. Поэтому слева иногда ставят рвалуе, чтобы точно не опечататься.
      Впрочем, меня и без этого варнинги несколько раз спасли.
      Ответить

      • ava Lure Of Chaos 06.11.2012 22:48 # −2

        дело даже не в том.
        сравни:
        final REQUEST="request";
        ...
        if (REQUEST.equals(args[argInx++]))
        Ответить

      • ava 3.14159265 07.11.2012 15:47 # +1

        >в котором одиночное равно означало присваивание
        От NPE тебя это не спасёт.
        >Это из-за уёбищного сишного дегенеративного наследия
        Это от непонимания тобой специфики.
        Вернее отчасти из-за уёбищного крестонаследия, когда для вызова функций сделали такой вот сахарок.
        eq(a,b) на порядок красивей выглядит и лучше работает.
        Ответить

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