Mensajes

Hola, tengo que hacer que la máquina se ha vuelto más prudente y se plantará cuando su puntuación sea superior a la del humano o, en caso de que estén empatados, si la probabilidad de pasarse supera el 50%.


He creado esto pero no se para cuando la probabilidad de pasarse es mayor a un 50%.

Código (cpp) [Seleccionar] Expandir

bool esProbablePasarse(double puntosMaquina, const tCartasPorAparecer cartas)
{
int variable = 0, variable2 = 0;
bool pasarse,probabilidad;
double dato = 7.5 - puntosMaquina;

for (int i = 0; i < 8; i++)
{
variable = variable + cartas[i];
variable2 = cartas[i] + variable2;
}


probabilidad = variable / variable2;

if (probabilidad < 0.5)
{
pasarse = false;
}
else
pasarse = true;

return pasarse;
}

Par empezar el tipo de dato de la variable Dato es double no Int, no puedes o mas bien vas a tener errores por guardar un dato entero cuando realemnte esperas un tipo de dato flotante o double.

Segundo segun veo tu Screenshot.

Código [Seleccionar] Expandir


for (int i = dato; i < 8; i++)
{
variable = variable + cartas[i];
}


Iguales i al valor de dato pero cuando realizas una Operacion en dato como:

Código [Seleccionar] Expandir

dato = 7.5 - puntosMaquina;

El resultado no va a ser siempre el que esperes, a no ser que estes 100% seguro que la operacion siempre va a devolver un numero entero.

ahora el error que te aparece es que estas tratando de acceder a una region de memoria no valida, ya sea que te excediste del limite del arreglo o la variable cartas es apuntador No valido. (Cosas que no podemos saber por que no tenemos acceso al resto del codigo.

Te recomiendo colocar Mejores nombres a tus variables por que los nombres de "dato" y "variable" son muy escuetos.

Saludos

Gracias, como puedo haberme pasado si yo he definido el array como

Código (cpp) [Seleccionar] Expandir

typedef int tCartasPorAparecer[8];  y con el for he hecho que lea hasta el 8 a si que no creo que me haya pasado... y lo de no valido no se como ver si es eso...

Código [Seleccionar] Expandir

int dato;

Me sigue dando el error

Buenas, me sale un error al compilar y luego en la puntuación me salen numeros negativos cuando deberían salir numeros pequeños...Es un error que nunca me habia salido y no se por que es.

https://gyazo.com/dd40d2b77b7fa13ba755a169e3b819fe

Código (cpp) [Seleccionar] Expandir

bool esProbablePasarse(double puntosMaquina, const tCartasPorAparecer cartas)
{
int dato, variable = 0, variable2 = 0, probabilidad;
bool pasarse;
dato = 7.5 - puntosMaquina;

for (int i = dato; i < 8; i++)
{
variable = variable + cartas[i];
}

for (int i = 0; i < 8; i++)
{
variable2 = cartas[i] + variable2;
}

probabilidad = variable / variable2;

if (probabilidad < 0.5)
{
pasarse = false;
}
else
pasarse = true;

return pasarse;

}

Primero si no es para citar algo concreto de otro mensaje no incluyas una cita a cada respuesta porque cada mensaje se hace larguísimo. Además de eso un tema se crea para resolver una duda específica, no para hacer un programa entero. Ya sé que es mucho más cómodo abrir un tema y que esté activo hasta que tu programa funcione completamente pero no es así. Ese programa cada vez es más largo y no vamos a estar revisándolo entero. Cuando tengas una duda, específica la duda y manda el código necesario para resolverla, a veces será el programa entero, otras veces sólo será necesario mandar una función o un bloque. Si hace falta más código ya te lo diremos.

Segundo tu función de generar un número aleatorio en un sitio tiene una definición y en otro sitio otra. Te recomiendo que si la función se encarga de generar números aleatorios, tenga dos parámetros que indiquen el mínimo y el máximo para determinar el rango.Así la función será más polivalente que si simplemente cogemos un trozo del <main> lo metemos en una función y lo dejamos igual, hay que aprovechar las ventajas de las funciones.

Un prototipo que yo creo que estaría bastante bien para generar números aleatorios puede ser:

Código (cpp) [Seleccionar] Expandir


int generarNumeroAleatorio(int minimo, int maximo);
Si no quieres que tu función sea así y tenga otra definición, es tu decisión pero fíjate en lo que le pasas como parámetros ya que primero has puesto un código donde le pasas un entero y luego en el programa entero tiene otro prototipo sin parámetros.

Precisamente ahí si te da un problema al usar la función eso es lo que tienes que mandar para que te ayudemos, la función y explicar el error que te da, no decir que te da algún error y soltar 357 líneas de código.
Hay que tener un poco de consideración en lo que se pide, no estamos aquí para salvarte de cada problema que tengas. Aquí el que más tiene que poner de su parte eres tú, no nosotros.

En otro tema posterior a este he comentado como usar <typedef>, me había confundido así que ahora voy a modificar ese mensaje y de momento te dejo la respuesta correcta por aquí:

Código (cpp) [Seleccionar] Expandir


typedef int array10[10];
int main(){
   array10 miArray;
   for(int i = 0; i < 10; i++)
       miArray[i] = i;
}
Ahí tienes como se usa el <typedef> para lo que tú querías hacer y un ejemplo de uso para que veas como se usan los índices, no tiene mucha complicación eso.

Por mi parte si tienes una duda concreta en un trozo de código concreto igual puedo ayudarte, revisar 357 líneas de código para solucionar todos tus problemas es algo que no voy a hacer. Suerte.

PD: Tampoco recortes el programa y mandes lo mismo con menos código. Dudas concretas es preguntar por qué te sale un error o cómo puedes hacer tal cosa y te podremos dar una idea ya sea un ejemplo de código o una explicación en pseudocódigo para que tú después te encargues de implementarlo.

Vale perdon, soy bastante nuevo, pero gracias por todo en serio, todavia me queda bastante pero gracias por todo hasta ahora

No es que la puntuación de la máquina salga al final, simplemente al ser un bucle en el que el usuario no tiene que hacer nada el programa se ejecuta muy rápido y por eso se muestra la puntuación y de seguido el ganador. Si quieres ver la puntuación en cada iteración lo que puedes hacer es meter una pausa por ejemplo con <cin.get()>. Así se va a parar en cada vuelta hasta que pulses enter.

Código (cpp) [Seleccionar] Expandir


double modoBmaquina(ifstream & archivo, int numeroAleatorio, double puntosHumano){
int final = 0, aux;
double puntos = 0;

while (final < numeroAleatorio && puntosHumano < puntos && puntos < PUNTUACION_LIMITE){
archivo >> aux;
if (aux > 7)
puntos += 0.5;
else
puntos += aux;
cout << "La maquina ha robado : " << aux << endl;
cout << "La puntuacion de la maquina es : " << puntos << endl;
                cin.get();
        }
return puntos;
}

El tema de la <PUNTUACION_LIMITE> si lo único que has hecho ha sido declarar la variable y sustituir en cada <7.5> por <PUNTUACION_LIMITE> el programa no tiene que cambiar. Puede que sea que si usas números aleatorios para la cantidad de cartas que se roban y has aplicado alguno de los métodos que te comenté para robar cartas desordenadas haya coincidido que el humano haya tenido 7 puntos con las cartas que ha robado y la máquina se haya pasado en la última iteración a 8.
Prueba a quitar los trozos de código aleatorios para que las cartas que se cogen siempre sean las mismas y así poder controlar los resultados.

Un par de consejos más:

Código (cpp) [Seleccionar] Expandir


if (determinaGanador(puntosHumano, puntosMaquina) == HUMANO) {
cout << "Ha ganado el HUMANO" << endl;
cin.get();
}
else{
cout<<"El ganador es la maquina"<<endl;
cin.get();
}
Si tienes un <if> que ejecuta una instrucción y un <else> que ejecuta la misma instrucción, esa instrucción se ejecutará siempre. Entonces se pone después del <else> y así sólo se pone una vez.

Código (cpp) [Seleccionar] Expandir


if (determinaGanador(puntosHumano, puntosMaquina) == HUMANO)
cout << "Ha ganado el HUMANO" << endl;
else
cout<<"El ganador es la maquina"<<endl;
cin.get();

Y esto:

Código (cpp) [Seleccionar] Expandir


while (responder < 1 || responder > 2)
Sería más correcto usar el operador "distinto" <!=>.

Código (cpp) [Seleccionar] Expandir


while(responder != 1 && responder != 2)

Por lo menos dime por que me da error en tCartasPorAparecer (error de sintaxis) y en "cartas"(en las primeras lineas)

Agradeceria mucho la ayuda.

Código [Seleccionar] Expandir

[quote author=YreX-DwX link=topic=489919.msg2180460#msg2180460 date=1544368262]
No es que la puntuación de la máquina salga al final, simplemente al ser un bucle en el que el usuario no tiene que hacer nada el programa se ejecuta muy rápido y por eso se muestra la puntuación y de seguido el ganador. Si quieres ver la puntuación en cada iteración lo que puedes hacer es meter una pausa por ejemplo con <cin.get()>. Así se va a parar en cada vuelta hasta que pulses enter.
[code=cpp]
double modoBmaquina(ifstream & archivo, int numeroAleatorio, double puntosHumano){
int final = 0, aux;
double puntos = 0;

while (final < numeroAleatorio && puntosHumano < puntos && puntos < PUNTUACION_LIMITE){
archivo >> aux;
if (aux > 7)
puntos += 0.5;
else
puntos += aux;
cout << "La maquina ha robado : " << aux << endl;
cout << "La puntuacion de la maquina es : " << puntos << endl;
               cin.get();
       }
return puntos;
}

El tema de la <PUNTUACION_LIMITE> si lo único que has hecho ha sido declarar la variable y sustituir en cada <7.5> por <PUNTUACION_LIMITE> el programa no tiene que cambiar. Puede que sea que si usas números aleatorios para la cantidad de cartas que se roban y has aplicado alguno de los métodos que te comenté para robar cartas desordenadas haya coincidido que el humano haya tenido 7 puntos con las cartas que ha robado y la máquina se haya pasado en la última iteración a 8.
Prueba a quitar los trozos de código aleatorios para que las cartas que se cogen siempre sean las mismas y así poder controlar los resultados.

Un par de consejos más:

Código (cpp) [Seleccionar] Expandir


if (determinaGanador(puntosHumano, puntosMaquina) == HUMANO) {
cout << "Ha ganado el HUMANO" << endl;
cin.get();
}
else{
cout<<"El ganador es la maquina"<<endl;
cin.get();
}
Si tienes un <if> que ejecuta una instrucción y un <else> que ejecuta la misma instrucción, esa instrucción se ejecutará siempre. Entonces se pone después del <else> y así sólo se pone una vez.

Código (cpp) [Seleccionar] Expandir


if (determinaGanador(puntosHumano, puntosMaquina) == HUMANO)
cout << "Ha ganado el HUMANO" << endl;
else
cout<<"El ganador es la maquina"<<endl;
cin.get();

Y esto:

Código (cpp) [Seleccionar] Expandir


while (responder < 1 || responder > 2)
Sería más correcto usar el operador "distinto" <!=>.

Código (cpp) [Seleccionar] Expandir


while(responder != 1 && responder != 2)
[/quote]


   srand(time(NULL));
   numeroAleatorio = 3 + rand() % 5 - 3 + 1;

   return numeroAleatorio;
}[/code]

Primero para la función que tienes no se usa un <while> sino un <if>. Es muy común traducir una sentencia como "mientras..." pero en realidad es "si...".Un <while> se usa si vas a repetir un trozo de código hasta que se cumpla una condición, por lo tanto los valores de la condición tienen que ir cambiando dentro del <while>, por ejemplo:

Código (cpp) [Seleccionar] Expandir


while(puntosMaquina < puntosHumano && puntosMaquina < 7.5)
   puntosMaquina += carta;
Eso sería un <while>. Mientras los puntos de la máquina no superen al humano ni el 7.5 entonces le sumamos otra carta. No digo que lo tengas que hacer así ya que tienes tus propias funciones para robar cartas. Es sólo para que veas el uso del <while>.

En tu caso si es para determinar un ganador es con un <if>. Además en programación lo que se suele hacer para dar menos vueltas es asumir un caso y si la condición es la opuesta, modificar el resultado, tu caso:

Código (cpp) [Seleccionar] Expandir


int determinaGanador(int puntosHumano, int puntosMaquina){
   int ganador = HUMANO;
   if(puntosHumano > 7.5 || (puntosMaquina < 7.5 && puntosHumano < puntosMaquina))
       ganador = MAQUINA;
   else if(puntosHumano == puntosMaquina)
       ganador += rand() % 2;
   return ganador;
}
Esa es tu función que una vez tiene los puntos de ambos, te dice quien es el ganador. No contempla el caso de que los dos jugadores se hayan pasado del 7.5. Eso ya te lo dejo a ti.  

Si lo que quieres es interrumpir el programa en cuanto uno de los dos pasa de 7.5. Entonces la modificación la tienes que hacer al momento en que van robando cartas. He mirado un poco tu programa por encima y en cada función del modo de juego ya compruebas que no se haya pasado de 7.5, eso está bien. Lo que puedes hacer en el <main> sería algo así:

Código (cpp) [Seleccionar] Expandir


puntosHumano = modoA(archivo, numeroAleatorio);
if(puntosHumano < 7.5)
   puntosMaquina = modoA(archivo, numeroAleatorio);
//...
Así si el humano se pasa de 7.5 tal y como te he hecho la función anterior ganaría la máquina sin necesidad de robar cartas. Esto tendrías que implementarlo para cada uno de los modos. Además te recomiendo no usar números mágicos, es decir, números que aparecen literalmente por ahí y no se sabe lo que son. Dale un nombre a 7.5 y sustitúyelo en el programa por ejemplo:

Código (cpp) [Seleccionar] Expandir


const int PUNTUACION_LIMITE = 7.5;
// y en el resto del programa poner PUNTUACION_LIMITE donde ponga 7.5
Así cambiando sólo ese número puedes cambiar el límite del juego sin tener que ir cambiando todo el programa. Suerte.

Para eso tienes varias alternativas:
- Una sería usar ficheros de acceso aleatorio mediante funciones como <seekp()> y <seekg()>. Puedes buscar más información buscando "ficheros de acceso aleatorio c++" o buscando directamente las funciones que te he comentado y ver cómo funcionan.

Si no tienes ni idea de trabajar con ficheros de acceso aleatorio, yo te recomendaría usar la segunda opción que consistiría en aplicar lo que ya sabes para conseguir una lectura aleatoria:
- Yo generaría un número aleatorio pequeño para no irme al final del fichero y saltaría esos números. Así no coges siempre los n-primeros.

Código (cpp) [Seleccionar] Expandir


int cartasASaltar = generarAleatorio(0,5);
int cartasSaltadas = 0;
int carta;
while(!fichero.eof() && cartasSaltadas <= cartasASaltar){
    fichero >> carta;
    cartasSaltadas++;
}
cout << "Tu carta es: " << carta << endl;

Ahí tienes un trozo de prueba para que veas a lo que me refiero. Con la función que te comenté hace un tiempo para generar números aleatorios le pasas el mínimo y el máximo (como coges como mucho 5 cartas, saltando un máximo de 5 cartas en cada vuelta es seguro que no te sales del fichero, podrías saltarte más eso ya te dejo que lo calcules tú si quieres) Para hacerlo más aleatorio puedes generar un número aleatorio distinto en cada vuelta para que no siga el mismo patrón. Suerte.

Mientras escribía esto me he dado cuenta de que también puedes empezar con un fichero vacío y el propio programa genere el fichero (simulando que se está barajando). Creas números aleatorios entre [1,10] y los escribes en el archivo. Así la baraja no empezaría ordenada. En este caso tendrías que controlar que no se repitan más de 4 veces cada valor ya que en la baraja cada valor numérico se repite hasta 4 veces. Mi recomendación si lo haces así que la verdad me parece bastante interesante es que hagas un array y cada vez que generes un número, compruebes que no has generado ya 4 veces ese número y  le sumes 1 al (i-1)-elemento. Me explico por si no se entiende:
- Tienes un array <maximos[10] = {0,0,0,0,0,0,0,0,0,0}> que indica cuantas veces has generado cada valor.
- Generas x valor (imagina un 1).
- Compruebas que (maximos[x-1] < 4) Si no es menor que 4 generas otro número distinto.
- Si es menor que 4, lo agregas al fichero y haces <maximos[x-1] += 1>, es decir le sumas 1 en la posición correspondiente.

Suerte y espero que te animes a intentar el último método que te he comentado. O incluso puedes mezclarlos, eso ya depende de la imaginación del programador. Personalmente creo que es mejor que resuelvas un problema con las herramientas que conoces hasta que estudies otras y no que uses herramientas que no sabes como funcionan para salir del paso. La decisión final es tuya. Suerte.

Okey gracias, por ultimo, me gustaría hacer que al pasarse de 7.5 tanto la maquina como el humano, pierdan la partida automáticamente con un cout<<"Has perdido la partida", para ello habia creado esto, pero no me funciona.

Código (cpp) [Seleccionar] Expandir

int determinaGanador(double puntosHumano, double puntosMaquina)
{
int ganador;

while (puntosHumano || puntosMaquina < 7.5)
{
if (puntosHumano > puntosMaquina)
{
ganador = HUMANO;
}
else if (puntosHumano < puntosMaquina)
{
ganador = MAQUINA;
}

else if (puntosHumano == puntosMaquina)
{
ganador = HUMANO + rand() % +(MAQUINA - HUMANO);
}
}

while (puntosHumano || puntosMaquina > 7.5)
{
if (puntosHumano > 7.5)
{
cout << "Has perdido ";
}
if (puntosMaquina > 7.5)
{
cout << "La maquina ha perdido ";
}
}
return ganador;
}

Según tu código la carta que robas la guardas en <aux> en cada una de las funciones <modoA()> y <modoB()>. Entonces para ver la carta sólo tendrías que hacer un <cout> de <aux>.

Un par de consejos:
- Si no recuerdo mal lo de generar el número aleatorio tenías que hacerlo con una función, no en el <main>.
- En vez de usar <if> anidados para cada opción del menú, es mejor usar el <switch>.
- En el <modoBHumano()> si la variable <continuar> le asignas <true> al empezar, luego no le asignes <true> otra vez cada vez que el usuario quiere otra carta. Sólo asígnale <false> cuando no quiera más cartas, hasta entonces ya vale <true>.
- La variable <continuar> de <modoBMaquina()> más de lo mismo.
- En la función <determinaGanador()> en caso de empate siempre gana el humano por lo que te he comentado antes de los números aleatorios. Estás haciendo <rand() % 1> y eso siempre es 0, por lo que HUMANO + 0 = HUMANO.

Aparte de esas cosillas, como el archivo se lee en orden, siempre van a salir los <numeroAleatorio>-primeros números del archivo.

Si esas cosillas las estuve corriegiendo gracias, pero como puedo hacer para que los numeros del archivo de texto se lean de forma aleatoria?