Zapis do pliku z pętli :: 4programmers.net

Patryk Wielgosz

2020-12-02 16:46

Patryk Wielgosz2020-12-02 16:46

Rejestracja:ponad 6 lat
Ostatnio:ponad 3 lata
Postów:68

Napisałem kod do poniższego zadania, wiem , że problem pewnie jest w pętli while, nie wiem tylko jak go poprawić

"Napisz program umożliwiający zapis do pliku w pętli łańcucha znaków
wczytanego ze standardowego wejścia. Wprowadzenie pustego tekstu będzie
oznaczać koniec wykonywania się pętli"

Kopiuj

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

int main()
{
    char lancuch[100];
    int i;


    FILE *zapis;
    zapis=fopen("tekst.txt","w");

    if (zapis==NULL)
    {
        printf("Błąd pliku\n");
        exit(1);

    }

    while((gets(lancuch))!="/0")
    {
        fputs(lancuch,zapis);

    }
    return 0;

}

AnyKtokolwiek

2020-12-02 17:13

AnyKtokolwiek2020-12-02 17:13

Rejestracja:prawie 7 lat
Ostatnio:około miesiąc
Postów:3561

Wg manulu gets na końcu pliku zwraca nie pusty string (jak, zresztą źle - z dwóch powodów - sprawdzasz) ale NULL
https://pl.wikibooks.org/wiki/C/gets

generalnie funkcja ma "złą sławę" przez to, że pozwala przejechać poza bufor.

edytowany 1x, ostatnio: AnyKtokolwiek 2020-12-02 17:13

Patryk Wielgosz

2020-12-02 17:18

Patryk Wielgosz2020-12-02 17:18

Rejestracja:ponad 6 lat
Ostatnio:ponad 3 lata
Postów:68

@AnyKtokolwiek:
Jak mogę to poprawić ?

AnyKtokolwiek

2020-12-02 17:23

AnyKtokolwiek2020-12-02 17:23

Rejestracja:prawie 7 lat
Ostatnio:około miesiąc
Postów:3561

Ja chyba źle odczytałem zadanie, nie do EOF a do pustej linii
najczytelniej, bez żadnej "hackerskości" (bo można kombinowac, ale po co)

Kopiuj

while((gets(lancuch)) != NULL )   // mój warunek na EOF
    {
        if(strlen(lancuch)==0)             // tak jak rozumiuem zadanie po drugim czytaniu
                break;
        fputs(lancuch,zapis);
    }

fclose(zapis);   // lepiej będzie z tym, niż bez tego