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

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)

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

+130

001
002
003
004
005
006
007
008
009
010
011
012
013
014
015
016
017
018
019
020
021
022
023
024
025
026
027
028
029
030
031
032
033
034
035
036
037
038
039
040
041
042
043
044
045
046
047
048
049
050
051
052
053
054
055
056
057
058
059
060
061
062
063
064
065
066
067
068
069
070
071
072
073
074
075
076
077
078
079
080
081
082
083
084
085
086
087
088
089
090
091
092
093
094
095
096
097
098
099
100

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Text.RegularExpressions;
using System.IO;

namespace SimpleLex
{
    class Lexical
    {
        //Сюда передается путь к файлу конфигурации
        public string Path = "";
        //Определяем ключевые слова 
        string regular = "Color|Size|Name";
        public void lexical()
        {
            //Создаем массив строк для дальнейшего заполнения
            string[] conf = new string[1024];
            //Переменная счетчик
            int i = 0;
            //Заполняем массив из файла 
            //Path мы присваиваем значение при создании
            //объекта нашего класса.
            conf = File.ReadAllLines(Path);
            //Запускаем цикл чтения 
            while (conf.Length - 1 >= i)
            {
            // Создаем новый объект класса Regex
            // и передаем ему в качестве конструктора
            // cписок ключевых слов
            Regex RegularExp = new Regex(regular);
            //Начинаем поиск совпадений в текущей строке
            Match match = RegularExp.Match(conf[i]);
            while (match.Success)
            {
                //Ожидаем наличия совпадений
                break;
                //Если нашли то выпрыгиваем 
            }

            switch (match.Value) // Смотрим что же мы обнаружили и вызываем соответствующий метод
            {
                case "Color":
                    //вызываем метод Color_
                    //и передаем ему текущую строку для разбора
                    //Предварительно удалив все пробелы с двух сторон
                    //если они были метод Trim()
                    Color_(conf[i].Trim());
                    i++;
                    break;
                case "Size":
                    size_(conf[i].Trim());
                    i++;
                    break;
                default:
                    //все другое пропускаем
                    i++;
                    break;
            }
            }
        }
        void Color_(string str)
        {
            int i = 0;
            while (str.Substring(i,1)!= "=")
            {
                //Ищем разделитель в данном случае "="
                i++;
            }
            //Находим и извлекаем нужную нам информацию
            string value_ = str.Substring(i + 1, str.Length - (i+1)).Trim();
            //Теперь все зависит от вашей фантазии хотите 
            //Создайте класс который будет устанавливать цвет шрифта
            //И.т.д

            Console.WriteLine(value_);
            Console.ReadKey();
        }
        void size_(string str)
        {
            /*
             * В этом методе я хочу показать как быть если вы используете
             * в своем файле для каждой команды разные разделители
             * в теории конечно можно все это засунуть в один метод
             * Но я советую в дальнейшем если вы будите применять
             * именно такую структуру передавать в метод тип разделителя
             * и соответственно искать именно его другое дело если вы имеете
             * различный формат входных данных как у нас параметр size имеет
             * вид shize X=100,Y=500
             */
            //Есть прекрасный метод 
            int i = 0;
            while (str.Substring(i,1) != " ")
            {
                i++;
            }
            str = str.Substring(i, str.Length - i).Trim();
            string[] commandMas = new string[3];
            commandMas = str.Split(',');

(GetPrivateProfileString? Не, не слышал)
Лютый песдеzz с сайта vbbook.ru

brutushafens, 24 Апреля 2014

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

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

+127

1
2
3
4
5

#define PIXEL_PARTS_NUM 3

typedef struct _pixel {
        char parts[PIXEL_PARTS_NUM];
} pixel;

evg_ever, 24 Апреля 2014

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

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

+124

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

import elements
import functions
import system
error = 'Ошибок нет'
add = input ('Введите элемент, который вы хотите добавить в таблицу')
table = element.table
addToTable : add
if addTo.table : Successful {
 put 'Ошибок нет' }
else 
put error
system.exit

На случай, если друг подумает, что ничего не работает..
Писалось на языке без названия, не придумал ещё :)

Mobac, 24 Апреля 2014

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

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

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

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

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

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

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

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

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

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

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

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