1. C# / Говнокод #5138

    +125

    1. 1
    m_PRICEURL = m_PRICEURL.Replace("{0}", "2010");

    Новый Год принёс не только подарки, но и коллекцию говнокода вроде такого

    Запостил: Shokker, 03 Января 2011

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

    • булочки свежие из Новой Зеландии!
      они ж прошлогодние!
      replace....а теперь?
      Ответить
    • показать все, что скрытоговнокод ещё и в том, что "{0}" может встречатся несколько раз в строке
      тогда лучше вызывать ReplaceAll
      Ответить
      • Запах любого говнокода заключается в том, что задача, поставленная говнокодеру решена полностью и\или с небольшими недочетами но очень говнисто. А данный быдлокод не решает поставленную задачу вовсе, хоть и криво. Значит это не говнокод вовсе.
        Ваш кэп (с).
        Ответить
      • В C# метод Replace заменяет все вхождения первого значения на второе. И никакого ReplaceAll там нет. Если конечно, это тип System.String, а не какой-то пользовательский класс.
        Ответить
        • эм, тут речь о том будет ли замена если в результате замены получилась искомая подстрока. типа того.
          Ответить
    • показать все, что скрытоWEBKILL OTAKE111
      Ответить
    • вообщем m_PRICEURL был что-то типа "year {0}, month {1} ...". юзался как строка формата для WriteLine или еще какой-нить унылой функции по типу
      Console.WriteLine(m_PRICEURL, Date.Year(), Date.Month());

      так год то сменился, а нужен старый -> меняем {0} на 2010. ???. PROFIT!!
      Ответить

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