Mensajes

Sugiero hacer un cambio en esta parte

Código (c) [Seleccionar] Expandir

if(puntosHumano == puntosMaquina)
        ganador += rand() % 2;

pues depende mucho del valor de las constantes, y que valor tiene "ganador" antes de llegar aqui, y si una de las constantes esta separada exactamente en 1 de la otra.

Cambiando ligeramente el codigo todas estas dependencias ya no estaran.

Ejemplo:

Código (c) [Seleccionar] Expandir

if(puntosHumano == puntosMaquina) {
       if( rand() % 2 ) {
          ganador = MAQUINA;
       }
}

Con esta modificacion, no importa el valor de las constantes, o que esten separadas en 1 una de otra, podrian ser otros valores y estar separadas en mas de 1 sus valores.

[nombre]

Fíjate en el nombre que le pusiste a la función y como la invocaste. Hay una diferencia allí.

Dices sentencias pero no das argumentas.
Que es lo que "no va a funcionar" y por que dices que no va a funcionar?

El cout no es para funciones?
No se que quisiste decir.
Algo así quise decir yo.

Código (cpp) [Seleccionar] Expandir

void pedir(char *mensaje) {
   // Posiblemente algunas instrucciones antes
   cout << mensaje;
   // Otras instrucciones antes
}

Instrucciones antes o después pueden ser los fflush, etc.

Para complementar la didáctica respuesta anterior:

Cada vez que se usa esto:

Código [Seleccionar] Expandir

*(p+i) puede cambiarse por

Código [Seleccionar] Expandir

p[i] si te acomoda más la notación de arreglos.

crear una funcion que reciba el string a mostrar y que internamente haga el cout, el fgets o gets y el fflush.

Tengo la impresion que este do - while se ejecutara por siempre. La condicion es una tautologia.

Código (c) [Seleccionar] Expandir

do{

}while (strcmp(str,"cadena 1")!=0 || strcmp(str,"cadena 2")!=0);

Si no necesitas conservar el orden, al momento de borrar puedes copiar el ultimo en el lugar del borrado, en vez de mover todos 1 lugar.

Ejemplo: Borrar "c"

[ a b c d e f ]    moviendo f al lugar de c.

queda:

[ a b f d e ]

En codigo:

Código (c) [Seleccionar] Expandir

     variosClientes.Clientes[i] = variosClientes.Clientes[variosClientes.numClientes-1];
     --variosClientes.numClientes;


Depende de lo que entiendas por "liberar".
Nada impide que los punteros apuntes a direcciones de memoria asignadas o no. El que tengas un puntero a zona de memoria liberada no la hace estar asignada, solo se convierte en un peligro para la ejecucion de tu programa el que alguna instruccion posterior, haciendo uso de ese puntero no actualizado quiera ir a esa direccion de memoria y leer o escribir alli.

En resumen:
Si "liberar" incluye mantener el estado de la lista, entonces es necesario la asignacion del puntero final. Si "liberar" solo se preocupa de memoria utilizada, no es necesario.

Parece que la logica original hay que modifcarla un poco, pues si existen 2 coliumnas iguales, pero justo las ultimas 2 que se comparan no son iguales, la funcion va a retornar 0.

Es lo que se quiere?