Lista - sposób zapisu kodu

Question

Hejka. Chciałabym się Was zapytać, czy prawidłowo rozwiązałam to zadanie z listy, o treści:
Wyznaczenie liczby elementów o nazwie "mleko"
i o strukturze:
 

typedef struct
{
	char nazwa[50];
	int ilosc;
	float cena_jedn;
	struct tTowar *next;
}tTowar;
typedef tTowar *wTowar;

wTowar poczatek, koniec;

Próba napisania wypadła tak:
 

int Wyszukaj(tTowar *P)
{
	int suma = 0;
	while(P!=NULL && P->nazwa!="mleko")
	{
		P=P->next;
	}
	while(P!=NULL && P->nazwa == "mleko")
	{
		if(strcmp(nazwa, "mleko")==0)
		{
			suma += P->nazwa;
			P=P->next;
		}
	}
	return suma;
}

Czy taki zapis jest poprawny?
I takie pytanko dodatkowe, w niektórych strukturach danych stosuje się w funkcji typ argumentu (tTowar *P) albo (wTowar P). Czym się to różni i czy któreś z nich jest błędem? Byłabym wdzięczna za sprawdzenie rozwiązania i odpowiedź :)

Answer 1

Jakoś bez sensu to robisz. W pierwszej pętli idziesz do momentu aż napotkasz mleko, w drugiej dopóki są mleka pod rząd to dodajesz do sumy ich nazwę xd, już pomijając to że 2 razy sprawdzasz czy nazwa to mleko.

int count = 0;
while(p!=NULL){
  if(p->nazwa == "mleko")
     count+=p->ilosc;
  p=p->next;
}

Chyba bardziej chciałaś mieć coś takiego.

 

Edit: A odnośnie tego wTowar i *tTowar, to jest to chyba to samo, bo robisz typedefa:

typedef tTowar *wTowar;

Comments

i jeszcze chyba ten wskaźnik na kolejny element musisz w konstruktorze ustawić na null

i nazwa zmień na string żeby jakoś normalnie dało się to porównywać

---

Hmm... pewnie tak, tylko... zastanawiałam się, czy przy liście jest ten wyjątek, że robi się to trochę inaczej, nie ukrywam, że kodowanie jest dla mnie jak jazda samochodem we mgle bez prawa jazdy, a po prostu to prawdopodobnie ostatnie zapytanie dotyczące tego języka. Tak czy inaczej bardzo dziękuję za upewnienie :) Nie wiedziałam, że można to w taki krótki sposób zapisać, w końcu najprostsze rozwiązania czasem są najtrudniejsze :P

Answer 2

Nie mam pojęcia po co jest ta pętla:

    while(P!=NULL && P->nazwa!="mleko")
    {
        P=P->next;
    }

 Btw. porównanie `P->nazwa!="mleko"` porównuje wskaźniki (adresy), ciągi znaków w C porównuje się za pomocą strcmp.

W drugiej pętli warunek `P->nazwa == "mleko"` jest zarówno niepoprawny (wyjaśnione powyżej) jak i niepotrzebny (tutaj nie wiem co ci powiedzieć, bo serio - nie wiem po co).

suma += P->nazwa;

Pomijając, że to się w ogóle nie skompiluje: wystarczy `++suma;`. suma ma zawierać liczbe elementów o nazwie mleko, tak?

To 

P=P->next;

Nie powinno być wewnątrz ifa. Chcemy przejść do następnego elementu zawsze. Obecnie masz możliwość wpadnięcia w nieskończoną pętlę.

 w niektórych strukturach danych stosuje się w funkcji typ argumentu (tTowar *P) albo (wTowar P). Czym się to różni i czy któreś z nich jest błędem?

wTowar to typedef (alias dla nazwy typu) oznaczający dokładnie tTowar*. Więc nie - niczym się nie różni i żadno nie jest błędne.