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

    +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
    private static final String QUERY = 
        new StringBuffer("SELECT p.id FROM policysummary p ") 
            .append(" WHERE p.currentRevisionInd     = 1 ") 
            .append("AND p.policyStatusCd          IN ('issued') ") 
            .append("AND p.timedPolicyStatusCd     IN ('inForce') ") 
            .append("AND p.txType                  IN ('policy','renewal','endorsement','reinstatement') ") 
            .append("AND p.expiration <= :requestDate ") 
            .append("AND :requestDate <= (p.expiration + :requestTo) ") 
            .append("AND NOT EXISTS ") 
            .append("  (SELECT prs.id ") 
            .append("  FROM PolicyReportStatus prs ") 
            .append("  WHERE prs.policyId  = p.id ") 
            .append("  AND prs.mvrOrderDt IS NOT NULL ") 
            .append("  )") 
            .toString();

    Недавно выдавили заграничные коллеги

    Запостил: roman-kashitsyn, 24 Февраля 2012

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

    • ava absolut 24.02.2012 13:58 # +3

      ещё и с выравниванием по-тарасовски
      Ответить

      • ava defecate-plusplus 24.02.2012 14:23 # +2

        однозначно нужен термин
        тарастайл?
        тарасыравнивание?
        тарасотступ?
        тарасбивка?
        Ответить

      • ava TarasB 24.02.2012 14:25 # +1

        чё
        Ответить

        • ava absolut 24.02.2012 14:59 # 0

          строки 4-6 и твоя идея с автовыравниванием
          Ответить

          • ava TarasB 24.02.2012 15:17 # 0

            Понял.
            По-моему, хороший пример.
            Ответить

          • ava 3.14159265 24.02.2012 15:18 # +2

            Ну и что. Я тоже так частенько ровняю
            руками
            Ответить

          • ava roman-kashitsyn 24.02.2012 15:48 # 0

            только тут ручками всё, ручками
            Ответить

          • ava UncleAli 25.02.2012 00:03 # +2

            Я сам так выравниваю, весьма эстетично получается. Для этого даже elastic tabstops придумали.
            Но есть минус: если надо дописать более длинное выражение в том же блоке, то надо остальные тоже пробелами подогнать. Неудобно.
            Ответить

    • ava roman-kashitsyn 24.02.2012 15:57 # 0

      Интересно, почему в нашем проекте никто (кроме меня) не использует @NamedQuery, всё через конкатенацию строк...
      Ответить

      • ava FAKYOUINTIRNEAT 24.02.2012 19:17 # +3

        судя по всему, и на говнокод никто из вашего проекта не заходит (кроме тебя)
        Ответить

      • ava tir 27.02.2012 14:50 # 0

        Так ведь в @NamedQuery вроде как один фиг текст запроса придется забивать?
        Да и не всегда они удобны. Даже хотел бы услышать в чем их преимущества =)
        Ответить

        • ava roman-kashitsyn 27.02.2012 16:05 # +1

          1. Они разбираются один раз на старте приложения, потому выполняются быстрее (не надо каждый раз разбирать текст запроса и строить АСД).
          2. Соответственно, ошибки в таких запросах обнаруживаются сразу, а не случайным образом в рантайме.
          Ответить

          • ava tir 27.02.2012 16:49 # 0

            +1 за ответ.

            К сожалению, мое приложение не позволяет из активно юзать. =)
            Ответить

    • ava someone 25.02.2012 01:52 # 0

      Самое забавное, что через + будет быстрее, потому что + использует StringBuilder, который быстрее StringBuffer.

      А вообще советую этим товарищам открыть для себя QueryDSL.
      Ответить

      • ava roman-kashitsyn 25.02.2012 07:59 # +7

        мы тут недавно этот вопрос выясняли: конкатенацию константных строк выполняет компилятор.
        Ответить

        • ava gorky 27.02.2012 08:37 # 0

          а из-за двойных кавычек они не перестают быть константными? или только при появлении в них переменных?

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

          • ava SmackMyBitchUp 27.02.2012 09:25 # 0

            "Ровняю код наложением рук"
            Ответить

          • ava roman-kashitsyn 27.02.2012 09:39 # +2

            В одинарных кавычках в жабе может находиться только один символ ('a', 'b', но не 'ab'). Строки окружаются двойными кавычками. Автоподстановки строковых переменных ("Hello, ${user}!") в жабе нет, для этого нужно использовать String.format(template, args). Под константными строками я имел в виду строки, извесные на этапе компиляции (включая именованные константы). По идее все строки в жабе иммутабельны. Пример, когда конкатенацию не сможет выполнить компилятор: 
            public String f(String p) {
    return "The param is " + p;
}
            Тут p заранее неизвестно, и код будет преобразован компилятором в конструкцию 
            public String f(String p) {
    return new StringBuilder("The param is ").append(p).toString();
}
            Ответить

        • ava guest 20.03.2015 00:37 # 0

          Строковых литералов.
          Ответить

          • ava roman-kashitsyn 20.03.2015 09:29 # +2

            Ну вот если не знаешь, зачем лезешь?
            $ cat Test.java
public class Test {
    public static final String A = "Hello";
    public static final String B = "World";
    public static final String C = A + ", " + B + "!";
};

$ javap -constants Test.class 
Compiled from "Test.java"
public class Test {
  public static final java.lang.String A = "Hello";
  public static final java.lang.String B = "World";
  public static final java.lang.String C = "Hello, World!";
  public Test();
}
            Ответить

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