1

estoy haciendo una asignación con matrices en C y me gustaría saber si es posible guardar cadenas dentro de una matriz, pero mediante entradas del usuario, lo he intentado caracter por caracter, pero al ejecutar no muestra nada. Aquí está mi código

int main()
{
    int cp = 0, i, j, ll = 50;
    char palabra[cp][ll];
    printf("indique cantidad de palabras: ");
    scanf("%d", &cp);

    for(i = 0; i<cp; i++)
    {
        for(j = 0; j<ll; j++)
        {
            printf("\nindique una palabra:");
            scanf("%c", palabra[i][j]);
        }
    }
    printf("%s", palabra[0]);
    return 0;
}
1
  • No necesitas el bucle interno del contador j; tienes que indicar a scanf que lees una cadena, en este caso debes utilizar %s (ya que %c es para un unico carácter)
    – Jakala
    Commented el 13 oct. 2019 a las 21:27

2 respuestas 2

2

Cuando en usas formaciones1 de tamaño dinámico (tamaño que se conoce durante la ejecución del programa) debes usar malloc para pedir memoria y free para liberarla:

int cp = 0;
printf("indique cantidad de palabras: ");
scanf("%d", &cp);

char **palabra = (char**)malloc(sizeof(char**) * cp);
//                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- Pedir memoria para 'cp' cadenas

Dado que vamos a crear una formación1 de cadenas, necesitaremos un puntero a puntero. Posteriormente pediremos cada palabra por separado, no es necesario un bucle anidado:

for (int i = 0; i < cp; ++i)
{
    palabra[i] = (char*)malloc(50);
//                      ~~~~~~~~~~ <--- Pedir memoria para una cadena de 50 caracteres
    printf("\nindique una palabra:");
    scanf("%s", palabra[i]);
}

El usar una variable para declarar una formación1 se conoce como Formación de Tamaño Variable (Variable Lenght Array en inglés) y no es código estándar por lo que los compiladores no están obligados a darle soporte a esta característica. El código no estándar puede no funcionar igual en todos los compiladores puede no funcionar en algunos compiladores y puede dejar de funcionar en el futuro en aquellos compiladores en que esté funcionando en el presente. En resumen: NO USES Formaciones de Tamaño Variable.

Otras cosas a tener en cuenta.

  1. En las funciones que no deben recibir argumentos deben definirse con void en la lista de parámetros, de lo contrario se considera que reciben una cantidad desconocida de argumentos, así que en lugar de int main() debes escribir int main(void).
  2. Procura favorecer el preincremento frente al postincremento, lee este artículo para saber más.
  3. Declara las variables de indizado de bucles for en el propio bucle; se aconseja que las variables tengan el ámbito más pequeño posible pues ayuda a entender el código y puede favorecer optimizaciones del compilador.

Teniendo en cuenta los puntos anteriores, tu código podría parecerse a:

#include <stdio.h>
#include <stdlib.h>

int main()
{
    int cp = 0;
    printf("indique cantidad de palabras: ");
    scanf("%d", &cp);
    
    char **palabra = (char**)malloc(sizeof(char**) * cp);

    for (int i = 0; i < cp; ++i)
    {
        palabra[i] = (char*)malloc(50);
        printf("\nindique una palabra:");
        scanf("%s", palabra[i]);
    }

    printf("Las %d palabras introducidas son:\n", cp);

    for (int i = 0; i < cp; ++i)
    {
        printf("%s\n", palabra[i]);
        free(palabra[i]);
    }
    free(palabra);

    return 0;
}

Puedes ver el código funcionando en Try it online!.


1También conocidas como arreglos o en inglés array.

3
  • Los VLA si es estándar, ANSI C lo introdujo a partir de C99. Ahora, el problema radica que no todos los desarrolladores de compiladores están de acuerdo con esta característica, así que depende del desarrollador si lo toma en cuenta o no.
    – MrDave1999
    Commented el 14 oct. 2019 a las 14:46
  • Caray Dave, me lo has hecho buscar. En el estándar C99 (6.7.5.2 Array declarators) en el apartado 4 explica lo que es un VLA. Tienes razón: están estandarizados. Commented el 14 oct. 2019 a las 14:57
  • Tenía mis dudas sobre la validez de los VLA y me sonaba que se habían estandarizado para después des-decirse. Si consultamos el estándar C11 (6.7.6.1 punto 4) vemos que los VLA han pasado de ser estándar a ser condicionalmente soportados por el compilador: (Variable length arrays are a conditional feature that implementations need not support. Commented el 14 oct. 2019 a las 15:07
1

El problema radica aquí:

int cp = 0, i, j, ll = 50;
char palabra[cp][ll];

La variable cp vale 0 y lo estás usando como longitud en la primera dimensión de la matriz palabra.

Debes declarar el arreglo después de pedir el tamaño por teclado:

int cp = 0, i, j, ll = 50;
printf("indique cantidad de palabras: ");
scanf("%d", &cp);
char palabra[cp][ll];

De este modo, se define correctamente la cantidad de filas que tendrá la matriz.

Ahora, si lo que quieres es pedirle al usuario que ingrese caracter por caracter, deberías cambiar el mensaje \nindique una palabra: por \nindique una letra:.

También recalco que el uso de los VLA (Variable Length Arrays) podrían causar problemas a tu programa, dado que no es posible detectar si hay fallas en la asignación de memoria. Para más información: The Danger Of Variable Length Arrays In C99+.

La manera adecuada de pedir tamaños en tiempo de ejecución, es usando la función malloc.

En tu programa, podríamos crear una matriz dinámica de tipo char de esta forma:

#include <stdio.h>
#include <stdlib.h>

int main(void)
{
    int cp, i, j, ll = 50;
    char** palabra;
    printf("indique la cantidad de palabras: ");
    scanf("%d", &cp);

    //Reserva memoria para las filas (que en realidad es un arreglo de punteros).
    palabra = malloc(cp * sizeof *palabra); 
    if(palabra == NULL)
    {
        printf("Error: No se pudo reservar memoria");
        return 1;
    }
    for(i = 0; i != cp; ++i)
    {
        //Reservamos memoria para las columnas (que en realidad estamos haciendo que cada puntero del arreglo de punteros, apunte a la dirección base de un ARRAY DINÁMICO).
        palabra[i] = malloc(ll * sizeof(char));
        if(palabra[i] == NULL)
        {
            //Si hay una falla, debemos liberar la memoria de cada columna que se haya reservado
            for(j = 0; j != i; ++j)
                free(palabra[j]);
            //Liberamos todas las filas reservadas 
            free(palabra);
            return 1;
        }
    }
    for(i = 0; i < cp; i++)
    {
        for(j = 0; j < ll; j++)
        {
            printf("\nindique una letra del STRING %d:", i + 1);
            scanf("%c", palabra[i][j]);
        }
    }
    printf("%s\n", palabra[0]);

    //Liberamos la matriz dinámica
    for(i = 0; i != cp; ++i)
        free(palabra[i]);
    free(palabra);
    return 0;
}
0

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.