- 1
- 2
- 3
if (Skin == null || ((Skin != null && Skin.Value == null) || (Skin != null && Skin.Value != null && Skin.Value.Length == 0))) {
Skin = new LocalString("...");
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+137.1
if (Skin == null || ((Skin != null && Skin.Value == null) || (Skin != null && Skin.Value != null && Skin.Value.Length == 0))) {
Skin = new LocalString("...");
}
проверочко.. ^_^
+154
if ( isset( $criterias['name'] ) && ($criterias['name'] = trim( $criterias['name'] )) )
{
$conditions .= empty( $conditions ) ? '(' : ' AND ';
$conditions .= "name LIKE '%".addslashes( $criterias['name'] )."%'";
}
if ( isset( $criterias['address'] ) && ($criterias['address'] = trim( $criterias['address'] )) )
{
$conditions .= empty( $conditions ) ? '(' : ' AND ';
$conditions .= "address LIKE '%".addslashes( $criterias['address'] )."%'";
}
// если условие непустое, то закрываем скобку
$conditions .= empty( $conditions ) ? '' : ')';
Вместо использования массива для условий и последующего его объединения
+151
public virtual Type GetFields() {
//Must be overridden!!!
return null;
}
public virtual Enum[] GetCompareFields() {
//Must be overridden!!!
return null;
}
Про абстракиные методы нам ещё не рассказывали :-D
+158.4
<?php
//...
if (!count($error)) {
if(strpos($msg,'samp.ucoz')) $msg = htmlSpecialChars('>>>>>>>>>>>{ Я ДАЛБОЁБ }<<<<<<<<<<<<<');
if(strpos(strtoupper($msg),'GAMES.SHOP777')) $msg = htmlSpecialChars('>>>>>>>>>>>{ Я ДАЛБОЁБ }<<<<<<<<<<<<<');
$DB->query("INSERT INTO `guestbook` (`user_id`,`user_name`,`msg`,`add_date`,`ip`) VALUES ('$user_id','$user_name','$msg',NOW(),'$ip')");
header("Location: guestbook.php"); exit;
}
//...
?>
Защита от спама :))
Не, ну а чё, задолбали!
+151.4
if (isset($_SESSION) && !is_array($_SESSION)){
exit("Hacking attempt");
}
просто улыбнуло
симпатичная антихакерская защита
+142.2
void Session_Start(object sender, EventArgs e)
{
Session["SessionID"] = Session.SessionID; //to avoid session regeneration
}
Вот так борются с устареванием сессии
+139
#define true false
попробуй найди ошибку :)
+151
private function GetUnitType($unitType)
{
if($unitType)
if(!UnitType::Exist($unitType))
{
Debug::error("WebPartCommentsList:: UnitType не найден");
exit;
}
else
$this->UnitType = UnitType::GetUnitTypeId($this->UnitType);
}
проверка на существование сущности внутри класса, причём реально $unitType == $this->UnitType
писал ...
+160.5
//...
$br="
";
//...
+148.5
function rgb_to_htmlcolor($rgb) {
$red = $rgb['red'];
$green = $rgb['green'];
$blue = $rgb['blue'];
$red = 0x10000 * max(0,min(255,$red+0));
$green = 0x100 * max(0,min(255,$green+0));
$blue = max(0,min(255,$blue+0));
// convert the combined value to hex and zero-fill to 6 digits
return "#".str_pad(strtoupper(dechex($red + $green + $blue)),6,"0",STR_PAD_LEFT);
}
Преобразование RGB в #RRGGBB
Чувак не ведает что есть sprintf('#%02X%02X%02X')