Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+70.8
if (tp != null) {
DocElement d = (DocElement)tp.getLastPathComponent();
if (d.getElementType() == Constants.TYPE_FOLDER) {
context.setCursor(DragSource.DefaultMoveDrop);
if (d.getElementType()==Constants.TYPE_FOLDER) {
tree.setSelectionPath(tp);
tree.expandPath(tp);
}
}else {
context.setCursor(DragSource.DefaultMoveNoDrop);
}
}
моё, так сказать носом ткнули)
условие if (d.getElementType()==Constants.TYPE_FOLD ER)
maxt,
29 Марта 2010
+76.8
public void isChangenabledExc(){
if(isNew){
return;
}
if(true)return;//TODO
if(!isChangenabled()){
throw new ChangeEIsNotLockedException();
}
}
В этой функции находится единственный коментарий из всего файла.
Видимо в планах эта функция должна была делать нечто хорошее, но
планы так и не были осуществелны.
В результате получилась функция, которая ничего не делает, а только проверяет какие то условия.
поиском обнаружил 5 вызовов этой функции.
maxt,
26 Марта 2010
+87.2
else {
if(codeName == SummaryConstants.TYPE_COST) {
return new Cost(rtOBJ);
}
else {
if(codeName == SummaryConstants.TYPE_EQUIPMENT) {
return new Equipment(rtOBJ);
}
}
}
}
}
}
}
}
}
}
}
}
}
}
не стал утомлять еще и километром вложенных if ов..
а говнокод в том, что здесь можно было использовать switch
maxt,
25 Марта 2010
+82.8
for (int i = 0; i < list.size(); i++) {
SpecialItem item = list.get(i);
if (item != null) {
item = null;
}
}
list.clear();Неиндийский способ очистить java.util.List от вместимого.
xrobak,
23 Марта 2010
+71.2
public class ObjectList<T> extends ErrorStatus
Morbo,
18 Марта 2010
+75
private static byte[] getByteArrayFromInt(int number) {
ByteArrayOutputStream out = new ByteArrayOutputStream();
DataOutputStream o = new DataOutputStream(out);
try {
o.writeInt(number);
} catch (IOException e) {
e.printStackTrace();
}
return out.toByteArray();
}Интересный способ переводить Int в массив байт )
Logarithm,
18 Марта 2010
+81.4
if(condition) {
function(false);
} else {
function(true);
}
danilissimus,
17 Марта 2010
+86.4
//#ifdef Slowpoke
private static final int MILLIS = 1100;
//#else
private static final int MILLIS = 1000;
//#endifСколько миллисекунд в секунде?
thePooh,
17 Марта 2010
+144.6
public class Main {
public static void main(String[] args) {
f0();//this is call of f0() from line 4
f0();//this is call of f0() from line 4
}
public static void f0(){
System.out.println("hi, I am f0");
Exception e = new Exception();
if(e.getStackTrace()[1].getLineNumber()==3){
f1();
} else if(e.getStackTrace()[1].getLineNumber()==4){
f2();
}
}
public static void f1(){
System.out.println("hi, I am f1");
}
public static void f2(){
System.out.println("hi, I am f2");
}
}вот кусочек, который выполняется по разному в зависимости от того с какой строчки в исходном коде он вызван. Сам сочинил.
pcherkas,
17 Марта 2010
+81.6
public void calculate(){
jTextField1.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner1.getValue()))
*Integer.valueOf(String.valueOf(jSpinner13.getValue()))));
jTextField2.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner2.getValue()))
*Integer.valueOf(String.valueOf(jSpinner14.getValue()))));
jTextField3.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner3.getValue()))
*Integer.valueOf(String.valueOf(jSpinner15.getValue()))));
jTextField4.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner4.getValue()))
*Integer.valueOf(String.valueOf(jSpinner16.getValue()))));
jTextField5.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner5.getValue()))
*Integer.valueOf(String.valueOf(jSpinner17.getValue()))));
jTextField6.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner6.getValue()))
*Integer.valueOf(String.valueOf(jSpinner18.getValue()))));
jTextField7.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner7.getValue()))
*Integer.valueOf(String.valueOf(jSpinner19.getValue()))));
jTextField8.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner8.getValue()))
*Integer.valueOf(String.valueOf(jSpinner20.getValue()))));
jTextField9.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner9.getValue()))
*Integer.valueOf(String.valueOf(jSpinner21.getValue()))));
jTextField10.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner10.getValue()))
*Integer.valueOf(String.valueOf(jSpinner22.getValue()))));
jTextField11.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner11.getValue()))
*Integer.valueOf(String.valueOf(jSpinner23.getValue()))));
jTextField12.setText(String.valueOf(Integer.valueOf(String.valueOf(jSpinner12.getValue()))
*Integer.valueOf(String.valueOf(jSpinner24.getValue()))));
}x_X
ChakuZa,
14 Марта 2010
Follow us!