Leer bloc de notas. URGEE :(

[only_skating]

Bueno pues hice un logg-in a mi manera y resulta que la profesora me dijo que asi no valia, que me buscase otra manera mejor, el problema esque no me explicó como hacerlo de otra manera, solo tengo un breve esquema de lo que entendi que me puso, pero el programa no me funciona, unicamente recoge el primer valor entero que encuentra, si ustedes pueden ayudarme a entenderlo se lo agradezco un montón:

Código C:

#include<stdio.h>
#include<stdlib.h>
 
void comprobar( int *id);
 
void main()
{
    int id=9999; // Usar el id del loggin el id
    comprobar(&id);
 
 
    system("pause");
 
}
 
void comprobar(int *id)
    {
        int id1,dia1,mes1;
        char nombre1[50];
        FILE *rd;
        rd = fopen("pacientes.txt","a+");
        if(rd==NULL)
        {
            printf("No se puede abrir el archivo\n");
            exit(1);
        }
 
        fprintf(rd,"9999 22 5 julio");
 
        fscanf(rd,"%d", &id1); // Te va leyendo todos los enteros
 
        if(*id==id1) // Y si alguno cumple esta condicion, es que esta registrado
        {
            fscanf(rd,"%d %d %d %s",&id1,&dia1,&mes1,nombre1[50]); // Ahora, va leyendo datos que esten en esta disposicion: entero(espacio)entero... y cuando encuentra el valor del primer entero, te guarda el de los dos enteros y cadena siguientes.
        
            printf("El usuario %s esta registrado, nacido el %d del mes %d \n" ,nombre1[50], dia1, mes1); 
        }
 
        if(*id!= id1)
        {
            printf("El usuario no esta registrado\n");
        }
 
}
 

[vosk]

Tal como lo tienes solo funciona cuando el archivo contiene 1 o mas entradas; me explico siguiendo tu codigo linea por linea suponiendo que el archivo 'pacientes.txt' está vacio:

Código:

rd = fopen("pacientes.txt","a+");

Ahora tienes un puntero al archivo (si existe lo abre y si no existe lo crea y abre). El modo 'a+' te permite leer y escribir teniendo en cuenta que los punteros de lectura y escritura (RW) avanzan a la vez: esto significa que ahora tienes el RW en posicion 0 ok?

Código:

fprintf(rd, "9999 22 5 julio");

Escribes 15 bytes, ahora el RW esta en posicion 15.

Código:

fscanf(rd, "%d", &id1);

Supongo que ahora ya ves donde tienes el error no? El RW sigue estando en posicion 15, es decir que cualquier lectura esta en EOF. Puedes solucionarlo resituando el puntero de lectura al principio:

Código:

fseek(rd, 0L, SEEK_SET);
fscanf(rd, "%d", &id1);

Ya tienes el 9999 en 'id1'

Saludos
vosk

[only_skating]

Ya probé a quitar lo de la escritura y buscar simplemente dentro del bloc de notas pero solo me cogia el primer int que encontraba y no pasaba a buscar mas.

¿Como hago para que recorra todos los int hasta encontrar el que busco de la manera anterior? yo lo hice de esta manera y me dijeron que la habia mucho mas facil.

Código C:

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#include<math.h>
void logg(int *id); //función en la que compruebo los datos introducidos por el usuario
void iniciosesion(int *id,char tsanitaria[], int longitudtsanitaria);//funcion en la que, una vez introducidos correctamente, creamos el puntero *id
void main(){
int id;
logg(&id);
printf("usuario con id %d",id); //prueba sin mas
system("pause");
}
 
void logg(int *id){
    char datos[10000],nacimiento[11]; //en la variable datos meto todo el archivo pacientes.txt para analizarlo
    char tsanitaria[4];
    int i=0,longitudtsanitaria,longitudnacimiento,j=0,x=0,contador=0,correcto=0,ok=0, prueba=0;
    while(ok==0){ //este bucle se repetirá hasta que el paciente introduzca bien sus datos
    int i=0,x=0,contador=0,correcto=0,j=0;
    FILE *rd; //abro archivo
    rd=fopen("pacientes.txt", "r");
    while((datos[i]=fgetc(rd))!=EOF){ ++i; } i=0;//Aquí es donde paso toda la informacion a una variable
    printf("Introduce tu tarjeta sanitaria\n");
    scanf("%s",tsanitaria);
    printf("Introduce tu fecha de nacimiento de la forma dd-mm-aaaa\n");
    scanf("%s",nacimiento);
    longitudtsanitaria= strlen(tsanitaria); //calculo la longitud para posteriores calculos
    longitudnacimiento= strlen(nacimiento); 
    while(datos[i] != '\0'){  //Aquí comienza el bucle que va recorriendo caracter por caracter hasta encontrar coincidencias con lo introducido por el paciente
    
                                  if(datos[i]==tsanitaria[0]){ x=i; //Si un caracter analizado coincide con el primer caracter de la tsanitaria introducido compruebo si coincide entero
                                          for(j=0;j<longitudtsanitaria;j++){ if(datos[x]==tsanitaria[j]){ contador++; if(contador==longitudtsanitaria){ correcto++;}} x++;}} contador=0;
                                  if(correcto==1){ 
                                      if(datos[i]=='\n'){ if(datos[i+1]=='\n'){ correcto=0;}}} /*Si al seguir buscando hay un doble salto de linea significa que estamos mirando en otro usuario, 
                                                                                               por lo menos no seria correcto introducir la tsanitaria de uno y el nacimiento de otro, por ello igualo
                                                                                               correcto=0 para que no de el logg-in como correcto*/
                                  if(datos[i]==nacimiento[0]){ x=i; //compruebo lo mismo pero con el nacimiento
                                          for(j=0;j<longitudnacimiento;j++){ if(datos[x]==nacimiento[j]){ contador++; if(contador==longitudnacimiento){ correcto++;}} x++;}}    contador=0;                                       
    i++;}
    if(correcto==2){ printf("Logg-in Correcto\n"); ok=1; //Si la tsanitaria y el nacimiento coindicen sin la existencia de dobles saltos de linea correcto sera igual a 2 y por tanto el loggin se completará
    iniciosesion(id, tsanitaria,longitudtsanitaria);} //aqui llamo a la función iniciar sesión para que genere el puntero y podamos operar
    else{ printf("logg-in incorrecto, reintente\n\n");
    }}
}
 

mil gracias de verdad

[cesar_casla]

hola!

la profesora me dijo que asi no valia

mmm q un profesor te diga "no vale" la verdad me parece muy mal. Para mi gusto mientras un codigo funcione, esta bien; luego se puede discutir si el codigo se puede optimizar, controlar errores, etc...

Te marco algunas cosas mas del primer codigo:
- estas haciendo dos lecturas en id1, es decir:

Código vb:

fscanf(rd,"%d", &id1);
if(*id==id1) // Y si alguno cumple esta condicion, es que esta registrado
        {
fscanf(rd,"%d %d %d %s",&id1,&dia1,&mes1,nombre1[50]);
 

si entra al if entonces volves a querer almacenar algo en id1, pero el cursor ya avando por lo tanto estarias leyendo mal. Ademas no tenes q poner nombre1[50] solo va nombre1

- id no hace falta q lo pases por referencia, ya q no haces ningun cambio en la funcion

- Para recorrer todo el archivo y encontrar el id deseado, solo hace falta poner todo en un bucle(comprobando q no sea fin de archivo)


En fin hace mucho no hago un programa, asi q subo las modificaciones(el archivo de texto debe estar cargado con datos):

Código C:

#include<stdio.h>
    #include<stdlib.h>
     
    void comprobar( int *id);
     
    int main()
    {
        int id=9999; // Usar el id del loggin el id
        comprobar(&id);
     
     
     //   system("pause");
     return 0;
    }
     
    void comprobar(int *id)
        {
            int id1,dia1,mes1;
            char nombre1[50];
            FILE *rd;
            rd = fopen("pacientes.txt","rt");
            if(rd==NULL)
            {
                printf("No se puede abrir el archivo\n");
                exit(1);
            }
            
            fscanf(rd,"%d %d %d %s",&id1,&dia1,&mes1,nombre1);
            while(!feof(rd)){ 
                
                if(*id==id1) // Y si alguno cumple esta condicion, es que esta registrado
                {   
                    printf("El usuario %s esta registrado, nacido el %d del mes %d \n" ,nombre1, dia1, mes1);
                    break; /* si se encuentra el usuario entonces no hace falta q recorra TODO el arhcivo.
                    Por lo tanto obligo a salir del bucle con el break */
                }
                fscanf(rd,"%d %d %d %s",&id1,&dia1,&mes1,nombre1);
            }
            if(*id!= id1)
            {
                printf("El usuario no esta registrado\n");
            }
        return;
    }
 

Saludos

[Malenko]

Iniciado por cesar_casla

hola!
mmm q un profesor te diga "no vale" la verdad me parece muy mal. Para mi gusto mientras un codigo funcione, esta bien; luego se puede discutir si el codigo se puede optimizar, controlar errores, etc...

Un código puede funcionar pero no hacer lo que la profesora le pidió, por lo tanto no estaría bien ;)

[vosk]

Hagas lo que hagas los profesores siempre critican y lo encuentran todo mal :) A mi me tenian tirria, no es que suspendiera por no estudiar sino porque los profesores se ponian de acuerdo para catearme :))

Creo que lo mejor seria que trabajaras con structs: es mas limpio, mas facil, mas de todo.

Código:

#define MAXLEN_NAME 50

typedef struct {
	int id, dia, mes;
	char nombre[MAXLEN_NAME];
} PACIENTE;

Ahora solo tienes que leer tamaños de struct y comprovar el id, o el nombre o lo que quieras

Código:

PACIENTE current;

while(fread(&current, sizeof(PACIENTE), 1, rd) == 1) {
	if(current.id == *id) {
                //ya lo tienes
	}
}
if(ferror(rd)) {
        //error de lectura
}

Puedes enviar un puntero a una struct para que al buscar te copie los datos encontrados y lo que quieras.

Saludos
vosk

[only_skating]

cesar_casla, muchas gracias por la correción pero sigo teniendo una pega :S

El while tal y como lo pusiste se lo salta directamente, no entra en el if a menos que ponga
while(feof(rd)!=0){ el problema es que al poner eso no termina nunca de buscar :S una vez mas muchas gracias a todos

[cesar_casla]

Un código puede funcionar pero no hacer lo que la profesora le pidió, por lo tanto no estaría bien ;)

Si eso es verdad, pero yo entendi que only_skating habia echo un programa q estaba bien pero a la profesora no le gusto ¿y desde cuando se aprende a programar segun los gustos de los demas??? Al menos yo no comparto esa idea


Saludos

[cesar_casla]

El while tal y como lo pusiste se lo salta directamente, no entra en el if a menos que ponga
while(feof(rd)!=0){ el problema es que al poner eso no termina nunca de buscar :S una vez mas muchas gracias a todos

mmm q raro a mi me anda, igual la condicion no puede ser esa, ya que feof solo devuelve distinto de cero cuando se alcanza el fin de archivo de lo contrario devuelve siempre cero... Mira q el archivo debe estar creado, yo probe con algo como:
2222 5 4 alberto
9999 22 5 julio
1111 14 6 leo
7894 1 1 ale

Saludos

[only_skating]

Muchas gracias de verdad solo tuve que añadir muchos contactos, como una vez recogidas ciertas variables podria modificarlas y escribirlas en el mismo sitio en el bloc de notas? espero que me entiendan, osea recoger el nombre por ejemplo y que pueda escribir uno nuevo en el mismo lugar donde tenia el antiguo. millll gracias!

[only_skating]

Una vez mas me mandaron hacer algo que no sé hacer con fscanfs sino cargando todo el bloc de notas a una variable y tirando de bucles y sentencias de control. Hay alguna manera de hacerlo mas "facilmente". osea, leer una linea con un scanf, modificar una variable e imprimirla en el mismo lugar donde estaba, o al menos, poder borrarla=? milllll gracias

[only_skating]

he conseguido apañarmelas utilizando un bloc de notas auxiliar en el que guardo los datos de todos los usuarios menos el que deseo eliminar y del auxiliar lo repego en el bueno. gracias a todos! un saludo