[C] Tratando de crear array de strings

[huchoko]

Buenas, he estado tratando de crear un array de strings, para luego leer todas sus strings.

Código (c) [Seleccionar] Expandir


/* Reservamos 16 elementos de string */
char* file_names_in_dir[16];
for (int i = 0; i < FAT32_FILES_PER_DIRECTORY; ++i) {
        ...
if (drce[i].file_name[0] == 0x00)
break;
file_names_in_dir[i] = drce[i].file_name;
++files_in_directory;
}
...
printf("%s", file_names_in_dir[0]);
...

El tema es que funciona, pero luego de imprimir la string correctamente le sigue basura.
por ejemplo, file_names_in_dir[0] es "HELLO TXT", pero luego la string sigue, pero con basura.
Alguna idea? Gracias.

[@XSStringManolo]

Puedes publicar todo el código?
Estás usando variables como ++files_in_directory;
Qué no sé exactamente lo que hace.

Qué es la basura? Direcciones de memoria? Espacios? El caracter null terminator \o?

[K-YreX]

Ten en cuenta que no estás copiando los datos de <drce[indice].file_name> en <file_names_in_dir[indice]>... Lo que estás haciendo es que el puntero del array de punteros que has creado apunte al mismo lugar que el otro <char*>. Entonces si se modifica el valor de <drce[indice].file_name> antes de mostrar tu array, éste último mostrará el nuevo contenido que puede no ser lo que tú esperabas.

Y como bien dice @string Manolo, con ese trocillo de código no podemos ayudar mucho más.
Recordarte también que para que la cadena termine necesitas el caracter '\0' que igual no aparece y por eso la cadena no termina donde debería.

[huchoko]

Puedes publicar todo el código?
Estás usando variables como ++files_in_directory;
Qué no sé exactamente lo que hace.

Qué es la basura? Direcciones de memoria? Espacios? El caracter null terminator \o?

El files_in_directory no esta relacionado. Creo que la basura es un salto extraño de memoria, ya que de ahi me imprime una string que no tiene nada que ver. Ese string basura no se relaciona de ninguna manera con el array de string que trato de imprimir.
Ah y el código completo (comentarios en ingles, pero los traduzco):

Código (c) [Seleccionar] Expandir


#include <stdio.h>
#include "hd.h" /* No relacionado con la falla */
char* file_names_in_dir[16];
void read_directory(void)
{
/* lee el directorio */
hd_read(start_of_root, FAT32_FILES_PER_DIRECTORY * sizeof(struct DirectoryEntry), (uint8_t*)&drce[0]);

for (int i = 0; i < FAT32_FILES_PER_DIRECTORY; ++i) {
if (drce[i].file_name[0] == 0x00) {
printf("\n-- END --\n");
break;
}
/* el archivo tiene un nombre largo */
if ((drce[i].attributes & 0x0F) == 0x0F)
continue;

/* el seudoarchivo es en realidad un directorio */
if ((drce[i].attributes & 0x10) == 0x10){
printf("Directory name: %s\n", drce[i].file_name);
continue;
}

/* el archivo no existe */
if (drce[i].file_name[0] == FAT32_DELETED_FILE)
continue;
printf("File name: %s, %i bytes.\n", drce[i].file_name, drce[i].file_size);
file_names_in_dir[i] = drce[i].file_name;
++files_in_directory;
}
}

void read_bpb(uint32_t offset)
{
/* Read the BPB (BIOS parameter block) - Lee el BPB (Bloque de parámetros de la BIOS) */
hd_read(offset, FATBPB_SIZE, (uint8_t*)&bpb);

fat_start = (offset + bpb.reserved_sectors);
fat_size = bpb.sectors_per_fat;
start_of_data = fat_start + (fat_size * bpb.fats_number);
start_of_root = start_of_data + ((bpb.cluster_root - 2) * bpb.sectors_per_cluster);
root_dir_sects = ((bpb.root_dir_entries * 32) + (bpb.bytes_per_sector - 1) / bpb.bytes_per_sector);
data_sects = (bpb.reserved_sectors + bpb.fats_number * (bpb.sectors_per_fat + root_dir_sects));
cluster_count = data_sects / bpb.sectors_per_cluster;

read_directory();
printf("%s", file_names_in_dir[0]);
}


[K-YreX]

Diría que el error son los <continue>...
Piensa en una iteración <i = 0> y se cumple una condición y por tanto un <continue>. Ahora <i = 1> pero no se han ejecutado las líneas siguientes al <continue> en esa iteración por lo que no se ha ejecutado la línea 28.
Ahora no se ejecuta ningún <continue> (recordemos i = 1). Estás guardando <drce[1].file_name> pero lo estás guardando en <files_names_in_dir[1]>... Qué hay en <files_names_in_dir[0]>??? Basura.

Tienes que usar una variable auxiliar que sólo se incremente si se ejecuta la línea 28 para guardar siempre en posiciones consecutivas.

Por algo no se recomiendan usar ese tipo de sentencias <continue>, <break>, etc. Yo cambiaría eso e implementaría el mismo comportamiento con un <bool> o <int> si no quieres usar <bool>, pero vamos, evitando sentencias de salto.

[huchoko]

He tratado esto, pero sigue igual, imprime 16 veces la string correctamente y otras 16 basura.

Código (c) [Seleccionar] Expandir


int aaa = 0;
void read_directory(void)
{
/* Read the directory */
hd_read(start_of_root, FAT32_FILES_PER_DIRECTORY * sizeof(struct DirectoryEntry), (uint8_t*)&drce[0]);

for (int i = 0; i < FAT32_FILES_PER_DIRECTORY; ++i) {
if (drce[i].file_name[0] == 0x00) {
kputs("\n-- END --\n");
break;
}
/* File has a long file name */
if ((drce[i].attributes & 0x0F) == 0x0F)
continue;

/* "File" is a directory */
if ((drce[i].attributes & 0x10) == 0x10){
kputs("Directory name: %s\n", drce[i].file_name);
continue;
}

/* If the first byte of file_name is 0xE5, means that the file is deleted */
if (drce[i].file_name[0] == FAT32_DELETED_FILE)
continue;
/* This should print the names of the files that are in the root directory... */
printf("File name: %s, %i bytes.\n", drce[i].file_name, drce[i].file_size);
file_names_in_dir[aaa] = drce[aaa].file_name;
++files_in_directory;
++aaa;
}
}

O es que soy tonto o no he dormido nada que estoy bien palurdo. 

[K-YreX]

La nueva variable <aaa> (que espero que le pongas un nombre mejor ) controla sólo <files_names_in_dir>. Es decir que tendría que ser:

Código (c) [Seleccionar] Expandir


file_names_in_dir[aaa] = drce[i].file_name;
++aaa;

De todas formas, por si el error no es sólo ese y no consigues solucionarlo, para la próxima pon la salida que obtienes literalmente para que veamos mejor a qué se puede deber.
Yo quitaría los <continue> de todas formas y los <break> como ya te he comentado...

[huchoko]

La nueva variable <aaa> (que espero que le pongas un nombre mejor ) controla sólo <files_names_in_dir>. Es decir que tendría que ser:

Código (c) [Seleccionar] Expandir


file_names_in_dir[aaa] = drce[i].file_name;
++aaa;

De todas formas, por si el error no es sólo ese y no consigues solucionarlo, para la próxima pon la salida que obtienes literalmente para que veamos mejor a qué se puede deber.
Yo quitaría los <continue> de todas formas y los <break> como ya te he comentado...

El resultado es el mismo, sobre los continue después lo arreglo...
Bueno, aca la salida literal:

Código [Seleccionar] Expandir


HELLO   TXT (16 veces)
El feliz pájaro hindú comia feliz cardillo mientras la cigüeña tocaba el saxofón. (16 veces más)

Ni idea de donde salio la ultima string, talvez sucedio la gran casualidad que entre en la memoria de otro proceso que tiene ese string.

[K-YreX]

Es posible que esa segunda línea sea el contenido de alguno de los ficheros.
He de decir que no sé cómo funciona la siguiente instrucción por lo que doy por hecho que su funcionamiento es correcto:

Código (c) [Seleccionar] Expandir


hd_read(start_of_root, FAT32_FILES_PER_DIRECTORY * sizeof(struct DirectoryEntry), (uint8_t*)&drce[0]);

Tampoco sé si la salida que obtienes es a partir de los <printf()> y <kputs()> que se ven en la función o la obtienes mostrando después el contenido de <files_names_in_dir>.
Además la variable <aaa> la has declarado globalmente por lo que puede que estés modificando su valor en otro punto del programa o estés llamando más de una vez a esa función.
Y si de paso puedo conocer la estructura de directorios y ficheros con los que estás trabajando, es decir, su nombre. Y que compruebes que esa segunda salida no sea el contenido de alguno de los ficheros.

Lo siento pero es que hay demasiadas cosas que me estoy teniendo que imaginar...

[huchoko]

Igualmente gracias. Es raro por que antes si funcionaba sin problemas, y se rompio todo de la nada.
Saludos