- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
if ($_REQUEST['name']){
$nameProduct = $_REQUEST['name'];
$products = $db->products($_REQUEST['name']);
} else {
$products = 'error';
}
if ($products == 'error'){
$http->redirect('index.php');
}-
PHP / Говнокод #4176
+158
Из частного проекта.
Во-первых почему-бы не использовать $nameProduct второй раз вместо $_REQUEST['name']?
Во-вторых вместо $products = 'error'; можно сразу было сделать редирект, т.к. все последующие использования $products не имеют смысла, если редирект выполнен.
mikhailu,
06 Сентября 2010
Комментарии (5) -
PHP / Говнокод #4175
+164
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
if($result){ list($Title_block, $Sections, $Description,$Moderators) = mysql_fetch_row($result) //$row = mysql_fetch_assoc($result) echo "<form>\n"; echo "\t<table border='4' cellspacing='1' cellpadding='1' width='1000' height='60'>\n"; echo "\t\t<tr>\n"; echo "\t\t\t<td colspan='5' bgcolor='gray' align='leftmargin'>$Title_block</td>\n"; echo "\t\t</tr>\n"; do{ echo "\t\t<tr>\n"; echo "\t\t\t<td bgcolor='#A0A0A0'></td>\n"; echo "\t\t\t<td bgcolor='#A0A0A0' width='500'>Форум</td>\n"; echo "\t\t\t<td bgcolor='#A0A0A0'>Тем</td>\n"; echo "\t\t\t<td bgcolor='#A0A0A0'>Сообщений</td>\n"; echo "\t\t\t<td bgcolor='#A0A0A0'>Последних сообщений</td>\n"; echo "\t\t</tr>\n"; echo "\t\t<tr>\n"; echo "\t\t\t<td bgcolor='##D0D0D0'><img src='mus.png'></td>\n"; echo "\t\t\t<td bgcolor='#D0D0D0' width='500' class='row4'>\n"; echo "\t\t\t\t<b><a href='Click' style='color:black;'>$Sections</a></b>\n"; echo "\t\t\t\t<br />$Description<br />Модераторы: $Moderators</td>\n"; echo "\t\t\t<td bgcolor='#D0D0D0'></td>\n"; echo "\t\t\t<td bgcolor='#D0D0D0'></td>\n"; echo "\t\t\t<td bgcolor='#D0D0D0'></td>\n"; echo "\t\t</tr>\n"; }while(list($Title_block, $Sections, $Description,$Moderators) = mysql_fetch_row($result)); echo "\t</table>\n"; echo "</form>\n";Часть одного самописного форума
uadeveloper,
06 Сентября 2010
Комментарии (12) -
C# / Говнокод #4174
+129
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
private const int _multiple_cols = 0x0000060D; //0001000001101 - (Multiple view) private const int _single_cols = 0x000007F1; //0001111110001 - (Single view) private void SetGridColumnVisibility() { int bits = _view_type == NotificationContactViewType.Multiple ? _multiple_cols : _single_cols; DataControlFieldCollection cols = gvContacts.Columns; DataControlField col; for (int i = 0; i < cols.Count; i++) { col = cols[i]; int bit = (int)Math.Pow(2, i); col.Visible = ((bits & bit) == bit); } }Интересный способ установки видимости колонкам в гриде :)
olldman,
05 Сентября 2010
Комментарии (27) -
PHP / Говнокод #4173
+144
- 1
HTML разметка вот этого форума (http://science-noir.kzforum.info/forum.htm) доставляет настоящее эстетическое наслаждение :-D
ageron,
05 Сентября 2010
Комментарии (7) -
PHP / Говнокод #4172
+168
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
define('Min2Sec',60); define('Hour2Sec',3600); define('Day2Sec',86400); define('int','integer'); define('real','float'); define('bool','boolean'); define('str','string'); define('mix','array'); define('intmix','int_mix'); define('realmix','real_mix'); define('boolmix','bool_mix'); define('strmix','str_mix'); define('obj','object'); define('nil','null');Из ядра одной русской CMS, писал видимо Pascal'ист
Мартин,
05 Сентября 2010
Комментарии (12) -
Куча / Говнокод #4171
+134
- 1
http://youex.ru/skins/default/pages/search/В продолжение =)
hybroid,
05 Сентября 2010
Комментарии (10) -
Java / Говнокод #4170
+80
- 1
- 2
- 3
- 4
- 5
- 6
- 7
try { while (sender.isAlive()) { this.sleep(100);//static field usage(!) } } catch (java.lang.InterruptedException e) { Log.log(Log.ERROR, this, e); }типа Thread.join() не наш метод.
qnikst,
04 Сентября 2010
Комментарии (7) -
Куча / Говнокод #4169
+121
- 1
- 2
http://habrahabr.ru/blogs/algorithm/103513/ Советую всем посмотреть, очень воодушевляет.А теперь по теме, вторая часть видео ( http://video.yandex.ru/users/ya-events/view/128/?cauthor=ya-events&cid=10 ) 44:44 .
Александр Александрович: "У указателей не нужно определять операцию сравнения [....] равенство есть, а неравенства нет.
[..] Вы не можете теперь создать множество. Точнее можете, но оно будет очень медленным."
Какое-то чудило: " ээ, ну если вести хэшировние, и операцию equals, как в Java сделано"..
Интересно, как он хеширование сделает, если две сущности можно сравнивать только на равенство?
Да, и ещё, сразу виден развращённый( хешированием ) неокрепший детский мозг - видимо никогда не слышал про двоичные деревья поиска, что уже говорить по красно-чёрные деревья.
P.S. Где тут куча? это же Pascal
J0hnny,
04 Сентября 2010
Комментарии (186) -
PHP / Говнокод #4168
+158
- 1
- 2
- 3
- 4
- 5
- 6
- 7
$sql = "SELECT t.topic_id FROM ".Config::Get('db.table.topic')." as t, WHERE 1=1 LIMIT ?d, ?d";5-6 строки мешают уснуть. Из того же LiveStreet..
hybroid,
04 Сентября 2010
Комментарии (17) -
PHP / Говнокод #4167
+156
- 1
- 2
- 3
- 4
- 5
- 6
if (!$this->sCurrentEvent) { /** * Показывает дефолтную страницу */ //а это какая страница? }И всё. Никаких else и прочих вещей. Сижу, курю, пытаюсь восстановить поток мысли авторов.
Из того же LiveStreet. hybroid,
03 Сентября 2010
Комментарии (12)
Follow us!