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

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

0

1
2
3
4

$data = array_slice(array_filter(explode("|", trim($data))), 0, 2000);
if (count(array_filter(explode("|", trim($data)))) > 2000) {
    mail('****@gmail.com', '**** too much orders', count(array_filter(explode("|", trim($data)))));
}

Не знаю, как это прокомментировать. Видимо, индус не хочет получать сообщения на почту.

jbot, 02 Февраля 2016

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

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

+3

1
2
3
4
5
6
7

$url = get_permalink();
    $text = 'Подробнее';
    if(strpos($url, '/en/') !== false){
        $text = 'More';
    } else if(strpos($url, '/ua/') !== false){
        $text = 'Детальніше';
    }

l10n

Stefan_Hitskov, 01 Февраля 2016

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

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

−1

01
02
03
04
05
06
07
08
09
10
11

mId = core.Material.Materials.FirstOrDefault(x => x.Value.article == m.article && x.Value.title == m.title && x.Value.category_id == m.category_id).Key;
if (mId == 0)
	mId = core.Material.Materials.FirstOrDefault(x => x.Value.article == m.article && x.Value.title == m.title).Key;
if (mId == 0)
	mId = core.Material.Materials.FirstOrDefault(x => x.Value.article == m.article && x.Value.category_id == m.category_id).Key;
if (mId == 0)
	mId = core.Material.Materials.FirstOrDefault(x => x.Value.title == m.title && x.Value.category_id == m.category_id).Key;
if (mId == 0)
	mId = core.Material.Materials.FirstOrDefault(x => x.Value.article == m.article).Key;
if (mId == 0)
	mId = core.Material.Materials.FirstOrDefault(x => x.Value.title == m.title).Key;

Есть 3 поля.
Поиск сначала по 3. Затем по 2, затем по 2 и еще по 2. И от безысходности по 1
Реально ли это оптимизировать?

yakov_255, 29 Января 2016

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

Куча / Говнокод #19372

−2

1
2
3
4
5
6
7

{{#ifexpr:{{{1|0}}}<=1|{{ {{{2|}}}|{{#expr:{{{1|1}}}+{{{3}}} }} }}|{{Numbered sequence/1|{{#expr:floor ({{{1|0}}}/2)}}|{{{2|}}}|{{{3}}} }}{{Numbered sequence/1|{{#expr:-floor (-{{{1|0}}}/2)}}|{{{2|}}}|{{#expr:floor ({{{1|0}}}/2)+{{{3}}} }} }} }}

{{#ifexpr:{{{1|0}}}<=1|{{ {{{2|}}}|{{#expr:{{{1|1}}}+{{{3}}} }} }}|{{Numbered sequence/2|{{#expr:floor ({{{1|0}}}/2)}}|{{{2|}}}|{{{3}}} }}{{Numbered sequence/2|{{#expr:-floor (-{{{1|0}}}/2)}}|{{{2|}}}|{{#expr:floor ({{{1|0}}}/2)+{{{3}}} }} }} }}
...
{{#ifexpr:{{{1|0}}}<=1|{{ {{{2|}}}|{{#expr:{{{1|1}}}+{{{3}}} }} }}|{{Numbered sequence/9|{{#expr:floor ({{{1|0}}}/2)}}|{{{2|}}}|{{{3}}} }}{{Numbered sequence/9|{{#expr:-floor (-{{{1|0}}}/2)}}|{{{2|}}}|{{#expr:floor ({{{1|0}}}/2)+{{{3}}} }} }} }}

{{#ifexpr:{{{1|0}}}<=1|{{ {{{2|}}}|{{#expr:{{{1|1}}}+{{{3}}} }} }}|<span style="color:red">Превышено количество включений — 1024</span>}}

Циклы нельзя, никаких переменных нельзя, только параметры. Казалось бы, ничего страшного — так и рекурсии тоже нельзя!

tucvbif, 29 Января 2016

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

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

−1

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

public function IndexAction() {
    $this->_view->caption = 'Справочник сотрудников клиента';

    $this->_view->description = '<p>Справочник сотрудников клиента нужен, чтобы сократить время на оформление бронирований проживаний, авиа и ж/д билетов, 
				трасферов. Поэтому информация в этом справочнике должна быть всегда правильной и актуальной. Чтобы добиться этого, справочник должен быть удобным и информативным.
				Информативность должна достигаться за счёт удобного поиска и отображения данных. Удобство должно быть в мелочах: календарь выбора дат, маска воода паспорта, использование json.</p>';

    $this->_view->requirements = [
        'Справочник должен содержать следующие поля: фамилия, имя, отчество, должность, адрес почты, мобильный телефон, пол, дату рождения, паспорт.',
        'Поля фамилия, имя, отчество, адрес почты, мобильный телефон должны быть обязательными и проверяться на валидность.',
        'Поля базы данных должны иметь комментарии.',
        'Код должен содержать комментарии.',
    ];

    $this->_view->tasks = [
        '<b>Сделать таблицу в базе</b> (основное задание).',
        '<b>Сделать контроллер для управления справочником</b> (основное задание).',
        '<b>Сделать добавление / изменение / удаление записей</b> (основное задание).',
        '<i>Сделать историю изменений</i> (дополнительное задание).',
        '<i>Использовать json, маски ввода, javascript-календари</i> (дополнительное задание).'
    ];

    $this->_view->rating = [
        'Читабельность кода.',
        'Структура базы данных.',
        'Работоспособность кода.',
        'Отсутствие потенциально опасного кода.',
        'Работа в рамках ООП.',
    ];

    $this->_view->hints = [
        'Для работы с базой данных используется <a href="http://localhost/pma/" target="pma">phpMyAdmin</a>.',
        'Справочная информация по <a href="http://php.net/" target="php">PHP</a>, <a href="http://dev.mysql.com/doc/refman/5.5/en/" target="mysql">MySQL</a>.',
        'Очень желательно подключить и использовать <a href="http://jquery.com/" target="_blank">jQuery</a>.',
    ];
  }

> "Необходимо сделать справочник сотрудников, используя предоставленный код."

Мне на почту пришло тестовое задание от конторы про которую до этого письма я и не знал. Контора не маленькая, несколько отделов разработки, но 2016 год бля, пишем свой фреймворк с нуля без SMS! Только лучшие мировые практики из моего 2 недельного опыта разработки!

Или это хитрый тест на скилл умение разбираться в самописных фреймворках?

elitist_1van0ff, 29 Января 2016

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

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

−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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65

if ($Module == 'edit' and $_POST['enter']) {	
ULogin(1);
$_POST['name'] = FormChars($_POST['name']);
$_POST['femeli'] = FormChars($_POST['femeli']);
$_POST['den'] = FormChars($_POST['den']);
$_POST['month'] = FormChars($_POST['month']);
$_POST['year'] = FormChars($_POST['year']);
$_POST['pol'] = FormChars($_POST['pol']);
$_POST['country'] = FormChars($_POST['country']);
$_POST['region'] = FormChars($_POST['region']);
$_POST['city'] = FormChars($_POST['city']);


if ($_POST['name'] != $_SESSION['USER_NAME']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `name` = '$_POST[name]' WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_NAME'] = $_POST['name'];
}

if ($_POST['femeli'] != $_SESSION['USER_FEMELI']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `femeli` = '$_POST[femeli]' WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_FEMELI'] = $_POST['femeli'];
}

if ($_POST['den'] != $_SESSION['USER_DEN']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `den` = $_POST[den] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_DEN'] = $_POST['den'];
}

if ($_POST['month'] != $_SESSION['USER_MONTH']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `month` = $_POST[month] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_MONTH'] = $_POST['month'];
}


if ($_POST['year'] != $_SESSION['USER_YEAR']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `year` = $_POST[year] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_YEAR'] = $_POST['year'];
}

if ($_POST['pol'] != $_SESSION['USER_POL']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `pol` = $_POST[pol] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_POL'] = $_POST['pol'];
}



if ($_POST['country'] != $_SESSION['USER_COUNTRY']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `country` = $_POST[country] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_COUNTRY'] = $_POST['country'];
}


if ($_POST['region'] != $_SESSION['USER_REGION']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `region` = $_POST[region] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_REGION'] = $_POST['region'];
}

if ($_POST['city'] != $_SESSION['USER_CITY']) {
mysqli_query($CONNECT, "UPDATE `users`  SET `city` = $_POST[city] WHERE `id` = $_SESSION[USER_ID]");
$_SESSION['USER_CITY'] = $_POST['city'];
}


MessageSend(3, 'Данные изменены!');
}

никто не подскажет как сократить вот эту шнягу

viktor, 29 Января 2016

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

C++ / Говнокод #19365

0

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

float massiv[2];
	massiv[0] = 1.05;
	massiv[1] = 2;
	float summa;
	for(int i = 0; i < 2; i++)
	{
		summa = summa + massiv[i];
	}
	ShowMessage(summa);


VS


	float massiv[2];
	massiv[0] = 1.05;
	massiv[1] = 2;
	float summa = massiv[0] + massiv[1];
	ShowMessage(summa);

Бывают же люди...

не люблю вообще наговаривать на людей, но тут уж п..здец полный.
не знаю - не сдержался, а может сегодня звезды не так стали.
дали мне на сопровождение код, но это еще полбеды...
вообщем суть вопроса - человеку дали (по всей видимости)
задачу найти сумму двух чисел. вот что он написал: (см. в коде)

Это вместо: (см. ниже в коде под надписью VERSUS)

И так все 100 000 строк кода.
Как говорится программист от Б-г-а. Под Б-г я имею ввиду Бориса Гребенщикова.
(Не знаю как кто, а я лично не перевариваю его музыку)

P.S. Автор сего произведения сейчас работает помощником директора. Чего-то я
наверное не понимаю в этой жизни.

P.P.S. Все совпадения с реальными людьми и событиями случайны. Говорю потому
что страшно - еще уволят.

expdev, 28 Января 2016

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

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

+2

1
2
3
4

public ActionResult Exception()
{
  throw new Exception("You better not to execute this!");
}

нашел говнокод, оставленный прошлым архитектором)

govnokoder1488, 28 Января 2016

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

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