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

    +82

    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
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    @ValidationExpression(value = "if (inn != null && !\"\".equals(inn)) {\n" +
                              "char[] st = inn.toCharArray();\n" +
                              "for(int i=0; i < st.length; i++) {\n" +
                              "if (!Character.isDigit(st[i])) { return false; }\n" +
                              "}\n" +
                              "}" +
                              "return true;", message = Messages.WRONG_INN_FORMAT)
@Expression(value = "if (inn != null && !\"\".equals(inn)) {\n" +
                    "char[] st = inn.toCharArray();\n" +
                    "for(int i=0; i < st.length; i++) {\n" +
                    "if (!Character.isDigit(st[i])) { return false; }\n" +
                    "}\n" +
                    "}" +
                    "return true;", messageKey = Messages.WRONG_INN_FORMAT)
public String getinn() {
	return getPerson().getinn();
}

@Required(messageKey = Messages.REQUIRED)
@DictEntityExists(dictId = MDictionaries.DICT_CURRENCY, message = Messages.NO_DICT_ENTITY)
@Expression(value = "SystemCounter.DICTCURRENCY__RUR_CURRENCY_TYPE.equals(agentStatementCurrencyId) ? " +
                    "   (" +
                    "       SystemCounter.DICTCURRENCY__RUR_CURRENCY_TYPE.equals(currency_id) ? " +
                    "           true : " +
                    "           SystemCounter.DICTCONTRACTCLASS__CONTRACT_CLASS_XXX.equals(contract_class_id) ? " +
                    "               false : " +
                    "               is_currency_equivalent == 1" +
                    "   ) : (" +
                    "       SystemCounter.DICTCONTRACTCLASS__CONTRACT_CLASS_GENERIC.equals(contract_class_id) &&" +
                    "       is_currency_equivalent == 0 &&" +
                    "       agentStatementCurrencyId.equals(currency_id)" +
                    "   )", messageKey = Messages.CURRENCY_NOT_EQUIVALENT_WITH_AGENT_STATEMENT)
public String getcurrency_id() {return getContract().getcurrency_id();}

    нет слов

    Запостил: borka, 21 Июля 2010

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

    • ava Lure Of Chaos 21.07.2010 16:16 # +1

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

      • ava Bjarne_Stroustrup 22.07.2010 09:36 # +2

        Да ладно, любую фичу можно довести до абсурда.
        Ответить

      • ava borka 22.07.2010 23:13 # 0

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

        • ava Lure Of Chaos 23.07.2010 12:45 # 0

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

          • ava borka 23.07.2010 22:24 # 0

            Это было ясно всем кроме аффтара. Только лучше было бы указывать на классы валидаторов, а не на методы валидации.
            Ответить

          • ava Amok 05.08.2010 12:52 # +1

            Это пиздец, товарищи
            а) Писать надо для стороннего движка валидации (OVal, Hibernate Validator, Spring Validation etc). Судя по @DictEntityExists и другим аннотациям, тут делается для самопальнольного движка
            б) Expressions лучше вообще не использовать. Во всех движках есть аннотации типа оваловского @ValidateWithMethod
            в) имена геттеров наверное вводилось вручную ?!
            г) имена констант - жопа (чего только стоит DICTCONTRACTCLASS__CONTRACT_CLASS_XXX)
            Ответить

    • ava 3.14159265 21.07.2010 16:38 # 0

      вау, да это ж по сути evalы
      Ответить

      • ava borka 22.07.2010 23:02 # 0

        эти аннотации потом обрабатываются скриптовым движком, понимающим синтаксис java
        Ответить

      • ava Altravert 23.07.2010 05:02 # 0

        ebal'ы это сплошные.
        Ответить

    • ava Smoke 21.07.2010 18:08 # 0

      эта пять!
      Ответить

    • ava Lure Of Chaos 21.07.2010 21:39 # +1

      да, и почему то не public String getInn(), public String getCurrencyId(). аффтар точна третий как с ветки пхп слез
      Ответить

      • ava borka 22.07.2010 23:06 # 0

        ну подобный стиль еще в C использовался
        Ответить

        • ava Анонимус 22.07.2010 23:13 # 0

          Капетан очевидность сообщает:

          в си использовался вот_такой_стиль.

          А в джаве -- кемел кейс: вотТакойСтиль.

          а в C# -- паскаль стайл: ВотТакойСтиль.
          Ответить

          • ava Altravert 23.07.2010 05:03 # 0

            о_0 у нас контора пишет на c/c++ java-like style?
            Ответить

            • ava borka 23.07.2010 06:20 # 0

              в данном случае названия полей и методов соответствуют названиям колонок в базе
              Ответить

              • ava Altravert 23.07.2010 08:38 # 0

                Это понятно, я про
                >> А в джаве -- кемел кейс: вотТакойСтиль.
                Ответить

              • ava Lure Of Chaos 23.07.2010 10:10 # 0

                а у нас в конторе было принятно писать:
                @Entity 
public class ExEntity
{

@Column(name="COLUMN_NAME")
private String columnName;

public String getColumnName()
{
 return this.columnName;
}

public void setColumnName(String columnName)
{
 this.columnName=columnName;
}

}


                то есть не надо привязывать стиль именования в базе к стилю написания классов (в данном случае пример Hibernate entity)
                Ответить

                • ava borka 23.07.2010 22:26 # 0

                  Конечно, обычно так и делают. Но проект бородатый - он писался еще до Java 1.5.
                  Ответить

            • ava Анонимус 23.07.2010 14:04 # 0

              ))я говорил об обычном C.

              хотя например api у винды ВТакомСтиле (CreateProcess), а у никсов обычно в_таком_стиле (fork)
              Ответить

    • ava asolntsev 22.07.2010 09:14 # 0

      Да уж, жестянуха!
      Только я всё-таки не понимаю, как это работает. Видимо, я ещё не всё знаю о Яве.. :(
      Ответить

      • ava borka 22.07.2010 23:03 # 0

        эти аннотации потом обрабатываются скриптовым движком, понимающим синтаксис java
        Ответить

    • ava Altravert 23.07.2010 11:07 # +1

      Сижу за перевернутым столом с ружьем, жду прихода с++0x
      Это ж сколько кала из новых фич полезет... Сначала за собой начну это замечать, потом за другими...
      Это страшно, господа.
      PS: это насчет первого каммента
      Ответить

      • ava 3.14159265 23.07.2010 11:24 # 0

        блин, а ты прав... зато тут на сайте скучно не будет
        PS и эта ждем наплыва где здесь C++0xников? )))
        Ответить

      • ava Lure Of Chaos 23.07.2010 12:46 # 0

        зато сайт говнокод.ру будет ждать целая туча не-бойянов! )))
        Ответить

        • ava Altravert 23.07.2010 16:34 # +1

          Сдается мне, недолго будет это интересно. Ну что там? Пустая лямбда... Пицот лямбд в каждой строчке со срачем "как это скомпилит компилятор"... Автотип, который ВНЕЗАПНО окажется не тем типом.. Это Уже Сейчас бояны.
          Питонобояны еще частично переместятся в сибояны (смешное слово получилось, с "долбоебом" немного созвучно).
          Ну и раз в месяц обсуждение "а в гцц оно будет не так как у мелкомягких" с последующими проклятиями в адрес тех, кто что-нибудь не допилил.
          Еще будут холивары трусишеров (сокр. трусишек), которые сейчас вдруг стали появляться и тех, кто за Малах++0_X (это смайлик).
          Боюсь, что не будет это весело. Это будет куча навоза, произведенная нами и окружающими.
          Грустно это всё, грустно. И несет меня чето. Наверно потому, что грустно. Весна началась, хуле.
          Ответить

    • ava tinynick 29.07.2010 18:16 # 0

      Ад
      Ответить

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