"PHP" / Говнокод #6979 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
$value = $this->model->getById(DataUtil::fromGET("id"));
foreach($this->model->get(PaginationUtil::getStart(),PaginationUtil::$countOnPage) as $value){
   return NewsUtils::createNewsLayout_ND($value);
}

В проект был взят стажер, и начал появляться такой вот индусский код.

Запостил: Kos Kos, (Updated )

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

  • ava code_master # Ссылка на оригинал 0

    Пользуясь случаем спрошу. После таких вот выходов из цикла, много мусора остается? Понимаю суть сего говнокода не в мусоре а в красоте, читаемости и прочей по*бне, но всё же?
    Ответить

    • ava Vasiliy # Ссылка на оригинал 0 ⇈

      после таких циклов остается 1 вопрос "НАХУЯ ???"
      З.Ы. Минусует кто автор ?? Или кто то считает что так и надо писать ???
      Ответить

    • ava Int # Ссылка на оригинал 0 ⇈

      Не думаю, что этот скрипт долго провисит в памяти. В php мусор вообще не очень кому надоедает, большинство скриптов через секунду уже завершаются, и память освобождаетя.
      Ответить

      • ava Vasiliy # Ссылка на оригинал 0 ⇈

        нет. Может не освободиться вернее не сразу. При частых обращениях память может не успевать освобождаться.
        Ответить

  • ava yasenhui # Ссылка на оригинал 0

    Код не индусский, скорее наооборот.
    Количество строк меньше чем должно было быть, а не больше.

    Не вижу ничего страшно криминального, кроме длинны строк.
    return разве что. Но это вряд ли достойно данного сайта. Не говнокод, так, корявость.
    Ответить

    • ava Kos # Ссылка на оригинал 0 ⇈

      Суть в том что функция getById берёт только один элемент, и его засовывают в цикл (+2 строчки) вместо того что бы сразу вернуть результат. Потом пытаются пагинацию прикрутить (PaginationUtil::getStart(),PaginationUt il::$countOnPage) что создаёт имитацию деятельности.
      PS говнокод не обязан падать, он может работать, и даже работать правильно, но при этом ужасно выглядит и тянет память.
      Ответить

  • ava Int # Ссылка на оригинал 0

    Я вообще не понял, что оно делает. Куча каких-то неизвестных классов.
    Ответить

    • ava wvxvw # Ссылка на оригинал 0 ⇈

      На сколько я понял - берет первый элемент массива, и его и возвращает - зачем для этого цикл - не понятно, да и всю функцию можно было бы не писать, если задача сводится к получению первого элемента, и return - не ошибка, а так и задумано.
      Ответить

  • ava guest # 0

    еG(I:F?S(Z?T V)Z,J,I)R.Z G(ERE"RJ:T"G)G:T,O)Q,XR(Y!H!Z C?Z(N)U$Z,J$G:SO(G$P:ZYN!G(G"K"B?R,B)M!D$S"P(Q,D"N?B:MQ$F)Y.B,A!I:F$X"D?M:Q,ZS?I$O.T"O,E:D G:A W:S.D$C$B B"A,F(O$X?X,W,W"R$U,W"H"N:U,V?G(C!S.V)S,R.I:V?F(DZ.MU(SE"D)T B$C(Q W,Z,N!L(X C.D N,OV?R"U"A)L(E"Q,R,N(G)Y$WF,Y)N O!L)E Q!LJ(P.S,U,C?D)J"N Z"C)Y"E.F?WW.D(Q N:S:MQ"A!L!F!Y(R:E,E.E"F?O.G G.P JR?L G"F.C"S)D"J,G$X(US"X?O:R:IK.O)V:E(D.G$N?E,Q:Z D?D?V$M.DSHI"O(TWG(Y:S.K!F.S?A!C"O?Y Y.U?MD(M(N,S)Z N!X"V?S:E,O$I)B:D,H:V)N?I.P"N)V(M,H!DI?M(S$U$P.SZ:K:Q"W(Y MK"S$J K)RK.W:X:SV$I:N,G!A"L.VY,LG(C:R V RA.XD?U$P:R.R(RA$K:A.A.E(U?M?S,A:H?E!Z$Z"GЖTZTEBRKSKAMNCOSXZBXLFTUOSXEIBFPEHDYQOPSUMAXLXFJT
    Ответить

Добавить комментарий

Из-за тебя ушел bormand, guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓