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

    +81

    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
    public boolean hasNext() {
        if (vectorIndex  == vector.size()) 
            return false;
        return true;
    }
    
    public Object next() {
        try {
            return new Double(vector.getElement(vectorIndex));
        }
        catch(OutOfBoundsException e) {
            throw new RuntimeException(e);
        }
        finally {
            vectorIndex++ ;
        }
    }

    Такой вот итератор у начинающего жавера...

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

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

    • ava guest 15.10.2013 06:53 # +1

      Прэлестно!
      Ответить

    • ava someone 15.10.2013 09:13 # +1

      Это прямо какой-то фрактальный говнокод, тут прекрасно всё:

      * if (X) return false else return true вместо if (!X)
      * new Double(), хотя можно было просто вернуть элемент
      * RuntimeException вместо требуемого контрактом интерфейса NoSuchElementException
      * инкремент в finally
      * Object next(), хотя Iterator - это generic-интерфейс

      И, наконец, на закуску - непонятно, зачем изобретать велосипед, если Vector уже реализует Iterable и этот итератор просто (коряво) дублирует логику стандартного.
      Ответить

      • ava eth0 15.10.2013 17:14 # 0

        > вместо if (!X)
        return, не?
        Ответить

        • ava someone 15.10.2013 17:24 # 0

          Да, конечно.

          Конкретно здесь - return vectorIndex != vector.size().

          Только надо вынести инкремент из finally, а то после первой попытки выйти за границы hasNext будет возвращать true.
          Ответить

      • ava pingw33n 15.10.2013 17:22 # 0

        >* new Double(), хотя можно было просто вернуть элемент
        Может getElement() возвращает строку, а это хитрожопый VectorDoubleIterator :)

        >Vector уже реализует Iterable
        Это, походу, велосипедный Vector, у стандартного ведь нет getElement().
        Ответить

        • ava someone 15.10.2013 17:28 # 0

          Опа! А я и не знала, что у Double есть конструктор от строки. Всю жизнь писала Double.valueOf(), если нужен был именно объект Double.

          Пути Сана неисповедимы.

          > Это, походу, велосипедный Vector, у стандартного ведь нет getElement().

          Однако! Точно, у стандартного есть просто get и есть elementAt.

          Получается, этот самопальный вектор даже List не реализует? М-да...

          M-M-M-MONSTER SHITTYCODE!
          Ответить

          • ava anonimb84a2f6fd141 16.10.2013 02:34 # +3

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

            • ava 3.14159265 16.10.2013 21:45 # +3

              http://govnokod.ru/user/4722
              Его говнокоды. Движок говнокода в курсе.
              Ответить

              • ava bormand 16.10.2013 22:13 # +2

                > Движок говнокода в курсе.
                Движок говнокода не провести сиськами на аватарке.
                Ответить

                • ava LispGovno 16.10.2013 23:15 # +1

                  Ну вы последнюю ловушку прогоните. Кончайте издеваться. Что он вам такого сделал?

                  На говнокоде обсуждают только код и сиськи.
                  Ответить

                  • ava kegdan 16.10.2013 23:24 # 0

                    Ну вы последнюю ловушку прогоните.

                    Ты хотел сказать давушку? LOL

                    Или ты имел ввиду "трапа"?
                    Ответить

                    • ava LispGovno 16.10.2013 23:31 # 0

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

                      • ava kegdan 17.10.2013 05:58 # 0

                        Ты так говоришь, будто мне не все равно девушка это или трап
                        Ответить

                        • ava LispGovno 17.10.2013 07:53 # 0

                          вот смотри, если на тебя наехать, то ты сразу начинаешь питушиться. А семён просто молчит. похоже же на девушку? знает же, что её место у плиты и молчит. кузя себя также вёл. как-то к нам заглядывала настоящая ловушка. так вот она кричала как резанная, пока не ушла, поджав стоячий хвост между ног.
                          Ответить

                          • ava LispGovno 17.10.2013 07:56 # 0

                            Ну и вообще, суди людей по делам, а не по словам. Мало ли там малолетний тролль добавляет окончания к словам.
                            Ответить

                          • ava anonimb84a2f6fd141 17.10.2013 21:21 # 0

                            Нет, это похоже на то, что оно не хочет развивать эту тему.
                            Ответить

                      • ava crastinus 17.10.2013 09:57 # +4

                        >приведу ещё больше ссылок и пруфов.
                        Уговорил. Приводи.
                        Ответить

          • ava Lure Of Chaos 17.10.2013 01:02 # 0

            > Опа! А я и не знала, что у Double есть конструктор от строки. Всю жизнь писала Double., если нужен был именно объект Double.

            у всех Number есть, и даже BigDecimal. а писать .valueOf() правильно, ибо, в отличие от new, дает прирост в производительности.
            или вы не на джаве прогите?
            Ответить

            • ava kegdan 17.10.2013 03:27 # 0

              В шарпе у конструктора double нет параметров. Еще один предохранитель на вечном пистолете)
              Ответить

            • ava bormand 17.10.2013 05:40 # 0

              > в отличие от new, дает прирост в производительности
              Даблы тоже кешируются? Я думал только целые.
              Ответить

    • ava kegdan 15.10.2013 10:46 # 0

      А зачем hasnext

      подозреваю, что должно быть так
      public boolean hasNext() {
        return vectorIndex != vector.size()
        }
    
public Object next() {
	if ( hasNext())
		return vector.getElement(vectorIndex++);
	return null;
    }
      Ответить

      • ava roman-kashitsyn 15.10.2013 10:47 # +1

        > return null;
        нет, это выход за границы, нужно кидать NoSuchElementException
        Ответить

      • ava someone 15.10.2013 11:18 # +2

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

        • ava kegdan 15.10.2013 11:54 # 0

          Все. понял. Тогда правда только через исключение
          Ответить

    • ava guest 15.10.2013 14:23 # 0

      А где remove()?
      Ответить

      • ava DBdev 15.10.2013 15:48 # 0

        Итератор же...
        Ответить

        • ava bormand 15.10.2013 15:52 # 0

          http://docs.oracle.com/javase/7/docs/api/java/util/Iterator.html#remove()
          Ответить

          • ava DBdev 15.10.2013 16:46 # 0

            А... с дотнетиком попутал.
            Там, вроде, MoveNext, Current и Reset нужны.
            Ответить

      • ava bormand 15.10.2013 15:50 # +1

        Он же опциональный, и на read-only итераторах только выбрасывает исключение.
        Ответить

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