Program kompiluje sie poprawnie jednak podaje zly wynik

Question

Witam, jak w tytule, program sie kompiluje bez problemu jednak podaje zly wynik, nigdzie nie moge znalezc bledu :(

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

typedef struct {
int dzieci;
int mezczyzni;
int kobiety;
}LUDZIE;

int liczba_dzieci(LUDZIE osoby);
int liczba_kobiet(LUDZIE osoby);
int liczba_mezczyzn(LUDZIE osoby);

void main(void)
{
    LUDZIE osoby;

    osoby.dzieci;
    osoby.mezczyzni;
    osoby.kobiety;
    liczba_dzieci(osoby);
    liczba_mezczyzn(osoby);
    liczba_kobiet(osoby);
    printf("Liczba osob to: %d\n",osoby.dzieci+osoby.mezczyzni+osoby.kobiety);
}
int liczba_dzieci(LUDZIE osoby)
{
    printf("Prosze podac liczbe dzieci:");
    scanf("%d",&osoby.dzieci);
    return osoby.dzieci;
}
int liczba_mezczyzn(LUDZIE osoby)
{
    printf("Prosze podac liczbe mezczyzn:");
    scanf("%d",&osoby.mezczyzni);
    return osoby.mezczyzni;
}
int liczba_kobiet(LUDZIE osoby)
{
    printf("Prosze podac liczbe kobiet:");
    scanf("%d",&osoby.kobiety);
    return osoby.kobiety;
}

 

Answer 1

Żeby przy pomocy funkcji modyfikować zewnętrzny obiekt, musisz jako parametr podać ten obiekt przez wskaźnik. Poza tym linie 18-20 nic pożytecznego nie robią.

Comments

Czyli w nawiasach funkcji mam wpisac np: (LUDZIE *osoby)??

Ok, w takim razie usune linijki 18-20

---

Tak. A przy wywołaniu dopisz do argumentu operator &, żeby otrzymać adres obiektu i powinno śmigać.

---

No i oczywiście należy zmienić wtedy operator . na -> w definicjach Twoich funkcji. ;)

---

Prototyp wyglada tak: int liczba_dzieci(LUDZIE *osoby);

Wywolanie funkcji w main: liczba_dzieci(&osoby);

Wyskakuje blad przy pierwszym scanf'ie... Error: request for member 'dzieci' in something not a structure or union...

---

scanf("%d",&osoby -> dzieci);

 

---

Zamienilem, ale nadal mam ten sam blad i sie nie kompiluje. Wstawic caly program jak go poprawilem?

---

Pewnie, zobaczymy co tam jest grane.

---

Btw dzieki wielkie za pomoc, sam bym to ogarnal chyba za rok :p 

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

typedef struct {
int dzieci;
int mezczyzni;
int kobiety;
}LUDZIE;

int liczba_dzieci(LUDZIE *osoby);
int liczba_kobiet(LUDZIE *osoby);
int liczba_mezczyzn(LUDZIE *osoby);

void main(void)
{
    LUDZIE osoby;
    liczba_dzieci(&osoby);
    liczba_mezczyzn(&osoby);
    liczba_kobiet(&osoby);
    printf("Liczba osob to: %d\n",osoby.dzieci+osoby.mezczyzni+osoby.kobiety);
}
int liczba_dzieci(LUDZIE *osoby)
{
    printf("Prosze podac liczbe dzieci:");
    scanf("%d",&osoby -> dzieci);
    return osoby.dzieci;
}
int liczba_mezczyzn(LUDZIE *osoby)
{
    printf("Prosze podac liczbe mezczyzn:");
    scanf("%d",&osoby -> mezczyzni);
    return osoby.mezczyzni;
}
int liczba_kobiet(LUDZIE *osoby)
{
    printf("Prosze podac liczbe kobiet:");
    scanf("%d",&osoby -> kobiety);
    return osoby.kobiety;
}

 

---

To powinno wyglądać jakoś tak:

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

typedef struct {
int dzieci;
int mezczyzni;
int kobiety;
}LUDZIE;

int liczba_dzieci(LUDZIE *osoby);
int liczba_kobiet(LUDZIE *osoby);
int liczba_mezczyzn(LUDZIE *osoby);

int main(void)
{
    LUDZIE osoby;
    liczba_dzieci(&osoby);
    liczba_mezczyzn(&osoby);
    liczba_kobiet(&osoby);
    printf("Liczba osob to: %d\n",osoby.dzieci+osoby.mezczyzni+osoby.kobiety);
    return 0;
}
int liczba_dzieci(LUDZIE *osoby)
{
    printf("Prosze podac liczbe dzieci:");
    scanf("%d",&osoby -> dzieci);
    return osoby -> dzieci;
}
int liczba_mezczyzn(LUDZIE *osoby)
{
    printf("Prosze podac liczbe mezczyzn:");
    scanf("%d",&osoby -> mezczyzni);
    return osoby -> mezczyzni;
}
int liczba_kobiet(LUDZIE *osoby)
{
    printf("Prosze podac liczbe kobiet:");
    scanf("%d",&osoby -> kobiety);
    return osoby -> kobiety;
}

Ja spadam teraz na zajęcia, ewentualne poprawki dopiero po 18.00. :P

---

Ok i tak dzieki wielkie!

---

Wszystko smiga :D

---

Cieszy mnie to. Różnica między moją a Twoją wersją to zmiana operator wyłuskania, czyli kropki, na operator dereferencji-wyłuskania (czy jak ludzie to jeszcze zechcą nazwać), czyli "strzałkę". Od strony programisty - używa się ich tak samo, tylko wersja strzałkowa służy do pracy ze wskaźnikami na obiekty.