1. PHP / Говнокод #9542

    +157

    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
    if(intval($data[0])>0){
      $massart=explode(" ",$data[1]);
      if($massart[0]){
        $infoarticle=mysql_query("SELECT `Message_ID`, `ExternalData` FROM `Message114` where `Article`='".mysql_escape_string($massart[0])."';");
        if(mysql_num_rows($infoarticle)){
          while($infoarticlerow=mysql_fetch_array($infoarticle)){ 
            $arrowitem["".$massart[0]]['name']=$data[1];
            $arrowitem["".$massart[0]]['id']=$infoarticlerow['Message_ID'];
            $arrowitem["".$massart[0]]['info']=$infoarticlerow['ExternalData'];
            for ($c=5; $c < $num; $c++) {
              if(intval($data[$c])>0){
                $arrowitem["".$massart[0]]['size'][$mssize[$c]]=$data[$c];    
              }
            }   
            if(floatval($arrowitem["".$massart[0]]['price'])==0){
              $arrowitem["".$massart[0]]['price']=floatval($data[3]);
            }else{
              if(floatval($arrowitem["".$massart[0]]['price'])>floatval($data[3])){
                $arrowitem["".$massart[0]]['price']=floatval($data[3]); 
              }
            }           
          } 
        }else{
          $notnetcat[]=$data[1];  
        }
      }
    }

    От нехватки рабочих рук отдали одну правку на фриланс. Рвусь между желанием всё переписать по-человечески и со спокойной совестью отчитаться перед заказчиком либо же забить (парню заплатили, блеать!), но потом краснеть за глюки этого чудо-кода, которые, жопой чую, обязательно возникнут.

    Запостил: telnet, 25 Февраля 2012

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

    • Печально, но практика показывает что глюки в таких местах возникают нечасто. :( Я тоже в своём проекте стараюсь такое давить где вижу, но постоянно где-нибудь появляется свежак, и каким-то чудом работает. Проблемы _изредка_ возникают на проде только буквально за неделю до следующего релиза, за которую потом срочно приходится это говно переписывать. :)
      Ответить
    • вычисли его по айпишнику
      Ответить
      • Да смысл, я и в аську ему написать могу, и на почту. Просто ситуация получилась, что он мне написал, что всё сделал, а я заработался, прошло два дня и дошли руки по-нормальному проверять его работу, только когда заказчик написал гневное письмо ("ну где?!!111") и надо было срочно отвечать. Проверил - вроде пашет код, так и отчитался.
        Ответить
    • Хочешь, что то сделать хорошо не доверяй фрилансерам, у них мозг работает по другому. Их оценка качества работы "много". Лучше найти коллегу в отпуске.
      Ответить
      • Не обобщайте, фрилансеры очень хорошие бывают, просто надо школоло фильтровать.
        Ответить
        • Вроде большая в сказки веришь. Хороший код от фрилансера сродни подарка от "Деда Мороза".
          Ответить
          • Я не верю, я работала и знаю.
            Ответить
          • Вася, я тоже фрилансером был... Старался всегда сделать по-нормальному всё... Но вот как раз из-за такого предвзятого мнения пришлось таки вступить в ряды аутсорсной команды, ...и заниматься точно тем же. :)
            И, может не поверишь, но в аутсорсной команде в серьёзной компании тоже находятся чудики, которые пишут вещи вроде if(empty($val)) $val = $val = str_replace(' ', '', $val);
            Ответить
            • http://www.dv-reclama.ru/download/2009/2009-09-02_newyone_04.jpg
              Ответить
            • Аутсорс мало чем отличается от фриланса, разве что народа больше... У меня начальство вообще разницы между этими двумя словами не видит.
              Ответить
              • >У меня начальство вообще разницы между этими двумя словами не видит.
                охеренный довод
                Ответить
            • Ну если был. Я 100% уверен что софт писался так
              1. Получение ТЗ
              2. Начали писать. (берем код с похожего пректа и вставляем в него костыли)
              3. Запустили если в 30% код делал то что от него нужно. можно сдавать.

              Как пишут я не фрилансер пишу код.
              1. Получение ТЗ
              2. Проектирование
              3. Написание кода (берем модули с другого похожего проекта если подходит оставляем если нет пишем с 0).
              4. Тестирование и оптимизация.
              5. Просмотр кода. Смотрим всей группой.
              6. сдача кода заказчику.

              И не нужно говорить что код который пишет фрилансер оптимизированный и потрачено хотя бы 1 день на его тестирование.
              Ответить
              • Как обычно проект попадает к фрилансеру.
                А давайте мы сэкономим!
                Дальше писать?

                Даже если желания сэкономить изначально не было при виде толп школьников готовых работать за еду оно появляется. Как раз вчера читала статью как не лопухнутся на ебее, я бы то же сказала про фрил - не ведитесь на "сделаю быстро дешево качественно", читайте рекомендации, ищите через знакомых, устраивайте минимальное собеседование на профпригодность...
                Ответить
                • > лопухнутся на ебее
                  прочитал на ебле покраснел.
                  Ответить
              • Гуд у вас в конторе это дело поставлено, конечно если не врешь )
                Ответить
                • Эх сколько наших полегло, что бы это стало так как есть. Особенно этап ТЗ. Любят некоторые товарищи "хотеть" или "вспоминать" тоже любят за неделю до сдачи мелочь какуюнибуть типа выгрузки данных в 1С. "Ну вы же инженеры, КАК вы могли подумать что Интерент магазу не нужна синхронизация товара с 1С, это ты как главный в группе должен был мне сказать когда Я показал вам ТЗ, что Иван Иванович а вы вот не указали необходимость реализации синхронизации с 1С, так что это ВАША вина, а Ваши просчеты пожалуйста отвечайте сами" Примерно дословный монолог одного из писателей ТЗ большого человека в нашей конторе, с нашим руководителем проектов.
                  Ответить
                  • И как вы все таки добились "что бы это стало так как есть".
                    Судя по фразе "много наших полегло", раз в день один из программистов с разбегу убивался об стену, пока ваши требования не были приняты )))
                    Ответить
                    • Да сейчас вроде все как должно быть. Прогеры не убивались просто уходили.
                      Ответить
                  • Побыв с другой стороны баррикад могу сказать что старший тоже виноват - не пропараноил с неадекватным начальством/заказчиком и не спросил.
                    Хотя бывают такие что на вопросы устраивают истерику, тут лучше сразу уходить)
                    Ответить
                    • Решается прописыванием в договор строки вида "за дополнительный функционал - дополнительные деньги и разработка проводится в течение дополнительного времени, в противном случае можете соснуть бочкуоспорить это в судебном порядке". Если что, они потом неустойку платить и будут.
                      Ответить
                      • Сукабля, а я сначала думал, что мой первый коммент зохавал сотона. Ддосят нас штоле.
                        Ответить
                      • Есть два вида подрядчиков - простоработники и профи.
                        Первые требуют полное тз, даже не пытаясь задуматься - а нужно ли это или то заказчику. Ходят злые на тупых заказчиков.

                        Вторые понимают что если бы заказчик знал точно что и как то сделал бы сам и что пришли за их знаниями и опытом. И нужна ли выгрузка, зачем вообще им магазин - спрашивают. Имеют исключительно адекватных клиентов.

                        И да, вторые зарабатывают значительно больше.
                        Ответить
                        • Что это за ТП? Кто ей за слово дал?
                          Ответить
                          • Люр дал, но вроде не за слово
                            Ответить
                            • За место для проживания.
                              ОЧЕРЕДНАЯ ТЕОРИЯ ЗАГОВОРА:Может он футбольный фанат и таким макаром решил бесплатно съездить на Евро-2012.
                              Ответить
                              • На него еще хрен билеты купишь насколько я знаю.
                                Оччень депрессивно отношусь к евро 2012 потому что работаю в пяти минутах ходьбы от олимпийской(
                                Ответить
                          • Тюю, опять мужички прорезались, я уж соскучилась.
                            Как раз весенний депресняк, когонить уебать хочется.
                            Поэтому - заткнись и дрочи на себя в другом месте.
                            Ответить
                            • Какая женщина
                              ЛЮР крепись
                              Ответить
                              • в таком случае лучше не крепиться, а расслабиться и попытаться получить удовольствие
                                Ответить
                            • >когонить уебать хочется.
                              За чем же с кем попало этим заниматься?
                              Ответить
                        • Но это не повод быть телепатом. Да, нужно полностью прорабатывать и такие вещи тоже, только заказчики - чаще всего - такие суки, хотят на чужом горбу в рай въехать. А пожелания у них (вроде кнопки "сделать заебись") формализуются крайне плохо. Потому я сторонник чётких слов: да, да; нет, нет; а что сверх этого, то от лукавого. И покрывается отдельным соглашением.
                          Ответить
                          • Вот теперь согласна. С такими только жесткий договор.
                            Мой идеализм наверно связан с тем что я всегда могла и не сдерживалась на послать неадекватного заказчика / работодателя в лес.
                            Ответить
                            • > И да, вторые зарабатывают значительно больше
                              > не сдерживалась на послать неадекватного заказчика
                              Конфликт интересов. По законам рынка-сервиса, клиентов нужно облизывать и выполнять все их грязные фантазии. С другой стороны, время-деньги; если позволяют возможности, сэкономленное время и нервы позволят сделать половину другого проекта и заработать не меньше.
                              Есть ещё вариант. Производить массовый продукт, рассчитанный под среднего пользователя. И продавать его, а не услуги.
                              Ответить
                    • Решается прописыванием в договор строки вида "за дополнительный функционал явно не описанный в ТЗ - дополнительные деньги и доработка проводится в течение дополнительного времени, в противном случае можете соснуть бочкуоспорить это в судебном порядке". Если что, они потом неустойку платить и будут.
                      Ответить

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