[C++] Ayuda para ejercicio de un calendario usando arrays.

[nadales56]

Hola, buenas a todos, tengo una pequeña duda sobre el siguiente ejercicio, les dejo el enunciado en negrita.

Defina el tipo Matriz como un array de 6x7 números enteros, y diseñe un subprograma para crear un calendario de un mes genérico en una matriz que se devolverá como parámetro de salida.
El subprograma se denominará calendario.  El primer parámetro (m), de salida, es de tipo Matriz.  El segundo parámetro (ndias), de entrada, es de tipo int y representa el número total de días del mes del calendario (28, 29, 30 o 31). El tercer parámetro (pds), de entrada, es de tipo int y representa el índice del día de la semana donde comienza el primer día del mes (donde 0 corresponde a Lunes, 1 a Martes, y así sucesivamente hasta 6 que representa al Domingo).


El código que he implementado es el siguiente:

Código (cpp) [Seleccionar] Expandir


void calendario(Tipo_Matriz &m, const int ndias,const int pds){
//Iniciar calendario me pone todos los valores de la matriz a 0
   iniciar_calendario(m);
   
   int dia = 1;
   int j;
   for(int i = 0; i < 6; i++){
       if(i == 0){
           j = pds;
       }else{
           j = 0;
       }
       while(dia <= ndias && (j >= 0 && j < 7)){
           m[i][j] = dia;
           dia++;
           j++;
       }
   }
   //mostrar calendario
   cout << endl;
   for(int i = 0; i < 6; i++){
       for(int j = 0; j < 7; j++){
           cout << m[i][j] << " ";
       }
       cout << endl;
   }
}


Cuando introduzco dias = 30 y pds = 2 (por ejemplo), el resultado que obtengo es:
0 0 1 2 3 4 6
6 7 8 9 10 11 13
13 14 15 16 17 18 20
20 21 22 23 24 25 27
27 28 29 30 0 0 0
0 0 0 0 0 0 0
me como una serie de números y no sé exactamente donde está el error, por lo que si pudiesen ayudarme, me sería de gran ayuda.
Muchas gracias!




Mod: No escribir titulos en mayusculas.

[engel lex]

Código (cpp) [Seleccionar] Expandir

for(int i = 0; i < 6; i++){
       if(i == 0){
           j = pds;
       }else{
           j = 0;
       }
       while(dia <= ndias && (j >= 0 && j < 7)){
           m[i][j] = dia;
           dia++;
           j++;
       }

uno de tus errores (si es que hay mas) está aqui...
en el while
ciclo 1: j=2, dia=1 -> j=4, dia=2
ciclo 1: j=3, dia=2 -> j=5, dia=3
ciclo 1: j=4, dia=3 -> j=6, dia=4
ciclo 1: j=5, dia=4 -> j=7, dia=5
el ciclo se rompe porque  porque j pasa a ser 7 ergo no llega al dia 5

pero no estoy seguro por que se salta los siguientes dias

para ser sincero yo lo habría hecho linearmente con division y modulo

Código (cpp) [Seleccionar] Expandir

for(i=0; <=ndias; i++){
 int x = (i+pds) % 7
 int y = (i+pds) / 7
 m[y][x] = i+1;

}

y te ahorrar andar trabajando con mas de 1 dimensión gracias a la magia de las matemáticas XD

[nadales56]

Muchas gracias por la respuesta!
He probado el código que me has puesto, me ha parecido muy interesante y es una forma que no me hubiese planteado nunca, por lo que te agradezco mucho el aporte.

Tengo una pregunta al respecto, en qué tipo de situaciones (aparte de esta claro está) se ultiliza?, lo digo por si puedo aplicarlo en otro tipo de ejercicios, por tener esa opción en cuenta.

Por otra parte, el resultado que obtengo es el mismo que obtenía antes, me hace ese salto de número cuando llego a m[y][6].

Lo que he visto, que me ha parecido curioso es lo siguiente:

Código (cpp) [Seleccionar] Expandir



for(int i = 0; i < ndias; i++){
            int x = (i+pds) % 7;
            int y = (i+pds) / 7;
            m[y][x] = i+1;
            cout << m[0][6] << " "; // linea nueva para comprobar el estado de la matriz en esa posicion

}



el resultado es este:
0 0 0 0 5 6 6 6 6 6 6 6 6 6...
de manera que llega a poner el 5, pero luego me pasa al 6 y se queda con ese último valor.

iré echandole un vistazo a lo largo del dia, a ver si llego solo a la solución. En cuanto la consiga la postearé.

Repito, muchas gracias por el aporte! =)

[K-YreX]

Hola, buenas a todos, tengo una pequeña duda sobre el siguiente ejercicio, les dejo el enunciado en negrita.

Defina el tipo Matriz como un array de 6x7 números enteros, y diseñe un subprograma para crear un calendario de un mes genérico en una matriz que se devolverá como parámetro de salida.
El subprograma se denominará calendario.  El primer parámetro (m), de salida, es de tipo Matriz.  El segundo parámetro (ndias), de entrada, es de tipo int y representa el número total de días del mes del calendario (28, 29, 30 o 31). El tercer parámetro (pds), de entrada, es de tipo int y representa el índice del día de la semana donde comienza el primer día del mes (donde 0 corresponde a Lunes, 1 a Martes, y así sucesivamente hasta 6 que representa al Domingo).


El código que he implementado es el siguiente:

Código (cpp) [Seleccionar] Expandir


void calendario(Tipo_Matriz &m, const int ndias,const int pds){
//Iniciar calendario me pone todos los valores de la matriz a 0
   iniciar_calendario(m);
   
   int dia = 1;
   int j;
   for(int i = 0; i < 6; i++){
       if(i == 0){
           j = pds;
       }else{
           j = 0;
       }
       while(dia <= ndias && (j >= 0 && j < 7)){
           m[i][j] = dia;
           dia++;
           j++;
       }
   }
   //mostrar calendario
   cout << endl;
   for(int i = 0; i < 6; i++){
       for(int j = 0; j < 7; j++){
           cout << m[i][j] << " ";
       }
       cout << endl;
   }
}


Cuando introduzco dias = 30 y pds = 2 (por ejemplo), el resultado que obtengo es:
0 0 1 2 3 4 6
6 7 8 9 10 11 13
13 14 15 16 17 18 20
20 21 22 23 24 25 27
27 28 29 30 0 0 0
0 0 0 0 0 0 0
me como una serie de números y no sé exactamente donde está el error, por lo que si pudiesen ayudarme, me sería de gran ayuda.
Muchas gracias!




Mod: No escribir titulos en mayusculas.

Después de estar un buen rato buscando dónde podía estar el fallo, he terminado por probar el código personalmente y... Esta es la salida que he obtenido:

Código [Seleccionar] Expandir


0 0 1 2 3 4 5
6 7 8 9 10 11 12
13 14 15 16 17 18 19
20 21 22 23 24 25 26
27 28 29 30 0 0 0
0 0 0 0 0 0 0
Por lo que no hay ningún error. No sé si es que lo has escrito manualmente y entonces has "corregido" el error sin darte cuenta o es que has modificado algo y lo estás ejecutando sin volver a compilarlo pero ya ves que funciona. Obviamente se puede mejorar como ya te han demostrado pero correcto es. Por ejemplo esta parte:

Código (cpp) [Seleccionar] Expandir


if(i == 0)
    j = pds;
else
    j = 0;
La condición del <if> va a ser cierta una vez (en la primera iteración). Por lo que se puede poner directamente el valor de <pds> a la <j> antes del bucle.

Código (cpp) [Seleccionar] Expandir


int dia = 1;
int j = pds;
for(...){
    while(...){
        // el cuerpo del while
    }
    j = 0; // actualizamos siempre a 0 despues de la primera iteracion
}

Y respecto a la solución que te ha dado @engel lex, también es correcta. El resultado es el mismo que he mostrado antes. Comprueba que estés guardando y recompilando el código con cada modificación o si estás usando un IDE prueba a cerrarlo y volver a abrirlo.

Ya nos contarás si consigues solucionarlo y cuál era el problema al final. Suerte

[engel lex]

efectivamente como te dicen tienes algo malo y no en tu codigo...

sobre la solucion, es basicamente como el programa aplica una matriz, en realidad una matriz no deja de ser unidimensional (porque en general la ram lo es), lo que hace es calcular los saltos para saber donde debe estar al acceder, yo hice lo mismo,  se que cada sección tiene 7 de largo, ergo divido sin resto entre 7, los elementos está dentro de los 7 contiguos que es el resto de la division

[nadales56]

Buenas señor@s,
estaban ustedes en lo cierto, he descargado otro editor de texto y he actualizado la versión del compilador y no me da ningún problema.

Gracias igualmente por la ayuda que me habéis prestado. Un saludo!! =)