[MV-Sabios] Problema programacion

http://www.media-vida.net/verforo.php?fid=17

apaga tu pantalla y escribe la función en papel. No falla.

#include
#include <stdlib.h>
#include <time.h>
using namespace std;

const int MAXCLAVE=10;
typedef char TVector[MAXCLAVE];

bool CompruebaClave (TVector clave1)
{
int i=0, j=0;
bool hayelemrepe=false;
while (hayelemrepe && i<4) {
while (hayelemrepe && j<4) {
if (clave1==clave1[j]) {
hayelemrepe=true;
}
j++;
}
i++;
}
return hayelemrepe;
}

void CreaClave (TVector& clave1)
{
TVector auxiliar;
char letra='a';
int i, j, k=0, l=0, aux;
bool hayelemrepe=false;

 for (i=0 ; i<MAXCLAVE ; i++) {
     auxiliar[i]=letra;
     letra++;
     }
     
 do {
    for (i=0 ; i<4 ; i++) {
        for (j=0 ; j<100 ; j++) {
            aux=rand()%10;
            }
            clave1[i]=auxiliar[aux];
            }
            } while (CompruebaClave (clave1));
              
 for (i=0 ; i<4 ; i++) {
     cout << clave1[i] << " ";
     }
 cout << endl;

}

int main ()
{
srand(time(0));
TVector clave;
CreaClave (clave);
system("Pause");
return 0;
}

EDIT: Si quieres te lo paso en un .txt porque aqui no respeta los tabuladores ni nada de eso y es ilegible.

Al margen de que esto vaya en DEV...

No me he bajado el código (ponlo en www.pastebin.es ), pero te recomiendo que te crees un array booleano de 256 elementos y lo inicialices con un FOR a FALSE.

Cada vez que generes una letra de la clave mira si array_booleano[(int)letra] está a FALSE. Si es así, lo pones a TRUE y pasas a generar la siguiente letra de la clave y si no, vuelves a generar otra letra y haces lo mismo.

Salu2

ponlo en FAKE haber si va xD

k es lo k tengo k invertir?

#13, si. Ese ya está bien xD

y ahora por qué hostias me da este problema al compilar??

CreaClave (TVector clave[nivel+3], nivel);

dice "expected primary expression before nivel".

lo unico que he hecho ha sido ajustar la longitud de la cadena al nivel del juego. la longitud es el nivel+3.

nivel 1: 4 letras
nivel 2: 5 letras.
...
nivel 5: 8 letras.

Hay cosas que no entiendo:

for (i=0 ; i<MAXCLAVE ; i++) {
auxiliar=letra;
letra++;
}

(Aquí estás metiendole a auxiliar a,b,c,d,e,f,g,h,i,j. Esto no es que no lo entienda, es que es innecesario.)

do {
for (i=0 ; i<4 ; i++) {
for (j=0 ; j<100 ; j++) {
aux=rand()%10;
}
clave1=auxiliar[aux];


Esto es lo que no entiendo, haces dos for, uno dentro de otro y el de dentro se va a repetir 100 veces porque sí creando números aleatorios. Es decir, vas a crear 400 números para quedarte con 4 :/. Es que no entiendo muy bien el por qué has hecho eso xD

En vez de hacer eso, el crear una array con las 10 primeras letras del alfabeto, le añades directamente 97 al número aleatorio (97 es la 'a' en ASCII) y ya tienes una función que genera letras xD

clave=char(rand()%10 + 97);

de hecho sería mejor así:

clave=char(rand()%MAXCLAVE + 97);

(creo que no haría falta usar la función char pero tampoco estoy muy seguro ahora mismo xD)

Pero bueno, si lo quieres hacer con la auxiliar, lo que puedes hacer es que cada vez que saques una letra, la pones en NULL y así puedes comprobar directamente si ya ha sido cogida o no. Te ahorras el tener que hacer otra función que compruebe eso, ya que tienes una array auxiliar, aprovechala al máximo xD

También tienes algunas variables declaradas que no usas.

Y por cierto, el debugger es la mejor herramienta de ayuda para cualquier programador xD