Список говнокодов пользователя askell

Всего: 10

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

−41

1
2
3
4

String phone = mobilePhone;
phone = (phone == null || phone.isEmpty()) ? homePhone : phone;
phone = (phone == null || phone.isEmpty()) ? workPhone : phone;
return phone;

И какой телефон тут главный то? %)

askell, 14 Октября 2015

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

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

+71

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

private static final int IDX_OBJECT = 0;
private static final String PARTY_ACCOUNT_BALANCE_QUERIES = "Select party.id from Party party WHERE party.TaxID='615100175';Select account.id from PaymentAccount account, Party party WHERE party.TaxID='615100175';Select balance.id from PaymentBalance balance WHERE balance.BalanceType='10000';";
private static final int queryAmount = 3;
public void createPaymentAccountUpdate() {
	String[] QUERY = new String[queryAmount];
	int startIndex = 0, endIndex=0;
	IRecordset [] rs=null;

	for (int i = 0; i < queryAmount; i++) {
		endIndex = PARTY_ACCOUNT_BALANCE_QUERIES.indexOf(';', startIndex);
		QUERY[i] = PARTY_ACCOUNT_BALANCE_QUERIES.substring(startIndex, endIndex);
		startIndex = endIndex + 1;
	} 
	for (int i = 0; i < queryAmount; i++) {
		rs[i] = s_mgr.newQuery().execute(QUERY[i].toString());
	}
	if (rs[0].moveNext()) {
		IParty party = (IParty)rs[0].valueFromIndex(IDX_OBJECT);
		if (rs[1].moveNext()) {
			IPaymentAccount pa = (IPaymentAccount) rs[1].valueFromIndex(IDX_OBJECT);
			pa.setParty(party);
		}
		if (rs[2].moveNext()) {
			IPaymentBalance pb = (IPaymentBalance) rs[2].valueFromIndex(IDX_OBJECT);
			pa.setBalance(pb);
		}            
		pa.setComment("Test account update created"); 
		pa.Save();
	}
}

Лучше уж никаких тестов, чем такие

askell, 14 Февраля 2013

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

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

+80

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17

m_scrabblerProperties = Utils.loadPropertiesFile(m_args[1]);
if (null != m_scrabblerProperties)
{
	System.out.println("Executing DBScrubber with properties file " + m_args[1] + " loaded from the classpath");
}
else
{
	try
	{
		m_scrabblerProperties.load(new FileInputStream(m_args[1]));
		System.out.println("Executing DBScrubber with properties file " + m_args[1] + " loaded from the file system");
	}
	catch (IOException e)
	{
		System.err.println("Failed to load " + m_args[1] + " from the classpath or the file system");
	}
}

Utils.loadPropertiesFile - какая то хитрая поделуха которая ищет файл в кэше. Если он не найден в кэше, то автор видимо хотел попробовать считать файл с диска, но судя по всему оказался оказался дебилом.

askell, 18 Ноября 2011

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

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

+73

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23

Credential credential = (Credential) getCurrentObject();
if(isNew(credential)) {
	CredentialDetails details=credential.getCurrentDetails();
	if(details == null)
		throw new NullPointerException("Credential details object is null");

	CredentialDetails updatedDetails = credential.getCurrentDetails();
	updatedDetails.setStartDate(getCurrentDate());
	updatedDetails.setExternalID(credential.getExternalID());
}
if(isNew(credential)==false) {
		
	CredentialDetails details=credential.getCurrentDetails();
	if(details == null)
		throw new NullPointerException("Credential details object is null");
	CredentialDetails updatedDetails = credential.getCurrentDetails();
	updatedDetails.setStartDate(getCurrentDate());
	updatedDetails.setExternalID(credential.getExternalID());
	m_existingDetailsClone.setEndDate(updatedDetails.getStartDate());
	updatedDetails.setPreviousDetails(m_existingDetailsClone);
			
	getModCol().addModified( m_existingDetailsClone );
}

Такое впечатление что автор кода клинический идиот. Зачем так насиловать несчастные POJO Credential и CredentialDetailsю

askell, 11 Ноября 2011

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

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

+71

01
02
03
04
05
06
07
08
09
10
11
12
13

if (prodAttainment < payeeSalesGoal)
{
	cashComp = 1;
}
else if (prodAttainment < 1.25 * payeeSalesGoal
		&& prodAttainment >= payeeSalesGoal)
{
	cashComp = 0.75;
}
else if (prodAttainment >= 1.25 * payeeSalesGoal)
{
	cashComp = 0.5;
}

Меня гнет или и правда нужно столько проверок? Чтоб наверняка...

askell, 27 Сентября 2011

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

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

+78

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

/**
	* @param loginName
	* @return
	* This method is create a LoginName as Input data
	*/
public String getLoginName(String loginName)
{
	String userQuery="select u.LoginName from User u";
	Recordset rs_user=null;
	rs_user = CustomExternalServiceImplUtil.getInstance().executeQuery(userQuery);
	List<String> userList = new ArrayList<String>();
	while(rs_user.moveNext()){ 
		userList.add(rs_user.valueFromIndex(0).toString());
	}
	int i=1;
	String result = loginName;
	for(int j=0; j < userList.size(); j++){
		if(userList.get(j).equals(result))
		{
			result = loginName+i++;
			j=0;
		}
	}
	return result;
}

Рефаткоринг чужого кода. Минут пять втуплял, что же тут вообще делается. Еще столько же придумывал, как же это привести в божеский вид с сохранением прежней функциональности.

askell, 06 Сентября 2011

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

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

+89

1
2
3
4
5

while (m.find()) {
    String sentence = m.group(0);
    setterMethodName = "setElement" + sentence;
    break;
}

Чем автора не устроил оператор if остается загадкой.

askell, 08 Августа 2011

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

Говнокод: по колено в коде.