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

    +67

    1. 1
    2. 2
    3. 3
    4. 4
    String a = 1234567890
String b = 4
def c = a.split(b)
if c.size() > 1 //...

    Замена indexOf

    Запостил: robin, 30 Июня 2010

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

    • ava cfdev 30.06.2010 08:09 # 0

      Это что за язык?
      Ответить

    • ava Lure Of Chaos 30.06.2010 09:18 # 0

      кавычки не обязательны?
      Ответить

    • ava 3.14159265 30.06.2010 09:41 # 0

      может замена contains?
      кстати лично я всегда пишу вместо contains - indexOf(...)>-1
      Ответить

      • ava robin 30.06.2010 09:52 # 0

        Да, ошибся
        Ответить

        • ava HyperGeek 30.06.2010 13:12 # 0

          Кстати, а при чем здесь contains или indexOf? Судя по коду, проверяется, что исходная строка разбита на две или более частей. Тогда, выходит, самое вонючее ты не запостил, либо гк здесь вообще нет.
          Ответить

          • ava 3.14159265 30.06.2010 13:24 # +2

            КО
            притом что если строка разбита на несколько частей
            ОЧЕВИДНО что в ней есть "4", то есть строка содержит (то есть contains подстроку "4")
            Ответить

            • ava HyperGeek 30.06.2010 13:45 # 0

              Все, теперь вкурил )) Логика говнокодеров (точнее ее отсутствие) для меня порой непостижима...
              Ответить

      • ava Lure Of Chaos 30.06.2010 12:35 # 0

        а чем contains оплошал?
        Ответить

        • ava 3.14159265 30.06.2010 14:46 # 0

          а в его исходник глянь ))
          public boolean contains(CharSequence s) {
          return indexOf(s.toString()) > -1;
          }
          так получается чутка быстрее
          + contains @since 1.5
          а это иногда критично
          Ответить

          • ava turdman 30.06.2010 15:27 # 0

            Мучительно пытаюсь понять, как пройти-по-списку-разрезая-его-на-части может быть быстрее чем проити-по-списку.
            Ответить

            • ava 3.14159265 30.06.2010 15:35 # 0

              никак
              split()>1 - однозначно говно
              contains() - не нравится лично мне
              indexOf()>-1 - мой выбор он ничего не разрезает
              см. http://govnokod.ru/3597#comment36506
              Ответить

    • ava cfdev 30.06.2010 14:34 # −6

      показать все, что скрытоКстати, вполне в духе ФЯ.
      Ответить

      • ava tinhol 01.07.2010 08:44 # +4

        Что именно, если не секрет?
        Ответить

        • ava cfdev 01.07.2010 11:08 # −3

          Ну, всё. Там ведь особо не принято задумываться о реализации. Лишь бы работало. Это тоже алгоритм :)

          f a b = size(split(a b)) > 1
          Ответить

          • ava tinhol 01.07.2010 13:38 # +1

            ИМХО, дело не в ФЯ, а в части из тех, кто пишет на них. Я бы сказал что это в духе говнокодеров.
            Ответить

      • ava robin 01.07.2010 15:08 # 0

        Что есть ФЯ?
        Ответить

    • ava guest8 09.04.2019 18:19 # −999

      показать все, что скрытоvanished
      Ответить

    • ava guest8 09.04.2019 18:19 # −999

      показать все, что скрытоvanished
      Ответить

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