По колено в коде — Говнокод.ру

PHP / Говнокод #19960

−2

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42

<? require 'config/bd.php'; ?>
<?
if($_POST["title"]){
$id = intval($_POST["id"]);
$title = intval($_POST["title"]);
$num = mysql_num_rows(mysql_query("SELECT id FROM banner WHERE id = '".$id."'"));
if($num>0){
mysql_query("UPDATE banner SET url = '$url', img = '$img', title = '$title', day = '$day', active = '".$_POST["active"]."', active_to = '$active_to' WHERE id = '".$id."'");
?>
<div class="color='red'">Баннер отредактирован</div>
<?
}
}
if($_POST["id"]){
$id = intval($_POST["id"]);
$title = intval($_POST["title"]);
$num = mysql_num_rows(mysql_query("SELECT id FROM banner WHERE id = '".$id."'"));
if($num>0){
$row = mysql_fetch_array(mysql_query("SELECT * FROM banner WHERE id = '".$id."'"));
?>
<form method="post" action="">
<strong>ID:</strong> <?=$row['id'];?><br>
<strong>Ссылка перехода:</strong> <?=$row['url'];?><br>
<strong>Ссылка на баннер:</strong> <?=$row['img'];?><br>
<strong>Заголовок:</strong> <?=$row['title'];?><br>
<strong>Дней:</strong> <input type="text" size="5" name="id" value="<?=$row['day'];?>">
<strong>Активен?:</strong><select name="active"><option value="1" <? if($row["active"]==1){?>selected="selected"<? }?>>Да</option><option value="0" <? if($row["active"]==0){?>selected="selected"<? }?>>Нет</option></select><br>
<strong>Активен до:</strong> <input type="text" size="10" name="id" value="<?=$row['active_to'];?>">
<input type="hidden" name="id" value="<?=$row['id'];?>">
<input type="submit" value="Сохранить">
</form>
<?
}else{
?>
Баннер не найден
<?
}
}?>
<form method="post" action="">
Введите ID баннера: <input type="text" name="id">
<input type="submit" value="Поиск">
</form>

Что здесь не так? Первую часть поиск по ID проходит! Дале выскакивает форма редактирования, ввел данные нажимаю Сохранить но ничего не происходит! Просто игнор! Исправьте пж!

Andriu, 07 Мая 2016

Комментарии (22)

PHP / Говнокод #19954

+1

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16

// новый тестовый экшн в контроллере
        public function actionNew($alias)
    {
        $model=Partners::model()->model()->findByAttributes(array('alias'=>$alias));
               
                if($model==null)
            throw new CHttpException(404,'The requested page does not exist.');
                       
                $this->render('view',array(
            'model'=>$this->loadModel($model->id),
        ));
       
    }
 
// правило в конфиге
// '<module:\w+>/<controller:\w+>/<alias:\w+>' => '<module>/<controller>/new',

https://vk.com/echo_php?w=wall-175_189930%2Fall

Уи1

Keeper, 07 Мая 2016

Комментарии (1)

PHP / Говнокод #19947

0

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
29
30

<?php
namespace DoctrineExtensions;
use \Doctrine\ORM\Event\LoadClassMetadataEventArgs;
/**
 * Расширение для Doctrine ORM
 * Позволяет отслеживать и работать не со всей базой, а только с таблицами с префиксом
 * Необходимо для уживания с битриксом
 *
 * Class TablePrefix
 * @package DoctrineExtensions
 */
class TablePrefix
{
    protected $prefix = '';
    public function __construct($prefix)
    {
        $this->prefix = (string) $prefix;
    }
    public function loadClassMetadata(LoadClassMetadataEventArgs $eventArgs)
    {
        $classMetadata = $eventArgs->getClassMetadata();
        $classMetadata->setTableName($this->prefix . $classMetadata->getTableName());
        foreach ($classMetadata->getAssociationMappings() as $fieldName => $mapping) {
            if ($mapping['type'] == \Doctrine\ORM\Mapping\ClassMetadataInfo::MANY_TO_MANY) {
                $mappedTableName = $classMetadata->associationMappings[$fieldName]['joinTable']['name'];
                $classMetadata->associationMappings[$fieldName]['joinTable']['name'] = $this->prefix . $mappedTableName;
            }
        }
    }
}

Адепты битрикса добрались до Doctrine ORM. И вот что из этого получилось.
Заставь дурака ORM подключать, он и events задрочит.

Keeper, 06 Мая 2016

Комментарии (1)

JavaScript / Говнокод #19946

−2

01
02
03
04
05
06
07
08
09
10
11

this.params.IsCellEditable = function(rowNumber, cellNumber) {
	cellNumber == 1;
	this.params.ButtonList = this.params.ButtonList.filter(b=>b[0] === "OnRefresh");

	let textContr = new CTextArea('textContr');
	textContr.SourceName = "value";
	textContr.ViewName = "Params";
	textContr.ComEdit = true;
	this.params.arrEditObj[1] = textContr;

}

Найдено в нашем проекте в старом модуле, в авторстве никто не признаётся.
Во-первых, строка 2 бессмысленна. Во-вторых, всё последующее имело бы хоть какой-то смысл _вне_ этой функции, а внутри уже на строке 3 выкидывает ошибку, потому что контекст там и есть this.param из первой строчки. В-третьих, строка 3 призвана выкидывать из тулбара виджета this.param все кнопки, кроме OnRefresh, но на самом деле она там только одна и есть. В-четвёртых, строчки 7 и 8 просто лишние (ну, это из логики используемого в проекте движка следует). В-пятых, из названия метода можно предположить (и это действительно так), что он должен бы возвращать булевское значение, но он всегда возвращает только undefined и, таким образом, все ячейки виджета оказываются нередактируемыми — что совсем лишает смысла создание контрола для редактирования в строках 5—9.
Редкостная бредятина. Кто-то в полном затмении писал, и даже десяти секунд не потратил на тестирование.

torbasow, 06 Мая 2016

Комментарии (8)

PHP / Говнокод #19945

+2

1
2
3
4
5
6
7
8
9

$DB->query("SELECT `id`, `fio`, `pos_id`, `phones` FROM `users` WHERE ...");
if ($DB->errno()) return false;
if (!$DB->num_rows()) return false;
$uList = false;
while (list($id, $fio, $pos, $tel) = $DB->fetch()) {
    $uList[] = array($id, $fio, $pos, $tel);
}
$DB->free();
return $uList;

FETCH_ASSOC to FETCH_NUM converting by ©senior shaurma developer

pahhan, 06 Мая 2016

Комментарии (0)

PHP / Говнокод #19944

+6

1
2
3
4
5
6

$startDate = $dayCount != 7 ? ($_REQUEST["wDate"] ? $_REQUEST["wDate"] : date($dateFormat)) : ($_REQUEST["wDate"] ?
                (date('w', strtotime($_REQUEST["wDate"])) == 1 ? $_REQUEST["wDate"] : (date('w',
                    strtotime($_REQUEST["wDate"])) ? date($dateFormat, strtotime($_REQUEST["wDate"]) -
                    (date('w', strtotime($_REQUEST["wDate"])) - 1) * 24 * 60 * 60) : date($dateFormat,
                    strtotime($_REQUEST["wDate"]) - 6 * 24 * 60 * 60))) : ($cDate == "Sun" ? date($dateFormat,
                    strtotime('Mon last week')) : date($dateFormat, strtotime('Mon this week'))));

pahhan, 06 Мая 2016

Комментарии (20)

C# / Говнокод #19941

+5

01
02
03
04
05
06
07
08
09
10
11
12
13

class Program {
    static void Main(string[] args) {
        AppInstance.Get().DoMain(IOMain);
    }
    static IO<None> IOMain() {
        return
            from _ in IO.Do(() => Console.WriteLine("What is your name?"))
            from name in IO.Do(() => Console.ReadLine())
            let message = "Hi, " + name + "!"
            from r in IO.Do(() => Console.WriteLine(message))
            select r;
    }
}

https://habrahabr.ru/post/282940

vleschenko, 05 Мая 2016

Комментарии (11)

PHP / Говнокод #19940

−3

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

<?php
global $human; //заселяем
$inkey = $_GET['inkey'];

//обучаем:
$array['word']='kill_all_humans';
$array['line']='drink_all_booze';
//и далее...

function drink_all_booze()
{}
function kill_all_humans()
{//убить всех человеков
    //unset ($human);
    while($human[0]>0)
    {//пока в списке людей больше 0
        if(!$human[$human[0]]['name'] == 'Fray')
        {//кроме Фрая.
            unset($human[$human[0]]);
        }
    $human[0]--;
    }
}

//выбираем что делать:
$${array}[$inkey]();
?>

Говорят, что пишу не понятно. Насколько всё хреново?

wano987, 05 Мая 2016

Комментарии (23)

Говнокод: по колено в коде.