- 1
- 2
- 3
- 4
- 5
- 6
case 43: /* + */
sum_up += sum_up > 1 ? 0 : 1;
return 1;
case 45: /* - */
sum_up -= sum_up < 1 ? 0 : 1;
return 1;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+142
case 43: /* + */
sum_up += sum_up > 1 ? 0 : 1;
return 1;
case 45: /* - */
sum_up -= sum_up < 1 ? 0 : 1;
return 1;
Если выше/ниже лимита то добавляем/отнимаем ничего.
ЗЫ sum_up это переменная контролирующая уровень вербозности (0,1,2) специальной мониторной проги. кнопками +/- можно интерактивно контролировать. К слову, сверху это уже правленый код. Оригинальный оригинал:
if (c == 43) { sum_up = sum_up > 1 ? sum_up : ++sum_up; return 1; }
if (c == 45) { sum_up = sum_up < 1 ? sum_up : --sum_up; return 1; }
+147
if(entry.getName().equals("back.jpg"));{
}
просто смешная (но с последствиями) опечатка
+78
if (searchFB.getOrderNo()!=null && searchFB.getOrderNo().length()==6){
int drawingStatus = searchFB.getDrawingStatus();
int criticalStatus = searchFB.getCriticalStatus();
int blockingStatus = searchFB.getBlockingStatus();
int suspensionStatus = searchFB.getSuspensionStatus();
searchFB.setDrawingStatus(drawingStatus);
searchFB.setCriticalStatus(criticalStatus);
searchFB.setBlockingStatus(blockingStatus);
searchFB.setSuspensionStatus(suspensionStatus);
}
И это мне поддерживать...
−108
+ (NSUInteger)tagExpensiveCandyAnimationInteger:(NSUInteger)tagSprite
{
NSUInteger animationTag;
if(tagSprite == 2001)
{
animationTag = 2101;
}
else if(tagSprite == 2002)
{
animationTag = 2102;
}
else if(tagSprite == 2003)
{
animationTag = 2103;
}
else if(tagSprite == 2004)
{
animationTag = 2104;
}
else if(tagSprite == 2005)
{
animationTag = 2105;
}
else if(tagSprite == 2006)
{
animationTag = 2106;
}
else if(tagSprite == 2007)
{
animationTag = 2107;
}
else if(tagSprite == 2008)
{
animationTag = 2108;
}
else if(tagSprite == 2009)
{
animationTag = 2109;
}
else if(tagSprite == 2010)
{
animationTag = 2110;
}
else if(tagSprite == 2011)
{
animationTag = 2111;
}
else if(tagSprite == 2012)
{
animationTag = 2112;
}
return animationTag;
}
Того же автора:)
+77
public class SomeEntityBean implements javax.ejb.EntityBean {
private boolean loadMember;
private Wrapper list;
/* ... */
public void ejbLoad() {
this.loadMember = false;
load();
}
private void load() {
/* a LOT of code */
String[] attrList = this.loadMember ? a.attrListMember() : a.attrList();
this.list = new Wrapper(attrList);
/* a LOT of code */
}
public Something getSomething() {
/* a LOT of code */
this.loadMember = true;
load();
/* do something with this.list */
this.loadMember = false;
return something;
}
}
Имена персонажей были умышленно изменены.
Разумеется, loadMember больше нигде не используется.
Яркий пример повторного использования кода.
+158
if(isset($_POST['otmen'])){
die('1');
echo "<META HTTP-EQUIV='Refresh' CONTENT='0; URL=/admin/?page=pages'>";
exit();
}else{
$id=$_GET['id'];
$name=$_POST['name'];
$title=$_POST['title'];
$txt=$_POST['txt'];
$txt=str_replace("\\\"","\"", $txt);
$meta_desc=$_POST['meta_desc'];
$meta_key=$_POST['meta_key'];
$templ=$_POST['templ'];
mysql_query("set names cp1251");
$DB->query("UPDATE ".$pdb."_pages SET name='".$name."', title='".$title."', templ='".$templ."' WHERE id='".$id."'");
$DB->query("UPDATE ".$pdb."_pages SET txt='".$txt."' WHERE id='".$id."'");
$DB->query("UPDATE ".$pdb."_meta SET meta_desc_txt='$meta_desc' WHERE id='$id'");
$DB->query("UPDATE ".$pdb."_meta SET meta_key_txt='$meta_key' WHERE id='$id'");
echo "<center>Текст изменён!<br>Если вас не перенаправляет на другую странице, то нажмите на эту ссылку: <a href='/admin/?page=pages'>Заказы</a></center>";
if(isset($_POST['prim'])){
echo "<META HTTP-EQUIV='Refresh' CONTENT='0; URL=/admin/?page=pages'>";
}
if(isset($_POST['ok'])){
echo "<META HTTP-EQUIV='Refresh' CONTENT='0; URL=/admin/?page=pages&id=$id'>";
}
exit();
}
Модуль (?) редактирования страниц. $_POST['otmen'] - это при нажатии кнопки "отмена". $_POST['prim'] - это "применить", $_POST['ok'] - "сохранить". Иногда лучше не смотреть, в чем работаешь...
+157
print str_replace('collapsed', '', str_replace('expanded', '', str_replace('leaf', '', menu_tree($menu_name = 'menu-fabric'))))
Вывод меню через файл шаблона в Drupal…
−116
SELECT COUNT(DISTINCT p.productID) as cnt FROM (SELECT DISTINCT p.productID FROM SC_products p AND collection!=2 ) AS p
мало того, что тут синтаксические ошибки, так еще и суть просто шедевральная
+127
try
{
this.DBConn.Open();
}
catch (Exception)
{
try
{
this.DBConn.Close();
this.DBConn.Open();
}
catch (Exception ex)
{
System.Diagnostics.EventLog.WriteEntry("Agent", "Ошибка подключения к базе данных: " + ex.ToString(),
System.Diagnostics.EventLogEntryType.Error);
return;
}
}
Поразительная настойчивость
−106
- (BOOL)shouldAutorotateToInterfaceOrientation:(UIInterfaceOrientation)toInterfaceOrientation
{
if (rotateBag > 6) //после возврата, шудавторотейт вызывается 7 раз, это условие для погашения этих вызовов
{
if ([portret isEqualToString:@"horizontal"])
{
if (firstLaunch == 0) // блок для первого запуска (по умолчанию всегда выставляет ландскейпРайт, а это не всегда правильно)
{
if (interfacePosition == 0)
{ self.view.transform = CGAffineTransformMakeRotation(M_PI); }
else
{ self.view.transform = CGAffineTransformIdentity; }
firstLaunch++;
indexLandscape = interfacePosition;
}
else
{
if (indexLandscape == 1) //если изначальный вариант был правосторонний
{
if (toInterfaceOrientation == UIInterfaceOrientationLandscapeRight) // если ЛандскейпРайт
{ self.view.transform = CGAffineTransformIdentity;
interfacePosition = 1; }
if (toInterfaceOrientation == UIInterfaceOrientationLandscapeLeft) // если ЛандскейпЛефт
{ self.view.transform = CGAffineTransformMakeRotation(M_PI);
interfacePosition = 0; }
}
else //если изначальный вариант был левосторонний
{
if (toInterfaceOrientation == UIInterfaceOrientationLandscapeRight) // если ЛандскейпРайт
{ self.view.transform = CGAffineTransformMakeRotation(M_PI);
interfacePosition = 1; }
if (toInterfaceOrientation == UIInterfaceOrientationLandscapeLeft) // если ЛандскейпЛефт
{ self.view.transform = CGAffineTransformIdentity;
interfacePosition = 0; }
}
}
}
else
{
if (firstLaunch == 0) // блок для первого запуска (по умолчанию показывается правильно, поэтому ничего делать не надо)
{ firstLaunch++; }
else
{
if (toInterfaceOrientation == UIInterfaceOrientationPortrait)
{ self.view.transform = CGAffineTransformIdentity; }
if (toInterfaceOrientation == UIInterfaceOrientationPortraitUpsideDown)
{ self.view.transform = CGAffineTransformMakeRotation(M_PI); }
}
}
}
else {
rotateBag++;
}
return false;
}
Крутящаяся сумка в действии...
Поворот дисплея в 50 строк :\