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

    +74

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    private String stateChangedReason;
...

if (stateChangedReason != null && stateChangedReason instanceof String) {
            dealerManageInfo.setChangeStateReason(stateChangedReason.toString());
}

    Запостил: amarfey, 15 Февраля 2013

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

    • ava guest 15.02.2013 22:45 # 0

      А что если String это базовый класс? есть 
      String1 extends String
String2 extends String

      и выше по коду есть проверка
      if(stateChangedReason instanceof String1)
{
...
return;
}
      Ответить

      • ava wvxvw 15.02.2013 23:03 # 0

        Даже если бы кто-то написал свой String - все равно нужно было бы указывать пакет... Хотя, хз, может если в корень положить, то пользовательский класс заслонит собой встроенный, но проверять лень.
        Ответить

      • ava orion 16.02.2013 00:28 # +5

        стринг же в жабе вроде final, не?
        Ответить

        • ava absolut 16.02.2013 10:21 # +2

          угу. недавно уже обсуждалось.
          http://govnokod.ru/12583#comment169578
          Ответить

    • ava guest 16.02.2013 00:32 # −2

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

      • ava tir 16.02.2013 20:34 # 0

        поддерживаю комментарий. определенно тип раньше был другой, а это просто жертва рефакторинга. на то что тип был другой, косвенно указывает stateChangedReason.toString().

        Единственное, что до рефакторинга проверка на null не нужна была, т. к. instanceof для null всегда вернет false
        Ответить

    • ava someone 16.02.2013 09:32 # +4

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

    • ava Lure Of Chaos 17.02.2013 17:13 # +1

      #0: import com.mycompany.String;
      Ответить

      • ava wvxvw 18.02.2013 08:53 # 0

        Я уже об этом выше говорил: в таком случае нужно будет квалифицировать неродной стринг (я так думаю, но по-прежнему лень пробовать), потому что родное импортируется в не зависимости от желаний моей компании.
        Ответить

        • ava bormand 18.02.2013 09:31 # 0

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

          • ava wvxvw 18.02.2013 21:10 # 0

            Хех, ну вобщем-то логика в этом есть. Если явно импортировали - то и квалифицировать надо то, что неявно импортировано... Вопрос - можно ли будет квалифицировать неродной, если оба явно импортировать.
            Ответить

            • ava bormand 18.02.2013 23:19 # 0

              Импортировать два класса с одним именем нельзя. Да и зачем? Если бы можно было, то пришлось бы квалифицировать оба String'а, а это можно делать и вообще без импортов.
              Ответить

      • ava amarfey 18.02.2013 12:34 # +1

        стринг стандартный, джавовый
        Ответить

    • ava agfvTzv 25.08.2021 00:13 # 0

      Ну вот и всё, это конец. Скоро угарный газ, этот кровяной яд, сделает своё дело, и из-за пожара одним парнем на свете станет меньше. Но мне нечего терять, я и так уже потерял самое дорогое, что у меня было, так что я ни о чём не жалею. Маму с тётей только жаль, плакать они будут очень сильно. А друзья... Был ли у меня хоть один настоящий друг за всю мою короткую жизнь? Наверное, всё же нет. Хотя теперь это не важно. Теперь уже ничего не важно.
      Ответить

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