Je hebt verschillende problemen met je code. Ik zal het regel voor regel behandelen:
MemoryStream ms =new MemoryStream();
byte[] PhotoByte=null;
PhotoByte=ms.ToArray();
pictureBox1.Image.Save(ms, ImageFormat.Jpeg);
PhotoByte =ms.ToArray();
Hoewel het geen probleem is, heb je hier onnodige opdrachten. De bovenstaande code zou op deze manier duidelijker kunnen worden geschreven:
MemoryStream ms =new MemoryStream();
pictureBox1.Image.Save(ms, ImageFormat.Jpeg);
byte[] PhotoByte =ms.ToArray();
Vervolgens gebruikt de volgende code geen parameters. Altijd, altijd, ALTIJD parametriseer uw SQL-query's in plaats van de SQL dynamisch te bouwen. Nee, serieus, altijd. Ja, zelfs dan. (Ook, wat is de Str
variabel? Een soort hergebruikte instantievariabele? Doe dat niet.)
Str = "insert into Experimmm Values('" + PhotoByte + "','" + textBox1.Text + "')";
Conn.Open();
cmd.Connection = Conn;
cmd.CommandText = Str;
cmd.ExecuteNonQuery();
Conn.Close();
In plaats daarvan zou het dit moeten zijn:
Conn.Open();
using(SqlCommand cmd = connection.CreateCommand())
{
cmd.CommandText = "insert into Experimmm (column list) values(@data, @name)";
cmd.Parameters.Add("@data", SqlDbType.VarBinary).Value = PhotoByte;
cmd.Parameters.Add("@name", SqlDbType.VarChar, yourlength).Value = textBox1.Text;
cmd.ExecuteNonQuery();
}
Conn.Close();
Vervolgens gaan we verder met het ophalen. Opnieuw met de Str
variabel, doe dit soort dingen niet. U moet deze query ook parametriseren.
byte[] data;
string name;
Conn.Open();
using(SqlCommand cmd = Conn.CreateCommand())
{
cmd.CommandText = "select column_list from Experimmm where id = @id";
cmd.Parameters.Add("@id", SqlDbType.VarChar, field_length).Value = textBox2.Text;
using(SqlDataReader dr = cmd.ExecuteReader())
{
if (dr.Read())
{
data = (byte[])dr.GetValue(0);
name = (string)dr.GetValue(1);
}
}
}
Conn.Close();
label1.Text = name;
pictureBox2.Image = Image.FromStream(new MemoryStream(data));