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

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

+183

1
2
3
4
5
6

private static function IIf($itr, $truu, $falss) {
  if($itr)
      return $truu;
  else
     return $falss;
}

Думаю, такое надо вносить в coding conventions...

maxikar, 05 Мая 2010

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

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

+167

01
02
03
04
05
06
07
08
09
10

$fileContent=file_get_contents($this->rootPath.'app/'.$this->interface.$this->handler.$this->p_h.'.php');
            	if (preg_match('/class\s{1,}\b'.$this->handler.$this->p_h.'\b/i', $fileContent,$matches)) {
                    .............
                    if (preg_match('/function\s{1,}\b'.$this->action.$this->p_a.'\b/i', $fileContent,$matches)) {
            			$class=$this->handler.$this->p_h;
				$action=$this->action.$this->p_a;
            			$this->includeClass($class,$action);
                        return;

                     .............

Так мы проверяем наличие медота в классе ((

minisot, 05 Мая 2010

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

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

+163

1
2
3

if(preg_match("#puid=(\d+)#si",$_SERVER['REQUEST_URI'],$m)) {
                 $parent_user_id = $m[1]; 
                 ................

Получаем $_GET

minisot, 05 Мая 2010

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

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

+100

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24

int wmain()
{
                _rmCfg rmCfg;
                double tt;
                int i;

                        String *filePath=S"C:\\rm\\rm.cfg";
                        String * sqlstr;
                        rmCfg.isread=0;
                        String * dir[];
                        DateTime dt;
                        if (File::Exists(filePath))
                        {
                                StreamReader *sr= new StreamReader(filePath);
                                rmCfg.gosanPath=sr->ReadLine();
                                rmCfg.iakServer=sr->ReadLine();
                                rmCfg.iakUser=sr->ReadLine();
                                rmCfg.iakPassword=sr->ReadLine();
                                rmCfg.trackId=sr->ReadLine()->ToInt32(0);
                        //      rmCfg.trName=sr->ReadLine();

                                Console::WriteLine(S"Viewing directory: ");
                                Console::Write(rmCfg.gosanPath);
                                Console::WriteLine();

Писано было на втором фреймворке. Так бы и продолжал писать, если бы товарищ не намекнул про сериализацию.

ursus, 05 Мая 2010

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

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

+164

1
2
3

<?php
$count =300;
define('MAXPROCESS',$count);

$count дальше естественно нигде не используется..

null, 05 Мая 2010

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

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

+130

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

//Сам скрипт
#!/bin/sh
cat /opt/chtag/truechars > /opt/chtag/tf
printf " $1@" >> /opt/chtag/tf
mp3info -p "%t" "$1">> /opt/chtag/tf
/opt/chtag/a.out < /opt/chtag/tf > /opt/chtag/t3
sh /opt/chtag/t3

//truechars
абвгдеёжзийклмнопрстуфхцчшщьыъэюяАБВГДЕЁЖЗИЙКЛМНОПРСТУФХЦЧШЩЬЫЪЭЮЯ

//Исходники a.out
#include <iostream>
using namespace std;
int main()
{
    int i,j;
    string name,s,as="",truec;
    cin >> truec;
    cin >> s;
    while(!cin.eof()) {as+=" "+s; cin >> s;}
    name=as.substr(1,as.find("@",0)-1);
    as=as.substr(as.find("'")+1);
    for(i=0; i<truec.size(); i++)
    {
	if(as.find(truec[i],0)<truec.size())
	{
	    cout << "printf \"" << name << " OK!\\n\"" << endl;
	    return 0;
	}
    }
    cout << "printf \""<<name<<"\\'s tags will be changed...\\nBefore\\n\";\n"
	    "mp3info \""<< name << "\";\n"
	    "mid3iconv -e CP1251 -d --remove-v1 \""<< name << "\";\n"
	    "printf \"Now\\n\"\nmp3info \"" << name << "\""
	 << endl;
    return 0;
}

Вообще должен кодировку в русских тэгах в mp3 нормальной делать...

Tanger, 04 Мая 2010

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

Си / Говнокод #3154

+143

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

// народ на протяжении 5+ лет жаловался что постоянно что-то в этой библиотеке Х
// криво потому что часто (с их слов) попытки вызова ее функций просто обламываются:

int module_main_function()
{
  int rc = 0;
  // ...

  rc = libX_do_work( ... )
  if (rc != OK) {
    // error!!!
  }

  // ...
}

// теперь заглянем в инициализацию этого модуля:

int module_init()
{
   int rc; 
   // ... кучи дежурного копи-паста для инициализации скипнуты ...
   rc = libZ_init();
   if (rc != OK)
   {
     /* error handling */
   }

   rc = libY_init();
   if (rc != OK)
   {
     /* error handling */
   }

   libX_init();   /* лопата */
   if (rc != OK)
   {
     /* error handling */
   }

   rc = libA_init();
   if (rc != OK)
   {
     /* error handling */
   }

   // ... и т.д. и т.п.
}

меня жалобы одного отдела начали потиху доставать что наша библиотека (назовем ее libX) у них в модуле постоянно не работает и шо ваабще за херня. как встроили ее - так и не работает. иногда. в 80+ модулях работает как часы - ни один другой отдел не жалуется - а вот в ихнем очень специальном модуле нет.

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

ЗЫ ответ: на строке с лопатой, не хватает "rc = ". у библиотеки есть внешняя конфигурация на отсутствии коей она у них обламывалась постоянно. и исправить скрипты свои что бы чужая конфигурация не убивалясь им похоже религия не позволяет.

Dummy00001, 04 Мая 2010

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

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

+168

1
2

$date=date("d.m.y");
$data=date("d.m.y");

В header.php, который инклюдитя в каждом файле. И, что самое главное, в коде используются обе переменных!

azzz, 04 Мая 2010

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

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