Funkcja wczytuje dodatkowe zera nie mogę znaleźć błędu [Język C]

Question

Funkcja ma za zadanie usunąć jeden wiersz z pliku tekstowego. Zrobiłem to w taki sposób że po kolei sprawdzam czy dana linijka ma zostać usunięta, jeżeli nie to zapisuję ją do nowego pliku. Jeżeli znajdzie żądaną funkcję to po prostu jej nie zapisuje w nowym pliku. Następnie usuwam stary plik i zmieniam nazwę nowego na stary. Problemem jest to że na koniec każdej linii dopisuje mi 0. Tj.

plik stary:

• 1,but,12
• 2,kaptur,7
• 3,skarpeta,8

nowy plik po usunięciu rzeczy nr 2

• 1,but,12,0
• 3,skarpeta,8,0

                    printf("podaj numer ID towaru ktory chcesz usunac z listy ");
                    scanf("%d",&usunt);
                    bt=fopen("tow.txt","r");
                    zast=fopen("zast.txt","a");
                    for(i=0;i<ile;i++)
                    {
                        fscanf(bt,"%d,%s,%d",&id,nazwatow,&liczbaszt);
                        if(id!=usunt)
                        {
                            fprintf(zast,"%d,%s,%d\n",id,nazwatow,liczbaszt);
                        }
                    }
                    fclose (bt);
                    fclose (zast);
                        remove("tow.txt");
                        rename("zast.txt","tow.txt");
                        printf("Pomyslnie usunieto\n\n");

 

Answer 1

Sprawdź co masz w zmiennej nazwatow. Prawdopodobnie ląduje w niej wszystko od pierwszego przecinka do końca linii.
Prawdopodobnie będziesz musiał ręcznie znaleźć przecinek i podzielić napis na dwie części.

Comments

Dodatkowe informacje: http://www.cplusplus.com/reference/cstdio/fscanf/

s

String of characters

Any number of non-whitespace characters, stopping at the first whitespace character found. A terminating null character is automatically added at the end of the stored sequence.

---

rzeczywiście w zmiennej znajduje się cała reszta linii. Jak zmienić tak żeby wczytywało mi tylko do przecinku?

---

najpierw powinieneś znaleźć przecinek (strchr) i podzielić napis na dwa napisy, a następnie przekształcić drugi napis na liczbę atoi lub sscanf

---

Czy mógłbyś napisać mi gotowy kod?  Nie mogę sobie z tym poradzić mam 300 linii kodu i już po prostu nie działam. Jak zmienić

fscanf(bt,"%d,%s,%d",&id,nazwatow,&liczbaszt);

tak żeby działało to wczytywanie?

---

Mniej więcej coś takiego:

// nie mozna wczytać liczbaszt bezposrednio, 
// wczytujemy napis w formacie 'nazwa,ilosc' do zmiennej nazwatow
fscanf(bt,"%d,%s",&id,nazwatow);
// szukamy przecinka
char *przecinek = strchr(nazwatow, ',');
// zastepunemy go zerem (koniec lini)
// dzieki temu printf("%s\n", nazwatow) wyswietli tylko nazwe towartu (pomimo tego ze bufor danych jest dluzszy)
*przecinek = 0;
// przesuwamy wskaznik przecinek na nastepny znak w napisie (pierwsza cyfra)
przecinek++;
// przeksztalcamy napis zawierający liczbe na liczbe
liczbaszt = atoi(przecinek);

 

---

dzięki wielkie

Temat do zamknięcia

---

Mógłbyś mi jeszcze pomóc z jednym mam podobną funkcję tylko że więcej zmiennych

fscanf(bd,"%s,%s,%s,%s,%d",nazwafir,imie,nazwisko,miejsce,&identy);

Próbowałem to zrobić analogicznie do wcześniejszej odpowiedzi ale niestety nie działa napisałem coś takiego:
 

                  fscanf(bd,"%s",nazwafir);
                        char *przecinek = strchr(nazwafir, ',');
                        *przecinek = 0;
                        przecinek++;
                         *przecinek = strchr(imie, ',');
                        *przecinek = 0;
                        przecinek++;
                         *przecinek = strchr(nazwisko, ',');
                        *przecinek = 0;
                        przecinek++;
                         *przecinek = strchr(miejsce, ',');
                        *przecinek = 0;
                        przecinek++;
                        identy = atoi(przecinek);

Podpowiesz jak to rozpisać, tym razem prosiłbym o podpowiedź a nie gotowy kod :)

---

Najpierw przetestuj kod:
 

fscanf(bd,"%s",nazwafir);
printf("1: %s\n", nazwafir);
char *przecinek = strchr(nazwafir, ',');
*przecinek = 0;
 przecinek++;
printf("1: %s\n", przecinek);

Potem się zastanów jakie wartości mają zmienne imie, nazwisko i miejsce.
Polecam przetestowanie programu z użyciem debuggera, linijka po linijce i przyjrzenie się wartościom zmiennych.

---

Jesteś wielki :D 
Napisałem coś takiego

#include<stdio.h>
 #include<stdlib.h>
 #include<string.h>
int main()
{
FILE *bd;
FILE *bt;
FILE *zast;
char a,d,e,f,g,q,z,tow[20], znak,nazwatow[20],nazwatow2[20],nazwatow1[20],nazwafir[1000],imie[10],nazwisko[10],miejsce[20],usunfirme[30];
int ile=0, ch,i,v,usunt,id,id1,liczbaszt1,id2,liczbaszt2,liczbaszt,sprzedane,identy;
bd=fopen("dos.txt","r");
fscanf(bd,"%s",nazwafir);
printf("1: %s\n", nazwafir);
char *przecinek = strchr(nazwafir, ',');
*przecinek = 0;
 przecinek++;
 char *przecine = strchr(przecinek, ',');
 *przecine = 0;
 przecine++;
 char *przecin = strchr(przecine, ',');
 *przecin = 0;
 przecin++;
 char *przeci = strchr(przecin, ',');
 *przeci = 0;
 przeci++;
 identy = atoi(przeci);
printf("1: %s\n", przecinek);
printf("1: %s\n", przecine);
printf("1: %s\n", przecin);
printf("1: %d\n", identy);
printf("2: %s\n",nazwafir);
fclose(bd);


return 0;
}

Działa w 100% tylko za te przecinki powstawiać te moje wartości i mam to z głowy :D

---

Jesteś wielki :D 
Napisałem coś takiego

#include<stdio.h>
 #include<stdlib.h>
 #include<string.h>
int main()
{
FILE *bd;
FILE *bt;
FILE *zast;
char a,d,e,f,g,q,z,tow[20], znak,nazwatow[20],nazwatow2[20],nazwatow1[20],nazwafir[1000],imie[10],nazwisko[10],miejsce[20],usunfirme[30];
int ile=0, ch,i,v,usunt,id,id1,liczbaszt1,id2,liczbaszt2,liczbaszt,sprzedane,identy;
bd=fopen("dos.txt","r");
fscanf(bd,"%s",nazwafir);
printf("1: %s\n", nazwafir);
char *przecinek = strchr(nazwafir, ',');
*przecinek = 0;
 przecinek++;
 char *przecine = strchr(przecinek, ',');
 *przecine = 0;
 przecine++;
 char *przecin = strchr(przecine, ',');
 *przecin = 0;
 przecin++;
 char *przeci = strchr(przecin, ',');
 *przeci = 0;
 przeci++;
 identy = atoi(przeci);
printf("1: %s\n", przecinek);
printf("1: %s\n", przecine);
printf("1: %s\n", przecin);
printf("1: %d\n", identy);
printf("2: %s\n",nazwafir);
fclose(bd);


return 0;
}

Działa w 100% tylko za te przecinki powstawiać te moje wartości i mam to z głowy :D

 

EDIT

Mógłbyś zerknąć na ten kod

                    case 'u':
                        printf("podaj nazwe firm ktora chcesz usunac z listy ");
                    scanf("%s",&usunfirme);
                    bd=fopen("dos.txt","r");
                    zast=fopen("zast.txt","a");
                    for(i=0;i<ile;i++)
                    {
                        fscanf(bd,"%s",nazwafir);
                        char *przecinek = strchr(nazwafir, ',');
                        *przecinek = 0;
                        przecinek++;
                        char *przecine = strchr(przecinek, ',');
                        *przecine = 0;
                        przecine++;
                        char *przecin = strchr(przecine, ',');
                        *przecin = 0;
                        przecin++;
                        char *przeci = strchr(przecin, ',');
                        *przeci = 0;
                        przeci++;
                        identy = atoi(przeci);
                        if(strcmp(nazwafir, usunfirme)==0)
                                             {
                            printf("usunieto pomyslnie");
                        }
                        else
                        {
                            fprintf(zast,"%s,%s,%s,%s,%d\n",nazwafir,przecinek,przecine,przecin,identy);
                        }
                    }
                        fclose (bd);
                        fclose (zast);
                        remove("dos.txt");
                        rename("zast.txt","dos.txt");
                        printf("Pomyslnie usunieto\n\n");
                        break;

Coś mi tutaj nie działa. W sensie nie ta część z wczytywaniem tylko bardziej z pętlą lub po, do pętli mi wchodzi ale zaraz po wyświetleniu komunikatu usunięto pomyślnie program przestaje działać.

---

Dodaj dodatkowe printfy zeby znalezc w ktorym miejscu program pada.
printf("%d\n", __LINE__)
Dodatkowo możesz wyświetlać numer iteracji na początku pętli
printf("iteracja: %d\n", i)

---

znalazłem błąd w pętli for. Wczytywałem ile zamiast ileb obie są zadeklarowane ale do różnych rzeczy.

---

Tak na przyszłość: do wczytywania linii z pliku powinieneś używać fgets a nie fscanf.
Funkcja fscanf wczyta ci tylko część linii jeżeli w gdzieś w środku linii (np. w nazwie firmy, lub po przecinku) znajdzie się spacja.