Kolejka - pytanie

Question

Hejka. Mam takie pytanko, czy mogłabym poprosić o sprawdzenie mojego zadanka z kolejki? Nie mam pewności niestety, a nie mam się kogo poradzić.
Treść wygląda następująco:
Napisać funkcję do zadanej struktury realizującą zadanie:
- Wyznaczyć średnią cenę towarów przechodzących przez kasę nr 5. W przypadku braku elementów, należy wyświetlić stosowny komunikat.
Struktura do zadania:
 

typedef struct
{
    int id_towaru, nr_trans, nr_kasy;
    float cena;
    struct tZakup *next;
} tZakup;

typedef tZakup *wZakup;

wZakup poczatek, koniec;

Próba napisania tego wypadła tak, ale nie wiem, czy poprawnie:
 

float Srednia(wZakup *P, tZakup x)
{
    float srednia = 0.f;
    float cena = 0;
    int i = 0;
    while(P!=NULL)
    {
        if(P->x.id_towaru > 0 && P->x.cena >0 && P->x.nr_kasy == 5)
        {
            for(int i=0; i<x.id_towaru; i++ )
            {
                i++;
            }
            P=P->next;
        }
    }
    if(P=NULL)
    {
        printf("Brak elementow!");
        return 0;
    }
    else return srednia/i;
}

Byłabym wdzięczna za pomoc. :)

Answer 1

float Srednia(tZakup *p)
{
	float cena = 0;
	int i = 0;

	if(p == NULL)
	{
		printf("Brak elementow!");
		return 0;
	}

	while(p)
	{
		if(p->nr_kasy == 5)
		{
			++i;
			cena += p->cena;
		}

		p = p->next;
	}

	return cena / i;
}

 

Comments

Bardzo dziękuję za pomoc i zrozumienie :) Teraz wszystko jest jasne. :) Szczęśliwego Nowego Roku! :)

---

Chyba lepsze było by Srednia_z_kasy(tZakup * p, unsigned numer_kasy). Nie było by "wbitego na stałe" numeru kasy w kodzie ( 5 ) .

Lepiej także sprawdzić warunek odwrotny "czy p nie jest NULL'em". Kod będzie prostszy w zrozumieniu.

A tak marudzę...