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

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

+2

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
66
67
68
69
70
71
72
73
74
75
76
77

string[] array=new string[10];
array[0] = "?";
array[1] = "?";
array[2] = "?";
array[3] = "?";
array[4] = "?";
array[5] = "?";
array[6] = "?";
array[7] = "?";
array[8] = "?";

....

if (DS.Tables[0].Rows[i][3].ToString() == "1")
{
checkBox20.Checked = true;
}
if (DS.Tables[0].Rows[i][4].ToString() == "1")
{
checkBox23.Checked = true;
}
if (DS.Tables[0].Rows[i][5].ToString() == "1")
{
checkBox22.Checked = true;
}
if (DS.Tables[0].Rows[i][6].ToString() == "1")
{
checkBox25.Checked = true;
}
if (DS.Tables[0].Rows[i][7].ToString() == "1")
{
checkBox24.Checked = true;
}
if (DS.Tables[0].Rows[i][8].ToString() == "1")
{
checkBox27.Checked = true;
}
if (DS.Tables[0].Rows[i][9].ToString() == "1")
{
checkBox11.Checked = true;
}

...

if (checkBox17.Checked)
{
array[0] = "application/";
}
if (checkBox18.Checked)
{
array[1] = "audio/";
}
if (checkBox21.Checked)
{
array[2] = "example/";
}
if (checkBox20.Checked)
{
array[3] = "image/";
}
if (checkBox23.Checked)
{
array[4] = "message/";
}
if (checkBox22.Checked)
{
array[5] = "model/";
}
if (checkBox25.Checked)
{
array[6] = "multipart/";
}
if (checkBox24.Checked)
{
array[7] = "text/";
}
if (checkBox27.Checked)

Дали на рецензию одну научную работу...

sgerman, 18 Сентября 2015

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

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

+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

public SomeLongNameSpace.PolicyPremiumReturnType ProcessPremiums(
                    int userID, 
                    int divisionID, 
                    string ObjectCode, 
                    int iPolicyID, 
                    int iBulkEndorsementID, 
                    System.Data.DataTable dtClientPremiums, 
                    System.Data.DataTable dtClientPremiumInterests, 
                    System.Data.DataTable dtRiskCommissions, 
                    System.Data.DataTable dtBrokerages, 
                    System.Data.DataTable dtDiscounts, 
                    System.Data.DataTable dtApportionments, 
                    System.Data.DataTable dtUWGroups, 
                    System.Data.DataTable dtUWGroupSettings, 
                    System.Data.DataTable dtUWGroupPremiums, 
                    System.Data.DataTable dtRiskDeclarations, 
                    System.Data.DataTable dtRiskDeclarationsInterests, 
                    int clientPaymentIntervalDays, 
                    int uwDefaultPaymentIntervalDays, 
                    int uwPaymentIntervalDays, 
                    System.Data.DataTable dtSchedulesLimitDetails, 
                    XII.Integration.GlobalXB.v1_4.PolicyGL.CommissionProcessingFlags commissionProcessingFlags, 
                    bool isOverrideWarranties, 
                    string transactionDescription, 
                    string TechnicalContactName, 
                    bool isCreateDeclarationWithoutInterests, 
                    int clientRPPaymentIntervalDays,
                    System.Data.DataTable deferredPayments)
        {
            return base.Channel.ProcessPremiums(userID, divisionID, ObjectCode, iPolicyID, iBulkEndorsementID, dtClientPremiums, dtClientPremiumInterests, dtRiskCommissions, dtBrokerages, dtDiscounts, dtApportionments, dtUWGroups, dtUWGroupSettings, dtUWGroupPremiums, dtRiskDeclarations, dtRiskDeclarationsInterests, clientPaymentIntervalDays, uwDefaultPaymentIntervalDays, uwPaymentIntervalDays, dtSchedulesLimitDetails, commissionProcessingFlags, isOverrideWarranties, transactionDescription, TechnicalContactName, isCreateDeclarationWithoutInterests, clientRPPaymentIntervalDays, deferredPayments);
        }

few params

g10829780, 18 Сентября 2015

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

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

−1

1

continueSaving(item, item);

В node.js 3 вызова функции continueSaving, первый объект новый, второй старый из бд, в самой функции проверяется наличие старого объекта и если есть то он используется, в остальных вызовах все нормально.

styopdev, 18 Сентября 2015

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

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

+3

1

$imageBinding['isHidden'] = $image->isHidden() === true ? true : false;

bit0rez, 18 Сентября 2015

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

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

+7

01
02
03
04
05
06
07
08
09
10
11

editfobjectobject($fobject_id,$fobject_object_id,
$object_type_id,$fobject_status_id,$all_sq,
$live_sq,$kitchen_sq,$floor,$balcony_id,
$su_id,$windows_id,$repair_id,
$number,$plain,$tplains,
$fcomment,$cost,$all_cost,
$manager_id,$delplain,$dic_room_type_id,
$dic_gazplit_id,$dic_phone_id,
$dic_window_type_id,$dic_pstate_id,$dic_dfloor_id,
$dic_internet_id,$dic_park_id,$dic_secur_id,
$dic_door_id,$dic_lift_id,$full_cost,$is_torg,$task_shema_id);

а до конца самой функции я тогда даже недоскроллил)

dehrom, 18 Сентября 2015

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

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

+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

class DBUSLIB_EXPORT DBusLib : public QObject
{
    Q_OBJECT
public:
    DBusLib(QObject* parent = 0);
    void requestId(const QString &req, const int &reqTimeOut);

    QByteArray replyData;
    QString name;
    MyPair reqStruct;  // id, partsCount
    QVector<QByteArray> vecFile;
    int partsCount; //кол-во частей
    int parts; //кол-во пришедших
    QTimer *timer = new QTimer(this);

signals:
    void doneSignal(QByteArray& fileData);

public slots:
    void requestMap(const MyPair &dbm);
    void partsFile(const MyPair &part);
    void requestFail(const QDBusError& error);
    void timeIsOver();

};

Студент принес говна. Паблик морозов, ценные комментарии и все такое.
Вот только мне любопытна 14 строчка. C++11 не включен.
Конпелятор mingw492 выбрасывает ворнинг
D:\projects\test_dbus\client_v2\client_l ib\dbuslib.h:35: предупреждение: non-static data member initializers only available with -std=c++11 or -std=gnu++11
QTimer *timer = new QTimer(this);
^
Но при этом все компилится и работает. Значит, 03 стандарт разрешает такое?

scp, 17 Сентября 2015

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

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

+2

1

https://github.com/imarketua/dump

Поработал на одном проекте, не оплатили. Предыстория там же, в ридми. Код знатный, подходит по тематике данного сайта, смотрим по ссылке)

HiNeX, 16 Сентября 2015

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

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

+11

1
2
3
4

for ($i = 0; $i < 11; $i++) {
    if ($i < 5) continue;
    //остальной код
}

первые 5 не нужны

kropotor, 16 Сентября 2015

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

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

+2

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

class Order_OrderProcessorBuilder
{
    /**
     * @param $processorName
     * @return Order_Abstract_OrderAbstract
     */
    private static function createOrderProcessor($processorName)
    {
        $processorClassName = 'Order_Types_'.ucfirst($processorName);
        return new $processorClassName;
    }

    /**
     * @param $builderName
     * @param Order_Abstract_OrderAbstract $orderProcessor
     * @return Order_Abstract_BuilderAbstract
     */
    private static function createOrderBuilder($builderName, Order_Abstract_OrderAbstract $orderProcessor)
    {
        $builderClassName = 'Order_Builders_'.ucfirst($builderName);
        return new $builderClassName($orderProcessor);
    }

    /**
     * @param Order_Abstract_BuilderAbstract $orderBuilder
     * @return Order_Director
     */
    private static function createDirectorClass(Order_Abstract_BuilderAbstract $orderBuilder)
    {
        return new Order_Director($orderBuilder);
    }

    /**
     * @param $orderTypeName
     * @return Order_Abstract_OrderAbstract
     */
    public static function build($orderTypeName)
    {
        /** @var Order_Abstract_OrderAbstract $orderProcessor */
        $orderProcessor = self::createOrderProcessor($orderTypeName);
        /** @var Order_Abstract_BuilderAbstract $orderBuilder */
        $orderBuilder = self::createOrderBuilder($orderTypeName, $orderProcessor);
        $director = self::createDirectorClass($orderBuilder);

        $director->construct();
        return $orderBuilder->getResult();
    }
}

Вот такая у нас есть прослойка, которая работает с паттероном "строитель", честно говоря недавно понял зачем она нужна, не знаю.. Как вам решение? Код не мой

proweber1, 16 Сентября 2015

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

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

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