Buenas.
Vuelvo a tener un pequeño problema con comparar unos datos.
Este código si recibe 1111 entra en modo cambiar clave. Los siguiente 4 números que vuelvas a introducir, son la nueva clave y por tanto los guarda en la EEPROM.
if (cadena[0]==cambio[0]&&cadena[1]==cambio[1]&&cadena[2]==cambio[2]&&cadena[3]==cambio[3])
{ cambioclave=true;
}
if (cambioclave==true){
Serial.print("Introduce la nueva clave.");
EEPROM.write(0, cadena[0]);
EEPROM.write(1, cadena[1]);//Guardo cada uno en cada posicion
EEPROM.write(2, cadena[2]);
EEPROM.write(3, cadena[3]);
valor =EEPROM.read(0);
valor =EEPROM.read(1);
valor =EEPROM.read(2);
valor =EEPROM.read(3);
cambioclave=false;
}
}
if (cadena[0]==usuario[0]&&cadena[1]==usuario[1]&&cadena[2]==usuario[2]&&cadena[3]==usuario[3])//si la clave es correcta...
{ Serial.print("Acceso aceptado");
}else{
Serial.print("Acceso denegado");
}
}
El problema viene cuando meto la nueva clave para que me salga Acceso aceptado, no funciona.
Gracias por su ayuda y atención.
Creo que no has corregido esto, que tendría que empezar desde cero:
if (cadena[1]==cambio[1]&&cadena[2]==cambio[2]&&cadena[3]==cambio[3]&&cadena[4]==cambio[4])
Y para comparar dos cadenas tienes una función strcmp()
En lugar de
if (cadena[0]==usuario[0]&&cadena[1]==usuario[1]&&cadena[2]==usuario[2]&&cadena[3]==usuario[3])
puedes hacer
if(strcmp(cadena, usuario) == 0)
Perdón, sí estaba corregido, pero he copiado y pegado del foro...en cuanto a esa función quiero usarla más adelante una vez tenga claro esto.
¿Se te ocurre por qué no funciona? Yo creo que es por las ' ' que llevan... pero son necesarias ya que es una cadena. Digo que son las ' ' porque si hago un Serial.print de los valores, me salen muchísimos números.
Ahora los leo con serial y me salen en ASCII, les resto 48 para que sean enteros y los mando a comparar...
...y lo que pasa es, que al poner ' 'aquí -> char usuario[5]={'numero1','numero2','numero3','numero4'}; SE LÍA PARDA, porque esas' ' hacen algún tipo de conversión, y al condicional en vez de llegarle el código correcto le llegan muchísimos números, y claro, pues no lo acepta como válido.
int t='0';
int t1='0';
int t2='0';
int t3='0';
unsigned long num = t;
unsigned long num1 = t1;
unsigned long num2 = t2;
unsigned long num3 = t3;
char usuario[5]={num,num1,num2,num3};
En cambio, si hago así no:(lógico porque lo convierto a char...entonces,¿qué hago?)
int t=0;
int t1=0;
int t2=0;
int t3=0;
unsigned long num = 't';
unsigned long num1 = 't1';
unsigned long num2 = 't2';
unsigned long num3 = 't3';
char usuario[5]={num,num1,num2,num3};
El error es ese, que el char no pilla los 't', 't1','t2' y 't3'.
Ese comando, convierte a unsigned long no sé muy bien el qué. Yo el único uso que conozco de las comillas simples es el de asignar el valor de un caracter a algo, es decir, su valor ascii. En tu primer ejemplo:
int t='0';
int t1='0';
int t2='0';
int t3='0';
unsigned long num = t;
unsigned long num1 = t1;
unsigned long num2 = t2;
unsigned long num3 = t3;
char usuario[5]={num,num1,num2,num3};
Podrías, perfectamente, haber asignado directamente char usuario[5]={t,t1,t2,t3} (por cierto, usuario[4] no lo usas) y el array contendría los valores ascii de t-t3. Sin pasos intermedios, podrías almacenar directamente char usuario[5]={'0','0','0','0'} y contendría lo mismo, y si lo que quieres es almacenar directamente un valor, en lugar de caracteres ascii, char usuario[5]={0,0,0,0} (sin las comillas simples).
A partir de aquí, ¿Qué vas a comparar? Si vas a cotejar byte a byte caracteres, no es necesario hacer conversiones, ya que si, por ejemplo char a='8', a=='8' independientemente de que en la memoria no haya un 8, sino el valor ascii de '8'. Si lo que deseas es obtener el valor 'puro' del dígito para realizar algún cálculo, deberás restar del ascii del dígito el valor ascii del cero, osea int x=a-'0', pero esto no tiene mucho sentido, a no ser que quieras crear una función para convertir varios dígitos en un int, que se haría más o menos así:
int valor;
char digitos[]={'1','7','6','8'};
valor=(digitos[0]-'0')*1000;
valor+=(digitos[1]-'0')*100;
valor+=(digitos[2]-'0')*10;
valor+=(digitos[1]-'0'); // Aquí valor contendría 1768
Todo lo que has puesto lo he intentado. He convertido a ASCII por una función, luego por atoi.....Al final he hecho una función que devuelve el número que meto pero con 'x', que es lo que la string necesita.
Pero muchas gracias por tu ayuda, ahora necesito simplificar la función(ya que es muy extensa).
PascuaL:
Todo lo que has puesto lo he intentado. He convertido a ASCII por una función, luego por atoi.....Al final he hecho una función que devuelve el número que meto pero con 'x', que es lo que la string necesita.
Pero muchas gracias por tu ayuda, ahora necesito simplificar la función(ya que es muy extensa).
Perdona, pero me está constando a veces bastante entenderte. En el código de tu primer post, comienzas con unas declaraciones "raras", del tipo
int valor='valor-48';
¿Quiero entender que tienes valores de dígito directamente y quieres convertirlos a su caracter ascii?
en ese caso, si suponemos que tienes
int a= 1; int b=5; int c=8; int d=0;
podrías pasarlos a sus dígitos correspondientes así:
char micadena[4]={a+'0',b+'0',c+'0',d+'0']; // Mi cadena contendrá {'1','5','8','0'}
Perdona, pero me está constando a veces bastante entenderte. En el código de tu primer post, comienzas con unas declaraciones "raras", del tipo
int valor='valor-48';
Como bien he dicho, era para hacer la conversión de ASCII.
Entonces no te compliques la vida; quita las comillas simples:
Valor de dígito a caracter:
char caracter=valor+48; // O char caracter=valor+'0'; pues supongo que 48 es el código ascii de '0'
Caracter de dígito a valor:
int valor=caracter-48; // O int valor=caracter-'0';