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

    +118

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    SqlCommand command = new SqlCommand ();
command.Connection = connection;
command.CommandText = "select data from filestorage where id=@id;";            
command.Parameters.AddWithValue ("@id", fileId);
command.CommandType = CommandType.Text;

byte [] data = (byte []) command.ExecuteScalar ();

System.IO.Stream outStream = Response.OutputStream;

Response.ContentType = "Application/octet-stream";
Response.AppendHeader ("Connection", "keep-alive");

Response.AppendHeader ("Content-Disposition", " attachment; filename = \"" + fName+"\"" );

outStream.Write (data, 0, data.Length);
Response.End ();

    продолжение http://govnokod.ru/4227

    Запостил: akai_mirror, 12 Сентября 2010

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

    • ava asfalanaft 13.09.2010 08:50 # −3

      В чем конкретно говнокод? В приведении типов? В хранении данных в таблице и выборке по id без какой-либо проверки? В самом языке C#?
      Ответить

      • ava Altravert 13.09.2010 10:12 # 0

        Последнее. Точно. Готовьтесь: скоро весь код на шарпе будет здесь. А потом других языков, которые кому-то не нравятся. И всё: это коллектор кусков.
        Ответить

        • ava Анонимус 13.09.2010 19:03 # +1

          C# прекрасный язык. А код говно
          Ответить

          • ava Altravert 14.09.2010 06:55 # +1

            Я не спорю, что шарп прекрасен. Видимо, мой сарказм был как всегда не очевиден.
            Ответить

            • ava Lure Of Chaos 15.09.2010 00:07 # +1

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

              • ava Altravert 15.09.2010 05:16 # 0

                Ну, не совершенен, я этого не говорил)) Есть мнение про несколько языков, что они абсолютное говно, но шарп в них точно не входит.
                Ответить

                • ava Lure Of Chaos 15.09.2010 12:41 # +2

                  причем в свое время и эти языки не считались говном )
                  Ответить

                  • ava Altravert 22.09.2010 05:23 # 0

                    Когда-нить и c/c++ абсолютным говном станет, но наверно это уже не моё поколение будет, и я рад))
                    Ответить

      • ava Lexey 13.09.2010 11:38 # +1

        Как минимум в отсутствии вызова Dispose для команды.
        Ответить

      • ava akai_mirror 13.09.2010 14:51 # +1

        говнокод в том, что данные из бд (а там есть записи >100MB) сначала целиком читаются в память, и только потом отдаются клиенту. сервер уходит в глубокий своп при уже десятках запросов.
        Ответить

        • ava Анонимус 13.09.2010 19:03 # +1

          непонимание автором концепции стримов

          видимо пхппист
          Ответить

        • ava asfalanaft 15.09.2010 14:00 # −1

          То есть вы предлагаете читать кусочками через GetBytes, собирая файл, так? А где указано, что у вас именно столь большие данные?
          Ответить

          • ava Lure Of Chaos 16.09.2010 17:30 # +2

            зачем. вообще не надо нигде запоминать\собирать
            Ответить

            • ava asfalanaft 18.09.2010 11:50 # 0

              Расскажите, как отдать данные, хранящиеся в базе, не делая этого :)
              Ответить

      • ava Анонимус 13.09.2010 19:02 # +2

        В каше из SQL и HTTP в одном классе.
        В зашитом имени SQL клиента.
        В кешировании файла ввиде массива байт (хотя можно ведь было стримом связать)
        Ответить

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