- 1
 - 2
 - 3
 - 4
 
// Привет коллеги, у меня возникла делема, куда пихать код отправки
// на e-mail сообщения? В модель формы или в контроллере оставить?
// фреймворк Yii, в самой Yii доке, написано, что в контроллерах надо
// оставлять, а вы как делаете?
                                    Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 9
−4
// Привет коллеги, у меня возникла делема, куда пихать код отправки
// на e-mail сообщения? В модель формы или в контроллере оставить?
// фреймворк Yii, в самой Yii доке, написано, что в контроллерах надо
// оставлять, а вы как делаете?
                                    Привет коллеги, у меня возникла делема, куда пихать код отправки на e-mail сообщения? В модель формы или в контроллере оставить? Фреймворк Yii, в самой Yii доке, написано, что в контроллерах надо оставлять, а вы как делаете?
+2
class Order_OrderProcessorBuilder
{
    /**
     * @param $processorName
     * @return Order_Abstract_OrderAbstract
     */
    private static function createOrderProcessor($processorName)
    {
        $processorClassName = 'Order_Types_'.ucfirst($processorName);
        return new $processorClassName;
    }
    /**
     * @param $builderName
     * @param Order_Abstract_OrderAbstract $orderProcessor
     * @return Order_Abstract_BuilderAbstract
     */
    private static function createOrderBuilder($builderName, Order_Abstract_OrderAbstract $orderProcessor)
    {
        $builderClassName = 'Order_Builders_'.ucfirst($builderName);
        return new $builderClassName($orderProcessor);
    }
    /**
     * @param Order_Abstract_BuilderAbstract $orderBuilder
     * @return Order_Director
     */
    private static function createDirectorClass(Order_Abstract_BuilderAbstract $orderBuilder)
    {
        return new Order_Director($orderBuilder);
    }
    /**
     * @param $orderTypeName
     * @return Order_Abstract_OrderAbstract
     */
    public static function build($orderTypeName)
    {
        /** @var Order_Abstract_OrderAbstract $orderProcessor */
        $orderProcessor = self::createOrderProcessor($orderTypeName);
        /** @var Order_Abstract_BuilderAbstract $orderBuilder */
        $orderBuilder = self::createOrderBuilder($orderTypeName, $orderProcessor);
        $director = self::createDirectorClass($orderBuilder);
        $director->construct();
        return $orderBuilder->getResult();
    }
}
                                    Вот такая у нас есть прослойка, которая работает с паттероном "строитель", честно говоря недавно понял зачем она нужна, не знаю.. Как вам решение? Код не мой
+142
Как-то долго на PHP говнокода не появляется, все научились писать что ли?
                                    
+145
Привет, я вот сайт пишу, каталог, на заказ, ну короче, считаю себя говнокодером. Хочу попросить более опытных что ль, оценить. Сказать так это или нет. Ссылочку на гитхаб чуть позже присобачу.
                                    
+159
if (1) {
define("IMGBASE","");
define("IMGNUM","");
} else {
define("IMGBASE","imgs");
define("IMGFN","_rm");
}
                                    Может я плохой программист, а это не говнокод??? Но таких условий в проекте миллионы блядь!
+167
error_reporting(FALSE);
                                    Я пофиксил баги в вашем проекте :)
+158
<?PHP
	
	# Регистрация
	if(isset($_POST["login"])){
	
	if(isset($_SESSION["captcha"]) AND strtolower($_SESSION["captcha"]) == strtolower($_POST["captcha"])){
	unset($_SESSION["captcha"]);
	$login = $func->IsLogin($_POST["login"]);
	$pass = $func->IsPassword($_POST["pass"]);
	$rules = isset($_POST["rules"]) ? true : false;
	$time = time();
	$ip = $func->UserIP;
	$UserIP = $_SERVER['REMOTE_ADDR'];
	
	$email = $func->IsMail($_POST["email"]);
	$referer_id = (isset($_COOKIE["i"]) AND intval($_COOKIE["i"]) > 0 AND intval($_COOKIE["i"]) < 1000000) ? intval($_COOKIE["i"]) : 1;
	$referer_name = "";
	if($referer_id != 1){
		$db->Query("SELECT user FROM db_users_a WHERE id = '$referer_id' LIMIT 1");
		if($db->NumRows() > 0){$referer_name = $db->FetchRow();}
		else{ $referer_id = 1; $referer_name = "wolframavtor"; }
	}else{ $referer_id = 1; $referer_name = "wolframavtor"; }
	
		if($rules){
			if($email !== false){
		
			if($login !== false){
			
				if($pass !== false){
			
					if($pass == $_POST["repass"]){
						
						$db->Query("SELECT COUNT(*) FROM db_users_a WHERE user = '$login'");
						if($db->FetchRow() == 0){
						
						# Регаем пользователя
						$db->Query("INSERT INTO db_users_a (user, email, pass, referer, referer_id, date_reg, ip) 
						VALUES ('$login','{$email}','$pass','$referer_name','$referer_id','$time',INET_ATON('$ip'))");
						
						$lid = $db->LastInsert();
						
						$db->Query("INSERT INTO db_users_b (id, user, a_t, last_sbor) VALUES ('$lid','$login','1', '".time()."')");
						
						# Вставляем статистику
						$db->Query("UPDATE db_stats SET all_users = all_users +1 WHERE id = '1'");
						
						echo "<center><b><font color = 'green'>Вы успешно зарегистрировались. Используйте форму слева для входа в аккаунт</font></b></center><BR />";
						?></div>
						<div class="clr"></div>	
						<?PHP
						return;
						}else echo "<center><b><font color = 'red'>Указанный логин уже используется</font></b></center><BR />";
						
					}else echo "<center><b><font color = 'red'>Пароль и повтор пароля не совпадают</font></b></center><BR />";
			
				}else echo "<center><b><font color = 'red'>Пароль заполнен неверно</font></b></center><BR />";
			
			}else echo "<center><b><font color = 'red'>Логин заполнен неверно</font></b></center><BR />";
		}else echo "<center><font color = 'red'><b>Email имеет неверный формат</b></font></center>";
		}else echo "<center><b><font color = 'red'>Вы не подтвердили правила</font></b></center><BR />";
	
		}else echo "<center><font color = 'red'><b>Символы с картинки введены неверно</b></font></center>";
	}
	
	
?>
                                    ШЕДЕВРАЛЬНО!
+170
public function action_ajaxlogin()
	{
		// Если запрос поступил не через AJAX, то шлем нахуй
		if ( ! $this->request->is_ajax())
		{
			throw new HTTP_Exception_404(NULL);
		}
		// Вырубаем авто-рендер, ибо это хуев аякс запрос
		$this->auto_render = false;
		// Собираем информацию и пользователе в ёбанный массив
		$user_data = $this->request->post('user_data');
		// Если пользователь авторизирован, то заебато, и возвращаем статус 200!
		if (Auth::instance()->login($user_data['username'], $user_data['password'], (bool) isset($user_data['remember_me'])))
			return $this->response->status(200);
		// Если нет, то "Вася, все хуйня! Давай по новой!"
		return $this->response->status(400);
	}
	public function action_logout()
	{
		// Если запрос поступил не через AJAX, то шлем нахуй
		if ( ! $this->request->is_ajax())
		{
			throw new HTTP_Exception_404(NULL);
		}
		// Вырубаем сучий авто-рендер, нахуй, в пизду блядь
		$this->auto_render = false;
		// Выходим из аккаунта, если вышли, то ахуенно, 200-ый статус)
		if (Auth::instance()->logout())
			return $this->response->status(200);
		// Если все хуйня, то "Вася, давай по новой!"
		return $this->response->status(400);
	}
                                    Kohana фреймворк, и таких комментариев по проекту тьма :)
+161
// Сначала он считал рефералов на уровнях
$sql = mysql_query("SELECT `lvl1` FROM `таблица` WHERE `nick` = $nick");
while($row = mysql_fetch_array($sql)){
	$a++;
}
$sql = mysql_query("SELECT `lvl2` FROM `таблица` WHERE `nick` = $nick");
while($row = mysql_fetch_array($sql)){
	$b++;
}
$sql = mysql_query("SELECT `lvl3` FROM `таблица` WHERE `nick` = $nick");
while($row = mysql_fetch_array($sql)){
	$c++;
}
$sql = mysql_query("SELECT `lvl4` FROM `таблица` WHERE `nick` = $nick");
while($row = mysql_fetch_array($sql)){
	$d++;
}
$sql = mysql_query("SELECT `lvl5` FROM `таблица` WHERE `nick` = $nick");
while($row = mysql_fetch_array($sql)){
	$e++;
}
$sql = mysql_query("SELECT `lvl6` FROM `таблица` WHERE `nick` = $nick");
while($row = mysql_fetch_array($sql)){
	$f++;
}
$sql = mysql_query("SELECT `lvl7` FROM `таблица` WHERE `nick` = $nick");
while($row = mysql_fetch_array($sql)){
	$g++;
}
$sql = mysql_query("SELECT `lvl8` FROM `таблица` WHERE `nick` = $nick");
while($row = mysql_fetch_array($sql)){
	$h++;
}
// Потом он начал регистрировать пользователя на определенном уровне
if($a<3){
	mysql_query("INSERT INTO `таблица`(`lvl1`) VALUES ('$login')");
} 
elseif($b<5){
	mysql_query("INSERT INTO `таблица`(`lvl2`) VALUES ('$login')");
}
elseif($c<9){
	mysql_query("INSERT INTO `таблица`(`lvl3`) VALUES ('$login')");
}
elseif($d<17){
	mysql_query("INSERT INTO `таблица`(`lvl4`) VALUES ('$login')");
}
elseif($e<33){
	mysql_query("INSERT INTO `таблица`(`lvl5`) VALUES ('$login')");
}
elseif($f<65){
	mysql_query("INSERT INTO `таблица`(`lvl6`) VALUES ('$login')");
}
elseif($g<129){
	mysql_query("INSERT INTO `таблица`(`lvl7`) VALUES ('$login')");
}
elseif($h<257){
	mysql_query("INSERT INTO `таблица`(`lvl8`) VALUES ('$login')");
}
elseif($h>256){
	echo "ВЫ использовали всех рефералов, можете смело идти нахуй:)";
                                    Пишем проект, чувак сделал реферальную систему, епт...