Autor
|
Tema: [C] Tratando de crear array de strings (Leído 4,108 veces)
|
huchoko
|
Buenas, he estado tratando de crear un array de strings, para luego leer todas sus strings. /* 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.
|
|
|
En línea
|
|
|
|
@XSStringManolo
Hacker/Programador
Colaborador
Desconectado
Mensajes: 2.399
Turn off the red ligth
|
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?
|
|
|
En línea
|
Mi perfil de patrocinadores de GitHub está activo! Puedes patrocinarme para apoyar mi trabajo de código abierto 💖
|
|
|
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.
|
|
|
En línea
|
cout << "Todos tenemos un defecto, un error en nuestro código" << endl;
|
|
|
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): #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) { 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]); }
|
|
|
En línea
|
|
|
|
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.
|
|
|
En línea
|
cout << "Todos tenemos un defecto, un error en nuestro código" << endl;
|
|
|
huchoko
|
He tratado esto, pero sigue igual, imprime 16 veces la string correctamente y otras 16 basura. 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.
|
|
|
En línea
|
|
|
|
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: 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...
|
|
|
En línea
|
cout << "Todos tenemos un defecto, un error en nuestro código" << endl;
|
|
|
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: 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: 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.
|
|
« Última modificación: 8 Junio 2019, 23:22 pm por hextakatt »
|
En línea
|
|
|
|
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: 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...
|
|
|
En línea
|
cout << "Todos tenemos un defecto, un error en nuestro código" << endl;
|
|
|
huchoko
|
Igualmente gracias. Es raro por que antes si funcionaba sin problemas, y se rompio todo de la nada. Saludos
|
|
« Última modificación: 8 Junio 2019, 23:47 pm por hextakatt »
|
En línea
|
|
|
|
|
Mensajes similares |
|
Asunto |
Iniciado por |
Respuestas |
Vistas |
Último mensaje |
|
|
Reordernar array de strings aleatoriamente (Ayuda)
Programación Visual Basic
|
katanaia
|
2
|
1,867
|
25 Noviembre 2008, 19:27 pm
por cobein
|
|
|
Separación de caracteres en Strings y guardado en array
Java
|
Debci
|
4
|
6,065
|
18 Septiembre 2009, 06:39 am
por Skeletron
|
|
|
FindWindow con array de strings
Programación C/C++
|
while
|
4
|
2,624
|
9 Noviembre 2010, 21:56 pm
por while
|
|
|
(Ayuda) Array de strings.
Programación C/C++
|
S++
|
4
|
2,821
|
11 Junio 2015, 03:59 am
por rir3760
|
|
|
Problema con array de strings
Programación C/C++
|
JaimeGc05
|
3
|
1,692
|
20 Enero 2019, 20:44 pm
por JaimeGc05
|
|