Mensajes

Obviamente los mensajes no son exactamente iguales, eso ya sería para llevarse las manos a la cabeza, pero decis cosas muy similares sin ni tan siquiera hacer mención a la persona que lo ha dicho anteriormente, resulta desagradable ver como cada uno dice una cosa sin tener en cuenta lo que ya se ha dicho, un "tal y como ha dicho no se quien..." no vendría mal para establecer un debate humano.

Este post se está convirtiendo en un bucle, parece que cada uno postea su idea sin tener en cuenta lo que ya se ha dicho, engel lex ya ha dicho exactametne lo que acaba de decir do-while, vchar4 pone lo mismo aunq ya se le ha dicho que es un código un tanto trabajoso, etc... sería interesante para todos que tengamos en cuenta lo que dice cada uno, para poder crecer y esas cosas.

[EDIT]

La forma adecuada de leer enteros por teclado sin dar pie a problemas es así:

Código (cpp) [Seleccionar] Expandir


int num;
char buffer[100];//cadena de texto auxiliar para copiar el buffer de entrada
//------
fgets(buffer,100,stdin); //copiamos el buffer de entrada a la cadena auxiliar
sscanf(buffer, "%d", &num); //leemos el entero de la cadena auxiliar

Esto se hace precisamente para quitar ese retorno de linea que nos está haciendo la puñeta, ya que con fgets estamos vaciando stdin por completo.

EDIT: también puedes meter ese código en una función para no tener que andar repitiéndolo todo el rato:

Código (cpp) [Seleccionar] Expandir


int leerEntero(){
   int num;
   char buffer[100];//cadena de texto auxiliar para copiar el buffer de entrada
   fgets(buffer,100,stdin); //copiamos el buffer de entrada a la cadena auxiliar
   sscanf(buffer, "%d", &num); //leemos el entero de la cadena auxiliar
   return num;
}


También añado que leer una cadena de texto con gets es inseguro ya que gets no controla el numero máximo de caracteres que vas a leer, por tanto deberías usar fgets también. Es decir, en lugar de:

Código (cpp) [Seleccionar] Expandir


gets(nom_alum[i]);


Usa:

Código (cpp) [Seleccionar] Expandir


fgets(nom_alum[i],30,stdin);


Por ultimo, decirte que he puesto 30 pero lo correcto es usar un DEFINE arriba del todo para no tener que andar poniendo 30 por todas partes, es decir:

Código (cpp) [Seleccionar] Expandir


#define MAX_CHARS 30 //esto arriba del todo
int main(...){
//.....
char nom_alum[10][MAX_CHARS];
//.....otras declaraciones....
//.....hacer tareas.....

fgets(nom_alum[i], MAX_CHARS,stdin);
//.....
}


De entrada te diría que no usaras esa parte de wxWidgets, pues es bien sabido que son muy incompletas. Usa wxWidgets solo para lo que es la creación del GUI y poco más. Si quieres una biblioteca buena para hacer aplicaciones cliente/servidor usa BOOST.

Y respondiendo a tu pregunta, sí, deberías actuar de forma totalmente apartada.

Sí lo del retorno del línea es cierto, pero es que fflush(stdin); no deberia solucionarlo. ¿A ti te funciona? Porque eso es muy raro que funcione, pero bueno si te va bien así déjalo, aunque ya te digo que a mi eso nunca me ha funcionado, la solucion correcta es otra más compleja, si quieres más adelante te la explico.

Y sí, yo creo que con eso ya estaría todo arreglado, si no te funciona y no encuentras el error pues coméntalo.

[malloc]

Por cierto, creo que declarar un array de tamaño variable de esa forma no es muy adecuado, deberías usar malloc.

De hecho es que no se puede hacer. Si ese programa compila es que no es C, quiza es alguna versión rara de C, pero en C estándar ese código no puede compilar.

Código (cpp) [Seleccionar] Expandir


int n;
scanf("%d",&n);
int v[n];//esto no puede hacerse en C estándar


Lo correcto sería:

Código (cpp) [Seleccionar] Expandir


int n;
scanf("%d",&n);
int *v = (int*) malloc(n*sizeof(int));
//... hacer tareas
free(v);//al final del programa hay que liberar la memoria, si esto no se hace, quizá no pase nada ya
//que el sistema operativo se encargará de liberarla, pero lo mas adecuado es liberarla aquí.


Perfecto, seguimos:

Código (cpp) [Seleccionar] Expandir


fflush(stdin);

Esto tampoco tiene efecto, fflush sirve para vaciar los buffer de salida, los de entrada no puedes vaciarlos de esta manera. Asique elimina también estas líneas.

Si solo quieres 3 trayectos pues declara solo 3:

Código (cpp) [Seleccionar] Expandir


int  not_tra[3][10];


Y el primer trayecto sera el 0 el segundo el 1 y el tercero el 2. Acostumbrate a reccorrer los arrays de esa manera, así es como se debe hacer siempre en C.

Con respecto a esto:

Código (cpp) [Seleccionar] Expandir


                       for(j=1; j<cant_mate+1; j++){
                           printf("\n\nIngrese las notas parciales de %s\n", nom_mate[j]);

                               printf("\n===== Trayecto 1 =====\n\n");
                                   for (k=1; k<cant_nota+1; k++){
                                       printf("Nota %d: ", k);
                                       fflush(stdin);
                                       scanf("%d",&not_tra1[k]);
                           }
                               printf("\n===== Trayecto 2 =====\n\n");
                                   for (k=1; k<cant_nota+1; k++){
                                       printf("Nota %d: ", k);
                                       fflush(stdin);
                                       scanf("%d",&not_tra2[k]);
                           }
                               printf("\n===== Trayecto 3 =====\n\n");
                                   for (k=1; k<cant_nota+1; k++){
                                       printf("Nota %d: ", k);
                                       fflush(stdin);
                                       scanf("%d",&not_tra3[k]);
                           }
                       }
                   }


Lo puedes sustituir por:

Código (cpp) [Seleccionar] Expandir


   int tra;//variable contadora para el bucle de trayectos, la puedes declarar en otro sitio si quieres.
   for(j=1; j<cant_mate+1; j++){
       printf("\n\nIngrese las notas parciales de %s\n", nom_mate[j]);
       for(tra=0; tra<3; tra++){
            printf("\n===== Trayecto %d =====\n\n",tra+1);
            for (k=0; k<cant_nota; k++){
                 printf("Nota %d: ", k+1);            
                 scanf("%d",&not_tra[tra][k]);
            }              
       }
  }


Fijate en como cambia la forma de recorrer el array:

Código (cpp) [Seleccionar] Expandir


for (k=1; k<cant_nota+1; k++) //mal
for (k=0; k<cant_nota; k++) //bien


Esta parte del código está mal:

Código (cpp) [Seleccionar] Expandir


//Arrays para nombres de alumnos y materias ---Arrays de caracteres---//
   char nom_alum[cant_alum][30];
   char nom_mate[cant_mate][40];
//Arrays para notas de trayectos ---not_tra1 = notas del trayecto1--- //

   int  not_tra1[4][cant_nota];    
   int  not_tra2[4][cant_nota];
   int  not_tra3[4][cant_nota];

   //Llamando a las variables globales//
   cant_alum;
   cant_mate;
   cant_nota;


No puedes definir el tamaño de un array con una variable, es decir:

Código (cpp) [Seleccionar] Expandir

char nom_alum[cant_alum][30];
char nom_mate[cant_mate][40];
int not_tra1[4][cant_nota];

Deberia ser:

Código (cpp) [Seleccionar] Expandir

char nom_alum[10][30];
char nom_mate[10][40];
int not_tra1[4][10];

He puesto 10, pero eso no quiere decir que haya q usar las 10, sino que como maximo tenemos 10.

Otra cosa es que si ya estas creando varios arrays para cada trayecto entonces el [4] sobra, aunque quedaria más organizado en un solo array, es decir, en lugar de:

Código (cpp) [Seleccionar] Expandir


   int  not_tra1[4][cant_nota];    
   int  not_tra2[4][cant_nota];
   int  not_tra3[4][cant_nota];


Deberías poner:

Código (cpp) [Seleccionar] Expandir


   int  not_tra[4][10];    


Luego esta parte:

Código (cpp) [Seleccionar] Expandir


//Llamando a las variables globales//
   cant_alum;
   cant_mate;
   cant_nota;

No sirve para nada, no tiene efecto, esas lineas las puedes eliminar directamente.

Cuando entiendas eso respónde al post y seguimos.

A ver, supongamos que el array lo llenamos con unos y ceros (1:ocupado 0:libre), el usuario no tiene que ingresar ninguna coordenada, tan solo una columna, y tu debes recorres esa columna hasta encontrar un 1, una vez encuentres el 1 debes cambiar el valor superior por un 1.

Ese es el principio básico. Luego tendrás que establecer más condiciones, como por ejemplo que si llegas a la fila 10 y no has encontrado ningun 1 pues coloques un 1 en la fila 10. También que si hay un 1 en la fila 1 no te permita hacer la jugada, etc.

A ver, cuando escribes

Código (cpp) [Seleccionar] Expandir

char *comp; estás reservando en memoria una sola variable que almacenará un puntero a char. Sin embargo tu necesitas reservar una variable para cada carácter, por eso es necesario crear un array

Código (cpp) [Seleccionar] Expandir

char comp[40]; De esa manera reservas 40 variables en memoria y tienes espacio para 40 caracteres (en realidad para 39, ya que el ultimo debe dejarse libre para el carácter nulo que indica la finalización de cadena).

El nombre de un array devuelve la dirección de memoria del primer elemento del array, es decir en este caso escribir comp es lo mismo que escribir &comp[0]

Al llamar a scanf("%s",comp) estás haciendo que los caracteres introducido por teclado se guarden en la memoria, la primera dirección de memoria en la que se guarda el primer carácter es la que le pasamos por parámetro, en este caso es la apuntada comp, el segundo carácter se guardara en esa misma direcion+1 y así hasta terminar.

Por tanto, si declaras comp como puntero y no reservas espacio en memoria y luego intentas llenar la memoria sin haberla reservado previamente puedes dar lugar a error o funcionamientos impredecibles.