1. VisualBasic / Говнокод #2173

    −366.8

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    Public Overrides Function DataChanged() As Boolean
        Const CONST_PROC_NAME = "DataChanged"
        Try
            Return False
        Catch objExcp As Exception
            Me.DisplayMessageBox("Unexpected Exception in " & CONST_CLASS_NAME & "." & CONST_PROC_NAME & " : " & objExcp.Message.ToString, MessageBoxIcon.Error)
        End Try
    End Function

    По суровому закону нашего проекта каждая функция должна быть обернута ловлей эксепшиона. Каждая. Всегда.

    Запостил: 6istik, 24 Ноября 2009

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

    • ava guest 24.11.2009 17:01 # 0

      Должна быть обёрнута снаружи или внутри?
      Ответить

    • ava guest 24.11.2009 19:25 # +2

      Это мощно!
      Это не говнокод, а говноархитектура!
      Сначала в недоумении пялился на безумный код перехвата ошибки... (бррр... не хотел бы у таких работать!)

      А потом увидел это чудо:

      Try
      Return False

      Эйнштейн отдыхает...
      Ответить

      • ava vyalcevanet 24.11.2009 23:42 # 0

        жестоко...
        А откуда тут может взяться exception? Или это задел под переписывание функции, которая пока реализована в качестве заглушки?
        Ответить

        • ava 6istik 24.11.2009 23:55 # 0

          если его не обернуть то может случиться непредвиденное. вот еще примерчик опасного участка кода:
          btnOK.Visible= true
          а если кто забудет это сделать то тут же получает письмо от начальство полное гнева и возмущения.
          Ответить

          • ava vyalcevanet 25.11.2009 00:00 # 0

            а что такое страшное может быть с return false?

            ну c btnOK, там, понятно, кнопка может куда-то затеряться, но return false - он и в Африке return false...
            Ответить

            • ava 3.14159265 13.09.2010 16:42 # +1

              просто шедеврально
              но возникает вопрос, а почему не обернута Me.DisplayMessageBox(...)
              она-то посложнее ретурна будет
              Ответить

        • ava guest8 28.05.2019 11:44 # −999

          показать все, что скрытоvanished
          Ответить

    • ava guest 24.11.2009 19:28 # +1

      Видимо руководители проекта не знают что такое StackTrace
      Ответить

    • ava corristo 25.11.2009 00:21 # 0

      ява-кодеры с явным указанием эксепшнов всегда и везде смеются.
      Ответить

    • ava Oleg_quadro 25.11.2009 20:08 # 0

      Офигеть. И видимо По суровому закону вашего проекта каждая функция тормозит.
      Ответить

    • ava Oleg_quadro 25.11.2009 20:13 # +1

      А зачем функция, которая всё время возвращает false? Задел?
      Ответить

    • ava guest 27.11.2009 05:02 # 0

      Хм, дотнет вроде бы не делит функции на потенциально-обязательно-ловимые и "прощаемые", как в яве (забыл термины). Я имкю в виду, по сути может выскочить абсолютно любое исключение, вплоть до OutOfMemoryException или какого-нибудь "ExecutionException" (опять, забыл точные названия, нувыпоняли). Их наверное и ловят :)
      Ответить

      • ava guest 27.11.2009 05:03 # 0

        > вроде бы не делит функции

        "вроде бы не делит исключения"

        fxd
        Ответить

    • ava guest 27.11.2009 05:05 # +1

      Это называется паранойя, в нормальных странах за такое в психиатрическую больницу кладут
      Ответить

    • ava guest 27.11.2009 05:07 # +1

      Да, и ещё умиляет константа CONST_PROC_NAME. Ладно б там люди о принтСтэкТрэйсах не ведают, но делать локальную константу, чтобы использовать один раз через две строки -- это точно диагноз
      Ответить

      • ava guest 30.11.2009 18:48 # +1

        Так гораздо легче не забыть исправить эту константу, когда название функции меняется.
        Ответить

    • ava guest 01.12.2009 00:37 # 0

      Это придумал совсем безнадёжный дебил.

      В Application Events есть во чиво:
      Private Sub MyApplication_UnhandledException...
      Ответить

    • ava TEEN 16.11.2010 17:35 # 0

      Это звезда!
      Ответить

    • ava DPNET 03.02.2012 12:06 # 0

      Все бы хорошо, но зачем objExcp.Message.ToString, если оно и так String по определению, и почему не обычный сниппет?
      Ответить

    • ava OCETuHCKuu_nemyx 03.01.2020 12:43 # 0

      Может ты не так понял, и это только про верхнеуровневые гуишные функции?
      Ответить

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