- 1
- 2
- 3
- 4
private static final int LENGHT255 = 255;
private static final int LENGHT450 = 450;
private static final int LENGHT3 = 3;
private static final int LENGHT5 = 5;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+64.6
private static final int LENGHT255 = 255;
private static final int LENGHT450 = 450;
private static final int LENGHT3 = 3;
private static final int LENGHT5 = 5;
Константы для проставления длин в Entity.
+139
это твой щенок. теперь он твой навсегда.
и ты не хочешь, но считаешь его дни и года.
твоя любовь к нему огромна, с каждым днём она растёт.
она станет бесконечной,
когда он умрёт.
для виолет_салмон
+81.3
private final String zeroValueString="0";
private final String sevenValueString="7";
Если б можно, то ещё б и hundredElevenValueString = "111"; впиндюрили бы
+133.4
// parse from the uninstall, the actuall installation path
try {
int jarLoc = uninstallDir.indexOf("-jar");
String s = uninstallDir.substring(jarLoc + 5);
// turn it into a normalized file
int uniLoc = s.indexOf("uninstaller");
s = s.substring(1,uniLoc);
s = s.replace("\\", "/");
if(s.endsWith("/"))
s = s.substring(0,s.length()-1);
File f = new File(s);
if(!f.exists()) {
throw new Exception("installation directory does not exist: " + s);
}
return f;
}
catch(Exception e) {
throw new Exception("installation directory could not be parsed from: " + uninstallDir);
}
Typical code of one highly paid "tech expert" from USA
Exception handling makes me cry, this guy doesn't even now about any other exception classes
+89.9
static char DigitToChar(
int _Digit
)
{
switch (_Digit)
{
case 0: return '0';
case 1: return '1';
case 2: return '2';
case 3: return '3';
case 4: return '4';
case 5: return '5';
case 6: return '6';
case 7: return '7';
case 8: return '8';
case 9: return '9';
default: return ':';
}
}
в результате долгих размышлений всетаки заменили на return '0' + _Digit;
−50
String[] search_array;
int index = -1;
for (int y = 0; y < search_array.length; y++) {
if (search_array[y].equals("SEARCH_PATTERN") {
index = y;
y = 100;
}
}
−55.5
for (ComplectationItemTO complectation : previewFullTO.getItemComplectations()) {
for (Map.Entry entry : complectation.getInstances().entrySet()) {
if (entry.getKey().equals(SoundData.SOUND_COMPLECTATION_NAME)) {
SoundData soundData = getComplectationSoundData(complectation.getId(), previewFullTO);
soundsMap.put(complectation.getId(), soundData);
}
}
}
private SoundData getComplectationSoundData(Long complectatiomId, PreviewFullTO previewFullTO) {
for (ComplectationItemTO complectationSound : previewFullTO.getItemComplectations()) {
if (complectationSound.getId().equals(complectatiomId)) {
for (Map.Entry entrySound : complectationSound.getInstances().entrySet()) {
if (entrySound.getKey().equals(SoundData.SOUND_LENGTH)) {
String soundLength = complectationSound.getInstances().get(SoundData.SOUND_LENGTH);
String soundData = complectationSound.getInstances().get(SoundData.SOUND_COMPLECTATION_NAME);
String productNumber = previewFullTO.getItemTO().getFeatureInstances().get("Product number");
String productRewritedName = previewFullTO.getItemTO().getProductTypeRewrited();
return new SoundData(soundData, productRewritedName, Long.valueOf(productNumber), soundLength);
}
}
}
}
return null;
}
цикл в цикле и циклом погоняет
−45.4
if ((solution.indexOf("filter") != -1)
|| (solution.indexOf("Filter") != -1)
|| (solution.indexOf("FILTER") != -1)) {
solution = "Filter incoming traffic to port/service \""
+ serviceName + "\" if the service is not used";
} else if ((solution.indexOf("package") != -1)
|| (solution.indexOf("Package") != -1)
|| (solution.indexOf("PACKAGE") != -1)
|| (solution.indexOf("vendor") != -1)
|| (solution.indexOf("Vendor") != -1)
|| (solution.indexOf("VENDOR") != -1)
|| (solution.indexOf("service") != -1)
|| (solution.indexOf("Service") != -1)
|| (solution.indexOf("SERVICE") != -1)) {
solution = "AMBIQUITY_SOLUTION_START port/service \""
+ item.getServiceName() + "\'} " + solution;
}
//...............................................etc
Развиваю один старый проект. Не буду говорить о всех перлах того, что я там увидел, про великую архитектуру проекта, но кое-что хотелось бы показать.
З.Ы. Я бы хотя бы сделал так
solution.toLowerCase().indexOf("package" ) != -1
З.З.Ы Этот ужас таки порефакторил и его убрали :)
−19
public Calendar getTomorrow() {
Thread.sleep(1000*60*60*24);
return Calendar.getInstance();
}
как узнать дату завтрашнего дня :
+17.1
public boolean isSocialNWportal(){
int accountId = getCurrentAccountID();
if(accountId == XXX.ACCOUNT_ID_1 ||
accountId == XXX.ACCOUNT_ID_2 ||
accountId == XXX.ACCOUNT_ID_3
)
return true;
else
return false;
}
Ну и конечно же смотрим как был реализован метод isSocialNWportal() ранее. Тут все как всегда: хиндогавнологика одна...