[C] Funkcja łącząca pojedyncze znaki z konwersją typów.

Question

#include <string.h>
#include <stdio.h>

char zlap_wiersz[80]; 
char wiersz[80];

void func_zlap_wiersz(int znak){
	
	if (znak == 13){
		wiersz = zlap_wiersz; /* zrzucenie wyniku do innej zmiennej error: c:49.10 */
		zlap_wiersz = 0; /* wyzerowanie zmiennej tak abym mógł jej znowu użyć error: c:50.15 */
	}

	if (znak != 13){
		const const char *dodaj_znak = (char)znak; /* rzutowanie liczby na typ char error: c:54.34 */
		strcat(zlap_wiersz, dodaj_znak); /* scalenie znaków */
	}
	
}

main.c:49:10: error: assignment to expression with array type
   wiersz = zlap_wiersz;
          ^
main.c:50:15: error: assignment to expression with array type
   zlap_wiersz = 0;
               ^
main.c:54:34: warning: initialization makes pointer from integer without a cast [-Wint-conversion]
   const const char *dodaj_znak = (char)znak;

Potrzebuję łapać pojedyncze znaki przychodzące z pliku i scalać je w kompletny cały wiersz, a następnie zapisać go do dynamicznej tablicy. W wejściu funkcji przychodzi mi liczba która odpowiada kodowi ASCII mojej pojedynczej litery słowa. Tą funkcję chcę wywołać w pętli w której przyjmuje po kolei wszystkie znaki z pliku, ale mam parę problemów i chciałbym prosić o pełne ich zrozumienie, a tutaj może kuleć moja znajomość wskaźników.

1. Jak zadeklarować zmienne  "zlap_wiersz" , "wiersz" tak aby były przygotowane na przyjęcie zmiennej ilości znaków?

2. Jak poprawnie dokonać rzutowania czy jest ono tutaj konieczne czy zwykle przypisanie liczby do char spowoduje jej rzutowanie na ten typ?

3. Jak poprawić moje błędy i dlaczego w ten sposób? 

Answer 1

#include <string.h>
#include <stdio.h>

/* Masz powód by te bufory były widoczne poza tą jednostką translacji? 
 * Jeśli nie to static..
 * Dzięki static "za darmo" masz także zerowanie tych tablic.
 */
static char zlap_wiersz[80];
static char wiersz[80];

void func_zlap_wiersz(int znak){

    /* Bo znak 13 to inaczej '\r'. Tak jest jaśniej.
     * Jesteś pewien że nie chodzi Ci o '\n' ?
     */
    if (znak == '\r'){
        strcat(wiersz, zlap_wiersz);
        zlap_wiersz[0] = '\0';
    } else {
        strncat(zlap_wiersz, (const char *)&znak, 1);
    }
}

int main(void) {
    /* Naiwny test... */

    /* Pierwsze użycie */
    func_zlap_wiersz((int)'A');
    func_zlap_wiersz((int)'n');
    func_zlap_wiersz((int)'n');
    func_zlap_wiersz((int)'a');
    func_zlap_wiersz((int)'\r');
    printf("%s\n", wiersz);
    wiersz[0] = '\0';

    /* Ponowne użycie... */
    func_zlap_wiersz((int)'Z');
    func_zlap_wiersz((int)'u');
    func_zlap_wiersz((int)'b');
    func_zlap_wiersz((int)'i');
    func_zlap_wiersz((int)'\r');
    printf("%s\n", wiersz);
    wiersz[0] = '\0';

    return 0;
}

Answer 2

wiersz = zlap_wiersz;

strcpy()

zlap_wiersz = 0;

zlap_wiersz[0] = 0;

 const const char *dodaj_znak = (char)znak;

strcat(zlap_wiersz, dodaj_znak);

size_t n = strlen(zlap_wiersz);
zlap_wiersz[n++] = (char)znak;
zlap_wiersz[n] = 0;

 

Comments

/* Funkcja lapiaca wiesz */
void func_zlap_wiersz(int znak){
    
	size_t n = strlen(zlap_wiersz);
        zlap_wiersz[n++] = (char)znak;
        
	if (znak == 13){
		strcpy(wiersz, zlap_wiersz);
		zlap_wiersz[n] = 0;
	}
}

Wielkie dzięki za pomoc, ale jeszcze coś jest nie tak, bo piszę jeszcze jakoś systemowi po pamięci.

"Naruszenie ochrony pamięci" Drukuje mi tylko 15/10 000 wyrazów z pliku, a kompilator nie wywala żadnego błędu.

---

Jak już modyfikujesz kod, który podałem, to postaraj się go zrozumieć:

void func_zlap_wiersz(int znak){

        if (znak == '\r'){ // IMO powinno być '\n'
                strcpy(wiersz, zlap_wiersz);
                zlap_wiersz[0] = 0;
                return;
        }
     
        size_t n = strlen(zlap_wiersz);
        zlap_wiersz[n++] = (char)znak;
        zlap_wiersz[n] = 0;     // <--- to MUSI być tutaj.    

}

 

Musisz zadbać o to, żeby n nie przekroczyło zakresu tablicy zlap_wiersz. I wyzeruj gdzieś na początku programu tablice zlap_wiersz i wiersz.

---

Dobra dzięki kod działa i działa to dla 13, bo liczba 13 w moim pliku tekstowym jest znakiem końca lini.

Zastanawiałem się już czy malloca gdzieś nie wcisnąć, ale to co dałeś wygląda bardzo logicznie. Żeby zrozumieć ten kod co podałeś na początku musiałem trochę czytać dokumentację, ale to było na pewno bardzo dobre dla mnie, jeszcze raz wielkie dzięki.