Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+72
String isFeatureEnabled = object.get("featureEnabledParam");
boolean featureEnabled = app.isFeatureEnabled();
if (Boolean.parseBoolean(isFeatureEnabled) && !featureEnabled) {
return false;
}
if (!Boolean.parseBoolean(isFeatureEnabled) && featureEnabled)
{
return false;
}
myzone,
22 Октября 2013
+72
Дорогой участник! Не спеши покидать страницу. Я прошу тебя помочь:
не поленись, пройдись по всем постам, и поставь каждому участнику по минусу;
минусуя других участников ты воспитываешь в себе чувство меры и помогаешь обществу снижать ЧСВ.
Заранее спасибо, друг мой. Ты будешь вознагражден за это деяние!
Тот, кто покинул страницу, но не выполнил моей просьбы, умрет через 9 дней.
Дорогой участник! Не спеши покидать страницу. Я прошу тебя помочь:
не поленись, пройдись по всем постам, и поставь каждому участнику по минусу;
минусуя других участников ты воспитываешь в себе чувство меры и помогаешь обществу снижать ЧСВ.
Заранее спасибо, друг мой. Ты будешь вознагражден за это деяние!
Тот, кто покинул страницу, но не выполнил моей просьбы, умрет через 9 дней.
Stertor,
11 Июля 2013
+72
Double extractShare(Product product, ShareExtractor extractor) {
if (product != null) {
Equity equity = product.getEquity();
if (equity != null) {
Double shareMetric = extractor.extract(equity);
if (shareMetric != null) {
return shareMetric;
}
}
}
return 0.0;
}
private interface ShareExtractor {
ShareMetricExtractor PUBLIC_SHARE_OUTST_EXTRACTOR = new ShareExtractor() {
@Override
public Double extract(Equity equity) {
return equity.getPublicShareOutst();
}
};
ShareMetricExtractor SHARE_OUTST_EXTRACTOR = new ShareExtractor() {
@Override
public Double extract(Equity equity) {
return equity.getShareOutst();
}
};
Double extract(Equity equity);Архитектор головного мозга - это диагноз. Фанатзия нашего системного архитектора толи зашкалила, то ли дала сбой. Сей фердопердозный код был заменен следующим же коммитом, но свн помнит все! )
jericho,
22 Апреля 2013
+72
public class Test {
public static final int a;
static {
System.out.println(Main.a);
a = 42;
System.out.println(Main.a);
}
}
javac такой javac...
Так видит проблему: http://ideone.com/jhPN5X
А вот так - нет: http://ideone.com/qAMIvH
bormand,
14 Февраля 2013
+72
public void load()
{
for(Language lang : Language.VALUES)
{
strings.put(lang, new HashMap<String, String>());
File f = new File(Config.DATAPACK_ROOT, "data/string/strings_" + lang.getShortName() + ".properties");
if(!f.exists())
{
log.warn("Not find file: {}", f.getAbsolutePath());
continue;
}
LineNumberReader reader = null;
try
{
reader = new LineNumberReader(new FileReader(f));
String line = null;
while((line = reader.readLine()) != null)
{
if(line.startsWith("#"))
continue;
StringTokenizer token = new StringTokenizer(line, "=");
if(token.countTokens() < 2)
{
log.error("Error on line: " + line + "; file: " + f.getName());
continue;
}
String name = token.nextToken();
String value = token.nextToken();
while(token.hasMoreTokens())
value += "=" + token.nextToken();
Map<String, String> strings = this.strings.get(lang);
strings.put(name, value);
}
}
catch(Exception e)
{
log.error("Exception: " + e, e);
}
finally
{
try
{
reader.close();
}
catch(Exception e)
{
log.error("Cannot close reader", e);
}
}
}
log();
}java.util.Properties? Нет, не слышали :)
EvSpirit,
22 Ноября 2012
+72
if("all".equalsIgnoreCase(lvl)) {
setLevel(SimpleLog.LOG_LEVEL_ALL);
} else if("trace".equalsIgnoreCase(lvl)) {
setLevel(SimpleLog.LOG_LEVEL_TRACE);
} else if("debug".equalsIgnoreCase(lvl)) {
setLevel(SimpleLog.LOG_LEVEL_DEBUG);
} else if("info".equalsIgnoreCase(lvl)) {
setLevel(SimpleLog.LOG_LEVEL_INFO);
} else if("warn".equalsIgnoreCase(lvl)) {
setLevel(SimpleLog.LOG_LEVEL_WARN);
} else if("error".equalsIgnoreCase(lvl)) {
setLevel(SimpleLog.LOG_LEVEL_ERROR);
} else if("fatal".equalsIgnoreCase(lvl)) {
setLevel(SimpleLog.LOG_LEVEL_FATAL);
} else if("off".equalsIgnoreCase(lvl)) {
setLevel(SimpleLog.LOG_LEVEL_OFF);
}Внутренности конструктора org.apache.commons.logging.impl.SimpleLo g
POPSuL,
30 Октября 2012
+72
/*
* Called from a large number of places
* By Ofer
*/
static function getOrderStatus( $po_numb='', $supplier='', $shipping='', $id=0,$customer_id=0,$order='',$order_asc='',$item_status_id=0,$order_status=0, $start_date='',$end_date='',
$name_begins_with = '', $phone = '', $email = '', $list_mgr_id = 0, $is_corporate = '', $sales_id=0,
$first_name = '', $last_name='', $city='', $state='',
$zip='', $store_numb=0, $po_numb='', $release_date='',$supplier=0,
$tracking_numb='', $payment_method='', $shipped_balance='', $avs='', $last4='',
$brand_id=0, $model_numb='',$damage='', $delivery_issue='', $past_damage='',
$coming_back='', $file_claim_ups='', $file_claim_frt='', $rewview='', $balance='', $shipper=0,
$has_balance = 0, $sales_account_id=0, $reference_number="", $third_party_order='', $ebay_id='', $trucker_id='',$get_total=false,
$limit=0, $start=0, $use_dates=0) {Просто очаровательный комментарий!
paulrudy,
05 Сентября 2012
+72
private String getMessage(String prop, boolean suffixEnabled) {
String title = null;
if (prop.equals("headerTitle.suffix")) {
try {
title = messageSource.getMessage("headerTitle.suffix", null, locale);
} catch (NoSuchMessageException e) {
//e.printStackTrace();
}
if (title == null)
title = "";
} else {
try {
title = messageSource.getMessage(prop, null, locale);
if (suffixEnabled)
title += " " + messageSource.getMessage("headerTitle.suffix", null, locale);
} catch (NoSuchMessageException e) {
//e.printStackTrace();
}
if (title == null) {
try {
title = messageSource.getMessage("headerTitle.default", null, locale);
} catch (NoSuchMessageException ex) {
title = "";
}
}
}
return title;
}
welvet,
09 Августа 2012
+72
<?php if ($content): ?>
<?php if($is_page): ?>
<?php print $content; ?>
<?php else: ?>
<?php print $content; ?>
<?php endif?>
<?php endif?>Где-то, в глубоких кодах проекта (писанном на Drupal 7)
Edd,
07 Августа 2012
+72
function valid_gender($str){
switch($str){
case "male":
return "male";
break;
case "female":
return "female";
break;
default:
return "male";
break;
}
}Ультимативная проверка переменной на половую принадлежность. Also return'ам нынче доверия нет!
swiftfooth,
16 Июля 2012
Follow us!