- 1
length=length%8==0?0:length+8-length%8;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+67.3
length=length%8==0?0:length+8-length%8;
пытаемся округлить length до 8 в большую сторону...
краткость - сестра ... таланта?
+67.3
$today = intval(date("Y-m-d H:i:s"));
$db->query("UPDATE dle_seocontrol SET date='$today' where id = '{$id_upd}'");
© SOFTER
+67
enum MethodCallResult {
OK,
INTERNAL_ERROR,
INVALID_ARGUMENTS,
...
}
MethodCallResult result = ...;
switch (result.ordinal()) {
case 0:
...
break;
case 1:
...
break;
...
}
Натолкнулся в офигенном Ынтырпрайз-проекте, который спихнули на поддержку. Утверждали, что Sonar выдает 0 ошибок.
+67
public class Client extends Thread {
private final static Logger LOG = Logger.getLogger(Client.class);
@Override
public void run() {
System.out.println("Client " + name + " comes to restaurant " + restaurant.getName());
this.cashDesk = chooseCashDesk();
System.out.println("Client " + getClientName() + " choosed the cashDesk#"+ cashDesk.getNumber());
//...остальное говно
}
}
Что такое логгер ? ХЗ. Требуют - значит должно быть!
+67
public List<LDAPUserState> getAllLDAPUsersFromAD() {
try {
ldc.getAllLDAPUsers();
} catch (NamingException e) {
LOG.error(e.getMessage(), e);
}
return null;
}
+67
public boolean isUsing(AttributeType type) {
return (this.attributeTypes.indexOf(type) >= 0);
}
низкоуровневое программирование
+67
catch (...)
{
return -__LINE__;
}
+67
public class LoaderForMongoCompanies extends AbstractJdbcLoader {
@Override
protected String getTableSuffix() {
return "mongo";
}
}
нужно больше классов
+67
File unzippedDir = File.createTempFile("po.", null);
unzippedDir.delete();
unzippedDir = new File(unzippedDir.getAbsoluteFile() + ".dir");
ну да, а посмотреть сигнатуру метода, который вызываем, не судьба, лучше изобрести велосипед
public static File createTempFile(String prefix, String suffix) throws IOException
СУФФИКС!!!
+67
if (!entityType.equalsIgnoreCase("provider"))
branches.add(new TreeNode(true, true, "branch", oe.getBranchName(), oe.getBranchId(), null, null, "branch_"+oe.getBranchId()));
else
branches.add(new TreeNode(true, true, "branch", oe.getBranchName(), oe.getBranchId(), null, null, "branch_"+oe.getBranchId()));
Ну и как это понимать? Может автор на будущее заготовку сделал... Вы так делаете?