1. C++ / Говнокод #56

    +339.8

    1. 1
    2. 2
    3. 3
    4. 4
    if (result == true)
      return true;
    else
     return result;

    этпипец... из разряда индокода, только в реале

    Запостил: guest, 04 Декабря 2008

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

    • Sap:
      Похоже, сначала возвращалось не булевское значение, а како-0то код ошибки и было что-то вроде:
      if (result == true)
        return -1;
      else
       return result;

      А потом по ходу отладки решили возвращать булевское - поставили заплатку, да так она и осталась. В больший проектах и не такое встречется.
      Ответить
    • показать все, что скрытоили автор кода подразумевал оператор === вместо ==, тогда вполне вменяемый смысл приобретается :)
      Ответить
    • показать все, что скрытоtigerox:
      да с === код не скомпилируется-смысл понятен=)
      Ответить
    • показать все, что скрытоЗвездочёт:
      Ну да! Конечно! Автор же не знает значение result! Вот и надо узнать! А то он ещё возвратит result непонятно какой...)
      Ответить
    • показать все, что скрытоAndrei:
      мг...похожее обычно получается после нескольких мелких изменений в коде в разное время. А потом как-нибудь смотришь на весь фрагмент в целом и понимаешь что данунах.
      Ответить
    • показать все, что скрытоAntON-LINE:
      я иногда так куски html комментирую
      ..
      if(0){
      ?>
      <table><tr..<?=$var?>
      строк 30
      </table>
      <?
      }
      главное по view source всего этого не будет ))

      так что в процессе многоразового исправления и не такое "накапливается"
      Ответить
    • показать все, что скрытоИндокод реален, %username%
      Ответить
    • показать все, что скрытоне пиздить, код нормальн для языка где у есть неопределенное значение у логической переменной, например...
      Ответить
    • показать все, что скрытоЕсли для result переопределён опреатор ==, то смысл есть. Хотя, конечно:
      return result == true;
      Или ещё православней использовать преобразование типа
      Ответить
    • показать все, что скрытоДовольно часто так делают что в вставить берйкпоинт перед возвратом из функции
      Ответить
      • показать все, что скрытоПоддерживаю, иногда возникают задачи, когда BP нужно поставить по определенному условию. Еще ладно, когда это условие выполняется чаще, чем не выполняется... Но если только вероятность - 1%, то приходится прибегать к таким конструкциям... Возможно, человек просто забыл убрать :-)
        Ответить
    • показать все, что скрытоЕсли result == 1, тогде его результат вернет true ( typeof true == boolean ), а если просто заменить на return result, то получим 1 ( typeof true == number )

      Так-что все кто сразу заржали, что автор полный мудак ВОЗМОЖНО тоже мудаки
      Ответить
    • В некоторых языках это может быть нормальным кодом. В if (result == true) result не обязательно булев тип, а только приводится к нему. То есть если result равен true, то возвращаем true, иначе возвращаем result.

      А вообще надо в таких случаях писать так:
      return not result or true
      Ответить
    • Например в павне будет работать такая штука.
      new variable = 0;
      if(varible)
      {
      return varible = true;
      }
      else if(!varible)
      {
      return varible = false;
      }
      Ответить
    • показать все, что скрытоReplica Handbags
      Hand bags are the essential accessories for the women as this stuff can add the elegance and beauty in their outlook. The trend of using handbags is rising to the peak level from the time when the celebrities have been going to carry the stylish and branded bags with them.
      The celebrities are the real inspiration behind the rising demands of hands bags A woman who wants to look like a super star always tries to copy her but there is a problem; the branded bags are not very affordable and normally tagged with high price. So how to deal with this problem, very simple just go for replica handbags
      [url=http://www.handbagshermes.org.uk ] Replica Handbags[/url] are carbon copies of [url=http://www.voguesking.net ] replica handbags[/url] . What increase the demand of these [url=http://www.chanel-handbags.co.uk ] replica chanel handbags[/url] is the affordability as it is an accessible for all women. The women who can’t afford the branded and designer bags have an alternative in the form of replica handsbag . By carrying this bag a woman can look like a celebrity and it is what for which she desire always.
      Normally at the first look Free Reprint Articles,you can never find the difference between a [url=http://www.replicacartierhandbags.co.uk ]replica cartier handbags[/url] and its replica but when you look it closely and keenly you noticed several things. The first thing is the quality of material that is used for the production of such bags. By and large the material of these bags is of average quality. Still there are some companies that are using the same quality material as used by designers.
      Ответить
    • показать все, что скрытоOn top of that, you can place in the [url=http://www.rivanna-stormwater.org/home.htm]cheap patek philippe watches[/url] going forward to going to conventional features like political works or school meetings.In regards to being aware of and purchasing, [url=http://www.hookedonbooksfloridakeys.com/home.html]cheap replica rolex watches online[/url] is often the first option of those. All the about [url=http://firstprolegal.com/quiy.html]high quality replica watches[/url], why not obtain one coming from the fake retail outlet. You could possibly be questioning the distinction between [url=http://www.alkhora.com/indexi.htm]swiss rolex replica watch[/url] and genuine timepieces. Most importantly, there is not any bargain using the excellent whenever you are purchasing the [url=http://www.sustainedaction.org/uksale.html]aaa+ replica breguet watches[/url] from those web sites.
      Ответить
    • показать все, что скрытоWhen you consider how easy it used to be to,[url=http://www.coachoutlet-factoryonline.us.com/]Coach Factory Outlet[/url] find fake versions of extremely expensive bags on Canal Street (I remember it very,[url=http://www.polosralphlauren.us.com/]Ralph Lauren Outlet[/url] distinctly from a trip to,[url=http://www.coandtiffany.us.com/]Tiffany And Co[/url] New York as a teenager–there,[url=http://www.coach-outletstores.us.com/]Coach Factory Outlet[/url] was no pretense of,[url=http://www.marcsbymarcjacobs.com/]Marc By Marc Jacobs[/url] secrecy), it seems as though,[url=http://www.katespadehandbag.us.com/]Kate Spade Handbags[/url] the NYPD's,[url=http://www.outletcoach.us.com/]Coach Outlet[/url] and luxury brands'combined efforts to curb counterfeiting may be paying dividends.,[url=http://www.tiffany.us.org/]Tiffany.com[/url] If nothing else, counterfeiters are having to work harder to sell their wares.As for why the market has shifted to bags,[url=http://www.marc-jacobsoutlet.com/]Marc Jacobs Outlet[/url] that retail for only a couple hundred dollars,[url=http://www.coachoutlet-factorystore.us.com/]Coach Factory Outlet[/url] Women's Wear Daily doesn't,[url=http://www.outletgucci.us.com/]Gucci Outlet[/url] draw any,[url=http://www.michael-korsoutlet.com.co/]Michael Kors Outlet[/url] conclusions, but I do have,[url=http://www.michaelkorsoutletinc.us.com/]Michael Kors Outlet Online[/url]
      Ответить
    • показать все, что скрытоOkay I'm covenicnd. Let's put it to action.
      Ответить
    • показать все, что скрытоYou have more useful info than the British had colonies prWWe-II. http://djjxdiobsk.com [url=http://cxxrgw.com]cxxrgw[/url] [link=http://sxrruof.com]sxrruof[/link]
      Ответить
    • показать все, что скрытоWhat a great resucroe this text is. http://oemius.com [url=http://nldxpenufe.com]nldxpenufe[/url] [link=http://pkjpgsbf.com]pkjpgsbf[/link]
      Ответить
    • показать все, что скрытоhttp://www.ray-bansunglassessite.net/ Ray Ban Sunglasses
      http://www.nikeoutletfactorystore.com/ Nike Outlet Factory
      http://www.longchamp--outlet.com/ Longchamp Outlet
      http://www.nikeflyknit.org/ Nike Flyknit
      http://www.oakleysunglassesoutlet.name/ Oakley Outlet
      http://www.oakleyoutletstore.net.co/ Oakley Outlet
      http://www.tomsoutletstore.org/ Toms Outlet
      http://www.ralphlaurenoutletstoreonline.com/ polo ralph lauren outlet online
      http://www.katespadeoutletsstore.com/ Kate Spade Outlet
      http://www.air-max2015.org/ nike air max 2015
      http://www.adidasoutletstore.org/ Adidas Outlet Store
      http://www.yeezyboost350.us.com/ Yeezy Boost 350
      http://www.nikeoutletsstore.com/ Nike Outlet
      http://www.yeezyboost.org/ Yeezy Boost 350
      http://www.longchampoutletstore.us.com/ longchamp outlet
      http://www.katespade-outlet.org/ kate spade outlet
      http://www.nfAljerseyswhosale.us.com/ Cheap NFL Jerseys
      http://www.airmax2016.us.com/ Nike Air Max 2016
      http://www.raybanoutletsunglasses.net/ Ray Ban Outlet
      http://www.burberryoutletstoreonline.org/ Burberry Outlet
      http://www.adidasyeezyboost350.org/ Adidas Yeezy
      Ответить
    • показать все, что скрыто<a href="http://www.pradaoutlet.org"><strong>prada handbags</strong></a>
      <a href="http://www.vansshoes.in.net"><strong>vans sk8 hi</strong></a>
      [url=http://www.cheapjordans.com.co]cheap jordan shoes[/url]
      [url=http://www.airjordan-uk.org.uk]air jordan uk[/url]
      http://www.airforce1-nike.fr
      http://www.canadagooseuk.me.uk
      ghd straighteners

      burberry outlet
      Ответить

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