Manejo de Archivos duda!

Iniciado por Mr.Blue, 4 Mayo 2010, 04:31 AM

0 Miembros y 1 Visitante están viendo este tema.

taul

Cita de: Littlehorse en  4 Mayo 2010, 09:12 AM
PD: sleep no es estándar.

ja siempre crei q era estadar...grax por la info siempre se aprende algo nuevo.

Mr.Blue

#11
aca pongo el codigo es uno ya escrito que lo unico que quise es optimizarlo el codigo fue publicado ya pero para no mesclar las dudas cree un tema aparte
es el codigo que puse no puse todo el codigo ya que crei que no era necesario ya que por donde creo q viene el problema talvez sea por el ingreso de datos que es la informacion que quiero guardar.
Código (cpp) [Seleccionar]
#include <stdio.h>
#include <conio.h>
#include <stdlib.h>
#include <string.h>

struct datos {
char nombre [15];// nombre del usuario
char pass [6];//password del usuario
char acceso [21];//acceso del usuario
char tipo [2];//tipo de acceso
};
void main()
{ int i=1;

datos regin;

FILE *pf; // apunta al archivo


pf=fopen ("C:\\users.txt","at+");
// C:\Program Files\Counter-Strike\cstrike\addons\amxmodx\configs
if (pf==NULL)
 {
printf ("ERROR DE LECTURA");
getch();
exit(1);
 }

 for(i=1;i<=1;i++) {
printf ("\nINGRESE NOMBRE DE USUARIO:");
gets (regin.nombre);

printf ("\nINGRESE PASSWORD DE USUARIO:");
gets (regin.pass);

printf ("\nINGRESE ACCESO:");
gets (regin.acceso);
printf ("\nINGRESE TIPO DE ACCESO:");
gets (regin.tipo);

fwrite (&regin,sizeof(datos),1,pf);
}
fclose (pf);

//re leo grabacion y compruebo
pf=fopen ("C:\\users.txt","r");
if (pf == NULL){
printf ("ERROR DE ACCESO");
getch();
exit(1);
}
fread (&regin,sizeof (datos),1,pf);

while (!feof(pf)){
fread (&regin,sizeof (datos),1,pf);
printf("NOMBRE DE USUARIO:%s    PASS:%s ACCESO:%s TIPO DE ACCESO:%s",regin.nombre,regin.pass,regin.acceso,regin.tipo);
fread (&regin,sizeof (datos),1,pf);
 }
fclose (pf);
getch();
}




edito: bueno ahi por lo q veo que dice littlehorse  voy a cambiar el gets pro un fgets aver que pasa

strlen: calcula la longitud de la cadena, no incluye el "\0" del final
sleep: si no me equivoco es un tiempo de espera sleep (1000); un segundo aunque la diferencia con delay (1000); no se cual es tube varios problemas una vuelta a programar en tc me andabla el delay pero en borlard o dev ninguna de las dos jaja me tiraba error de compilacion en esas lineas

EI: juntando mensajes.

Sigo con el problema ultimamente viendo, por ahi es un problema en las cadenas
aver como explicarme
en primer lugar vamos ignorar el overflow, por ahora.
bueno tengo q puedo ingresar en nombre 15 caracteres en pass 6 caracteres, ect
ahora esto esta dentro de una estructura para mi el error esta en que si ingreso en nombre 10 caracteres me estan sobrando 5 caracteres talves rellena los suigientes 5 caracteres con basura del buffer osea por lo q tenia entendido al apretar la tecla enter este actua como \0 o \n dependiendo que use

bueno ahora mi pregunta como implemento algo para q coloque el \0 al final de la cadena ingresada
en este momento estoy pensando en un strlen para q me cuente la longitud de la cadena y colocar un \0 al final de la cadena ingresada pero no se como implementarlo i si es de esta manera q deberia hacerlo.

Littlehorse

A pesar que no deberías usar gets, gets debería finalizar la cadena con un NULL en forma automática. Obviamente los problemas del overflow no se pueden obviar así que te vuelvo a decir que pruebes con fgets.

Por lo pronto sigue habiendo errores en ese codigo, falta un typedef.

En el while, haces esto:

mientras no sea el final
            leo el archivo;
                  imprimo;
            leo el archivo;

Si lo miras bien, cuando se realiza la segunda vuelta del ciclo, hay una lectura que no la imprimes, ya que en vez de imprimir el contenido previamente leido, vuelves a leer.

Usa el buscador y revisa los códigos que ya existen sobre estructuras, y mira también el manejo de fgets. Lo tendrás claro rápidamente y podrás recodificar ese ejercicio.

Saludos!
An expert is a man who has made all the mistakes which can be made, in a very narrow field.

taul

Littlehorse te repito...no puede usar fgets porque no esta leyendo un archivo.esta leyendo desde el teclado y guardandolo en un buffer.

el code q puse yo esta bien..probalo

cambia esto:

scanf("%14s",&regin.nombre);
scanf("%5s",&regin.pass);

saludos.

Littlehorse

#14
Taul, vamos por partes:

scanf("%14s",&regin.nombre);

Primero: El ampersand esta de mas. Creo que ya te he dicho, el identificador del array es un puntero que apunta a la primera posición de dicho arreglo. De la forma que lo haces solo funciona porque el puntero direcciona el mismo lugar que la primera referencia, intenta hacer lo mismo pero reservando memoria dinámica para el arreglo y veras mas claro el error.

Segundo: Si tienes 20 llamadas a scanf, y luego necesitas modificar el tamaño del arreglo, tienes que modificar todas las llamadas a scanf. Con fgets, la cantidad a leer es un argumento que puede ser pasado mediante -por ejemplo- una variable, una constante, una llamada a otra función, o lo que sea que se te ocurra.

Tercero: Precisamente leer de un archivo es leer desde un stream, y el input del teclado es precisamente, un stream.

fgets(buff,size,stdin);

Saludos!
An expert is a man who has made all the mistakes which can be made, in a very narrow field.

leogtz

No te ofendas, taul, pero, ¿así quieres crear un sistema operativo?
Código (perl) [Seleccionar]

(( 1 / 0 )) &> /dev/null || {
echo -e "stderrrrrrrrrrrrrrrrrrr";
}

http://leonardogtzr.wordpress.com/
leogutierrezramirez@gmail.com

Mr.Blue

haber haber disculpa Littlehorse que sea un toque duro de entender  :xD,
en primer lugar lo del typedef ? en donde me falta eso? en la estuctura ?

Código (cpp) [Seleccionar]
typedef struct datos {
char nombre [15];// nombre del usuario
char pass [6];//password del usuario
char acceso [15];//acceso del usuario
char tipo [2];//tipo de acceso
};


asi? o es otra cosa a lo q te referis?
segundo lugar
intento ingresar con fgets pero no tengo resultado aver si esta mal la sintaxis
Código (cpp) [Seleccionar]
fgets (regin.nombre,15, stdin); // o esta
fgets (regin.nombre,15, fp); //pongo las dos porq con ninguna de las dos hay resultado

otra cosa aca tmb que veo es el fgets lee el archivo asociado en la cadena de caracteres s.
osea lee una cadena de caracteres desde el archivo. pero mi cadena esta en la stuctura


PD: lo del overflow digo de que no importa por ahora porq quiero consentrar en lo del ingreso ese y bueno despues agregare lo del overflow pero por el momento la persona que ejecuta el programa sabe que no tiene q probocar un overflow  :laugh:


Otra cosa tmb, el progrmaa en si funciona, osea hace lo q tiene q hacer y guarda los datos ahora bueno que los guarde mal, es por lo q pregunte nuevamente el porque pasa esto, osea no es que no guarde lo q se ingresa por lo contrario se guarda pero junto con basura.

Desde ya gracias por las respuestas y Littlehorse gracias por la paciencia  :P jaja


Littlehorse

Cita de: eeprom46 en  8 Mayo 2010, 02:03 AM
haber haber disculpa Littlehorse que sea un toque duro de entender  :xD,
en primer lugar lo del typedef ? en donde me falta eso? en la estuctura ?

Código (cpp) [Seleccionar]
typedef struct datos {
char nombre [15];// nombre del usuario
char pass [6];//password del usuario
char acceso [15];//acceso del usuario
char tipo [2];//tipo de acceso
};


Es que vos en el código declaras la estructura así:

datos regin;

Y eso si no pones un typedef es incorrecto, en todo caso seria:


typedef struct data
{
//lo que sea
}datos;


De no hacer eso, al declarar la estructura tendrías que hacer:

struct data regin;


Citarintento ingresar con fgets pero no tengo resultado aver si esta mal la sintaxis
Código (cpp) [Seleccionar]
fgets (regin.nombre,15, stdin); // o esta
fgets (regin.nombre,15, fp); //pongo las dos porq con ninguna de las dos hay resultado

otra cosa aca tmb que veo es el fgets lee el archivo asociado en la cadena de caracteres s.
osea lee una cadena de caracteres desde el archivo. pero mi cadena esta en la stuctura


A lo que voy con fgets es que la uses para leer desde el teclado, es decir con stdin. Si utilizas fgets con fp (el puntero al archivo) lees una cadena desde el archivo que maneja ese puntero. A lo que me refería, a grandes rasgos, es que uses fgets en reemplazo de gets, no en reemplazo de fread.
También tienes algunas confusiones entre el modo binario y el modo texto en el manejo de archivos, son dos cosas distintas. Se tratan de distinta forma, y se usan distintas funciones. Busca información sobre ello para tener ciertos conceptos un poco mas sólidos.

CitarOtra cosa tmb, el progrmaa en si funciona, osea hace lo q tiene q hacer y guarda los datos ahora bueno que los guarde mal, es por lo q pregunte nuevamente el porque pasa esto, osea no es que no guarde lo q se ingresa por lo contrario se guarda pero junto con basura.

Revisa lo que te dije del while, la lectura la haces mal, no te enteras cuando llegas al final del archivo y te salteas una lectura a la hora de imprimir (es decir, lees dos veces, imprimes una).

Citar
Desde ya gracias por las respuestas y Littlehorse gracias por la paciencia  :P jaja

De nada, para eso estamos!

Saludos!
An expert is a man who has made all the mistakes which can be made, in a very narrow field.

Mr.Blue

Buenisimo gracias por la ayuda, ya me estoy a modificar los errores no pude el finde pero ahora tengo tiempo jaja

;-)

taul

no me ofendo para nada Leo Gutiérrez...yo programo de una forma y a lo mejor vos de otra..si esta bien programado o no eso pasa por cada uno..creo q podemos opinar diferente o no? para eso estamos juntados aca en el foro..porque todos vemos las cosas desde un punto de vista diferente...quien decide cual es la mejor opinion? pues el q formulo la pregunta...yo no vine al foro a criticar los conocimientos de nadie..solo opino al igual q cualquier usuario. si vos Leo Gutiérrez pensas q no se programar...respeto tu opinion..nadie se puede ofender por la opinion de alguin no?


Saludos...Taul.