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

    0

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    try
       {
            await storageClient.DownloadObjectAsync(Bucket, fileName, stream).ConfigureAwait(false);
       }
    catch(Exception ex)
       {
            throw new FileStorageException($"File '{fileName}' not found in a bucket '{Bucket}'", ex) { StatusCode = StatusCodes.Status404NotFound };
       }

    Сеньйор дот нет девелопер

    Запостил: horil97821, 26 Марта 2020

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

    • ava gostinho 26.03.2020 22:43 # 0

      Что не так?
      Ответить

      • ava Fike 26.03.2020 22:59 # 0

        видимо await ... .ConfigureAwait(false). но зачем всё остальное - тоже не понял
        Ответить

        • ava phpBidlokoder2 27.03.2020 00:32 # 0

          Переведи на пхп, ничерта не понятно
          Ответить

          • ava Fike 27.03.2020 01:26 # 0 

            <?php
$socket = socket_create_listen(1223);
socket_set_nonblock($socket);

socket_accept($socket);
?>
            Ответить

            • ava MAPTOBCKuu_nemyx 27.03.2020 01:41 # +1

              Получилось намного короче! Именно поэтому я за «PHP».
              Ответить

            • ava MAPTOBCKuu_nemyx 27.03.2020 01:44 # 0

              А если серьёзно, то «bucket» — это слишком специфическое слово. Это из API какого-нибудь «AWS»?
              Ответить

              • ava KOPOHABuPYC 27.03.2020 02:16 # −1

                Когда апостол Пётр даст тебе серебряную указку и попросит прочесть тору, увидев алеф говори "ба" - и попадешь в рай.
                Евреев туда не пустят.
                Ответить

            • ava phpBidlokoder2 27.03.2020 09:21 # −1

              Зачем закрыл тег ? И что это за функции ? Аналоги strip_tags ?
              Ответить

      • ava Fike 27.03.2020 04:32 # +1

        ну и вообще стрелять "нихуя не найдено" на любое исключение - тоже говно
        Ответить

    • ava horil97821 27.03.2020 13:55 # 0

      то есть ничего не смущает тот факт, что в что в эксепшене создается еще один эксепшн?
      Ответить

      • ava gost 27.03.2020 14:06 # 0

        Это-то как раз норма. ConfigureAwait() может бросать какое-нибудь HttpException, а этот код просто переводит его в более удобный вид: во-первых, с понятным сообщением об ошибке, а во-вторых — вызывающему этот код не нужно будет импортировать HTTP-либу и разбираться в её деталях, вместо этого вызывающий просто ловит FileStorageException и течёт.

        А вот ловля «Exception ex» — это говно, да.
        Ответить

        • ava horil97821 27.03.2020 14:18 # 0

          иными словами отдебажить потом будет ооочень нелегко.... и по хорошему нужно не ошибку писать а писать в лог...
          Ответить

          • ava gost 27.03.2020 14:25 # 0

            Если заменить «catch (Exception ex)» на правильное «catch(HttpException ex)» (ну и обработать другие возможные вореанты) — то почему это должно быть сложно дебажить?
            Писать в лог нужно далеко не всегда, важен коньтекст. Если этот код — из либы для работы с какой-то сетевой ФС, то выбрасывания подробного, семантически верного, исключения вполне достаточно, либе совершенно не обязательно срать в лог, это оверинжиниринг. В лог писать должен тот код, который этот самый FileStorageException поймает.
            Ответить

            • ava horil97821 27.03.2020 14:28 # 0

              ога например креды сдохли и как ты поймешь это?) Выведешь эксепшн в который не входит это ошибка - ну ок)
              Ответить

              • ava gost 27.03.2020 14:32 # 0

                Очень просто:
                catch(HttpException ex) when (ex.StatusCode == 403)
{
    throw new FileStorageException($"Couldn't load the file {Filename}: access denied", ex) { StatusCode = StatusCodes.Status403AccesDenied };
}
                Ответить

        • ava eukaryote 28.03.2020 07:50 # 0

          Кстати, про ConfigureAwait(false). В «ASP.NET Core» нет никакого «SynchronizationContext» (в отличие от старого ASP.NET), поэтому ConfigureAwait(false) больше не нужен. Можно блокировать асинхронный код и не опасаться дедлоков. Именно поэтому я за «ASP.NET Core».
          Ответить

    • ava OlegUP 27.03.2020 23:21 # 0

      А почему все три параметра DownloadObjectAsync в разных нотациях?
      ex тоже в camel case
      а StatusCode в pascal case.
      Это у шарперов норм?
      Ответить

      • ava guest8 27.03.2020 23:24 # −999

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

        • ava kak 27.03.2020 23:32 # 0

          Какая-то говноконвенция, to be honest with you.
          Впрочем, это же Visual BRACIS...
          Ответить

        • ava bormand 27.03.2020 23:57 # +1

          > филды с большой

          Зачем? Зачем? Чем они от переменных отличаются?
          Ответить

          • ava kak 28.03.2020 00:08 # 0

            Причем он еще умудрился в штаны дриснуть обозвав метод полем.
            Ответить

            • ava KOPOHABuPYC 28.03.2020 01:44 # 0

              Скоро вы все будете лежать на поле, точнее, в поле, и разница между методами и свойствами будет для Вас уже не столь очевидна. Вороны будут клевать ваши высохшие глаза.
              Ответить

          • ava guest8 28.03.2020 02:21 # −999

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

          • ava eukaryote 28.03.2020 07:22 # +1

            Паблик филды пишутся с прописной буквы (но вообще, лучше вместо них использовать проперти).
            Для приватных строгого соглашения нет, и вот их как правило пишут со строчной, как обычную переменную; некоторые ещё андерскор впереди добавляют.
            https://docs.microsoft.com/en-us/dotnet/standard/design-guidelines/capitalization-conventions
            Ответить

    • ava bootcamp_dropout 28.03.2020 11:42 # 0

      Охуительная практика использовать эксепшены как часть логики программы
      Неужели у storageClient нет метода "проверить наличие файла"?
      Ответить

      • ava gost 28.03.2020 11:59 # +1

        > Неужели у storageClient нет метода "проверить наличие файла"?
        Судя по https://googleapis.github.io/google-cloud-dotnet/docs/Google.Cloud.Storage.V1/api/Google.Cloud.Storage.V1.StorageClient.ht ml — именно такого нет, разве что GetObject().

        Впрочем, он и не нужен: в распределённых приложениях паттерн «проверить наличие, потом использовать» — априори говно.
        Ответить

        • ava guest8 28.03.2020 15:04 # −999

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

          • ava Fike 29.03.2020 04:01 # 0

            строго говоря, в редких ситуациях проще проверить наличие/отсутствие, прежде чем начинать операцию, если первое в разы дешевле
            Ответить

        • ava bootcamp_dropout 29.03.2020 13:46 # 0

          >Впрочем, он и не нужен: в распределённых приложениях паттерн «проверить наличие, потом использовать» — априори говно.
          Если эта хуйня упакована в storageClient, можно было бы создавать какой-нибудь StorageClientInstance, который бы под капотом делал один запрос, но поддерживал бы оба метода, храня в себе состояние этого запроса. Разве получилось бы хуево?
          Ответить

          • ava gost 29.03.2020 14:00 # 0

            Да, хуёво. Потому что тогда метод «проверить наличие файла» должен скачивать весь файл, чтобы обеспечить работоспособность подхода «проверить наличие, потом использовать». Правда, в этом случае мы огребаем дичайшие проблемы синхронизации, когда в нашем локальном состоянии файл есть, а на сервере его уже нет, и даже какой-нибудь getFilesCount() возвращает 0.
            Ответить

            • ava KOPOHABuPYC 29.03.2020 22:02 # 0

              Не нужно нажимать на кнопочки и пачкать экран. Лучше выйди на улицу и сделай глубокий вдох.
              Ответить

      • ava eukaryote 28.03.2020 16:11 # 0

        Мне больше всего нравится, что любой эксепшен интерпретируется как «файл не найден». А если просто интернет отвалился или Роскомнадзор?
        Ответить

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