- 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 "ВЫ использовали всех рефералов, можете смело идти нахуй:)";
Пишем проект, чувак сделал реферальную систему, епт...