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

    +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
    34. 34
    protected string buildInsertString(ForecastTransactionsBean ft)
 {
    String sql = (new StringBuilder("insert into ")).append(getTableName()).append(" (").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append("year,version_num,corp,acct_num,dept_num,cc,").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append("epd_num,est_mar,est_apr,est_may,").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append("est_jun,est_jul,est_aug,est_sep,est_oct,est_nov,").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append("est_dec,est_jan,est_feb,inputby,batch_id,inputdate,").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append("manualetc,forecastnotes,acct_class,resource_group").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(") values ('").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getYear().toString()).append("',").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getVersionNum().toString()).append(",'").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getCorp().toString()).append("','").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getAcctNum().toString()).append("','").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getDeptNum().toString()).append("','").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getCc().toString()).append("','").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getEpdNum().toString()).append("',").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getEstMar().toString()).append(",").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getEstApr().toString()).append(",").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getEstMay().toString()).append(",").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getEstJun().toString()).append(",").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getEstJul().toString()).append(",").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getEstAug().toString()).append(",").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getEstSep().toString()).append(",").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getEstOct().toString()).append(",").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getEstNov().toString()).append(",").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getEstDec().toString()).append(",").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getEstJan().toString()).append(",").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getEstFeb().toString()).append(",'").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getInputBy().toString()).append("','").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getBatchId().toString()).append("','").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append(ft.getInputDate().toString()).append("',").toString();
    sql = (new StringBuilder(string.valueOf(sql))).append("0,'Clarity','','')").toString();
    return sql;
 }

    Запостил: denis90, 05 Июля 2012

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

    • ava bormand 05.07.2012 12:23 # +6

      Все верно. Сказано было программист использовать StringBuilder - он его и использует. Придраться не к чему ;)
      Ответить

    • ava roman-kashitsyn 05.07.2012 12:31 # +8

      о_О
      Никогда ещё не видел такой феерии. Это точно человек написал?
      Ответить

    • ava rat4 05.07.2012 13:00 # +2

      Усердный копи-пастинг.
      Ответить

    • ava Lure Of Chaos 05.07.2012 14:07 # +1

      а толку
      Ответить

    • ava 3.14159265 05.07.2012 16:27 # +9 

      String r="";
 r = (new StringBuilder(string.valueOf(r))).append("П").toString();
 r = (new StringBuilder(string.valueOf(r))).append("И").toString();
 r = (new StringBuilder(string.valueOf(r))).append("З").toString();
 r = (new StringBuilder(string.valueOf(r))).append("Д").toString();
 r = (new StringBuilder(string.valueOf(r))).append("Е").toString();
 r = (new StringBuilder(string.valueOf(r))).append("Ц").toString();
      Ответить

      • ava bormand 05.07.2012 16:37 # 0

        Кстати проблески понимания у автора есть. Он в некоторых строках вызвал append() аж целых 2 раза на одном билдере.
        Ответить

        • ava interested 05.07.2012 17:32 # +5

          Кошмар! Кошмар! Как он мог...
          Не видать ему звания "мисс говнокод".
          Ответить

    • ava someone 06.07.2012 09:54 # +2

      Кстати, а почему string с маленькой буквы? Это Java или C#?
      Ответить

      • ava carsten 06.07.2012 19:29 # 0

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

      • ava roman-kashitsyn 06.07.2012 22:19 # +2

        Выглядит как опечатка. Это не может компилироваться: возвращается sql, имеющий тип String, а в сигнатуре string.
        Ответить

      • ava Lure Of Chaos 07.07.2012 12:07 # 0

        это киллер фича, сразу в сигнатуре указываем что будем возвращать, хехе
        Ответить

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