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

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

+143

1
2

Небольшие программки :)
Советую посмотреть и написать отзыв о любой из них.

Скачать >> http://coun.at.ua/stuff/1-1-0-122

Mobac, 28 Апреля 2014

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

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

+136

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

SqlConnection con = new SqlConnection(connectionString);
            string sql = "SELECT idea.Id, idea.Small_text, idea.Full_text, idea.Example_text, ";
            sql = sql + " ul.FullName as Avtor, ref.Name as Status, even.date_act, (select count (*) from events ev where ev.id_action=2 and ev.id_target=idea.Id and ev.type_target=1) as CountComm, (select count (*) from events ev where ev.id_action=57 and ev.id_target=idea.Id and ev.type_target=1) as Voit, (select max(ev.date_act) from events ev where ev.id_action in (1,2,4,5,7,8,9,10,58,60,61,62,63,64,65,66,67,68,69,70,71,72,74) and ev.id_target=idea.Id and ev.type_target=1) as Last_action";
            sql = sql + " FROM Idea idea, UserLists ul, Refbook ref, Events even ";
            sql = sql + " WHERE idea.Avtor=ul.Id and idea.Status_id = ref.Id and even.id_action=1 and even.type_target=1 and even.id_target=idea.id ";
            if (id >= 0)
                sql = sql + "  and idea.Status_id=" + id;
            if (id_avtor >= 0)
                sql = sql + "  and idea.Avtor=" + id_avtor;
            sql = sql + " ORDER BY Last_action desc";

            SqlCommand cmd = new SqlCommand(sql, con);
            cmd.CommandType = CommandType.Text;
            List<Idea> Ideas = new List<Idea>();
            using (con)
            {
                con.Open();
                SqlDataReader reader = cmd.ExecuteReader();
                while (reader.Read())
                {
                    Idea idea = GetIdeaInfo(Int32.Parse(reader["Id"].ToString()));
                    Ideas.Add(idea);
                }
                reader.Close();
                return Ideas;
            }

Принял еще один проект. ASP.NET MVC4. Автор уехал в Новую Зеландию.

Vince, 28 Апреля 2014

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

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

+154

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

trait TPerson {
    public $fname;
    public $lname;
    public $mname;

    // ...
}

class BUser extends EMongoDocument {
    public $first_name;
    public $last_name;
    public $middle_name;

    // ...
}

class User extends BUser {
    public $firstname;
    public $lastname;

    // ...

}

Попалось такое наследие на одном рабочем месте. Было два типа юзеров, каждый из них наследовал BUser, в одном из них были firstname и lastname, в другом не были. В монге документы юзера, у которого свои firstname и lastname были вида:
{
_id: ObjectId("..."),
...
first_name: '',
last_name: '',
middle_name: '',
fname: '',
lname: '',
mname: '',
firstname: '',
lastname: ''
}

LubarVV, 26 Апреля 2014

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

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

+72

1
2
3
4
5

if(error == CABtEngineErrors.K_ERR_BTENGINE_CANCEL) {
	mObserver.btEngineEncryptError(CABtEngineErrors.K_ERR_BTENGINE_CANCEL);
} else {
	mObserver.btEngineEncryptError(error);
}

busylee, 26 Апреля 2014

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

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

+154

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

$massiv = array(1,2,3,4,5,6,7,8);
$her = 0;
$idRazdelov = array();
$idRoditela = $idrazdelf;
foreach($massiv as $chislo){
 
$db_list = CIBlockSection::GetList(Array(SORT=>"ASC"), $arFilter = Array("IBLOCK_ID"=>"7", "ID"=>$idRoditela), true); 
while($ar_result = $db_list->GetNext()){
    
    if ($her == 0){
    if ($ar_result["IBLOCK_SECTION_ID"]){
       $idRazdelov[].= $ar_result["ID"];
       $idRoditela = $ar_result["IBLOCK_SECTION_ID"]; 
    }else{
        $idRazdelov[].= $ar_result["ID"];
        $her = 1;
    }
    }
    
    }
}

Cotard, 25 Апреля 2014

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

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

+153

01
02
03
04
05
06
07
08
09
10
11

class Application_Model_Product extends Leto_Model
{
    ...
    public function getParts($cityId)
    {
       ...
        if (@$prods['Сезонный']) {
            $html[] = "<h2>" . _T(__FILE__, 'Сезонные') . "</h2><p>";
            ...
        }
        $html[] = '</p>';

HTML в модели - это сильно

DIX315, 25 Апреля 2014

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

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

+152

1
2
3
4
5
6
7

/** 
 * Yesterday,
 * All my troubles seemed so far away
 * Now it looks as though they're here to stay
 * Oh, I believe in yesterday
 */
$yesterday = date('Y-m-d', strtotime($_REQUEST['dateFrom']) - 86400);

DIX315, 25 Апреля 2014

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

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

+153

1
2
3
4
5
6
7

// Заполняем листы целиком одним махом. Что бы про PHP ни говорили, он весьма крут.
$curr_griddles = array_merge($curr_griddles, array_fill(0, $gqty, array('total_qty' => $piesPerList, $pieId => $piesPerList)));
                                
   ...

// Выбираем наименее загруженного работника. А вот тут PHP демонстрирует корявость. Правда непонятно, мою или свою...
asort($workersLoad); reset($workersLoad); $kv = each($workersLoad); $workerId = $kv['key'];

DIX315, 25 Апреля 2014

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

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

+129

1

http://archive.today/9LWN

From: Linus Torvalds
I think goto's are fine, and they are often more readable than large
amounts of indentation.

laMer007, 25 Апреля 2014

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

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

+150

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

<?php
	require_once("inc/mysql.php");
	sleep(3); //Для ajax запроса, потом удалить

	// Проверяем, что к нам идёт Ajax запрос
	if (isset($_SERVER['HTTP_X_REQUESTED_WITH']) && !empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest') {
		$city1 = $_POST['city1'];
	}else{
		exit(); // Заканчиваем работу скрипта, если это не ajax запрос
	}

	if (isset($city1)) {
		$query = "SELECT id_city, id_region, id_country FROM cities WHERE city_name_ru LIKE '$city1'";
		$result = mysqli_query($link, $query);// or trigger_error(mysql_error($link)." ".$query);
		$row = mysqli_fetch_array($result);
		$country = $row['id_country'];
		//echo $country;
		//echo '<br>';
		$region = $row['id_region'];
		//echo $region;
		//echo '<br>';
		$query1 = "SELECT country_name_ru FROM countries WHERE id_country = '$country'";
		$result1 = mysqli_query($link, $query1);// or trigger_error(mysql_error($link)." ".$query);
		$row1 = mysqli_fetch_array($result1);
		echo '<p class=\'country\'>Страна '.$row1['0'].'</p>';
		//echo '<br>';
		$query2 = "SELECT region_name_ru FROM regions WHERE id_region = '$region'";
		$result2 = mysqli_query($link, $query2);// or trigger_error(mysql_error($link)." ".$query);
		$row2 = mysqli_fetch_array($result2);
		echo '<p class=\'reqion\'>Область '.$row2['0'].'</p>';
	}
?>

Прокомментируйте, пожалуйста, как улучшить этот говнокод.

eprivalov1, 25 Апреля 2014

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

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

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

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

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

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

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

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

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

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

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

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