Mensajes

Obviamente mi código era de referencia, y el listview lo cree para listar las imágenes.

Pero bueno ya solucionaste no?

EDIT:

Ahora que veo tu código tienes varios problemas que, creo, deberías echarle un ojo:

1.- Considera usar una struct o clase, para almacenar la data en vez de un arreglo
2.- Qué pasa si la imagen no es jpeg?
3.- Si la aplicación final va a usar SqlServerExpress no pongas las fotos en la BD porque la versión express tiene un límite de almacenamiento.
4.- Considera usar una clase de acceso a datos en vez de hacerlo todo en un método.
5.- En:

Código [Seleccionar] Expandir

String sentenciaSQL = "SELECT * FROM contactos WHERE apodo = 'pavolino'"; Seleccionas * , por lo que la consulta se hace más lenta, considera seleccionar las columnas que necesitas, así sean todas. Además de que nada asegura de que estas usando la imagen correcta al momento de cargar la data ya que estas usando un foreach en vez de validar si hay registros en el datatable y luego usar el primero.
6.- Con respecto a la base de datos, siempre utiliza un primarykey decimal, autoincremental, es decir, identity, con eso creas una identidad para el sistema.

Un saludo!

hola, gracias. te comento:

1. lo intentaré hacer así xD
2. Eso es lo que me preguntaba yo unos post más atrás.
3. Seguramente termine usando mysql, de momento ando probando con imágenes pequeñajas. xD
4. Sí, en mi código lo hago por separado, ya que utilizo un patron dao y el acceso lo hago en otra clase.
5. Pero si quiero seleccionar todos los datos??? no entiendo muy bien como se haría..
6. Sí, ahora solo estoy de pruebas, luego tendré que usar un primary key para hacer las búsquedas.

@Hartigan en el insert solo inserta la imagen y un string para la consulta, luego prueba nuevamente que valores guarda.

Código (sql) [Seleccionar] Expandir

INSERT INTO <nombre_tabla>
[(<campo1>[,<campo2>,...])]
values
(<valor1>,<valor2>,...);

debes especificar el campo y el valor para hacerlo.

No entiendo muy  bien como sería.... dices que le pase la imagen y un array para los datos???...

Una cosa mas, estas utilizando:

Código (csharp) [Seleccionar] Expandir

'" + img + "'

saca las dobles comillas.

ok, voy a probar a ver.

Muchas gracias a los 2.

Obviamente mi código era de referencia, y el listview lo cree para listar las imágenes.

Pero bueno ya solucionaste no?

ya me imaigno. Que va tio, me sigue pasando lo mismo. El problema es como dice el compañero 43H4FH44H45H4CH49H56H45H  que los datos que guardo y los que recojo son completamente distintos. Al guardar los bytes son: 255 216 255 224....... y al leer de la base de datos, la variable img tiene los siguientes bytes: 83 121 115 116...... como puedes ver no se parecen en nada. Entonces debe ser o que guardo mal, o que recojo mal, pero esque he seguido vuestros códigos y debería estar bien, no veo el fallo por ningun lado... A ver si me podeis ayudar.

Saludos y perdonar todas las molestias que estoy causando.

Gracias.

Bueno pues efectivamente ahí está el problema.

Os publico como hago para guardar y para leer de la base de datos a ver si veis el error

Guardar:

Código (csharp) [Seleccionar] Expandir


private void button_AceptarNC_Click(object sender, EventArgs e)
       {
           bool exito = false;
           //Primero copiamos las strings
           string[] datos = new string[33];
           int imgLength = 0;

           /*string temp = Path.GetTempFileName();
           FileStream fs = new FileStream(temp, FileMode.OpenOrCreate, FileAccess.ReadWrite);*/

           System.IO.Stream derecho = new System.IO.MemoryStream();
           this.pictureBox_FotoImagen.Image.Save(derecho, System.Drawing.Imaging.ImageFormat.Jpeg);
           byte[] img = new byte[derecho.Length];
           derecho.Position = 0;
           derecho.Read(img, 0, System.Convert.ToInt32(derecho.Length));
           derecho.Close();
           this.textBox1.Text = img[0].ToString() + " " + img[1].ToString() + " " + img[2].ToString() + " " + img[3].ToString();
           

           
           //System.IO.Stream stream = new System.IO.MemoryStream();

           datos[0] = this.textBox_Titulo.Text;
           datos[1] = this.textBox_Nombre.Text;
           datos[2] = this.textBox_SegundoNombre.Text;
           datos[3] = this.textBox_PrimerApellido.Text;
           datos[4] = this.textBox_SegundoApellido.Text;
           datos[5] = this.textBox_Sobrenombre.Text;
           datos[6] = this.textBox_Direccion.Text;
           datos[7] = this.textBox_Ciudad.Text;
           datos[8] = this.textBox_Provincia.Text;
           datos[9] = this.textBox_CodigoPostal.Text;
           datos[10] = this.textBox_Pais.Text;
           datos[11] = this.textBox_Prefijo_Telefono.Text + this.textBox_Telefono.Text;
           datos[12] = this.textBox_Movil.Text;
           datos[13] = this.textBox_Fax.Text;
           datos[14] = this.listBox_Email.Text;
           datos[15] = this.textBox_Web.Text;
           datos[16] = this.textBox_Twitter.Text;
           datos[17] = this.textBox_Facebook.Text;
           datos[18] = this.textBox_Direccion_Trabajo.Text;
           datos[19] = this.textBox_Ciudad_Trabajo.Text;
           datos[20] = this.textBox_Provincia_Trabajo.Text;
           datos[21] = this.textBox_CP_Trabajo.Text;
           datos[22] = this.textBox_Pais_Trabajo.Text;
           datos[23] = this.textBox_Organizacion_Trabajo.Text;
           datos[24] = this.textBox_Puesto_Trabajo.Text;
           datos[25] = this.textBox_Departamento_Trabajo.Text;
           datos[26] = this.textBox_Oficina_Trabajo.Text;
           datos[27] = this.textBox_Telefono_Trabajo.Text;
           datos[28] = this.textBox_Fax_Trabajo.Text;
           datos[29] = this.comboBox_Sexo.Text;
           datos[30] = this.textBox_Pareja.Text;
           datos[31] = this.listBox_Hijos.Text;
           datos[32] = this.richTextBox_Comentario.Text;


           /*this.pictureBox_FotoImagen.Image.Save(fs, System.Drawing.Imaging.ImageFormat.Jpeg);
           fs.Position = 0;

           imgLength = Convert.ToInt32(fs.Length);
           byte[] img = new byte[imgLength];
           fs.Read(img, 0, imgLength);
           fs.Close();*/

           String sentenciaSQL = "insert into contactos values('" + datos[0] + "','" + datos[1] + "','" + datos[2] + "','" +
                    datos[3] + "','" + datos[4] + "','" + datos[5] + "','" + datos[6] + "','" + datos[7] + "','" + datos[8] + "','" +
                    datos[9] + "','" + datos[10] + "','" + datos[11] + "','" + datos[12] + "','" + datos[13] + "','" + datos[14] + "','" +
                    datos[15] + "','" + datos[16] + "','" + datos[17] + "','" + datos[18] + "','" + datos[19] + "','" + datos[20] + "','" +
                    datos[21] + "','" + datos[22] + "','" + datos[23] + "','" + datos[24] + "','" + datos[25] + "','" + datos[26] + "','" +
                    datos[27] + "','" + datos[28] + "','" + datos[29] + "','" + datos[30] + "','" + datos[31] + "','" + datos[32] + "','" + img + "')";

           SqlConnection conexion = null;

           try
           {
               conexion = new SqlConnection(cadena_conexion);
               conexion.Open();

               SqlCommand cmd = new SqlCommand(sentenciaSQL, conexion);
               
               if ((1 != cmd.ExecuteNonQuery()))
               {
                   MessageBox.Show("Se ha producido un error al acceder a la fuente de datos", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
                   exito = false;
               }
               else
               {
                   exito = true;
               }

           }
           catch (SqlException ex)
           {
               // MessageBox.Show(ex.InnerException.Message, "error", MessageBoxButtons.OK, MessageBoxIcon.Error);
               MessageBox.Show(" ha producido un error al acceder a la fuente de datos", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
               exito = false;
           }
           finally
           {
               if (conexion != null)
               {
                   if (conexion.State == ConnectionState.Open)
                   {
                       conexion.Close();
                   }
               }
           }
if (exito)
           {
               MessageBox.Show("Contacto añadido con exito", "Éxito", MessageBoxButtons.OK, MessageBoxIcon.Information);
               this.Close();
           }
           else
           {
               MessageBox.Show("No se ha podido añadir el contacto", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
               this.Close();
           }

           
           


       }
     

Ahora como leo:

Código (csharp) [Seleccionar] Expandir


public Interfaz_Datos_Contacto()
       {
           InitializeComponent();

           
           string cadena_conexion = @"Server= (local)\SQLEXPRESS; DataBase = agenda; Integrated Security = yes;";

           byte[] img = new byte[0];
           string[] datos = new string[33];

           

           String sentenciaSQL = "SELECT * FROM contactos WHERE apodo = 'pavolino'";

           SqlConnection conexion = null;

           try
           {
               conexion = new SqlConnection(cadena_conexion);
               SqlDataAdapter data_adapter = new SqlDataAdapter(sentenciaSQL, conexion);

               DataTable data_table = new DataTable();
               
               conexion.Open();
               data_adapter.Fill(data_table);

               datos = new string[33];
               datos[0] = data_table.Rows[0][0].ToString();
               datos[1] = data_table.Rows[0][1].ToString();
               datos[2] = data_table.Rows[0][2].ToString();
               datos[3] = data_table.Rows[0][3].ToString();
               datos[4] = data_table.Rows[0][4].ToString();
               datos[5] = data_table.Rows[0][5].ToString();
               datos[6] = data_table.Rows[0][6].ToString();
               datos[7] = data_table.Rows[0][7].ToString();
               datos[8] = data_table.Rows[0][8].ToString();
               datos[9] = data_table.Rows[0][9].ToString();
               datos[10] = data_table.Rows[0][10].ToString();
               datos[11] = data_table.Rows[0][11].ToString();
               datos[12] = data_table.Rows[0][12].ToString();
               datos[13] = data_table.Rows[0][13].ToString();
               datos[14] = data_table.Rows[0][14].ToString();
               datos[15] = data_table.Rows[0][15].ToString();
               datos[16] = data_table.Rows[0][16].ToString();
               datos[17] = data_table.Rows[0][17].ToString();
               datos[18] = data_table.Rows[0][18].ToString();
               datos[19] = data_table.Rows[0][19].ToString();
               datos[20] = data_table.Rows[0][20].ToString();
               datos[21] = data_table.Rows[0][21].ToString();
               datos[22] = data_table.Rows[0][22].ToString();
               datos[23] = data_table.Rows[0][23].ToString();
               datos[24] = data_table.Rows[0][24].ToString();
               datos[25] = data_table.Rows[0][25].ToString();
               datos[26] = data_table.Rows[0][26].ToString();
               datos[27] = data_table.Rows[0][27].ToString();
               datos[28] = data_table.Rows[0][28].ToString();
               datos[29] = data_table.Rows[0][29].ToString();
               datos[30] = data_table.Rows[0][30].ToString();
               datos[31] = data_table.Rows[0][31].ToString();
               datos[32] = data_table.Rows[0][32].ToString();
               
               

               
               
               foreach (DataRow myRow in data_table.Rows)
               {
                   img = (byte[])myRow["imagen"];
                   this.textBox1.Text = img[0].ToString() + " " + img[1].ToString() + " " + img[2].ToString() + " " + img[3].ToString();
                   
               }

           }
           catch (SqlException ex)
           {
               datos = null;
               MessageBox.Show("No se ha podido acceder a la fuente de datos", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
           }
           finally
           {
               if (conexion != null)
               {
                   if (conexion.State == ConnectionState.Open)
                   {
                       conexion.Close();              
                     
                   }
               }
           }
if ((datos != null) && ( img != null))
           {
               label_Nombre_Completo.Text = datos[1] + "" + datos[2] + "" + datos[3] + "" + datos[4];
               label_Titulo.Text = datos[0];
               label_Nombre.Text = datos[1];
               label_SegundoNombre.Text = datos[2];
               label_PrimerApellido.Text = datos[3];
               label_SegundoApellido.Text = datos[4];
               label_Sobrenombre.Text = datos[5];
               label_Direccion.Text = datos[6];
               label_Ciudad.Text = datos[7];
               label_EstadoProvincia.Text = datos[8];
               label_CodigoPostal.Text = datos[9];
               label_PaisRegion.Text = datos[10];
               label_Telefono.Text = datos[11];
               label_Movil.Text = datos[12];
               label_Fax.Text = datos[13];
               label_EMail.Text = datos[14];
               label_Web.Text = datos[15];
               label_Twitter.Text = datos[16];
               label_Facebook.Text = datos[17];
               label_Direccion_Trabajo.Text = datos[18];
               label_Ciudad_Trabajo.Text = datos[19];
               label_Provincia_Trabajo.Text = datos[20];
               label_CP_Trabajo.Text = datos[21];
               label_Pais_Trabajo.Text = datos[22];
               label_Organizacion_Trabajo.Text = datos[23];
               label_Puesto_Trabajo.Text = datos[24];
               label_Departamento_Trabajo.Text = datos[25];
               label_Oficina_Trabajo.Text = datos[26];
               label_Telefono_Trabajo.Text = datos[27];
               label_Fax_Trabajo.Text = datos[28];
               label_Sexo.Text = datos[29];
               label_Pareja.Text = datos[30];
               label_Hijos.Text = datos[31];
               richTextBox_Comentarios.Text = datos[32];
               
               

               MemoryStream stream = new MemoryStream(img);
               stream.Write(img, 0, img.Length);
               stream = new MemoryStream(img);
               this.pictureBox_FotoImagen.Image = Image.FromStream(stream);      

             

          }
           else
           {
               MessageBox.Show("Se ha producido un error al obtener los datos del contacto", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
           }



       }

He dejado el código donde compruebo los bytes y es ceirto que salen completamente distintos.

Salu2.

EDITO: la tabla de los contactos en la base de datos la tengo así:

Código (sql) [Seleccionar] Expandir


CREATE TABLE contactos(
titulo varchar(50),
nombre varchar(50) NOT NULL,
nombre2 varchar(50),
ape1 varchar(50) NOT NULL,
ape2 varchar(50),
apodo varchar(50) NOT NULL,
direccion varchar(150),
ciudad varchar(20),
provincia varchar(20),
cp varchar(10),
pais varchar(20),
telefono varchar(20),
movil varchar(20),
fax varchar(20),
email varchar(50),
web varchar(50),
twitter varchar(50),
facebook varchar(50),
direccion2 varchar(150),
ciudad2 varchar(20),
provincia2 varchar(20),
cp2 varchar(10),
pais2 varchar(20),
organizacion varchar(50),
puesto varchar(50),
departamento varchar(50),
oficina varchar(50),
telefono2 varchar(20),
fax2 varchar(20),
sexo varchar(10),
pareja varchar(50),
hijos varchar(100),
comentario varchar(500),
imagen image,

);

Bueno no hay manera. He seguido tropecientos tutoriales y esque hay algo que no va bien. En la parte en la que se hace la conversión de bytes al tipo image, toma el valor null, no se porqué y entonces no le asigna nada... Creo que voy a desistir y a probar el método de almacenar solo la dirección local de la imágen.

Puedes leer que almaceno en la BD sin asignar la imagen al picturebox.
Si leemos algunos bytes antes de almacenarlos:

Código (csharp) [Seleccionar] Expandir

System.IO.Stream derecho = new System.IO.MemoryStream();
this.pictureBox1.Image.Save(derecho, System.Drawing.Imaging.ImageFormat.Jpeg);
byte[] imgDerecha = new byte[derecho.Length];
derecho.Position = 0;
derecho.Read(imgDerecha, 0, System.Convert.ToInt32(derecho.Length));
derecho.Close();
this.textBox1.Text = imgDerecha[0].ToString() + " " + imgDerecha[1].ToString();

y luego con la BD antes de asignar la imagen leemos los mismos bytes para saber si se almaceno correctamente asi podemos comparar con los bytes originales.
Puedes hacerlo con el primero y el ultimo o hacer un foreach para saber donde existe alguna diferencia.
Me parece que no estas guardando correctamente los bytes en la BD.
Pero tienes que hacer las pruebas para estar seguro en que parte tienes el problema.

Tiene que ser eso, que se guardan mal los datos, porque incluso con el programa de D4N93R me sale el mismo error en el listview (parámetro no válido).

Voy a probar a ver. Muchas gracias.

Cómo que no tienes un listview? claro que si, arrástralo de la barra de herramientas. y le pones el mismo nombre que en mi código.

Vamos vamos que si puedes..

si tener lo tengo, pero en mi programa no tengo intención de añadirlo. xDD Probaré aunque sea por ver si funciona. xD

Bueno no hay manera. He seguido tropecientos tutoriales y esque hay algo que no va bien. En la parte en la que se hace la conversión de bytes al tipo image, toma el valor null, no se porqué y entonces no le asigna nada... Creo que voy a desistir y a probar el método de almacenar solo la dirección local de la imágen.

ya me funciona tu programa D4N93R, a ver si consigo aplicarlo al mio, pero veo un inconveniente que yo no tengo un listview de esos, y entonces como puedo usar el foreach????

Lool, no me funciona tu programa. xD

Acá te dejo esto que acabo de hacer, espero te funcione:

Código C#:
http://pastebin.com/Vp1XLfit

Código del component del formulario:

Código (csharp) [Seleccionar] Expandir


        private System.Windows.Forms.PictureBox pictureBox1;
        private System.Windows.Forms.Button button1;
        private System.Windows.Forms.ListView listView1;
        private System.Windows.Forms.Button button2;
        private System.Windows.Forms.TextBox m_imageTitle;
        private System.Windows.Forms.Label label1;


Base de datos
Una tabla, llamada Image, con las siguientes columnas:

- ID identiy decimal
- Title varchar
- ImageData de tipo Image.


PD: disculpa que no tiene comentarios, y los controles tienen el nombre por defecto, pero no tengo mucho tiempo ahora. Si tienes una duda no dudes en consultar..

Saludos

Muchas gracias voy a mirarlo a ver.

una pregunta. Es necesario guardar un nombre para imagen?? porque yo solamente guardo el campo imagen y veo que tu y más gente creais tres campos... esque yo tengo una tabla con todos los datos del contacto (nombre, apellidos... ) y uno de ellos es un campo (imagen image). Con eso debería servir no???.-

Salu2. y gracias,

Esta bien a menos que sepas leer una imagen en hexadecimal a simple vista

yo te digo que lo pongas en un picturebox ysi se ve la imagen, es porque el error esta en otro lado. Sino se ve, esporque pusiste mal la imagen

Esque no me carga, porque da error.

Cuando yo al principio selecciono la imágen en un picture box para guardarla luego en la base de datos lo hace bien. Luego al leerla y sacarla a otro picturebox ya no funciona bien. Y la imágen parece que la coge bien de la base de datos... pero vete tu a saber...