Member 14647607 Ответов: 1

Невозможно привести объект типа "system.int32" к типу "system.byte".?

The image is saved in access database in “Long binary data” format but when retrieving the same image cause an error in C#“
Unable to cast object of type 'system.int32' to type 'system.byte '.? ”

Error:

Unable to cast object of type 'system.int32' to type 'system.byte '.? ”


Что я уже пробовал:

View Image from access databse code:

private void ViewImage_Click(object sender, EventArgs e)
        {
            OleDbConnection connection = new OleDbConnection();
            connection.ConnectionString = @"Provider=Microsoft.ACE.OLEDB.12.0;Data Source =C:\Users\Shabbir\source\repos\Pricing\bin\Debug\Database\Pricing.accdb;
                Persist Security Info=False;";
            connection.Open();

OleDbCommand command = new OleDbCommand("select * From tbl_TestImage where Name='" + textBoxName.Text + "' ", connection);

         
            OleDbDataReader reader = command.ExecuteReader();

          while (reader.Read())
            {

                lbl_photoName.Text = reader[1].ToString();
                pictureBox1.Image = LoadPhoto((byte[])reader[3]);
            
               

            }
        }

        private Image LoadPhoto(byte[] photo)
        {
            MemoryStream ms = new MemoryStream(photo);
            return Image.FromStream(ms);
        }

    } }

1 Ответов

Рейтинг:
9

OriginalGriff

Боюсь, там полно плохих идей.

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

Когда вы объединяете строки, вы вызываете проблемы, потому что SQL получает такие команды, как:

SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
Цитата, добавленная пользователем, завершает строку в том, что касается SQL, и вы получаете проблемы. Но могло быть и хуже. Если я приду и наберу вместо этого: "x';DROP TABLE MyTable;--", то SQL получит совсем другую команду:
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Которые SQL видит как три отдельные команды:
SELECT * FROM MyTable WHERE StreetAddress = 'x';
Совершенно правильный выбор
DROP TABLE MyTable;
Вполне допустимая команда "удалить таблицу" 
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?

Во - вторых, никогда жестко не кодируйте строки соединений-они всегда должны храниться в конфигурационном файле той или иной формы.

В-третьих, вы не должны использовать SELECT * FROM, и вы очень особенно никогда не должны использовать SELECT * FROM а затем получите доступ к полученным данным с помощью числовых индексов: если база данных будет изменена, ваш код будет нарушен неприятным образом, что может привести к повреждению вашей базы данных без возможности восстановления.
И это также неэффективно - никогда не выбирайте столбцы, которые вы не собираетесь использовать!
Вместо этого используйте SELECT Column1, Column2 FROM и перечислите фактические столбцы, которые вас интересуют, а затем получите доступ к ним с помощью числового (или лучше текстового) индекса. Это делает программное обеспечение более надежным, а также более читаемым и ремонтопригодным.

И, скорее всего, это исправит вашу проблему - я подозреваю, что столбец с индексом 3, который вы возвращаете, вовсе не является данными изображения, а целочисленным значением...

phil.o

Похоже, что хорошие советы не так уж приветствуются новичками в наши дни...

OriginalGriff

Сценарий детишкам не нравится учить. :вздох:

Maciej Los

5ed!