Foro de elhacker.net

Tengo un problema con este codigo

#include <iostream>
#include <stdlib.h> 
#include <time.h> 
#include <conio.h>
#include <stdio.h>
FILE *doc;
using namespace std;
int His();
int	Cien();
int Depor();
int Geo();
int c=1;
int main()
	{
		int s,tot,u=0;
		srand(time(NULL)); 
		 while (c==1)
		{
		system ("cls");
		cout<<"Presione cualquier tecla para escoger su categoria"<<endl;
		getch(); 
		s=rand()% 4 + 1;
 
		switch (s)
		{
		case 1:
			{
				His();
				break;
			}
		case 2:
			{
				 Cien();
				break;
			}
		case 3:
			{
				 Depor();
				break;
			}
		case 4:
			{
				Geo();
				break;
			}
		}
		}
		system ("pause");
		return 0;
	}
 
int His()
{
int x,i=1,w;
	system ("cls");
	cout<<"<===============Su categoria es Historia===============>"<<endl;
	cout<<"Presione cualquier tecla para escoger su pregunta"<<endl;
	getch(); 
	doc = fopen("C:\Users\HERNAN\Desktop\prueba.txt","r");
	while(i<=5)
		w=getc(doc);
		if (w =='\n') printf ("\n");		
		else 
		printf("%c",w);
	fclose(doc);
	return 0;
}
int Depor()
{
	int x,i=1,w;
	system ("cls");
	cout<<"<===============Su categoria es Ciencia===============>"<<endl;
	cout<<"Presione cualquier tecla para escoger su pregunta"<<endl;
	getch(); 
	doc = fopen("C:\Users\HERNAN\Desktop\prueba.txt","r");
	while(i<=5)
		w=getc(doc);
		if (w =='\n') printf ("\n");		
		else 
		printf("%c",w);
	fclose(doc);
	return 0;
}
int Cien()
{
	int x,i=1,w;
	system ("cls");
	cout<<"<===============Su categoria es Deportes===============>"<<endl;
	cout<<"Presione cualquier tecla para escoger su pregunta"<<endl;
	getch(); 
	doc = fopen("C:\Users\HERNAN\Desktop\prueba.txt","r");
	while(i<=5)
		w=getc(doc);
		if (w =='\n') printf ("\n");		
		else 
		printf("%c",w);
	fclose(doc);
	return 0;
}
int Geo()
{
	int x,i=1,w;
	system ("cls");
	cout<<"<===============Su categoria es Geografia===============>"<<endl;
	cout<<"Presione cualquier tecla para escoger su pregunta"<<endl;
	getch(); 
	doc = fopen("C:\Users\HERNAN\Desktop\prueba.txt","r");
	while(i<=5)
		w=getc(doc);
		if (w =='\n') printf ("\n");		
		else 
		printf("%c",w);
	fclose(doc);
	return 0;
}
 
 

El problema en la ejecución es este
(http://fotos.subefotos.com/22e9cdc29500e5efb7c5de3e998cdd8eo.png)


[MOD] para publicar código, usa la etiqueta GeShi correspondiente. Gracias.

Debes poner titulos descriptivos del tema, "Ayuda!" no lo es, leete las Reglas del Foro.

1. O usas los mecanismos de entrada salida de C (printf, scanf) o los de C++ (cin, cout)... pero por favor, no los mezcles. Puede dar problemas.

2. No uses conio.h... más información al respecto aqui (http://foro.elhacker.net/programacion_cc/lo_que_no_hay_que_hacer_en_cc_nivel_basico-t277729.0.html)

3. No uses variables globales. Muy pocas veces es necesario su uso. Además hay que tener cuidado con ellas, pueden ser problemáticas y dificultan la lectura del código.

4. ¿Y si no se ha podido abrir el fichero?

	doc = fopen("C:\Users\HERNAN\Desktop\prueba.txt","r");
	while(i<=5)
		w=getc(doc); // Y si doc == NULL???

Te faltan comprobaciones ahí.

5. Si un if, un else, un while o un do-while ocupan más de una línea hay que usar llaves:

	while(i<=5)
		w=getc(doc);
		if (w =='\n') printf ("\n");		
		else 
		printf("%c",w);

Ese código se va a quedar atascado en las dos primeras líneas. Como no hay llaves el while afecta únicamente a w=getc(doc).

Además, aunque pusieses llaves no estás incrementando i... luego seguirías en un bucle sin fin que finalizaría con error al terminar de leer todo el archivo.

6. Los nombres de las variables salen gratis... pon nombres que te den una idea de la función de dicha variable.

No es lo mismo:

if ( i > j || x < w )

que:

if ( velocidad > velocidadMaxima || fechaITV < hoy )

Seguro que con la segunda opción es más facil hacerse una idea de qué es lo que hace el código.

Puede que haya algún otro error... pero como tienes que darle un repaso serio al programa no merece la pena seguir mirando.

Ya hice lo que me dijiste pero persiste el error incluso puse para cuando doc sea null porque ahora me pone siempre error en la apertura del archivo, pero dime como soluciono lo del fichero y la librería la necesito para que se pause el programa porque con system pause no me gusta lo del printf no lo uso porque no me deja imprimir, la variable la necesito porque si responde una pregunta mal, hare que se salga del ciclo el programa todavia no esta completo

El path esta mal, no mire nada mas ya que eso salta a la vista, la \ se usa para secuencias de escape (http://en.wikipedia.org/wiki/Escape_sequences_in_C), la que corresponde usar en tu caso es \\ .

Confiaba en que al ver que no se abrían los archivos, investigase un poco el tema y acabase viendo lo de las barras el solo.

Le dijiste, como corresponde, que no usara conio.h y ahi sigue ... yo no confiaria tanto  ;D

Haber el error ya no es ese nisiquiera ya lo solucione era el users que no debe ir ahí porque es un comando prederminado, lo del conio es porque lo necesito y ahora mi problema es que el programa cuando abre el archivo entra en un bucle infinito y no sale porque no se todavia

O no te sabes explicar o no sabes lo que estás haciendo... lo que te dijo Eternal es


Es decir, que sustituyeses los símbolos '\' por '\\', ya que '\' se utiliza para secuencias de escape, es decir, se interpreta ese carácter y el siguiente como uno solo que además tiene un efecto especial: '\n' salto de línea, '\t' tabulador, etc.

Dicho en cristiano. Tu ruta debería quedar: C:\\Users\\HERNAN\\Desktop\\prueba.txt


Lo necesitas por... espera, no me lo digas... getch??

hay mil alternativas a getch... tu por ejemplo que estás usando C++ puedes recurrir, por ejemplo, a "std::cin.ignore()". Tiene la ventaja de que es una solución estándar y multiplataforma. Ahora, si prefieres coger malos hábitos nada más empezar tú mismo, es tu decisión.

Por cierto, si respondo a un mensaje con dudas lo menos que espero es que el interesado se tome la molestia de leer mi respuesta completa... por respeto y deferencia más que nada. Te lo digo porque te puse el enlace a un post en el que se indican malas prácticas en C y C++, además de explicar los motivos explica también diferentes posibilidades y soluciones. Por tu forma de hablar entiendo que has pasado de ese punto completamente... si no vas a leer las respuetas por que no te gustan... ¿para qué pides ayuda?


Pues mira, si te molestases en leer las respuestas de la gente lo sabrías. Además también deberías aprender a usar el depurador de código... al principio cuesta cogerle el punto, pero en cuanto adquieres soltura descubres lo práctico y útil que es: te permite evaluar y modificar variables, ves la secuencia de ejecución, puedes saltar instrucciones...

Ok coregire lo del getch pero lo del bucle infinito ya puse a aumentar la i osea no estoy habalando del bucle del while de la funcion principal, estoy hablando del siclo que hay en las otras funciones el while y quitare la conio.h despues pero ese es mi problema mas grande si estoy aumentando cada vez que hay salto de linea porque no se detiene el programa?

si por aumentar i te refieres a i=1+1 ... que es como tienes en tu código... ahí i va a valer SIEMPRE 2... luego NUNCA va a alcanzar el valor necesario para que el código abandone el bucle... ahí es donde entra la parte de aprender a usar el depurador... si usas el depurador este problema lo ves en seguida porque te das cuenta de que i no cambia su valor en la vida.

bueno ya coregido todo lo que me dijeron quiero mostrarles una cosa haber si me pueden ayudar estudie lo de los strings como me dijeron y saque un programa mas de conocimiento lo hice con logica pero quiero que me digan lo del patcher y un pequeño error que me sale primero les mostrare el codigo

#include <iostream>
#include <stdlib.h> 
#include <time.h> 
#include <stdio.h>
#include <fstream>
using namespace std;
int main()
{
int f,i=1,s=1,w,r;
char x[1];
	system ("cls");
	printf("\n <===============Su categoria es Historia===============> \n");
	printf("\n Presione cualquier tecla para escoger su pregunta \n");
	cin.ignore();
	ofstream res;
	res.open("D:\\res.txt",ofstream::out | ofstream::trunc);
	ifstream resp;
	resp.open("D:\\res.txt");
	ifstream narchivos;
	narchivos.open("D:\\prueba.txt");
	ifstream enarchivos;
	enarchivos.open("D:\\prueba.txt");
	ifstream archivos;
	archivos.open("D:\\prueba.txt");
	while (!enarchivos.eof())
	{
	i++;
	f=enarchivos.get();
	}
	enarchivos.close();
	i=i-2;
	while (s<=i)
	{
	f=narchivos.get();
	s++;
	}
	i=i-1;
	s=1;
	while (s<=i)
	{
	r=archivos.get();
 
	if (r =='\n') 
			{
				printf ("\n");
		}
		else printf("%c",r);
		s++;
	}
	enarchivos.close();
	scanf("%s",x);
	res<<x<<endl;
	w=resp.get();
	if (w==f)
	{
		printf("Su respuesta fue correcta felicitaciones!");
		cin.ignore();
 
	}
	else 
	{
		printf("Respuesta incorrecta es el turno del otro jugador");
		cin.ignore();
	}
	res.close();
	resp.close();
	enarchivos.close();
	narchivos.close();
	archivos.close();
	cin.ignore();
	system("del c:\\res.txt");
	return 0;
 
}

Cuando termino el programa me dice que la variable x la corrompieron (ya lo depure mucho tiempo) y el archivo que quiero que se me borre no se borra :'( y lo otro es lo del path yo quiero que en en vez del prueba.txt pongo la f que esta como variable como hago para que me lea lo que hay dentro de la variable y no que se abra el archivo con nombre f esto dentro del patcher si pueden ayudarme muchas gracias por adelantado

Una cadena de caracteres, al estilo C que es la que estás usando, se componen de una secuencia de caracteres que termina con el carácter nulo '\0'. Esto quiere decir que si quieres almacenar un texto de 30 caracteres, necesitas un buffer que tamaño igual o superior a 31.

En tu caso tienes char x[1] ... es decir, estás definiendo un arreglo de caracteres de tamaño 1... lo que te permite almacenar, bien un caracter suelto, bien una cadena vacía (solo el carácter nulo). Al intentar almacenar una cadena de caracteres en x, tu código escribirá fuera del espacio reservado para X, por eso te sale el mensaje de memoria corrupta. Debes modificar ese '1' para que tenga un valor igual o superior a "LongitudMaximaDelTextoAAlmacenar+1"

La opción b es usar la clase string y olvidarse de tener que gestionar las cadenas de caracteres a tan bajo nivel.


Deberías fijarte un poco más en tu código:

res.open("D:\\res.txt",ofstream::out | ofstream::trunc);
 
// ...
 
system("del c:\\res.txt");

Es el problema de tener valores constantes "a pelo" en el código, sobretodo cuando están repetidos...

pista: Fíjate en los dos path y busca diferencias.


Deberías aprender a expresarte un poco mejor.

¿Pretendes no tener que poner el path "a pelo" al abrir los ficheros?

¿Quieres usar 'f' para elegir que fichero tienes que abrir?
Esto no lo vas a poder hacer porque f es de tipo int, no es un string.

¿Quieres que el contenido del fichero se vuelque en 'f'?
Recuerda que 'f' solo puede guardar un entero por vez.

¿Es otra cosa?
Tendrás que replantear la forma de expresar tu problema.

Uyyy si muchas gracias bien ahora debo adivinar como poner el path para que se abra desde la ruta donde esta el archivo cpp esto se puede hacer?

dir\file.txt