Goran Bibic Ответов: 2

Сохранение и извлечение изображения из SQL C#

Everything working fine i I have picture in picture box when insert into sql.

If is picturebox empty have error

Object reference not set to instanceof an object.  Line 152

Line 152 is:

dataPictureBox.Image.Save(ms, dataPictureBox.Image.RawFormat);

 

Reciving image from database is ok...

 

Some help please

Complete code down


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

private void button5_Click(object sender, EventArgs e)
        {
            byte[] img_arr = null;
            MemoryStream ms = new MemoryStream();
            dataPictureBox.Image.Save(ms, dataPictureBox.Image.RawFormat);
            img_arr = ms.GetBuffer();

            if (imeTextBox.ReadOnly == false)
            {



                if (string.IsNullOrEmpty(idTextBox.Text))
                {

                    using (SqlConnection openCon = new SqlConnection(cs))
                    {
                        string saveStaff = "declare @maxNo integer = 0 select @maxNo = isnull(max(redni_broj), 0) from [dbo].[roba_usluge]; Set @maxNo=@maxNo+1; INSERT into dbo.roba_usluge (redni_broj, ime, data) VALUES (@maxNo,@ime,@data)";

                        using (SqlCommand querySaveStaff = new SqlCommand(saveStaff))
                        {

                            querySaveStaff.Connection = openCon;

                            querySaveStaff.Parameters.Add("@ime", SqlDbType.VarChar, 255).Value = imeTextBox.Text;

                            querySaveStaff.Parameters.AddWithValue("@data", img_arr);


                            openCon.Open();
                            querySaveStaff.ExecuteNonQuery();
                            openCon.Close();

                        }

                    }
                }
                else
                {

                    using (SqlConnection openCon = new SqlConnection(cs))
                    {
                        string saveStaff = "UPDATE  dbo.roba_usluge SET redni_broj=@redni_broj, ime=@ime, data=@data  WHERE id= " + idTextBox.Text;



                        using (SqlCommand querySaveStaff = new SqlCommand(saveStaff))
                        {
                            querySaveStaff.Connection = openCon;
                            querySaveStaff.Parameters.Add("@redni_broj", SqlDbType.Int).Value = redni_brojTextBox.Text;
                            querySaveStaff.Parameters.Add("@ime", SqlDbType.VarChar, 255).Value = imeTextBox.Text;                           

                            querySaveStaff.Parameters.AddWithValue("@data", img_arr);



                            openCon.Open();
                            querySaveStaff.ExecuteNonQuery();
                            MessageBox.Show("Uspješno ste izmenili stavku!", "Informacija", MessageBoxButtons.OK, MessageBoxIcon.Information);
                            openCon.Close();
                        }
                    }
                }

            }
            else
            {

                MessageBox.Show("Dokument je već potvrđen! Unesite novi ili izmjenite postojeci!", "Obavještenje", MessageBoxButtons.OK, MessageBoxIcon.Information);

            }




        }

F-ES Sitecore

dataPictureBox - это null или dataPictureBox.Изображение равно нулю. Из кода, который вы опубликовали, невозможно сказать, что, или почему, или что вам нужно сделать, чтобы исправить это.

Richard Deeming 

declare @maxNo integer = 0 select @maxNo = isnull(max(redni_broj), 0) from [dbo].[roba_usluge]; Set @maxNo=@maxNo+1; ...

Не делай этого так. Если несколько пользователей одновременно обращаются к вашей системе, вы в конечном итоге попытаетесь вставить один и тот же первичный ключ несколько раз. Использовать IDENTITY вместо этого колонка.

string saveStaff = "UPDATE  dbo.roba_usluge SET redni_broj=@redni_broj, ime=@ime, data=@data  WHERE id= " + idTextBox.Text;

Еще раз повторюсь, SQL-инъекция:
Все, что вы хотели знать о SQL-инъекции (но боялись спросить) | Трой Хант[^]
Как я могу объяснить SQL-инъекцию без технического жаргона? | Обмен Стеками Информационной Безопасности[^]
Шпаргалка по параметризации запросов | OWASP[^]

Вы уже знаете, как использовать параметры, так как вы уже используете их в том же запросе. Так почему же вы продолжаете писать уязвимый код снова и снова?!

2 Ответов

Рейтинг:
2

Goran Bibic

Решено спасибо

Replace this Line 
dataPictureBox.Image.Save(ms, dataPictureBox.Image.RawFormat);
To
bool isNullOrEmpty = dataPictureBox == null || dataPictureBox.Image == null;
if(!isNullOrEmpty)
dataPictureBox.Image.Save(ms, dataPictureBox.Image.RawFormat);

Рейтинг:
15

OriginalGriff

Это одна из самых распространенных проблем, которые нам задают, и это также та, на которую мы меньше всего готовы ответить, но вы больше всего готовы ответить сами.

Позвольте мне просто объяснить, что означает ошибка: вы пытались использовать переменную, свойство или возвращаемое значение метода, но оно содержит null - что означает, что в переменной нет экземпляра класса.
Это немного похоже на карман: у вас есть карман в рубашке, который вы используете, чтобы держать ручку. Если вы сунете руку в карман и обнаружите, что там нет ручки, вы не сможете подписать свое имя на листе бумаги - и вы получите очень смешные взгляды, если попытаетесь! Пустой карман дает вам нулевое значение (здесь нет ручки!), поэтому вы не можете сделать ничего, что обычно делали бы, когда вы извлекли свою ручку. Почему он пуст? Вот в чем вопрос - может быть, вы забыли взять ручку, когда уходили из дома сегодня утром, или, возможно, вы оставили ручку в кармане вчерашней рубашки, когда снимали ее вчера вечером.

Мы не можем сказать, потому что нас там не было, и, что еще важнее, мы даже не можем видеть вашу рубашку, не говоря уже о том, что находится в кармане!

Вернемся к компьютерам, и вы каким - то образом сделали то же самое-и мы не можем увидеть ваш код, а тем более запустить его и узнать, что содержит null, когда это не должно быть.
Но вы можете - и Visual Studio поможет вам здесь. Запустите свою программу в отладчике, и когда она выйдет из строя, VS покажет вам строку, в которой она обнаружила проблему. Затем вы можете начать просматривать различные его части, чтобы увидеть, какое значение равно null, и начать просматривать свой код, чтобы узнать, почему. Поэтому поставьте точку останова в начале метода, содержащего строку ошибки, и снова запустите программу с самого начала. На этот раз VS остановится перед ошибкой и позволит вам изучить, что происходит, пройдя через код, глядя на ваши значения.

Но мы не можем этого сделать - у нас нет вашего кода, мы не знаем, как его использовать, если бы он у нас был, у нас нет ваших данных. Так что попробуйте - и посмотрите, сколько информации вы можете узнать!