Tema: Alguien q sepa C++ ¿?

Ya se que este foro no toca esta parte de la informática, pero bueno, seguro que hay alguien q me pueda sacar d dudas.
bien, tengo un txt en el cual hay lo siguiente:

Juan
Perez
55581953

Soy nuevo en este lenguaje, y tengo que hacer un programa en C++ que me lea los datos. A priori pense que era muy facil, pero no se pq no me funciona bien el programa:

#include <stdio.h>

int main () {
FILE *fp;
char nom[15];
char cog[15];
double dni;
//int letra;
//char tabla[24]="TRWAGMYFPDXBNJZSQVHLKE";
//char letradni;
//int i;
fp = fopen ( "miDNI.txt", "r" );
if(fp==NULL) printf("Error, no es troba miDNI.txt");
fscanf(fp,"%s",&nom);
fscanf(fp,"%s",&cog);
fscanf(fp,"%d",&dni);

printf("%s \n", nom);
printf("%s \n", cog);
printf("%d", dni);

// letra=dni%23;
// letradni=tabla[letra];
// printf("%c \n",letradni);
fclose ( fp );
return 0;
}

Lo que está comentado no le hagáis caso. Leyendo ya sabeis de que va el programa ^^. Pues bueno, compilando y ejecutanto esto obtengo lo siguiente:

C:\TC\BIN>dni.exe
Juan
Perez
7425

Pq me sale un 7425 en vez del 55581953¿? Incluso he probado con un float (el resultado es 0.0000000) o con un long y me da lo mismo. Alguien sabe pq?
Gracias!

A mi me funciona como deberia, de todos modos, debes tener en cuenta que el fscanf lee hasta que se encuentra una nueva linea, si en el archivo miDNI.txt al final del dni no pones un salto de linea (un intro vamos) el fscanf leera lo que le de la gana. Otra cosa en el fscanf y en el printf el %d se usa con int (con un int debe sobrarte), para los float usa %f y para el double %Lf (creo).

Esta pagina te vendra bien: http://www.cplusplus.com/reference/

Hasta otra.

Lo de limpiar el buffer del teclado no se lo que es, y el resto q me habeis dicho lo he intentado y nada. Añadí incluso algo de texto en la siguiente linea para ver si me hacia algo, pero tp.

Acabo de probar declarando float como dni y poniendo "%f" en vez de "%d" en sus respectivos sitios y si que me lo lee, pero como float, y por tanto me añade: 55581953.000000000

Sinceramente, no lo entiendo -.-
Com0 no sonsiga hacerlo le meto ahora un cast! xDD

¿Y cogiendo DNI como String? Así puedes además añadir la letra xDD

El C ya lo tengo muy olvidado, pero bueno, como te han dicho un entero te sobra (hasta 2*10^9 aprox.). De todas formas creo que hay un fallo, y los Double se muestran de la misma forma que los Float, con %f, no con %d (enteros).

También deberías probar el fwrite/fread (en este caso no, ya que el archivo lo creas tu) y el fput/fget.

Por cierto, yo veía más cómodo el D.e.v.C.p.p [sin puntos, error 403] que el TurboC, ¿lo has probado?

Un saludo

No, no lo he probado. Y uso TutboC más que nada pq és el mismo que usamos en las prácticas de la carrera.
Y lo cogerlo como string ya lo pense, pero tendre que pasarlo con el atoi o con alguna funcion a int, pq no se si se ve muy bien (no esta todo el codigo) pero lo que hare despues es hacer un mod 23, y si pones el resultado en la tabla, la letra resultante es la letra del DNI. Lo que quiero dcir es que necesitare como valor numerico esa variable, para poder interactuar con ella.

Ajam, pués luego casi mejor intentar cogerlo como integer, ya que a mí personalmente el atoi nunca me entusiasmó... Sí fuese al revés, de int a char no habría complicación.

Yo creo que así:

#include <stdio.h>
#include <stdlib.h>

int main (void) {

FILE *fp;
char nombre[15];
char apellidos[15];
double dni;
int letra;
char tabla[24]="TRWAGMYFPDXBNJZSQVHLKE";
char letradni;

fp = fopen ( "miDNI.txt", "r" );

if(fp==NULL)
{
printf("Error, no se ha encontrado miDNI.txt");
}
else
{
fscanf(fp,"%s",&nombre);
fscanf(fp,"%s",&apellidos);
fscanf(fp,"%f",&dni);
fclose (fp);
}

printf("Nombre: %s \n", nombre);
printf("Apellidos: %s \n", apellidos);
printf("DNI: %d", dni);

letra=dni%23;
letradni=tabla[letra];
printf("- %c \n",letradni);

return 0;
}

A ver...

Un saludo

Quete, de hecho he visto un fallo, pero no ese; el no quiere escribir en el fichero, simplemente leer (por lo menos, de escribir no ha dicho nada). El fallo que he visto es que debería cerrar el else justo antes del return, es decir, no mostrar por pantalla nada más que el error.

De todas formas échale un ojo, por que yo esto lo tengo muy olvidado xD (ahora uso PHP y JS, ni del .Net me acuerdo apenas).

Un saludo

No me has entendido; no tiene que escribir en el fichero (lo que sería con el fprintf, fwrite, fputs o fputc), sino leer.

La mejor opción, sí antes de leer tiene que escribir en el fichero (es decir, utilizando la aplicación) es utilizar fwrite/fread mediante una estructura de datos.

En el caso de que el fichero lo cree manualmente, fscanf es lo recomendable, siendo más cómodo que fgetc/fgets.

Viendo el código, diría que al menos por ahora es la segunda opción, se trata de leer datos de un fichero de texto debidamente estructurado (por lo que el fread no nos vale), así que queda como mejor opción fscanf.

El fallo de la primera versión que nos ha puesto era que intentaba capturar una variable Double como Integer/Decimal (%d), cuando el método correcto es capturarla como Float (%f). Las opciones son corregir eso, o como dices utilizar un Int, más que suficiente para el número del DNI.

Un saludo

Iniciado por Ly0n

Viendo el código, diría que al menos por ahora es la segunda opción, se trata de leer datos de un fichero de texto debidamente estructurado (por lo que el fread no nos vale), así que queda como mejor opción fscanf.

Si, yo de momento no pretendo escribir ningun archivo, solo leerlo. Y el archivo ya esta debidamente extructurado, como comentas. Ademas, tengo que utilizar la funcion "fscanf" pq me lo piden así.

Iniciado por Ly0n

El fallo de la primera versión que nos ha puesto era que intentaba capturar una variable Double como Integer/Decimal (%d), cuando el método correcto es capturarla como Float (%f). Las opciones son corregir eso, o como dices utilizar un Int, más que suficiente para el número del DNI.

Un saludo

Haciendo las modificaciones que dices, es decir:

#include <stdio.h>

int main () {
FILE *fp;
char nom[15];
char cog[15];
int dni;
fp = fopen ( "miDNI.txt", "r" );
if(fp==NULL) printf("Error, no es troba miDNI.txt");
fscanf(fp,"%s",&nom);
fscanf(fp,"%s",&cog);
fscanf(fp,"%d",&dni);

printf("%s \n", nom);
printf("%s \n", cog);
printf("%d", dni);
fclose ( fp );
return 0;
}

El resultado q me muestra por pantalla es el siguiente:


C:\TC\BIN>dni
Juan
Perez
7425

Como ya dije antes, la única forma que obtengo un valor "normal", es declarando como "float dni" y leyendo y escribiendo
"/f" respectivamente. En este caso el resultado es:

C:\TC\BIN>dni
Juan
Perez
55581952.000000

Podria aplicarle un CAST y listos, pero no me parece adecuado cuando por logica con un INT bastaria. Y me pica mucho la curiosidad de saber qué está fallando.

¿Puedes probar a declarar dni como double, sin tocar el resto del código? Sólo para probar una cosa.

Un saludo

Edito: lo olvidaba, tanto en el fscanf como en el printf ponle %f a dni, cuando esté como double.

Otro saludo

He probado el ultimo código que tiene, y funciona perfectamente :?

¿Quizás el buffer del teclado? Métele un fflush.

Un saludo

PD: Olvida lo del double xD

como es eso del buffer del teclado?

He probado poniendo eso un par de veces y sigue funcionando =.

Respecto a tu codigo, lo he probado y el resultado es el mismo que el mio:

Juan
Perez
7425

Ya empezaré a sospechar que es mi compilador, pero el caso es que en un ordenador de la uni funcionó "solo la primera vez" utilizando un int.

Iniciado por pritt

El problema creo que puede venir por aquí:

fscanf(fp,"%d",&dni);

Si en el archivo de origen tienen un simple espacio te puede dar un error.

Un saludo.

Puede ser, pero de ser asi, pq si me reconoce el DNI entero cuando uso un FLOAT en vez de un INT?

Tb me he dado cuenta que si pongo sólo los 4 primeros dígitos del DNI, es decir: 5558 me los lee a la perfeccion como int. Si pongo una cifra más, ya me los lee mal y el resultado es un numero negativo.

Yo de nuevo! xDD Pues resulta que me he puesto a jugar a la "patata caliente" con el programa y creo que en C++ el integuer no es tan grande como decis! xD (bueno, al menos eso he deducido de los siguiente). He ido probando valores en el dni, asi que el valor maximo que me lee a la perfeccion como integer es el 32767. A partir de aki, es dcir, si pusiera 32768, sería el -32768 al leerse. El 32769 es el -32767, por lo q supongo q el int va del 32767...0...-32768, no? Y si no recuerdo mal en java es lo mismo, qué cabeza la mía! Tendriamos tb el long int, pero no creo q sirviera (seria el doble de 32767 pq no acepta negativos). El siguiente valor ya es el float (q es el unico q me funcionaba)...
Lo q no entiendo es pq todos estamos diciendo que con el int ya tendria suficiente, o nos ekibocamos todos (me incluyo) o aun entiendo menos el problema de mi programa xDDDD

He estado buscando información y en C++ un integer son 2bytes, es dcir, 16bits; 2^16=65536, entonces lo que puse arriba era correcto.

Iniciado por Pagina Web vía Google

1-Un byte es la menor unidad de informacion que pueden direccionar la mayoria de las computadoras. En la mayoria de las arquitecturas el tipo char ocupa un solo byte, por lo tanto es la unidad minima. Un bool admite solo dos valores diferentes, pero es almacenado como un byte. El tipo integer ocupa generalmente 2 bytes, un long 4, double 8, y asi con el resto de los tipos.

Pues si, ni yo había caído hasta que fui probando. En prácticas ensamblamos sobre un emulador del viejo Intel 8086, así que debe ser por eso que la versión del compilador tb es en arquitectura de 16b. Por cierto, que deberia darme el sizeof(int)? el valor en Bytes? Me da 2.

Perdonad por el lio, tp yo sabía que estaba trabajando en 16b hasta que probé... No tengo más remedio que declararlo como float.

Me podríais recomendar algun compilador para C++ q no sea demasiado engorroso? Quiero dcir, el eclipse por ejemplo esta muy bien y tal para trabajos grandes, pero prefiero algo similar al TurboC, simple y sencillo. Un solo click y tienes el exe.

DevC-pp (sin el guión).

Para mí de los más sencillos, interfaz gráfica y cuando quieras compilar y ejecutar, F9. Sin más rodeos. Además, incluye gran cantidad de ejemplo.

Un saludo

Acabo de crear el programa completo, calculando la letra y tal con el compilador que me has dicho, LyOn, y funciona a la perfección. Gracias
Para ir practicando por mi cuenta usaré este, y para prácticastendre que usar el otor, que remedio! :o

Es q es algo tan raro q ni yo me esperaba que a estas alturas me fueran a hacer trabajar con un compilador que trabaja en 16b

Tiene gracia... pq es precisamente esa asignatura. Hasta ahora todo ha sido en ensamblador, pero no se pq hemos cambiado a C++. Sólo se que algo tiene que ver con el 8086 xD. Yo empecé con java, y ahora los q empiezan carrera estudian algo llamado Paiton o algo asi, no lo conozco sinceramente.

Bueno, es como hoy día C, se estudia por su estructura y por que lo aprendido en el se puede exportar de una u otra forma a la mayoría de lenguajes (o por lo menos por eso lo estudié yo, aunque por lo que tengo entendido es un lenguaje muy potente). Pero a nivel práctico, dudo que en cualquier empresa te pidan un programa en C, ya que no es orientado a objetos.

Sobre la facilidad de los medios actuales, realmente programar en .Net con el Visual Studio es una gozada, pero bueno, hay que empezar por abajo.

De hecho yo ahora me dedico de forma exclusiva a lenguajes utilizados en la creación de webs, y por desgracia no conozco ningún editor para PHP lo suficientemente decente, así que la opción que me queda es Dreamweaver (porque es más cómodo para visualizar el código fuente, coloreado y eso) y activar los warnings, y aún así te dejas media hora revisando una maldita línea para que al final el fallo sea una simple comilla, una coma, un punto y coma, una variable mal escrita...

Un saludo a todos y suerte con el proyecto, MacSpawn.

------------

Ups, me acaba de llegar el aviso de respuesta, no había visto tu último mensaje

No había visto este post XD

El problema era ese efectivamente... con TurboC int = 2bytes (±32768) por lo que para el DNI se te quedaba corto (con un unsigned int tendrías hasta +65536).
La manera más sencilla de solucionarlo es poniendo un unsigned long por ejemplo (cuando se lo quieras asignar a una variable basta con poner %ld en lugar de %d) y así te cubrirá toda la cifra sin decimales

#include <stdio.h>

int main () {
FILE *fp;
char nom[15];
char cog[15];
unsigned long dni;
clrscr();
fp = fopen ( "miDNI.txt", "r" );
if(fp==NULL) printf("Error, no es troba miDNI.txt");
fscanf(fp,"%s",&nom);
fscanf(fp,"%s",&cog);
fscanf(fp,"%ld",&dni);

printf("%s \n", nom);
printf("%s \n", cog);
printf("%ld", dni);
fclose ( fp );
getch();
return 0;
}