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

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 telnet, (Updated )

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

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

    Печально, но практика показывает что глюки в таких местах возникают нечасто. 🙁 Я тоже в своём проекте стараюсь такое давить где вижу, но постоянно где-нибудь появляется свежак, и каким-то чудом работает. Проблемы _изредка_ возникают на проде только буквально за неделю до следующего релиза, за которую потом срочно приходится это говно переписывать. 🙂
    Ответить

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

      Да смысл, я и в аську ему написать могу, и на почту. Просто ситуация получилась, что он мне написал, что всё сделал, а я заработался, прошло два дня и дошли руки по-нормальному проверять его работу, только когда заказчик написал гневное письмо ("ну где?!!111") и надо было срочно отвечать. Проверил - вроде пашет код, так и отчитался.
      Ответить

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

    Хочешь, что то сделать хорошо не доверяй фрилансерам, у них мозг работает по другому. Их оценка качества работы "много". Лучше найти коллегу в отпуске.
    Ответить

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

      Не обобщайте, фрилансеры очень хорошие бывают, просто надо школоло фильтровать.
      Ответить

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

        Вроде большая в сказки веришь. Хороший код от фрилансера сродни подарка от "Деда Мороза".
        Ответить

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

          Вася, я тоже фрилансером был... Старался всегда сделать по-нормальному всё... Но вот как раз из-за такого предвзятого мнения пришлось таки вступить в ряды аутсорсной команды, ...и заниматься точно тем же. 🙂
          И, может не поверишь, но в аутсорсной команде в серьёзной компании тоже находятся чудики, которые пишут вещи вроде if(empty($val)) $val = $val = str_replace(' ', '', $val);
          Ответить

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

            Аутсорс мало чем отличается от фриланса, разве что народа больше... У меня начальство вообще разницы между этими двумя словами не видит.
            Ответить

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

              >У меня начальство вообще разницы между этими двумя словами не видит.
              охеренный довод
              Ответить

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

            Ну если был. Я 100% уверен что софт писался так
            1. Получение ТЗ
            2. Начали писать. (берем код с похожего пректа и вставляем в него костыли)
            3. Запустили если в 30% код делал то что от него нужно. можно сдавать.

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

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

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

              Как обычно проект попадает к фрилансеру.
              А давайте мы сэкономим!
              Дальше писать?

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

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

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

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

                  И как вы все таки добились "что бы это стало так как есть".
                  Судя по фразе "много наших полегло", раз в день один из программистов с разбегу убивался об стену, пока ваши требования не были приняты )))
                  Ответить

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

                    Да сейчас вроде все как должно быть. Прогеры не убивались просто уходили.
                    Ответить

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

                  Побыв с другой стороны баррикад могу сказать что старший тоже виноват - не пропараноил с неадекватным начальством/заказчиком и не спросил.
                  Хотя бывают такие что на вопросы устраивают истерику, тут лучше сразу уходить)
                  Ответить

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

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

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

                      Сукабля, а я сначала думал, что мой первый коммент зохавал сотона. Ддосят нас штоле.
                      Ответить

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

                      Есть два вида подрядчиков - простоработники и профи.
                      Первые требуют полное тз, даже не пытаясь задуматься - а нужно ли это или то заказчику. Ходят злые на тупых заказчиков.

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

                      И да, вторые зарабатывают значительно больше.
                      Ответить

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

                            За место для проживания.
                            ОЧЕРЕДНАЯ ТЕОРИЯ ЗАГОВОРА:Может он футбольный фанат и таким макаром решил бесплатно съездить на Евро-2012.
                            Ответить

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

                              На него еще хрен билеты купишь насколько я знаю.
                              Оччень депрессивно отношусь к евро 2012 потому что работаю в пяти минутах ходьбы от олимпийской(
                              Ответить

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

                          Тюю, опять мужички прорезались, я уж соскучилась.
                          Как раз весенний депресняк, когонить уебать хочется.
                          Поэтому - заткнись и дрочи на себя в другом месте.
                          Ответить

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

                        Но это не повод быть телепатом. Да, нужно полностью прорабатывать и такие вещи тоже, только заказчики - чаще всего - такие суки, хотят на чужом горбу в рай въехать. А пожелания у них (вроде кнопки "сделать заебись") формализуются крайне плохо. Потому я сторонник чётких слов: да, да; нет, нет; а что сверх этого, то от лукавого. И покрывается отдельным соглашением.
                        Ответить

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

                          Вот теперь согласна. С такими только жесткий договор.
                          Мой идеализм наверно связан с тем что я всегда могла и не сдерживалась на послать неадекватного заказчика / работодателя в лес.
                          Ответить

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

                            > И да, вторые зарабатывают значительно больше
                            > не сдерживалась на послать неадекватного заказчика
                            Конфликт интересов. По законам рынка-сервиса, клиентов нужно облизывать и выполнять все их грязные фантазии. С другой стороны, время-деньги; если позволяют возможности, сэкономленное время и нервы позволят сделать половину другого проекта и заработать не меньше.
                            Есть ещё вариант. Производить массовый продукт, рассчитанный под среднего пользователя. И продавать его, а не услуги.
                            Ответить

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

                    Решается прописыванием в договор строки вида "за дополнительный функционал явно не описанный в ТЗ - дополнительные деньги и доработка проводится в течение дополнительного времени, в противном случае можете соснуть бочкуоспорить это в судебном порядке". Если что, они потом неустойку платить и будут.
                    Ответить

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

Где здесь C++, guest?!

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

    ↑
    8
    ↓