Школоло / Говнокод #2173 Ссылка на оригинал

0

  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 6istik, (Updated )

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

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

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

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

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

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

    Try
    Return False

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

      fxd
      Ответить

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

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

    ↑
    8
    ↓