Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+142
private int[] ListToArray(List<int> list)
{
try
{
int[] res = new int[list.Count];
for (int i = 0; i < list.Count; i++)
{
res[i] = list[i];
}
return res;
}
catch (Exception) { return new int[0]; }
}А ещё он заполняет List<> в цикле for
rumpowered,
01 Июля 2015
+141
foo() ->
Du = os:cmd("du -b "++?PM_LOGS_PATH),
[DiscSize|_] = string:tokens(Du, "\t"),
case list_to_integer(DiscSize) > 10000000000 of
...Вся мощь эрланга. В строке может быть не только число? Let it crash!
lesmugfrog,
01 Июля 2015
−84
BEGIN
select count(*) INTO var FROM table_name;
EXCEPTION WHEN OTHERS THEN
var:=0;
END;
Если что-то пойдет не так...
P.S.: count() всегда возвращает какое-либо значение. Ошибке не откуда взяться.
mishok,
01 Июля 2015
−176
and (ss_1<>'SS0001' or ss_1<>'SS0002' or ss_1<>'SS0010')Сегодня на продакшене было обнаружено гениальное выражение в одном из запросов.
mishok,
01 Июля 2015
+142
public int MapToLowerPageBoundary(int rowIndex)
{
return (rowIndex / RowsPerPage) * RowsPerPage;
}Нашел этот метод в проекте... Интересно о чем думал автор...
alex493049469,
01 Июля 2015
+142
<?php
$self=$_SERVER['PHP_SELF'];
if (isset($_GET['page'])) { $page=$_GET['page'];} else {$page=1;}
$pages=25;
$page_offset=5;
$unique_links=9;
if ($page != 1) { // Don't show back link if current page is first page.
$back_page = $page-1;
echo("<a href=\"$self?page=$back_page\">prev</a> \n");}
for ($i=1; $i <= $pages; $i++) {
if ($i == $page) {
echo("<b>$i</b>\n");// If current page don't give link, just text.
} else {
if (($i>$page-$page_offset) && ($i<$page+$page_offset) && ($page>$page_offset)) {
echo("<a href=\"$self?page=$i\">$i</a> \n");
}
elseif (($i<=$unique_links) && ($page<=$page_offset)) {
echo("<a href=\"$self?page=$i\">$i</a> \n");
}
elseif (($i>$pages-$unique_links) && ($page>$pages-$page_offset)) {
echo("<a href=\"$self?page=$i\">$i</a> \n");
}
}
}
if (!($page >= $pages) && $pages != 1) { // If last page don't give next link.
$next_page = $page + 1;
echo("<a href=\"$self?page=$next_page\">next</a>");}
?>
Адовый аромат (хоть и рабочий), понимаю, но помогите упростить, пожалуйста. Условия внутри цикла скорее всего избыточны.
Вкратце: "плавающий" пагинатор, 9 уникальных ссылок.
nurfed362,
30 Июня 2015
+143
if($scope.model.table){
if($scope.model.table.cells == null){
if($scope.model.table.cols){
$scope.model.table.cells = $scope.model.table.cols;
}
}
if($scope.model.table.heads && $scope.model.table.cols == null){
$scope.model.table.cols = [];
$scope.model.table.heads.forEach(function(head){
$scope.model.table.cols.push({head:head});
});
$scope.model.table.cells.forEach(function(cell, i){
$scope.model.table.cols[i] = {cell:cell};
});
}
}
Andruxa,
30 Июня 2015
+66
Map<Organization, Collection<IFormula>> comissionFormulasMap = getComissionFormulae(bank, service, a);
transIterator:
for (Transaction T : afterBillingMemorandum) {//для каждой транзакции
if (comissionFormulasMap.isEmpty()) {
//System.out.println("Нет формулы для комиссии");
afterComissionSet.add(T); //если нет формулы для определения комиссии транзакция попадает в список на расщепление
} else {
BigDecimal transactionStartValue = T.getStartValue();
Set<Transaction> afterComissionForCurrent = new HashSet<>();
BigDecimal fullTax = BigDecimal.ZERO;
Collection<IFormula> comissionsAKT = new LinkedList<>();
for (Map.Entry<Organization, Collection<IFormula>> comissionsByBank : comissionFormulasMap.entrySet()) {
Organization comissionBank = comissionsByBank.getKey();
for (IFormula f : comissionsByBank.getValue()) {//для каждой формулы по комиссии
GeneratedTransactionTypes transactionType = f.getGeneratedTransactionTypes(comissionBank);
if (transactionType == GeneratedTransactionTypes.AKT) {
comissionsAKT.add(f);
} else {
BigDecimal commie = f.commission(transactionStartValue);//Определяем значение комиссии
if (commie.compareTo(BigDecimal.ZERO) == 0)
continue; //Комиссия 0 нам не интересна, переходим к следующей формуле
fullTax = fullTax.add(commie);//складываем комиссии
if (fullTax.compareTo(transactionStartValue) == 1) {
log.error("Помилка зняття комісії. Сума комісій більша 100 %. Транзакція не буде оброблена. Транзакція: {}", T.toBaseString());
continue transIterator;
} else {
afterComissionForCurrent.add(T.getComissionTransaction(f.getOrganization(), commie, transactionType));
}
}
}
}
//
// afterComissionSet.addAll(afterComissionForCurrent);
// afterComissionForCurrent.clear();
//Далі зняття по акту
BigDecimal afterComissionStartValue = transactionStartValue.subtract(fullTax);
BigDecimal aktFullTax = BigDecimal.ZERO;
for (IFormula f : comissionsAKT) {
GeneratedTransactionTypes transactionType = GeneratedTransactionTypes.AKT;
BigDecimal commie = f.commission(afterComissionStartValue);
if (commie.compareTo(BigDecimal.ZERO) == 0)
continue;
BigDecimal newFullTax = aktFullTax.add(commie);
if (newFullTax.compareTo(afterComissionStartValue) > 0) {
continue;
}
aktFullTax = newFullTax;
afterComissionForCurrent.add(T.getComissionTransaction(f.getOrganization(), commie, transactionType));
}
afterComissionSet.add(T.getRestTransaction(fullTax.add(aktFullTax)));
afterComissionSet.addAll(afterComissionForCurrent);
}
}Писал один товарещ, притом взрослый и сформировавшийся как личность )
ekh,
30 Июня 2015
−160
ALTER TABLE db.table ADD CONSTRAINT cu_contents UNIQUE (contents(128));
ALTER TABLE db.table DROP INDEX cu_contents;Создал констрейнт, удал индекс. Загадки Майсиквела.
wvxvw,
30 Июня 2015
+167
if ($result = mailer($data)) echo "<p>Коммерческое предложение отправлено к вам на электронную почту.</p>";
else echo "<p>Коммерческое предложение отправлено к вам на электронную почту.</p>";//да простит меня БогКовыряю сайта заказчика. Нашел гениальный код в скрипте обработки формы.
jankefir,
30 Июня 2015
Follow us!