Говнокод

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.07.2012 (Updated 25.03.2018)

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

bormand 14 лет ago # 0

Все верно. Сказано было программист использовать StringBuilder - он его и использует. Придраться не к чему 😉
Ответить
roman-kashitsyn 14 лет ago # 0

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

Усердный копи-пастинг.
Ответить
Lure Of Chaos 14 лет ago # 0

а толку
Ответить
3.14159265 14 лет ago # 0
```
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();
```
Ответить
- bormand 14 лет ago # 0 ⇈
  
  Кстати проблески понимания у автора есть. Он в некоторых строках вызвал append() аж целых 2 раза на одном билдере.
  Ответить
  - interested 14 лет ago # 0 ⇈
    
    Кошмар! Кошмар! Как он мог...
    Не видать ему звания "мисс говнокод".
    Ответить
someone 14 лет ago # 0

Кстати, а почему string с маленькой буквы? Это Java или C#?
Ответить
- carsten 14 лет ago # 0 ⇈
  
  Пхоже он вызывает статический метод через экземпляр класса.
  Ответить
  - carsten 14 лет ago # 0 ⇈
    
    А, ещё и возвращается string. Тогда хз.
    Ответить
- roman-kashitsyn 14 лет ago # 0 ⇈
  
  Выглядит как опечатка. Это не может компилироваться: возвращается sql, имеющий тип String, а в сигнатуре string.
  Ответить
- Lure Of Chaos 14 лет ago # 0 ⇈
  
  это киллер фича, сразу в сигнатуре указываем что будем возвращать, хехе
  Ответить