1

soy nuevo en la programación, estoy haciendo un programa que obtenga las calificaciones N alumnos, sus nombres y determinar su condición, todo en funciones diferentes, la cosa es que estoy teniendo problemas para poner la condición de los alumnos ya que el ejercicio pide que se pase a la función un arreglo de condiciones vacio.

El arreglo condicion lo cree a partir del numero de alumnos y como la palabras Satisfactorio y Sobresaliente son las mas largas y ambas contienen 13 caracteres (14 mas el carácter nulo según entiendo) puse 15 en el otro tamaño para redondear.

Me da error dentro de los condicionales if en donde estoy tratando de asignarle las palabras al arreglo y el error que me lanza el compilador (Codeblocks) es el sigiuente:

error assignment to expression with array type

Espero me puedan orientar, gracias.

void calificar(int calis[ALUMNOS], char condicion[ALUMNOS][15])
{
    int i, j;
    for(i = 0; i < ALUMNOS; i++)
    {
        if (calis[i] < 6)
        {
            condicion[i] = "Reprobado";
        }
        if (calis[i] > 6 && calis[i] <= 8)
        {
            condicion[i] =  "Satisfactorio";
        }
        if (calis[i] >= 9)
        {
            condicion[i] = "Sobresaliente";
        }
    }

    for(j = 0; j < ALUMNOS; j++)
    {
        printf("\nLa condicion de alumno %d es: %s", j+1, condicion[j]);
    }
}

2 respuestas 2

2

La forma en la que asignas la cadena no es correcta, usa strcpy de la libreria string.h y asigna la cadena de esta forma:

strcpy(condicion[i], "Reprobado");
5
  • string.h no es una librería sino un archivo de encabezado.
    – MrDave1999
    el 26 mar. 2020 a las 16:17
  • Creo que es correcto también llamarlas librerías, no se si me estoy perdiendo en algo, pero yo aprendí a llamarlas "librerías" más que como "archivos de encabezado".
    – 4lrdyD
    el 26 mar. 2020 a las 18:54
  • Tuve esa misma duda hace mucho tiempo. Mira este hilo.
    – MrDave1999
    el 26 mar. 2020 a las 18:58
  • 1
    Definitivamente tiene sentido, puede ser más correcto llamarlos "archivos de encabezado" para no confundirlas con las bibliotecas estáticas o también dll's que usamos en muchos proyectos, aunque tampoco estoy seguro si sea totamente incorrecto llamar a los archivos de encabezado también librerías, aquí parece que no las diferencian mucho.
    – 4lrdyD
    el 26 mar. 2020 a las 19:30
  • En la wiki lo explica claramente, una biblioteca contiene código binario.
    – MrDave1999
    el 26 mar. 2020 a las 19:48
0

Primero que nada, en C no es posible asignar una cadena literal a un array o matriz (de tipo char) después de su declaración.

Por ejemplo:

char name[24];
name = "Ivan";//error

char names[100][24];
names[0] = "Ivan";//error

El ejemplo de arriba dará un error de compilación, debido a que, es ilegal asignar una cadena después de su declaración.

Por lo tanto, la solución más simple es la de @4lrdyD, sin embargo, no es recomendable usar la función strcpy para copiar cadenas constantes (es decir, siempre es la misma durante la ejecución del programa) en un búfer, debido a que, harás el programa lento y consumirá memoria.

¿Por qué es ineficiente?

Porque en cada iteración estarías copiando la misma cadena (por eso es constante) en X fila de la matriz condicion.

Por ejemplo, si la condición fuera para 100 alumnos. La matriz condicion estaría consumiendo 1500 bytes.

Y esto lo compruebas de esta manera:

char condicion[100][15];
printf("%d\n", sizeof condicion);

Entonces sería un desperdicio de memoria estar copiando la misma cadena en cada fila de la matriz condicion, por esa razón es ineficiente.

Para optimizar este código podríamos usar un arreglo de punteros a char:

#include <stdio.h>
#include <stdlib.h>
#define ALUMNOS 100

void calificar(int[], char*[]);

//Code example:
int main (void)
{
    int calis[ALUMNOS] = {9};
    char* condicion[ALUMNOS];
    calificar(calis, condicion);
    return 0;
}

void calificar(int calis[], char* condicion[])
{
    int i, j;
    for(i = 0; i < ALUMNOS; i++)
    {
        if (calis[i] < 6)
            condicion[i] = "Reprobado";
        if (calis[i] > 6 && calis[i] <= 8)
            condicion[i] =  "Satisfactorio";
        if (calis[i] >= 9)
            condicion[i] = "Sobresaliente";
    }
    for(j = 0; j < ALUMNOS; j++)
        printf("\nLa condicion de alumno %d es: %s", j+1, condicion[j]);
}

¿Por qué es más eficiente?

Porque no estamos copiando en cada iteración la misma cadena. La cadenas literales: Reprobado, Satisfactorio y Sobresaliente están alojados en un bloque de memoria, por ende, cada puntero del arreglo de punteros apuntarán a cualquiera de esos tres bloques de memoria. Por lo tanto, estaríamos ahorrando memoria y tiempo.

Recalco esta es una optimización que hace el compilador. Por lo tanto, se debe asegurar que las optimizaciones que haga el compilador estén activadas.

Por ejemplo, el compilador cl.exe (de Visual Studio) no hará esta optimización hasta que le pases el comando /Ox (ver este hilo).

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.