La primera funcion de pasar un array a una funcion, eso OK, y luego retornar la posicion de la primera celda libre sino retornar -1, eso no lo sabia hacer.
jorcoval escribió:Tú has dicho que hay que recorrer un array, no un struct:La primera funcion de pasar un array a una funcion, eso OK, y luego retornar la posicion de la primera celda libre sino retornar -1, eso no lo sabia hacer.
así que recorres un array. Sea de lo que sea. OJO: que un array puede ser de structs... que creo que es lo que te están pidiendo y es lo que te estás liando.
Vamos a diferenciar, de nuevo, a lo guarro:
- Un array es una estructura de memoria, donde las direcciones son CONSECUTIVAS y contienen TODAS el mismo TIPO DE DATOS.
-Un struct es una estructura de memoria donde las direcciones PUEDEN CONTENDER DISTINTOS tipos de DATOS
Por supuesto, un array puede contener structs (lo que normalmente se llama en C punteros a struct).
Que una celda esté vacía, significa que no contiene un struct, no que contenga un struct y debas comprobar sus distintos campos (matrícula, número de ruedas... ).
Estoy tratando de no meterme en particularidades del lenguaje (las celdas contienen null, etc) para tratar de que entiendas QUÉ hay que hacer.
Cuando esté claro, nos metemos en el CÓMO
(dibujarte las distintas estructuras de datos te ayudarán a entenderlas)
http://www.etnassoft.com/biblioteca/
![[+risas]](/images/smilies/nuevos/risa_ani3.gif "más risas")
pero no pierdas un tiempo desorbitado en ver por qué no compila una cosa si no tienes ni el más mínimo indicio de dónde puede estar el error.Aceskies escribió:Aprender a programar consiste en echar horas, horas y horas al código en casa. Al principio, que es donde estás tú, lo más normal es estar bastante atascado, sobre todo si la documentación que te dan no es la adecuada para un principiante, cosa que por experiencia suele ser así. Pero en cualquier caso, programar es una actividad bastante autodidacta.
Eso sí, un consejo te doy. Haz ejercicios. Si te quedas atascado en algo, por tonto que sea, y no logras salir del problema, PREGUNTA. Al profesor, a tus compañeros, o a google
Como decía mi profesor de IA, la tarea de programar es una de las más difíciles que hay, porque por muchas estimaciones de tiempo que te pongas para terminar una tarea, NUNCA se van a cumplir. Para agilizar el trabajo siempre tienes que focalizarte en resolver las incógnitas lo máximo posible, e ir disminuyendo la incertidumbre del "por qué no funciona". Te podría decir, como algunos han hecho, la solución a tu examen, y eso te vendrá genial para aprender. Pero a largo plazo, lo que necesitas es aprender a debuggear el código. Pon prints allá donde lo necesites, experimenta con trozos de código problemáticos, probando a cambiar valores, variables, estructuras de datos y modificando el algoritmo. Cuanto más te pelees con tu código, más vueltas le des y más maneras de solucionarlo intentes, más cerca estarás.
Ánimo con ello. Y te aseguro que a todos nos ha costado aprender a programar. Y de hecho es algo que siempre se está aprendiendo. Programar es fácil, cualquiera puede hacerlo, programar de manera correcta es muy difícil.
case 2:
printf("\nMODIFICAR DATOS");
printf("Marca de la moto: ");
gets(mar);
printf("Modelo: ");
gets(mod);
do{
printf("Cilindrada: ");
scanf("%d",&cilin);
if(cilin<50 || cilin>1200){
printf("\nError: la cilindrada debe estar entre 50 y 1200\n\n");
}
}while(cilin<50 || cilin>1200);
if(modificar(moto,mar,mod,cilin)==true){
printf("\nPrecio de la moto actualizado.\n");
printf("");
}else{
printf("\nMoto inexistente.\n");
}
break;
bool modificar(struct motos *moto, char *mar, char *mod, int cilin, float *preu){
int i=0;
while((strcmp(moto->marca,mar)!=0 || strcmp(moto->modelo,mod)!=0 || moto->cilindrada!=cilin) && i<MAX){
i++;
moto++;
}
if(i<MAX){ //se ha salido del bucle porqué se ha encontrado la moto
do{
printf("\nIntroduce el nuevo precio: ");
scanf("%f",&moto->precio);
if(moto->precio<400 ||moto->precio>70000){
printf("\nError: el precio debe ser mayor de 400 €\n\n");
}
}while(moto->precio<400 || moto->precio>70000);
*preu=moto->precio;
return true;
}
else
return false;
}
YonkiKong escribió:Gracias por los consejos, he tenido otros examenes e por medio pero ahora solo me queda ir a tope con el de programación, volviendo al problema que os expuse, ya se hacer la funcion que os expuse y solo me falt una funcion que no entiendo muy bien como hacer.
Ahí va:
(Acordaos que habiamos hecho una struct con marca,modelo,cilindrada y precio)
Función que recibe un array de motos, marca, modelo y cilindrada y devuelve un booleano indicando si se encontró la moto y el nuevo precio de la moto (anteriormente se tenia que hacer una función donde rellenabamos la informacion de una moto)
Bien, como no se me ocurria como empezar finalmente tuve que irme a la solucion, y el código de la funcion es:
Fuera de la funcion, en el void main.case 2:
printf("\nMODIFICAR DATOS");
printf("Marca de la moto: ");
gets(mar);
printf("Modelo: ");
gets(mod);
do{
printf("Cilindrada: ");
scanf("%d",&cilin);
if(cilin<50 || cilin>1200){
printf("\nError: la cilindrada debe estar entre 50 y 1200\n\n");
}
}while(cilin<50 || cilin>1200);
if(modificar(moto,mar,mod,cilin)==true){
printf("\nPrecio de la moto actualizado.\n");
printf("");
}else{
printf("\nMoto inexistente.\n");
}
break;
En la funcionbool modificar(struct motos *moto, char *mar, char *mod, int cilin, float *preu){
int i=0;
while((strcmp(moto->marca,mar)!=0 || strcmp(moto->modelo,mod)!=0 || moto->cilindrada!=cilin) && i<MAX){
i++;
moto++;
}
if(i<MAX){ //se ha salido del bucle porqué se ha encontrado la moto
do{
printf("\nIntroduce el nuevo precio: ");
scanf("%f",&moto->precio);
if(moto->precio<400 ||moto->precio>70000){
printf("\nError: el precio debe ser mayor de 400 €\n\n");
}
}while(moto->precio<400 || moto->precio>70000);
*preu=moto->precio;
return true;
}
else
return false;
}
Aqui vienen mis dudas, la primera es... si te piden que la funcion recibe un array de motos, marca, modelo y cilindrada, porque te preguntan por el precio????
Ahí va:
(Acordaos que habiamos hecho una struct con marca,modelo,cilindrada y precio)
Para empezar aunque no se use "for" si pone "i++" y "moto++" funciona igual que un "for" no? , luego la forma de usar strcmp no la entiendo (se que es para comparar cadenas), se hace strcmp y entre parentesis pone la estructura señalando a marca (este lo habremos rellenado antes, por eso se llama el problema MODIFICAR) y luego mar por separado, diferente a 0, porque diferente a 0? Se supone que marca y mar tienen que coincidir para encontrar la moto.
El resto de comparaciones tampoco las entiendo, a ver si hay alguien que lo entienda.
Muchísimas gracias.
void main(){
char saludo[10];
printf("introduce saludo\n");
//el usuario introduce "hola que tal estas yo muy bien y tu"
gets(saludo);
printf(saludo);
}
jorcoval escribió:Si te sale basura, es que probablemente tengas un desbordamiento de buffer.
Y veo que utilizas gets, que no controla el tamaño de los strings, así que es muy probable que te esté pasando.
1) No uses gets o controla muy muy bien el tamaño de las cadenas introducidas (vamos, que no uses gets)
2) Controla los tamaños de las cadenas que recorres (sobre todo si usas gets... no uses gets)
3) Controla los tamaños de las cadenas ANTES de unirlas
Para que entiendas lo que te pasa:void main(){
char saludo[10];
printf("introduce saludo\n");
//el usuario introduce "hola que tal estas yo muy bien y tu"
gets(saludo);
printf(saludo);
}
printf ("\nIntroduce a partir de que posicion quieres que se copie la segunda frase: ");
scanf ("%d",&i);
return i;i=n;
while (cad1[i]!='\0')
{
aux[j]=cad1[i];
i++;
j++;
} . Y un consejo para hacer debug de todo esto, empieza a poner printf en ciertos puntos y realmente iras viendo que vas leyendo, copiando, etc... con programas asi es bastante simple hacer debugging.Mistercho escribió:Lo primero que no entiendo es que hace esto en la funcion de contarprintf ("\nIntroduce a partir de que posicion quieres que se copie la segunda frase: ");
scanf ("%d",&i);
return i;
A mi entender forma parte de los valores a introducir por el usuario por lo que iria en el main.
Mistercho escribió:Otra cosa que veo es que si no piden la version mas optimizada, usa mas variables y no reutilices el integer i, no tiene porque ser un problema, pero si tienes libertad, para que complicarse.
Mistercho escribió:Por ultimo, sobre tu forma de cambiar las cadenas, yo lo haria asi, en la cadena aux, que a ser posible la defino del mismo tamaño que las cadenas 1 y 2, pongo un contador y copio los primeros n caracteres, segun haya indicado el usuario en el numero, de la cadena 1, despues copio la cadena 2 entera hasta \0 y por ultimo vuelvo a copiar hasta \0 de la cadena 1, con 2 whiles anidados y varios contadores lo haces facil de una pasada.
Mistercho escribió:Ademas, revisando un poco:i=n;
while (cad1[i]!='\0')
{
aux[j]=cad1[i];
i++;
j++;
}
Estas copiando caracteres desde la posicion n en aux, es decir, basura post \0, cuando deberias empezar desde la posicion 0 hasta n.
En fin, yo que no me dedico a esto sino a administracion te comento lo que veo
Un saludo.