Bueno, mi problema es que el programa me crashea al poner 1 para que genere el codigo. El code no está terminado pero no puedo terminar de resolver eso:
int main()
{
int a, i, j, k, f;
int b[10] = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9};
char c[] = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z', '\0'};
printf("==========================\n== Generador de codigos ==\n==========================\n\n* Ingrese 1 si quiere generar un codigo\n");
scanf("%i", &a);
char letra[5];
if (!(a = 1))
{
printf("Necesitas ingresar el numero 1 para generar un codigo\n");
}
if (a = 1)
{
printf("En breve se generara tu codigo\n");
for(j=0; j=5; j++)
{
i++;
k = 0;
k = 1 + rand() % 23;
c[k] = letra[i];
}
}
return 0;
}
int main()
{
int a;
scanf("%i", &a);
if (!(a = 1))
{
}
if (a = 1)
{
}
}
Eso está mal se mire por donde se mire:
* !(a=1) es equivalente a !1... luego ese if NUNCA se ejecutará
* (a=1) es equivalente a 1... el segundo if se ejecutará SIEMPRE.
La forma correcta de ponerlo es usando los operadores de comparación:
if ( a != 1 )
{
}
if (a == 1)
{
}
}
o mejor aún:
if ( a != 1 )
{
}
else
{
}
}
Luego, quitando eso, creo que deberías fijarte en la variable "i"... que no se inicializa en la vida... al no inicializarse, en "letra" vas a acceder a una posición de memoria incorrecta y el programa acabará con un bonito pantallazo.
char letra[5]; que valores tiene?
Cita de: Stakewinner00 en 12 Marzo 2014, 17:09 PM
char letra[5]; que valores tiene?
Tiene valores aleatorios... pero esas 5 posiciones están reservadas para la aplicación... otra cosa es que intentes acceder a letra[4561234]...
Sinceramente a mí me da la impresión que más que letra
quería poner c.
Cita de: eferion en 12 Marzo 2014, 17:12 PM
Tiene valores aleatorios... pero esas 5 posiciones están reservadas para la aplicación... otra cosa es que intentes acceder a letra[4561234]...
Sinceramente a mí me da la impresión que más que letra quería poner c.
Por eso hice la pregunta, por que me da que se descuido de algo ahí xD.
De paso podrías usar un if else en vez de dos if como te dijo eferion
Saludos
hola wiD^ estaba escribiendo los errores de sintaxis por ejemplo en la igualdad de la condición pero ya te fueron muy bien indicados; con respecto a la función aleatoria, deberías chequear la misma ya que al sumarle 1 no estás contemplando todas las k para el char c. (no tendrás nunca el valor para 'a' y luego creo que superas la cantidad de posiciones para el tope) saludos.
Gracias por toda la ayuda, pero me sigue crasheando el programa. Ahora lo tengo así:
#include <stdio.h>
#include <stdlib.h>
int main()
{
int a, i, j, k, f;
int b[10] = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9};
char c[] = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z', '\0'};
printf("==========================\n== Generador de codigos ==\n==========================\n\n* Ingrese 1 si quiere generar un codigo\n");
scanf("%i", &a);
char letra[5];
i = 0;
if (a != 1)
{
printf("Necesitas ingresar el numero 1 para generar un codigo\n");
}
if (a == 1)
{
printf("En breve se generara tu codigo\n");
for(j=0; j=5; j++)
{
i++;
k = 0;
k = 1 + rand() % 23;
c[k] = letra[i];
}
}
return 0;
}
La condición del for es erronea.
¿Porque usas letra? Usa letra[j].
Como bien dice amchacon,
La condición for es errónea,
Estás dandole un valor a j = 0 y luego dandole de nuevo otro valor a j = 5.
Y para el for sería asín:
for(j=0; j < 5; j++)
{
}
Da un valor a j = 0, y si j es menor que 5, se incrementará.
Un cordial saludo.
Aparte de todo lo dicho falta inicializar la semilla para los números aleatorios, esto puede hacerse con:
#include <stdlib.h>
#include <time.h>
/* ... */
int main(void)
{
/* ... */
srand((unsigned) time(NULL));
/* Uso de rand ... */
}Y en el bucle la primera asignación a la variable "k" sobra, la asignación "c[ k ] = letra[ i ];" esta al revés (debería ser "letra[ i ] = c[ k ];") y por ultimo si se trata de una cadena falta agregar el terminador '\0'.
Cita de: wiD^ en 12 Marzo 2014, 16:50 PMEl code no está terminado pero no puedo terminar de resolver eso
En buen plan deberías conseguir un buen curso o libro, para recomendaciones solo tienes que utilizar el motor de búsqueda de los foros.
Un saludo
Tengo el libro de como programar en C/C++ de deitel. Pero en el libro nunca especifica de terminar el rand.
PD: Seguí adelante con el programa, pero me sigue crasheando, en cuanto ingreso 1 me crashea:
#include <stdio.h>
#include <stdlib.h>
int main()
{
int a, i, j, k, f;
int b[] = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9};
char c[] = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z', '\0'};
printf("==========================\n== Generador de codigos ==\n==========================\n\n* Ingrese 1 si quiere generar un codigo\n");
scanf("%i", &a);
char letra[5];
int numero[5];
i = 0;
if (a != 1)
{
printf("Necesitas ingresar el numero 1 para generar un codigo\n");
}
if (a == 1)
{
for(j=0; j < 5; j++)
{
i++;
k = 0;
k = 1 + rand() % 23;
letra[i] = c[k];
}
i = 0;
j = 0;
f = 0;
for(j=0; j < 5; j++)
{
i++;
f = 0;
f = 1 + rand() % 9;
numero[i] = b[f];
}
printf("%s %s %s %s %s %i %i %i %i %i", letra[0], letra[1], letra[2], letra[3], letra[4], numero[0], numero[1], numero[2], numero[3], numero[4]);
}
return 0;
}
si haces
printf("%s", letra[0] );
estás imprimiendo una cadena... una cadena es una secuencia de caracteres que acaba en el caracter '\0'. Dado que en tu caso, letra tiene una secuencia de caracteres que no termina en '\0', el programa seguirá imprimiendo el contenido de la memoria adyacente a letra y eso hace que el programa se muera.
Si lo que quieres es imprimir un caracter tienes que cambiar el printf por
printf( "%c", letra[0] );
Yo le aconsejaría revisarlo todo ya que hay variables de más, falta la librería time.h, no inicializa rand, haría falta bajo mi punto de vista un bucle por si no introduce 1.
En fin, te lo dejo funcionando. Ya es cosa tuya mejorarlo:
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
int main(){
srand((unsigned) time(NULL));
int a, j, k=0, f=0;
int b[] = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9};
char c[] = {"abcdefghijklmnopqrstuvwxys"};
char letra[5];
int numero[5];
do{
printf("==========================\n== Generador de codigos ==\n==========================\n\n* Ingrese 1 si quiere generar un codigo\n");
scanf("%d", &a);
if (a != 1)
printf("Necesitas ingresar el numero 1 para generar un codigo\n");
}while(a != 1);
for(j=0; j < 5; j++) {
k = 1 + rand() % 25;
letra[j] = c[k];
}
for(j=0; j < 5; j++){
f = 1 + rand() % 9;
numero[j] = b[f];
}
printf("%c %c %c %c %c %d %d %d %d %d", letra[0], letra[1], letra[2], letra[3], letra[4], numero[0], numero[1], numero[2], numero[3], numero[4]);
return 0;
}¡¡¡¡ Saluditos! ..... !!!!
(http://st.forocoches.com/foro/images/smilies/aaaaa.gif)
Dos preguntas:
Para que sirve el include time.h y que hace esto?
srand((unsigned) time(NULL));
Deberías conseguir y leer con tranquilidad un manual de C. Te dará muchas más satisfacciones que darte de cabezazos con estas cosas. Es un consejo.
los includes sirven para enlazar con código externo.
#include <time.h>
se sustituye por el contenido del archivo "time.h". De esta forma el compilador es capaz de encontrar las funciones definidas dentro de este archivo ( en este caso la función time().
Para saber qué hace la línea
srand((unsigned)time(NULL));
Deberías descomponerla en instrucciones simples:
// Esta linea te devuelve la fecha y la hora actuales.
// Realmente almacena el numero de segundos desde el 01/01/1970 a las 00:00
time_t tiempo = time(NULL);
// Esta linea convierte el valor anterior en un tipo basico de C, un entero sin signo.
unsigned int valor = (unsigned)tiempo;
// Esta linea inicializa el generador de numeros aleatorios.
// Pasarle el tiempo es una forma de que la semilla inicial sea mas o menos aleatoria.
srand(valor);
En cualquier caso, en serio, consigue un buen manual de C y empieza a leerlo sin prisa. Adquirir una buena base es más importante que aprender a tirar líneas de código.
Cita de: wiD^ en 13 Marzo 2014, 21:37 PMDos preguntas:
Para que sirve el include time.h y que hace esto?
srand((unsigned) time(NULL));
Tienes que leer con mas cuidado el tema, eso lo respondí en mi primer mensaje:
Cita de: rir3760 en 13 Marzo 2014, 02:40 AMAparte de todo lo dicho falta inicializar la semilla para los números aleatorios, esto puede hacerse con:
#include <stdlib.h>
#include <time.h>
/* ... */
int main(void)
{
/* ... */
srand((unsigned) time(NULL));
/* Uso de rand ... */
}
Antes de utilizar rand y srand se debe incluir <stdlib.h>, antes de time <time.h>.
Un saludo
Una pequeña observación: no guardas el código entero en ninguna parte.
Esto lo puedes solventar usando el mismo array letra con las letras y los números.
Otra cosita: el código puede considerarse débil, en el sentido de que van letras seguidas de números. Podrías hacerlo más fuerte entremezclando los números con las letras como te indico a continuación:
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
int main(){
srand((unsigned) time(NULL));
int a, j, k=0, f=0,cont=0;
char c[] = {"abcdefghijklmnopqrstuvwxys0123456789"};
char letra[11],codigo[11];
for(j=0; j < 11; j++)
codigo[j]='A';
do{
printf("==========================\n== Generador de codigos ==\n==========================\n\n* Ingrese 1 si quiere generar un codigo\n");
scanf("%d", &a);
if (a != 1)
printf("Necesitas ingresar el numero 1 para generar un codigo\n");
}while(a != 1);
for(j=0; j < 5; j++) {
k = rand() % 26;
letra[j] = c[k];
}
for(j=5; j < 10; j++){
f = 26 + rand() % 10;
letra[j] = c[f];
}
letra[j]='\0';
printf("%s\n", letra);
///printf("%d %s\n", j,letra);
for (j=0;j<10;j++){
j=0;
do{
k=rand ()%10;
if (codigo[j]=='A' && letra[k]!='A'){
codigo[j]=letra[k];
letra[k]='A';
j++;
}
}while (j<10);
codigo[10]='\0';
}
printf("%s\n", codigo);
return 0;
}¡¡¡¡ Saluditos! ..... !!!!
(http://st.forocoches.com/foro/images/smilies/aaaaa.gif)