1. Лучший говнокод

    В номинации:
    За время:
  2. PHP / Говнокод #19019

    +2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    class/View.php
    
    <?php
    
    class View {
    
    ...
    
      public function compile() {
        $view_src = file_get_contents(VIEW_SRC_PATH . $this->full_name . EXT);
        // echo
        $view_src = preg_replace("/\{\{(\w+)\}\}/", "<?php echo \$$1; ?>", $view_src);
        $view_src = preg_replace("/\{\{(\w+)\|(\w+)\}\}/", "<?php echo \$$1['$2']; ?>", $view_src);
        $view_src = preg_replace("/\{\{(\w+)\.(\w+)\}\}/", "<?php echo \$$1->$2; ?>", $view_src);
        // foreach
        $view_src = preg_replace("/<!--each\s+(\w+)\s+in\s+(\w+)-->/", "<?php foreach(\$$2 as \$$1): ?>", $view_src);
        $view_src = preg_replace("/<!--each\s+(\w+)\s+in\s+(\w+)\|(\w+)-->/", "<?php foreach(\$$2['$3'] as \$$1): ?>", $view_src);
        $view_src = preg_replace("/<!--each\s+(\w+)\s+in\s+(\w+)\.(\w+)-->/", "<?php foreach(\$$2->$3 as \$$1): ?>", $view_src);
        $view_src = preg_replace("/<!--each\s+(\w+)\s+(\w+)\s+in\s+(\w+)\.(\w+)-->/", "<?php foreach(\$$3->$4 as \$$1 => \$$2): ?>", $view_src);
        $view_src = preg_replace("/<!--each\s+(\w+)\s+(\w+)\s+in\s+(\w+)-->/", "<?php foreach(\$$3 as \$$1 => \$$2): ?>", $view_src);
        $view_src = preg_replace("/<!--each-->/", "<?php endforeach; ?>", $view_src);
        // switch
        $view_src = preg_replace("/<!--select\s+(\w+)\.(\w+)-->\s*<!--when\s+(.+)-->/", "<?php switch(\$$1->$2): case $3: ?>", $view_src);
        $view_src = preg_replace("/<!--when\s+(.+)-->/", "<?php break; ?><?php case $1: ?>", $view_src);
        $view_src = preg_replace("/<!--otherwise-->/", "<?php break; ?><?php default: ?>", $view_src);
        $view_src = preg_replace("/<!--select-->/", "<?php endswitch; ?>", $view_src);
        if (!file_exists(VIEW_PATH . $this->path)) mkdir(VIEW_PATH . $this->path, 0777, true);
        $fp = fopen(VIEW_PATH . $this->full_name . EXT, 'w');
        fwrite($fp, $view_src);
        fclose($fp);
      }
    
    ...
    
    ?>
    
    view-src/ShadeType.php
    
    {{type.name}}
    <!--each k d in type.diameters-->
    <h1>{{k}}</h1>
      <!--each s in d-->
      <div>
        {{s.bottom}} × {{s.top}} × {{s.height}}
        <div>
          <!--select s.mode-->
          <!--when 'u'-->(на ушках)
          <!--when 'p'-->(только подвес)
          <!--otherwise--> 
          <!--select-->
        </div>
      </div>
      <!--each-->
    <!--each-->
    
    view/ShadeType.php
    
    <?php echo $type->name; ?>
    <?php foreach($type->diameters as $k => $d): ?>
    <h1><?php echo $k; ?></h1>
      <?php foreach($d as $s): ?>
      <div>
        <?php echo $s->bottom; ?> × <?php echo $s->top; ?> × <?php echo $s->height; ?>
        <div>
          <?php switch($s->mode): case 'u': ?>(на ушках)
          <?php break; ?><?php case 'p': ?>(только подвес)
          <?php break; ?><?php default: ?> 
          <?php endswitch; ?>
        </div>
      </div>
      <?php endforeach; ?>
    <?php endforeach; ?>

    На хостинге PHP 5.2, а старые версии фреймворков не хочется. Решил запилить свой.

    kgm-rj, 13 Ноября 2015

    Комментарии (149)
  3. Куча / Говнокод #19011

    +2

    1. 1
    Как избавиться от говнокода?

    Вот есть большой проект в большой корпорации. Ты приходишь и видишь, что архитектура говно, комментариев нет, стиль кода различается в разных частях одного файла, да и в остальных мелочах полный говнокод.
    С другой стороны, нужно добавить много разных фич и пофиксить баги, все программисты и ты тоже активно заняты этим, вы приделываете новые костыли и заплатки.

    Как с этим бороться? Убеждать менеджера, что вам нужно несколько недель, а то и месяцев на полный рефакторинг всего? Заниматься этим в свободное время? Забить и говнокодить дальше? Уйти в другое место (не факт ведь, что там будет лучше)?

    imihajlov, 12 Ноября 2015

    Комментарии (113)
  4. PHP / Говнокод #19008

    +2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    public function generateRequestXml()
    {
        echo "Start generate request xml\n";
        $dt = date("c");
        $output = "";
        $output .= "<?xml version=\"1.0\" encoding=\"utf-8\"?>\n";
        $output .= "\t<request>\n";
        $output .= "\t<requestTime>" . $dt . "</requestTime>\n";
        $output .= "\t<operatorName>ИМЯ</operatorName>\n";
        $output .= "\t<inn>98989898989</inn>\n";
        $output .= "\t<ogrn>998545454545</ogrn>\n";
        $output .= "\t<email>[email protected]</email>\n";
        $output .= "</request>";
    
        if (!file_put_contents(self::REQUEST_FILE, $output))
            return false;
    }

    SimpleXMLElement? Не, не слышал

    podvzbzdnul, 12 Ноября 2015

    Комментарии (13)
  5. Куча / Говнокод #19004

    +2

    1. 1
    2. 2
    -- Hehehe
    typeGroups ll = (\(t, tt) -> printf "\n~~~~~~~~~~~~~~~\n%s\n~~~~~~~~~~~~~~~\n%s" t (show tt))  >>= (nub . map (_measInfoId &&&(snd .unzip . M.toList . _measTypes)) . (>>= _measInfo) . (>>= _measData)) $ (concat `fmap` mapM (\file -> runX $ readXml file >>> parseFile)) ll

    скрипт, высранный в спешке для обработки данных в одной задаче перерос в утилиту

    CHayT, 10 Ноября 2015

    Комментарии (13)
  6. PHP / Говнокод #18994

    +2

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    <?
    $key = crc32("release") ^ 0x817990;
    $header_bg = '#' . substr(base_convert(crc32($DBName) ^ $key, 10, 16), 0, 6);
    ?>
    <div id="top_header" style="background:<?= $header_bg ?>">

    На доменах с похожими названиями живут продакшн и версии для разработки. Заказчик путается при тестировании, а потом разрабы удивляются тестовым данным в рабочей базе.
    Вот так была реализована цветовая дифференциация штанов сайтов в зависимости от имени БД. Подобрать им имя базы, которое даст коллизию, что ли...

    clauclauclau, 09 Ноября 2015

    Комментарии (1)
  7. JavaScript / Говнокод #18977

    +2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    (function ($) {
        Drupal.behaviors.kdating = {
            attach: function (context, setting) {
    
                var links = $('#field-slideshow-1-pager').find('a');
                links.colorbox({ rel: 'gal' });
    
                $.each(links, function (index, value) {
                    var val = $(value);
                    var href = val.find('img').attr('src');
                    var path = href.split('/'),
                        newPath = path[0] + '//'
                            + path[2] + '/'
                            + path[3] + '/'
                            + path[4] + '/'
                            + path[5] + '/'
                            + path[9];
    
                    val.attr('href', newPath);
                });
    
            }
        };
    })(jQuery);

    Парсим ссылки вместе

    kdating, 06 Ноября 2015

    Комментарии (0)
  8. PHP / Говнокод #18974

    +2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    if ($this->getShowTemplateHints()) {
        echo <<<HTML
    <div style="position:relative; border:1px dotted red; margin:6px 2px; padding:18px 2px 2px 2px; zoom:1;">
    <div style="position:absolute; left:0; top:0; padding:2px 5px; background:red; color:white; font:normal 11px Arial;
    text-align:left !important; z-index:998;" onmouseover="this.style.zIndex='999'"
    onmouseout="this.style.zIndex='998'" title="{$fileName}">{$fileName}</div>
    HTML;
        if (self::$_showTemplateHintsBlocks) {
            $thisClass = get_class($this);
            echo <<<HTML
    <div style="position:absolute; right:0; top:0; padding:2px 5px; background:red; color:blue; font:normal 11px Arial;
    text-align:left !important; z-index:998;" onmouseover="this.style.zIndex='999'" onmouseout="this.style.zIndex='998'"
    title="{$thisClass}">{$thisClass}</div>
    HTML;
        }
    }

    Правильная реализация hover эффекта. Яркости для картины добавляет то, что это все написано в блоке MVC модели и понятное дело, на живом проекте.

    bayanets, 05 Ноября 2015

    Комментарии (16)
  9. PHP / Говнокод #18973

    +2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    if (count($not_enough_rights) == 1 and !empty($not_enough_rights['user_extra'])) {
    					// если нужен доступ только к user_extra, то дополнительного доступа не надо, но скрываем сами права user_extra
    
    				} else {
    
    					if (count(array_merge($user_rights, $EDIT_ADMIN_GROUPS)) != count($EDIT_ADMIN_GROUPS)) {
    						$GLOBALS['VALID_GROUPS'] = array('root');
    					} else {
    						$GLOBALS['VALID_GROUPS'] = array('superadmin');
    					}
    
    				}

    Если $user_rights не пустой, то ты просто админ, если пустой, то суперадмин. Не проставил в БД права - суперадмин!

    dimka3210, 05 Ноября 2015

    Комментарии (1)
  10. JavaScript / Говнокод #18962

    +2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    var data = [];
    if(!Ext.isEmpty(val[i][Object.keys(value[i])[0]])) {
     data['duration'] = val[i][Object.keys(value[i])[0]]['duration'];
     data['count'] = val[i][Object.keys(value[i])[0]]['count'];
     data['acd'] = val[i][Object.keys(value[i])[0]]['acd'];
     data['asr'] = val[i][Object.keys(value[i])[0]]['asr'];
    } else {
     data['duration'] = '';
     data['count'] = '';
     data['acd'] = '';
     data['asr'] = '';
    }
    i++;
    if(i>=val.length) {
     i = 0;
    }
    return tpl.apply(data);

    user-dob, 03 Ноября 2015

    Комментарии (12)
  11. C# / Говнокод #18949

    +2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    public int RowSpan {
                get {
                    int p = HtmlText.IndexOf("rowspan");
                    if (p > -1) {
                        int index = HtmlText.IndexOf(" ", p + 8);
                        int index1 = HtmlText.IndexOf(">", p + 8);
                        int index2 = HtmlText.IndexOf("/", p + 8);
                        if ((index > index1) && (index1 != -1))
                            index = index1;
                        if ((index > index2) && (index2 != -1))
                            index = index2;
                        string s = HtmlText.Substring(p + 8, index - p - 8);
                        return Convert.ToInt16(s);
                    } else
                        return -1;
                }
                set {
                    if (value == 0) {
                        HtmlText = HtmlText.Replace("rowspan", string.Empty);
                    }
                }
            }
            public int ColSpan {
                get {
                    int p = HtmlText.IndexOf("colspan");
                    if (p > -1) {
                        int index = HtmlText.IndexOf(" ", p + 8);
                        int index1 = HtmlText.IndexOf(">", p + 8);
                        int index2 = HtmlText.IndexOf("/", p + 8);
                        if ((index > index1) && (index1 != -1))
                            index = index1;
                        if ((index > index2) && (index2 != -1))
                            index = index2;
                        string s = HtmlText.Substring(p + 8, index - p - 8);
                        return Convert.ToInt16(s);
                    } else
                        return -1;
                }
            }

    а тут регулярки он не осилил

    Lokich, 02 Ноября 2015

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