Говнокод: по колено в коде.

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5703)
2. C++ (2740)
3. Куча (2337)
4. JavaScript (2030)
5. C# (1929)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   −21

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5

   try {
                   shadow_page.insert_record(index, record);
               } catch ( ... ) {
                   throw;
               }

   Запостил: Koshak90, 07 Февраля 2017

   Tweet

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

   • Bobik 08.02.2017 10:08 # 0

     Наглядно показывает, что эта часть кода будет часто кидать исключения?

     Ответить

     • Antervis 08.02.2017 10:47 # +3

       shadow_page.insert_record(index, record); // срет throws

       Ответить

   • Steve_Brown 08.02.2017 11:11 # +4

     Когда поймал исключение и не знаешь, что с ним делать дальше.

     Ответить

     • bormand 21.02.2017 21:24 # 0

       Заметает следы в бектрейсе.

       Ответить

       • guestinho 21.02.2017 21:27 # 0

         throw c параметрами разве не кидает исключение с оригинальным трейсом?

         Ответить

         • bormand 21.02.2017 21:29 # 0

           Трабла в том, что в крестовых исключениях нет бектрейса (если его туда не положить). Но после этого catch-throw, емнип, даже в дебаггере концов не найдёшь.

           Ответить

           • guestinho 21.02.2017 21:31 # 0

             Не пизди давай много.

             Ответить

           • doctor_stertor 21.02.2017 22:20 # 0

             [propovednik]...не мечите жемчуга (бисера) перед свиньями, чтобы они, подняв рыла, не растерзали Вас в клочья. Лучше в ломбард отнесите.[/propovednik]

             Ответить

   • isnullxbh 21.02.2017 13:28 # 0

     Если бы помимо throw там было бы еще что-нибудь - было бы понятнее. Мне кажется, такая конструкция полезна, когда основная логика обработки исключения находится где-то "выше", но при этом нужно сделать какие-то небольшие измнения на уровне данного класса.
     Вроде:
     try
     {
     shadow_page.insert_record(index, record);
     }
     catch ( ... )
     {
     m_someoneCounter = 0;
     throw;
     }

     Ответить

     • huesto 21.02.2017 20:11 # 0

       Это надо делать с помощью RAII.

       Ответить

       • guestinho 21.02.2017 21:21 # 0

         Что-то мне подсказывает, что спринцовкой каноничнее будет.

         Ответить

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

   Ошибка компиляции комментария:

   1. Гости могут высказаться только в понедельник, среду, четверг или воскресение

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

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

   • [b]жирный[/b] — жирный
   • [i]курсив[/i] — курсив
   • [u]подчеркнутый[/u] — подчеркнутый
   • [s]перечеркнутый[/s] — перечеркнутый
   • [blink]мигающий[/blink] — мигающий
   • [color=red]цвет[/color] — цвет (подробнее)
   • [size=20]размер[/size] — размер (подробнее)
   • [code=<language>]some code[/code] (подробнее)

   Проверочный код: *