Java — Говнокод.ру

Java / Говнокод #10284

+66

1
2
3
4
5
6

catch (com.google.gwt.event.shared.UmbrellaException ex) {
					Iterator<Throwable> it = ex.getCauses().iterator();
					while (it.hasNext()) {
						Throwable msg = it.next();
						System.out.println(msg.getStackTrace());
					}

обрабатываем exception......

sermolaev, 16 Мая 2012

Комментарии (6)

Java / Говнокод #10257

+71

1
2
3
4
5
6
7

int buttonState = R.id.power_widget_button_indic; // Зачем-то объявили лишнюю переменную, ссылающуюся на константу
ImageView indic = (ImageView) mView.findViewById(R.id.power_widget_button_indic); // Но здесь используем константу, а не новую, более короткую переменную!
if ((Settings.System.getInt(context.getContentResolver(), Settings.System.EXPANDED_HIDE_INDICATOR, 0)) == 1) {
	indic.setVisibility(8); // Для установки видимости существуют константы View.VISIBLE, View.INVISIBLE и View.GONE, а циферка 8 вообще ни о чем мне не говорит
} else {
	indic.setVisibility(0); // И еще раз
}

Это - кусок кода из Android 4, вернее, из CyanogenMod 9. Репозиторий frameworks_base, файл packages/SystemUI/src/com/android/systemui/statusbar/powerwidget/PowerButton.java. И такого говнокода в исходниках Android - полно :(

RankoR, 13 Мая 2012

Комментарии (76)

Java / Говнокод #10252

+69

01
02
03
04
05
06
07
08
09
10
11

//JAVA
request.setAttribute("sexForRegi", person.getSex().getLogicConstant());
//А теперь и JSTL: 
<c:choose>
   <c:when test="${sexForRegi == 'MALE'}">
      <div class="sys-icon-128x128 icon-128x128-RegEndMan"></div>
   </c:when>
   <c:when test="${sexForRegi == 'FEMALE'}">
      <div class="sys-icon-128x128 icon-128x128-RegEndWoman"></div>
   </c:when>                  
</c:choose>

Sex - это у меня сущность, которая определяет половую принадлежность.
Regi - Action, отвечающий за регистрацию.
Вот и назвал переменную, чтобы определять какую картинку выводить "sexForRegi".
Вдумавшись, такое название наталкивает на мнение, что регистрация возможна только через секс =)

Dvelopp, 12 Мая 2012

Комментарии (18)

Java / Говнокод #10246

+72

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

final String sqlString=
					" select distinct 1 "+
					" , c_rep.card_id as rep_id "+
					" , av_rep_xml.long_binary_value as rep_xml "+
					" , av_sname_nm.string_value as name "+
					" , av_c_date.date_value as date \n"+
					" from	card c_res \n"+
					" 	join 	attribute_value av_res_rep on (av_res_rep.card_id = c_res.card_id and av_res_rep.attribute_code = 'JBR_RIMP_REPORT') \n"+
					" 	join	attribute_value av_res_exec on (av_res_exec.card_id = c_res.card_id and av_res_exec.attribute_code = 'JBR_INFD_EXEC_LINK') \n"+
					" 	join 	card c_rep on (c_rep.card_id = av_res_rep.number_value ) \n"+
					" 	left outer join attribute_value av_rep_xml on (av_rep_xml.card_id = c_rep.card_id and av_rep_xml.attribute_code = 'ADMIN_702354') \n"+
					" 	left outer join attribute_value av_c_date on (av_c_date.card_id = c_rep.card_id and av_c_date.attribute_code = 'CREATED') \n"+
					" 	left outer join	attribute_value av_rep_exec on (av_rep_exec.card_id = c_rep.card_id and av_rep_exec.attribute_code ='ADMIN_702335') \n"+
					" 	left outer join	person person_card on (person_card.person_id=av_rep_exec.number_value) \n"+
					" 	left outer join attribute_value av_sname_nm on (av_sname_nm.card_id = person_card.card_id and av_sname_nm.attribute_code = 'JBR_PERS_SNAME_NM') \n"+
					" 	left outer join attribute_value av_on_control on ( av_on_control.attribute_code = 'JBR_TCON_ONCONT' and av_on_control.card_id = c_res.card_id) \n"+
					" where	c_res.card_id = " + sResId+ "\n"+
					" 	and av_res_exec.number_value = av_rep_exec.number_value \n"+
					" 	and av_on_control.value_id=1449 \n"+
					" 	and c_res.status_id in (103, 206) \n"+

					// UNION
					" UNION select 51 \n"+
					" , c_rep.card_id as rep_id \n"+
					" , av_rep_xml.long_binary_value as rep_xml \n"+
					" , av_sname_nm.string_value as name \n"+
					" , av_c_date.date_value as date \n"+
					" from card c_prnt \n"+
					" 	left outer join attribute_value av_res on ( av_res.attribute_code = 'JBR_IMPL_RESOLUT' and av_res.card_id = c_prnt.card_id) "+
					" 	left outer join card c_res on (c_res.card_id=av_res.number_value)"+
					" 	join 	attribute_value av_res_rep on (av_res_rep.card_id = c_res.card_id and av_res_rep.attribute_code = 'JBR_RIMP_REPORT') "+
					" 	join	attribute_value av_res_exec on (av_res_exec.card_id = c_res.card_id and av_res_exec.attribute_code = 'JBR_INFD_EXEC_LINK') "+
					" 	join 	card c_rep on (c_rep.card_id = av_res_rep.number_value ) "+
					" 	left outer join attribute_value av_rep_xml on (av_rep_xml.card_id = c_rep.card_id and av_rep_xml.attribute_code = 'ADMIN_702354') "+
					" 	left outer join attribute_value av_c_date on (av_c_date.card_id = c_rep.card_id and av_c_date.attribute_code = 'CREATED') "+
					" 	left outer join	attribute_value av_rep_exec on (av_rep_exec.card_id = c_rep.card_id and av_rep_exec.attribute_code = 'ADMIN_702335') "+
					" 	left outer join	person person_card on (person_card.person_id=av_rep_exec.number_value) "+
					" 	left outer join attribute_value av_sname_nm on (av_sname_nm.card_id = person_card.card_id and av_sname_nm.attribute_code = 'JBR_PERS_SNAME_NM') "+
					" 	left outer join attribute_value av_on_control on ( av_on_control.attribute_code = 'JBR_TCON_ONCONT' and av_on_control.card_id = c_res.card_id)  "+
					" where "+
					" 	c_prnt.card_id = "+ sResId+ "\n"+
					" 	and c_prnt.template_id <> 324 \n"+
					" 	and av_res_exec.number_value = av_rep_exec.number_value \n"+
					" 	and av_on_control.value_id=1449 \n"+
					" 	and c_res.status_id in (103, 206) \n"+
	
.... и еще 8 UNION, куча строчек в том же стиле и с тем же смыслом

Формирование SQL запроса в методе, который возвращает строковое представление хода исполнения поручения (кто сделал, что сделал, когда и в какой последовательности). Сам метод, если кому интересно дергается из Jasper report.

Мало того, что запросик имеет длину 250 строк кода (большая часть которых копи/паст с небольшими изменениями), так он ещё и возвращает полную ахинею вместо того что нужно. Чего хотел сказать автор - теперь уже не узнать...

amberLord, 11 Мая 2012

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

Java / Говнокод #10238

+79

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
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83

import java.util.*;

class Euler32 {
    public static void main(String[] args) {
	int total=0;

	LinkedList<Integer> remDigits=new LinkedList<Integer>();
	for(int n=1;n<=9;n++)
		remDigits.add(new Integer(n));
	
	for(int n9=2;n9<9;n9++){ //starting digit can't be 1 or 2
	  int thouC=(remDigits.get(n9)).intValue();
	  remDigits.remove(n9);
	  for(int n8=0;n8<8;n8++){
	    int hundC=(remDigits.get(n8)).intValue();
	    remDigits.remove(n8);
	    for(int n7=0;n7<7;n7++){
	      int tenC=(remDigits.get(n7)).intValue();
	      remDigits.remove(n7);
	      for(int n6=0;n6<6;n6++){
		int oneC=(remDigits.get(n6)).intValue();
		remDigits.remove(n6);

		int c=1000*thouC+100*hundC+10*tenC+oneC;
		boolean found=false;
		int n5=0;
		while((n5<5)&&(found==false)){
		    int ab1=(remDigits.get(n5)).intValue();
		    remDigits.remove(n5);
		    int n4=0;
		    while((n4<4)&&(found==false)){
		      int ab2=(remDigits.get(n4)).intValue();
		      remDigits.remove(n4);
		      int n3=0;
		      while((n3<3)&&(found==false)){
			int ab3=(remDigits.get(n3)).intValue();
		      	remDigits.remove(n3);
		      	int n2=0;
			while((n2<2)&&(found==false)){
			  int ab4=(remDigits.get(n2)).intValue();
			  remDigits.remove(n2);
			  int ab5=(remDigits.get(0)).intValue();

			  int a3=100*ab1+10*ab2+ab3;
			  int a4=1000*ab1+100*ab2+10*ab3+ab4;
			  int b2=10*ab4+ab5;
			  int b1=ab5;

			  if((a3*b2)==c){
			    found=true;
			    total+=c;
			    System.out.println(c+" = "+a3+"x"+b2); 
			    }
			  else if((a4*b1)==c){
			   found=true;
			   total+=c;
			   System.out.println(c+" = "+a4+"x"+b1); 
			   }

			  remDigits.add(n2,new Integer(ab4));
			  n2++;
			  }
			remDigits.add(n3,new Integer(ab3));
		      	n3++;
			}
		      remDigits.add(n4,new Integer(ab2));
		      n4++;
		      }
		    remDigits.add(n5,new Integer(ab1));
		    n5++;
		    }

		remDigits.add(n6,new Integer(oneC));
		}
	      remDigits.add(n7,new Integer(tenC));
	      }
	    remDigits.add(n8,new Integer(hundC));
	    }
	  remDigits.add(n9,new Integer(thouC));
	  }
	System.out.println(total);	
    }
}

http://projecteuler.net/problem=32
http://projecteuler.net/thread=32;page=2

>My code is absolutely hideous, but it works and it's fast.

Извиняюсь за длинный пост, но это просто шедевр, я не мог это не запостить!

TheHamstertamer, 10 Мая 2012

Комментарии (15)

Java / Говнокод #10232

+79

1

for (; e.hasMoreElements();)

смайлокод.
вот почему не while, объясните мне?

Lure Of Chaos, 10 Мая 2012

Комментарии (10)

Java / Говнокод #10211

+86

1
2
3

int createStageCycles = 0;
createStageCycles = Integer.parseInt(strCycle);
int nextVal = new Integer(createStageCycles) + 1;

int двойного отжима

zloizerg, 07 Мая 2012

Комментарии (7)

Java / Говнокод #10185

+86

1
2
3
4
5

if (bindingContext != null && files != null) {
 if (files != null && bindingContext != null) {
  // bla-bla
 }
}

I want to be sure!

Art, 04 Мая 2012

Комментарии (7)

Java / Говнокод #10175