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

    +132

    1. 1
    public virtual int ReadByte()

    Тут в соседнем треде появилась такая тема:

    http://msdn.microsoft.com/ru-ru/library/system.io.stream.readbyte.aspx
    
http://govnokod.ru/12311#comment164854

    Запостил: LispGovno, 20 Декабря 2012

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

    • ava LispGovno 20.12.2012 07:08 # −2

      >12321
      21.12.1212
      Ответить

    • ava krypt 20.12.2012 13:45 # 0

      Зато лентяям можно делать так:
      while (bt = stream.ReadByte() > 0)
{
    //...
}
      Ответить

      • ava roman-kashitsyn 20.12.2012 13:51 # 0

        Что-то мне подсказывает, что в bt постоянно будет лежать true
        Ответить

        • ava Steve_Brown 20.12.2012 14:22 # 0

          Не, здесь будет что-нибудь типа "Error: cannot convert from 'bool' to 'byte'". Тут вам не там.
          Ответить

        • ava krypt 20.12.2012 15:08 # 0

          Вы хоть по ссылке сходили?
          Возвращаемое значение
          [qoute]Тип: System.Int32
          Байт без знака, приведенный к Int32, или значение -1, если достигнут конец потока.[/qoute]
          Другой вопрос, что писал без IDE и есть ошибки.

          int bt = 0;
FileStream stream = new FileStream(@"d:\test.txt", FileMode.Open);
while ((bt = stream.ReadByte()) >= 0)
{
    Console.WriteLine(bt);
}
          Ответить

      • ava LispGovno 20.12.2012 15:13 # +1

        >Зато лентяям можно делать так:
        >while (bt = stream.ReadByte() > 0)
        Зато ты лентяй и сразу ошибся. Вот что я называю нихрена непродуманным интерфейсом, что приводит к таким банальным глупым ошибкам.
        Ответить

        • ava krypt 20.12.2012 15:20 # 0

          Ну подумаешь, забыл, что у присваивания самый низкий приоритет )
          Конкретно этот код даже не скомпилируется, выдаёт ошибку "Cannot implicitly convert type 'bool' to 'int'" (строгая типизация, это вам не сишка )
          И из того, что я привёл этот пример ещё не следует, что я его использую.
          Ответить

        • ava 3.14159265 20.12.2012 17:22 # +3

          Не знаю. У меня в независимо от языка руки так и чешутся поставить там скобки, чтоб понятней было.
          Это раз.
          Во-вторых, используешь сишный side-effect инкрементов и присваиваний, тогда yoda-style твой лучший друг.
          В-третьих, читать по байту - мелковато.
          В-четвертых, есть LINQ.
          Ответить

          • ava 3.14159265 20.12.2012 17:27 # +2

            В-четвертых, для лентяев есть LINQ.
            Ответить

            • ava bormand 20.12.2012 17:51 # +2

              Lazy Ignorant Noob Query?
              Ответить

              • ava 3.14159265 20.12.2012 18:04 # +1

                Language for Infamous Noobs Queries
                Ответить

              • ava 3.14159265 20.12.2012 18:13 # +2

                http://www.urbandictionary.com/define.php?term=linq
                3. Scumbag
                The dirt under the fingernails of progress as some may say. Scumbags, also known as scummies are immoral and have no sense of right or wrong in the world due to poor parenting.

                Avoid scummies at all costs.
                Scummie: Hey do you have a skeet AssParallel?
                Person: fuck off, LINQscumbfag.

                Лол.
                Ответить

              • ava 3.14159265 29.12.2012 23:10 # +1

                Lame Incompetent Newb Quirks
                Ответить

    • ava Steve_Brown 20.12.2012 13:47 # +2

      Ну дык... конец файла — это нормальная рабочая ситуация. А исключения предназначены для исключительных ситуаций.
      В Питоне другая философия, там даже из цикла выход осуществляется через исключение.
      Ответить

      • ava LispGovno 20.12.2012 15:09 # 0

        >Ну дык... конец файла — это нормальная рабочая ситуация.
        Ты не поверишь:
        while(stream.NotEnd())
{
  byte bt = stream.ReadByte();
  ...
}
        Ответить

        • ava myaut 20.12.2012 15:41 # 0

          Ну получается тогда такая бяка:
          while(stream.NotEnd())
{
  byte bt = stream.ReadByte();
  if (stream.eos()) 
    break;

  /*process byte*/
}
          Ответить

          • ava Steve_Brown 20.12.2012 16:06 # 0

            Не понял. Предполагается же, что раз NotEnd(), то байт гарантированно* прочитать можно.
            Другой вопрос, что в Stream такого свойства нет. А почему так сделано... Ну, возможно, для облегчения реализации интерфейса. Да и в общем случае не всегда можно узнать, достигнут ли конец данных, не фактически прочитав байт.
            Ответить

            • ava myaut 20.12.2012 20:06 # 0

              > байт гарантированно* прочитать можно.
              И что вы можете гарантировать в сокете?
              Ответить

              • ava 3.14159265 20.12.2012 20:06 # +1

                [KO]Поток может быть буферизованным[/KO]
                Ответить

                • ava myaut 20.12.2012 20:11 # 0

                  Когда буфер чтения исчерпается, но сокет еще открыт и сообщение еще не до конца принято, то что должна возвращать NotEnd()?
                  Ответить

                  • ava 3.14159265 20.12.2012 20:15 # 0

                    Должна ждать.
                    Ответить

                    • ava guest 14.11.2015 15:54 # 0

                      Son of a gun, this is so heulflp!
                      Ответить

                    • ava guest 20.11.2015 07:35 # 0

                      This is exactly what I was looking for. Thanks for <a href="http://cpuodnzih.com">wrtngii!</a>
                      Ответить

                    • ava guest 20.11.2015 15:09 # 0

                      Time to face the music armed with this great inraomftion. http://niftloaxpyc.com [url=http://ekgszqs.com]ekgszqs[/url] [link=http://ffurzr.com]ffurzr[/link]
                      Ответить

                    • ava guest 24.11.2015 09:43 # 0

                      Hey, that's a clever way of thnkiing about it. http://seovxcg.com [url=http://kjealeev.com]kjealeev[/url] [link=http://uqqhtz.com]uqqhtz[/link]
                      Ответить

                  • ava 3.14159265 20.12.2012 20:17 # +2

                    Ну вот если байт нету, а сокет открыт, то что еще надо делать?
                    Очевидно же.
                    Пока буфер вновь не наполнится хотя байтом или не закроется сокет.
                    Ответить

                    • ava myaut 20.12.2012 20:19 # 0

                      А. Теперь понятно объяснил.
                      Просто я привык, что блокирующий у нас read()
                      Ответить

                      • ava 3.14159265 20.12.2012 20:24 # +2

                        В таких делах вроде как любое чтение блокирующее.
                        Или я что-то упустил?
                        Ответить

        • ava guest 14.11.2015 13:01 # 0

          This could not polisbsy have been more helpful!
          Ответить

        • ava guest 20.11.2015 15:05 # 0

          In the colapicmted world we live in, it's good to find simple solutions. http://kvgydlm.com [url=http://dnmtkzqfq.com]dnmtkzqfq[/url] [link=http://enjjyltk.com]enjjyltk[/link]
          Ответить

        • ava guest 24.11.2015 09:40 # 0

          You know what, I'm very much innicled to agree. http://isvkmiju.com [url=http://xhpasnztnvc.com]xhpasnztnvc[/url] [link=http://yohbkdeu.com]yohbkdeu[/link]
          Ответить

      • ava bormand 20.12.2012 17:21 # +4

        На самом деле есть две ситуации:

        В первой я читаю файл, и хочу знать когда он закончится - этот случай и имелся в виду в шарпе.

        А другая ситуация - я читаю из файла определенную структуру, вызывая всякие-там getInt, getFloat, getDouble и т.п. И вот тут конец файла до окончания структуры это именно исключительная ситуация. И именно ее я хотел показать как пример об удобстве исключений в соседнем треде.
        Ответить

    • ava LispGovno 20.12.2012 15:12 # 0

      В конце концов напиши:
      public virtual byte? ReadByte()
Что типа public virtual Maybe<byte> ReadByte()
      Но зачем городить такое говно, что они сделали?
      Ответить

      • ava krypt 20.12.2012 16:55 # 0

        "?" во фреймворке нигде не используется, насколько я знаю.
        Ответить

        • ava Steve_Brown 20.12.2012 17:19 # 0

          http://msdn.microsoft.com/en-us/library/System.Nullable.aspx
          Supported in: 4.5, 4, 3.5, 3.0, 2.0
          А Stream был изначально (с 1.0)
          Ответить

          • ava krypt 20.12.2012 18:11 # 0

            Да, что-то типа. Только 1.0 и 2.0 не совместимы, и что им мешало использовать его, не понятно. Nullable там встречается, но только в неявном виде.
            Ответить

            • ava Steve_Brown 20.12.2012 19:44 # 0

              То есть заменить int ReadByte() на byte? ReadByte() ? Это было бы слишком кардинальное изменение для такого базового API.
              Ответить

              • ava krypt 20.12.2012 21:28 # 0

                Я не конкретно про этот случай, я вообще имел ввиду. Имхо, тут всё вполне нормально.
                Ответить

    • ava myaut 20.12.2012 15:40 # +1

      Не городить, а плодить.

      http://docs.oracle.com/javase/1.4.2/docs/api/java/io/InputStream.html#read%28%29

      public abstract int read()

      Reads the next byte of data from the input stream. The value byte is returned as an int in the range 0 to 255. If no byte is available because the end of the stream has been reached, the value -1 is returned.
      Ответить

      • ava bormand 20.12.2012 17:23 # +4

        А вот откуда это все пошло:

        fgetc() reads the next character from stream and returns it as an unsigned char cast to an int, or EOF on end of file or error.

        #define EOF (-1)
        Ответить

      • ava 3.14159265 20.12.2012 19:52 # +1

        Но в жабе оксюморона-то нету.
        http://docs.oracle.com/javase/1.4.2/docs/api/java/io/DataInputStream.html

        >getc() reads the next character from stream and returns
        Прикол в том что для чара байта и даже джвух может не хватить. А если мы будем будем читать в нативный 16-битный int, то...
        Ответить

        • ava LispGovno 21.12.2012 06:17 # +2

          > IOException - if an I/O error occurs.
          > EOFException - if this input stream has reached the end.
          Ответить

    • ava LispGovno 21.12.2012 06:19 # +1

      Что-то все попрятались и на уютненький не заходят.
      Конца Светы что-ли ждут?
      Ответить

      • ava guest 21.12.2012 07:07 # +1

        bormand только что #
        Разлогинило ;(
        А пароль, в который раз, забыл сохранить на рабочей машине.
        Ответить

    • ava Dummy00001 21.12.2012 19:48 # 0

      @bormand:
      > В нормальных компиляторах (аля гцц) плата за экцепшн берется только при его вбросе и раскрутке [...]

      Производительность - да. Но есть еще проблема с размером кода.

      На текущем C++ модуле с которым я работаю и который активно пользуется исключениями, 75% кода (центральных функций) приходится именно на обработку исключений. Другими словами, этот тот код который никогда в нормальном случае и даже не будет затронут.

      Самая центральная функция: ~5К асма на саму логику против ~15К асма (после последнего `ret`а) на обработку исключений. Если сильно в эти 15К всматриваться, то можно реально разглядеть что тут есть кусок кода соответствующий каждому месту функции которое может бросать исключения. Подавляющая часть этого кода: вызов деструкторов для временных/локальных объектов.

      И это я здесь говорю про код. Exception handling tables по старой памяти так много места не занимают - да и в любом случае IIRC они лежат в своем собственном сегменте и сегмент кода не засоряют.

      ЗЫ Я сомневаюсь что C# может это сделать намного лучше.
      Ответить

      • ava 3.14159265 21.12.2012 20:06 # 0

        Вы так гойворите будто ifы не увеличивают размер кода.
        Я уже говорил и повторю еще раз: нужно ключевое слово, оператор, флаг или там аннотация, чтобы включать и выключать исключения на блоке кода.
        Ответить

        • ava bormand 21.12.2012 20:43 # +2

          > Выключать исключения на блоке кода
          Чтобы превратить "ой упало, вот вам стектрейс" в "кровь-кишки-молча-распидорасило-внутреннее-состояние-проги-и-об-этом-узнали-через-год"?

          Разве что добавить на функции и блоки атрибут типа nothrow, показывающий, что она не может вбросить никаких исключений, даже RuntimeException. И разрешать из таких функций вызывать только меченные... Тогда будет гарантия, что хуйню не напишешь.
          public void do1() nothrow {
}
...
public void test() {
    nothrow {
        do1();
        do2();
        do3();
    }
}
          P.S. К сожалению любой блок, в котором есть new, пометить как nothrow уже не получится ;(
          Ответить

          • ava 3.14159265 21.12.2012 20:54 # 0

            И получим java checked exceptions. В самой хуёвой форме.
            Моё мнение такое если и уж делали checked exceptions, то надо было сделать их настраиваемыми как опция компилятора ignore/warn/error. Вместо того чтобы тупо запрещать коду компилится.

            Хороший пример - переполнение. При расчетах оно нужно и хорошо бы его вовсе не замечать, но обычно же лучше включать проверку.
            >"распидорасило-внутреннее-состояние-проги-и-об-этом-узнали-через-год"

            В жабе если run() в треде кидалось исключение (куда ему дальше идти? тоже игнорится) то стектрейс писался в System.err.
            Несколько версий спустя туда добавили возможность задать default handler для таких вот неуловимых ошибок.
            Так что разрулить можно.
            Ответить

          • ava TarasB 21.12.2012 21:45 # 0

            > P.S. К сожалению любой блок, в котором есть new, пометить как nothrow уже не получится ;(

            Почему? Ааа, жаба...
            Ответить

            • ava bormand 21.12.2012 22:32 # +2

              Ну в крестах ты можешь попросить, чтобы new возвращало тебе null. Но будешь ли ты его проверять при каждом вызове? Что ты будешь делать, если там null? Вызовешь ассерт? Будешь закрывать ресурсы и возвращать ошибку на уровень выше? Так и исключениями можно сделать тоже самое... только меньшими усилиями.
              Ответить

            • ava guest 14.11.2015 15:58 # 0

              I hate my life but at least this makes it bealbare.
              Ответить

            • ava guest 20.11.2015 07:36 # 0

              Your story was really <a href="http://stbzfrkf.com">inomvfatire,</a> thanks!
              Ответить

            • ava guest 20.11.2015 15:09 # 0

              I was really confused, and this answered all my quesniots. http://syxktru.com [url=http://lhqwimdfcjo.com]lhqwimdfcjo[/url] [link=http://vhvgky.com]vhvgky[/link]
              Ответить

            • ava guest 24.11.2015 09:43 # 0

              It's wouferdnl to have you on our side, haha! http://jsqiuru.com [url=http://jxrpexcqa.com]jxrpexcqa[/url] [link=http://yfrrrmjplmw.com]yfrrrmjplmw[/link]
              Ответить

        • ava Dummy00001 21.12.2012 20:57 # 0

          "Вы так гойворите будто ifы не увеличивают размер кода."

          if'ы есть часть нормального кода. обработка исколючений - 99% кода генирится самим компилером и никакого отношения к остальному коду или логике не имеет.

          "Я уже говорил и повторю еще раз: нужно ключевое слово, оператор, флаг или там аннотация, чтобы включать и выключать исключения на блоке кода."

          это достаточно просто: не пользуйся ОО побрякушками в куске кода - временными объектами на стэке, статически аллоцироваными объектами внутри подконтекста - и у него не будет никаких специальных довесков для обработки исключений.

          еще раз повторю - "Подавляющая часть этого кода: вызов деструкторов для временных/локальных объектов." другими словами этот авто-сгенерированый код как бы никакого непосредственного отношения к исключениям не имеет, он имеет непосредственное отношение к коду функции. но этот код нужен только исключительно для обработки исключений, что бы гарантировать что даже в этом случае объекты/етц ведут себя согласно С++ стандартам.
          Ответить

          • ava 3.14159265 21.12.2012 21:20 # +1

            Я о ключевом слове, которое позволит тупо игнорить исключения и просто продолжать своё выполнение, в отдельно взятом за жопу куске кода. Ближайший аналог приснопамятный On Error Resume Next.
            Сишный аналог - каждая операция обернута в try, с кетчем который вызывает единый, заданный кодером обработчик ошибок.

            >этот код нужен только исключительно для обработки исключений, что бы гарантировать что объекты ведут себя согласно С++ стандартам.

            За удобство в любом случае придется чем-то заплатить, будь то скорость и/или размер программы. Не спорю.
            Ответить

          • ava bormand 21.12.2012 22:18 # +2

            > if'ы есть часть нормального кода
            Вот кстати не всегда... Ифы, проверяющие коды возврата, закрывающие всякие ресурсы и освобождающие память это как раз обработка исключений. Только она, к сожалению, не генерируется компилятором, а пишется руками.

            > согласно С++ стандартам
            Согласно здравому смыслу. Иначе приходится городить конструкции с goto в конец и закрыванием всех файлов. А готофобам и того хуже...
            Ответить

    • ava mvuBfZP 25.08.2021 07:55 # 0

      - Что тут у вас творится? Даня заперся в комнате и, судя по голосу, рыдает.
      Ответить

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