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

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

+46

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
78
79
80
81
82
83
84

//В хедере
    QVector<double> stateStack;
//Сохраняем
    stateStack.push_back((double)renderInclusionsAs3DMeshSpheres);
    stateStack.push_back((double)transparencyForClosestSidePolies);
    stateStack.push_back((double)renderFEMGrid);
    stateStack.push_back((double)NEST_results_detail_per_unit);
    stateStack.push_back((double)sourceLineActive);
    stateStack.push_back((double)renderScale);
    stateStack.push_back((double)c2DZoom1To1);
    stateStack.push_back((double)render2DInclusions);
    stateStack.push_back((double)render3DInclusions);
    stateStack.push_back((double)renderNESTResults);
    stateStack.push_back((double)renderFEAResults);
    stateStack.push_back((double)NEST_anumTime);
    stateStack.push_back((double)isAnimRun);
    stateStack.push_back((double)NEST_timeIterToRender);
    stateStack.push_back((double)NEST_RenderResultsIn3D);
    stateStack.push_back((double)NEST_renderTestPoints);
    stateStack.push_back((double)nestNewResults);
    stateStack.push_back((double)fibers_renderFormedRegions);
    stateStack.push_back((double)sourceLineStarted);
    stateStack.push_back((double)renderSourceLine);
    stateStack.push_back((double)heatQuadsModifyMode);
    stateStack.push_back((double)renderTopHeats);
    stateStack.push_back((double)sliceModifyMode);
    stateStack.push_back((double)useSlicePlane);
    stateStack.push_back((double)showSlicedPart);
    stateStack.push_back((double)cRenderMenuOpended);
    stateStack.push_back((double)cForbidRenderResultsOverLayers);
    stateStack.push_back((double)renderCoordLines);
    stateStack.push_back((double)renderDimensions);
    stateStack.push_back((double)renderLimits);
    stateStack.push_back((double)cResTransparency);
    stateStack.push_back((double)cRenderResultsOverLayers);
    stateStack.push_back((double)cActiveLayer);
    stateStack.push_back((double)renderMiniMapAtTop);
    stateStack.push_back((double)subCompositeMode);
    stateStack.push_back((double)subCompositeMapRendering);
    stateStack.push_back((double)fibersRenderable);
    stateStack.push_back((double)renderFibersAsMesh);
    stateStack.push_back((double)droplets_detailedSpheres);
    stateStack.push_back((double)cViewType);
//Загружаем
 renderInclusionsAs3DMeshSpheres   =(bool)stateStack[0];
    transparencyForClosestSidePolies=        stateStack[1];
    renderFEMGrid=(bool)                     stateStack[2];
    NEST_results_detail_per_unit=            stateStack[3];
    sourceLineActive=(bool)                  stateStack[4];
    renderScale=(bool)                       stateStack[5];
    c2DZoom1To1=(bool)                       stateStack[6];
    render2DInclusions=(bool)                stateStack[7];
    render3DInclusions=(bool)                stateStack[8];
    renderNESTResults=(bool)                 stateStack[9];
    renderFEAResults=(bool)                  stateStack[10];
    NEST_anumTime=                           stateStack[11];
    isAnimRun=(bool)                         stateStack[12];
    NEST_timeIterToRender=                   stateStack[13];
    NEST_RenderResultsIn3D=(bool)            stateStack[14];
    NEST_renderTestPoints=(bool)             stateStack[15];
    nestNewResults=(bool)                    stateStack[16];
    fibers_renderFormedRegions=(bool)        stateStack[17];
    sourceLineStarted=(bool)                 stateStack[18];
    renderSourceLine=(bool)                  stateStack[19];
    heatQuadsModifyMode=(bool)               stateStack[20];
    renderTopHeats=(bool)                    stateStack[21];
    sliceModifyMode=(bool)                   stateStack[22];
    useSlicePlane=(bool)                     stateStack[23];
    showSlicedPart=(bool)                    stateStack[24];
    cRenderMenuOpended=(bool)                stateStack[25];
    cForbidRenderResultsOverLayers=(bool)    stateStack[26];
    renderCoordLines=                        stateStack[27];
    renderDimensions=(bool)                  stateStack[28];
    renderLimits=                            stateStack[29];
    cResTransparency=                        stateStack[30];
    cRenderResultsOverLayers=(bool)          stateStack[31];
    cActiveLayer=                            stateStack[32];
    renderMiniMapAtTop       =(bool)         stateStack[33];
    subCompositeMode         =(bool)         stateStack[34];
    subCompositeMapRendering =(bool)         stateStack[35];
    fibersRenderable=(bool)                  stateStack[36];
    renderFibersAsMesh=(bool)                stateStack[37];
    droplets_detailedSpheres=                stateStack[38];
    cViewType=                               stateStack[39];

Структуры для педиков.

Abbath, 25 Февраля 2014

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

Ruby / Говнокод #14953

−131

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

def sum_all_orders
    sum = 0
    Order.where(user_id: self.id).each do |order|
      sum = sum + order.total_sum
    end
    sum
  end

  # вместо
  def sum_all_orders
     orders.sum(&:total_sum)
  end

В коде магазина, таких ляпов полон проект. Задрался dd жмакать.
Так сложно API active_support почитать???

alexes_dev, 25 Февраля 2014

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

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

+151

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

$s = $this->getAdapter()->select()->from(array('b' => $this->info('name'), array(
   'text',
   'link',
   'image',
            'show_register',


  )))->joinInner(array('s' => 'banner_size'), 's.id=b.size_id', array('x', 'y'))
   ->where('b.show_it = ?', 1)
   ->where('s.x = ?', $sizeX)
   ->where('s.y = ?', $sizeY)
   ->limit(1)
   ->order('RAND()');


        if (!empty($uid)){
            $s->where('(b.show_register= ?, 1)' OR '(b.show_register= ?, 0)');
        } else {
            $s->where('(b.show_register= ?, -1)' OR '(b.show_register= ?, 0)');

        }
      echo $s-> __toString();
      //  var_dump(!empty($uid));
  return $this->getAdapter()->fetchRow($s);

мега крутое условие
$s->where('(b.show_register= ?, 1)' OR '(b.show_register= ?, 0)');

kolyabres, 25 Февраля 2014

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

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

+131

01
02
03
04
05
06
07
08
09
10

// Сериализуем
var outStream = new MemoryStream();
serializer.Serialize(outStream, storedSplitInfo);
string output = outStream.ToArray().Aggregate(string.Empty, (current, simbol) => current + (char) simbol);

...

// Десериализуем
var serializer = new XmlSerializer(typeof(StoredSplitInfo));
return (StoredSplitInfo)serializer.Deserialize(new MemoryStream(docNumber.SplitInfo.Select(c => (byte)c).ToArray()));

Вот таким изощренным способом мы умеем сериализовывать объекты в XML

Govnokoder555, 25 Февраля 2014

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

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

+37

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

long* Service::qByteToLongArray(QByteArray qba, long r)
{
    long* larr = new long[r];

    char *carr;
    carr = (char*)larr;

    int len = r * sizeof(long);//length in bytes

    for(int i=0; i < len; i++)
        carr[i] = qba.at(i);

    return larr;
}

QByteArray Service::longToQByteArray(long **larr2, long r, long c)
{
    QByteArray qba;// = new QByteArray();

    char **carr2;
    carr2 = (char**)larr2;

    int rlen = c * sizeof(long);//length of row in bytes

    for(int i=0; i < r; i++){
        qba.append(carr2[i], rlen);//add next row of matrix to the QByteArray
    }

    return qba;
}

Нашли это, только когда собрали под x64.

Abbath, 25 Февраля 2014

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

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

+152

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

...
## $config['components'] - массив компонентов(>25) для Yii
foreach ($config['components'] as $name => $component) {
    if ($name != 'db') {
        unset($config['components'][$name]);
    }
}
...

# потом был баг фикс 
foreach ($config['components'] as $name => $component) {
    if ($name != 'db' && $name != 'db_stats') {
        unset($config['components'][$name]);
    }
}

Сделал один из наших программистов, потом другой добавил свое, и уже я , с утра, в безсознательном состоянии, хотел было добавить ещё условие " && $name != 'format'" , но увидев результат, в голове бзынкнуло - WTF I AM DOING!!!
cделал так
$config['components'] = array_intersect_key($config['components'], array('db'=>1, 'db_stats'=>1, 'format' => 1));

xakki, 25 Февраля 2014

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

SQL / Говнокод #14941

−122

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

select CMPNPFCOD into c_CMPCOD from SPECPART.CHSCMPALL where INSNMB = l_INSNMB fetch first 1 rows only;
select CMPPRFCOD into c_CMPPRF from SPECPART.CHSCMPALL where INSNMB = l_INSNMB fetch first 1 rows only;
select CMPDCL_YER into c_CMPDCL_YER from SPECPART.CHSCMPALL where INSNMB = l_INSNMB fetch first 1 rows only;
select DPTCOD into c_DPTCOD_CHS from SPECPART.CHSCMPALL where INSNMB = l_INSNMB fetch first 1 rows only;
select DCINMB into c_DCINMB_CHS from SPECPART.CHSCMPALL where INSNMB = l_INSNMB fetch first 1 rows only;
select DCIDCK into c_DCIDCK_CHS from SPECPART.CHSCMPALL where INSNMB = l_INSNMB fetch first 1 rows only;
select DCICPY into c_DCICPY_CHS from SPECPART.CHSCMPALL where INSNMB = l_INSNMB fetch first 1 rows only;
select DPTCOD into c_DPTCOD_AGR from SPECPART.AGRNPFALL where INSNMB = l_INSNMB fetch first 1 rows only;
select DCINMB into c_DCINMB_AGR from SPECPART.AGRNPFALL where INSNMB = l_INSNMB fetch first 1 rows only;
select DCIDCK into c_DCIDCK_AGR from SPECPART.AGRNPFALL where INSNMB = l_INSNMB fetch first 1 rows only;
select DCICPY into c_DCICPY_AGR from SPECPART.AGRNPFALL where INSNMB = l_INSNMB fetch first 1 rows only;
select PROCDATE into c_PROCDATE from SPECPART.CHSCMPALL where INSNMB = l_INSNMB fetch first 1 rows only;

Глумление над DB2

rush, 25 Февраля 2014

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

SQL / Говнокод #14938

−163

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

CREATE PROCEDURE DEPTORS_EURO AS	-- Но сначала отключу ТРИГГЕРА 
   ALTER TABLE DEPTORS_REFERENCE DISABLE  TRIGGER Tr_for_DEPTORS_REFERENCE
   ALTER TABLE DEPTORS_DOC_EURO DISABLE  TRIGGER Tr_for_DEPTORS_DOC_EURO
   ALTER TABLE DEPTORS_Remander DISABLE  TRIGGER Tr_for_DEPTORS_REMANDER
   ALTER TABLE DEPTORS_Payment DISABLE  TRIGGER Tr_for_DEPTORS_Payment -- Создаю документ DEPTORS_DOC_EURO_old
                     Она будет нужна для поиска оплативших долги АК и появившихся новых */
 IF EXISTS(SELECT name  -- Удаляю, если существует в БД
	  FROM 	 sysobjects WHERE  name = N'DEPTORS_DOC_EURO_old' AND 	 type = 'U') DROP TABLE DEPTORS_DOC_EURO_old
            SELECT * INTO DEPTORS_DOC_EURO_old -- Это DEPTORS за предыдущий месяц
            FROM DEPTORS_DOC_EURO     -- После копирования данных - уничтожим DEPTORS_DOC_EURO
DELETE  DEPTORS_DOC_EURO -- Т.О. по FK-PK обнуляютя таблицы  
DELETE FROM STATEMENT_UKRSTA WHERE mvt_date = 
(SELECT DISTINCT LATEST__DATE -- выбираю дату 
FROM STATEMENT_UKRSTA , CALENDAR_BILL_DATE WHERE
 DATEPART(mm,FLIGHT_PERIOD) = DATEPART(mm, (SELECT DATEADD(mm,-1, MAX(stac_date)) FROM STATEMENT_UKRSTA WHERE type = 'FB'))  and
 DATEPART(yy,FLIGHT_PERIOD) = DATEPART(yy, (SELECT DATEADD(mm,-1, MAX(stac_date)) FROM STATEMENT_UKRSTA WHERE type = 'FB'))  and
 type = 'FB') -- Удаляю из таблицы STATEMENT_UKRSTA  все АК, которые не имеют долгов
DELETE FROM STATEMENT_UKRSTA WHERE corr IN (SELECT corr FROM STATEMENT_UKRSTA a GROUP BY a.corr HAVING Sum(a.saldo) <= 0 ) 
-- Создаю псевдовременную таблицу по выборке DEPTORS_REFERENCE_Vrem
IF EXISTS(SELECT name  -- Удаляю, если существует в БД
	  FROM 	 sysobjects WHERE  name = N'DEPTORS_REFERENCE_Vrem' AND type = 'U') DROP TABLE DEPTORS_REFERENCE_Vrem  
SELECT  corr, type, SUBSTRING(reference,1,6) AS Account, Sum(saldo) AS Saldo INTO DEPTORS_REFERENCE_Vrem FROM STATEMENT_UKRSTA GROUP BY corr, type, reference HAVING SUM(saldo) > 0  
-- Т.К. FK-PK то сначала внесем User_Ref всех должников в DEPTORS_DOC_EURO
IF EXISTS(SELECT name  -- Удаляю, если существует в БД
	  FROM 	 sysobjects WHERE  name = N'USER_REF' AND 	 type = 'U')
    DROP TABLE USER_REF  
SELECT DISTINCT corr INTO USER_REF FROM DEPTORS_REFERENCE_Vrem 
INSERT INTO DEPTORS_DOC_EURO SELECT User_Ref=a.corr,DEBT_AMOUNT=Null,Name= Null,Country=Null,Code_ICAO=Null,Final_Date=Null,Period_of_Dept=Null,Saldo_D_Facte=0,Saldo_D_Ure=0 FROM USER_REF a  
DROP TABLE USER_REF -- Заполняю таблицу DEPTORS_REFERENCE (User_Ref уже есть - поэтому позволит внести)
INSERT INTO DEPTORS_REFERENCE SELECT User_Ref=a.corr,reference=a.Account,type=a.type,Old_Saldo=a.Saldo,Saldo_D_Facte =0,Saldo_D_Ure=0 -- первично  
 FROM DEPTORS_REFERENCE_Vrem a   -- Удалим псевдовременную таблицу DEPTORS_REFERENCE_Vrem
DROP TABLE  DEPTORS_REFERENCE_Vrem -- Корректирую счета (Выставлено - Оплачено)
IF EXISTS(SELECT name  -- Удаляю, если существует в БД
	  FROM 	 sysobjects WHERE  name = N'Vrem_Stat_Negativ_Saldo' AND 	 type = 'U') DROP TABLE Vrem_Stat_Negativ_Saldo  
SELECT  corr, type, SUBSTRING(reference,1,6) AS Account, Sum(saldo) AS Saldo INTO Vrem_Stat_Negativ_Saldo FROM STATEMENT_UKRSTA GROUP BY corr, type, reference HAVING SUM(saldo) < 0  -- Сравниваю две таблицы (DEPTORS_REFERENCE и Vrem_Stat_Negativ_Saldo)
UPDATE a SET Old_Saldo = a.Old_Saldo + b.Saldo FROM DEPTORS_REFERENCE a INNER JOIN Vrem_Stat_Negativ_Saldo b ON a.User_Ref = b.corr and a.reference = b.Account
DROP TABLE Vrem_Stat_Negativ_Saldo      -- Создаю новый документ DEPTORS_DOC_EURO 
IF EXISTS(SELECT name  -- Удаляю, если существует в БД
	  FROM 	 sysobjects WHERE  name = N'Vrem_DEPTORS_DOC_EURO' AND 	 type = 'U')    DROP TABLE Vrem_DEPTORS_DOC_EURO
   SELECT User_Ref, SUM(Old_Saldo) AS Old_Saldo,SUM(Saldo_D_Facte) AS Saldo_D_Facte,SUM(Saldo_D_Ure)   AS Saldo_D_Ure
 INTO Vrem_DEPTORS_DOC_EURO FROM DEPTORS_REFERENCE GROUP BY User_Ref 
UPDATE b
SET 
  DEBT_AMOUNT  = a.Old_Saldo, Saldo_D_Facte = a.Old_Saldo, Saldo_D_Ure   = a.Old_Saldo  FROM Vrem_DEPTORS_DOC_EURO a INNER JOIN DEPTORS_DOC_EURO b ON a.User_Ref = b.User_Ref
  DROP TABLE  Vrem_DEPTORS_DOC_EURO
UPDATE a SET Name= b.corr_name  FROM DEPTORS_DOC_EURO a INNER JOIN STATEMENT_UKRSTA b 
           ON a.User_Ref = b.corr              
UPDATE a SET Code_ICAO = ICAO_CODE FROM DEPTORS_DOC_EURO a INNER JOIN AIRCOMPANYS b
     ON (a.User_Ref = b.CRCO_NUMBER) OR (a.User_Ref = b.CRCO3)
UPDATE a SET Country = c.FULL_NAME FROM DEPTORS_DOC_EURO a INNER JOIN AIRCOMPANYS b 
     ON (a.User_Ref = b.CRCO_NUMBER) OR (a.User_Ref = b.CRCO3)
                        INNER JOIN COUNTRYS c ON b.COUNTRY_ID = c.ID
 IF EXISTS(SELECT name  -- Удаляю, если существует в БД
	  FROM 	 sysobjects WHERE  name = N'DEPTORS_DOC_EURO_NewDolg' AND 	 type = 'U')
    DROP TABLE DEPTORS_DOC_EURO_NewDolg
               SELECT * -- Делаю выборку в таблицу   
               INTO DEPTORS_DOC_EURO_NewDolg FROM DEPTORS_DOC_EURO
WHERE User_Ref Not IN (SELECT User_Ref FROM DEPTORS_DOC_EURO_old)
IF EXISTS(SELECT name  -- Удаляю, если существует в БД
	  FROM 	 sysobjects WHERE  name = N'DEPTORS_DOC_EURO_OplatDolg' AND 	 type = 'U')
    DROP TABLE DEPTORS_DOC_EURO_OplatDolg
               SELECT * -- Делаю выборку в таблицу   
               INTO DEPTORS_DOC_EURO_OplatDolg FROM DEPTORS_DOC_EURO_old
WHERE User_Ref Not IN (SELECT User_Ref FROM DEPTORS_DOC_EURO)
   ALTER TABLE DEPTORS_REFERENCE ENABLE  TRIGGER Tr_for_DEPTORS_REFERENCE
   ALTER TABLE DEPTORS_DOC_EURO ENABLE  TRIGGER Tr_for_DEPTORS_DOC_EURO
   ALTER TABLE DEPTORS_Remander ENABLE  TRIGGER Tr_for_DEPTORS_REMANDER
   ALTER TABLE DEPTORS_Payment ENABLE  TRIGGER Tr_for_DEPTORS_Payment

курсор головного мозга 2.2

bahamot, 24 Февраля 2014

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

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

+142

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

static OSStatus
SSLVerifySignedServerKeyExchange(SSLContext *ctx, bool isRsa, SSLBuffer signedParams,
                                 uint8_t *signature, UInt16 signatureLen)
{
    OSStatus        err;
    ...

    if ((err = SSLHashSHA1.update(&hashCtx, &serverRandom)) != 0)
        goto fail;
    if ((err = SSLHashSHA1.update(&hashCtx, &signedParams)) != 0)
        goto fail;
        goto fail;
    if ((err = SSLHashSHA1.final(&hashCtx, &hashOut)) != 0)
        goto fail;
    ...

Говно с яблочным привкусом.
http://habrahabr.ru/post/213525/

P.S.: Не уверен Си это или плюсы.

Vindicar, 24 Февраля 2014

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

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

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