[C++] Funkcja zwracajaca obiekt struktury

[mickolo]

napisac bezparametrową funkcję o nazwie losuj zwracającą obiekt typu elipsa wypełniony losowymi
wartościami. Pola \(\displaystyle{ x}\) i \(\displaystyle{ y}\) mają zawierać się w przedziale \(\displaystyle{ [-10.0; 10.0]}\) . Pola \(\displaystyle{ a}\) i \(\displaystyle{ b}\) mają zawierać się w przedziale \(\displaystyle{ [0; 5.]}\).

mnie udało sie stworzyc cos takiego:

Kod: Zaznacz cały

elipsa fun()
{
       
       elipsa o;
       o.x=rand()%(10 +10+1)-10;
       o.y=rand()%(10 +10+1)-10;
       o.a=rand()%(5-0+1)+0;
       o.b=rand()%(5-0+1)+0;
       return o;
}

lecz po wyswietleniu np cout<< o.x pojawiaja sie jakies smieci.

Z góry dzieki z apomoc

[Jacek_Karwatka]

nie bardzo rozumiem dlaczego w Twojej funkcji jest:

o.x=rand()%(10 +10+1)...
moim zdaniem powinno być:

o.x=rand()%(10 +10)

dobrym sposobem na rozwiązanie tego typu problemu jest sprawdzanie i wypisywane krok po kroku co liczy komputer.
np:

Kod: Zaznacz cały

cout<<o.x; // jeśli są śmieci to jakie?
o.x=rand();
cout<<o.x;
cout<<(o.x%20);
cout<<(o.x%20-10);

itp.

oczywiście można skorzystać z debuggera

[KaBaSZo]

Randy są ok, kod wydaje się być ok. Tworzysz obiekt lokalny, ale zwracasz go przez wartość, czyli powinna być zwrócona kopia tego obiektu.

Może pokaż jak wygląda klasa/struktura elipsa(masz tam jakieś konstruktory kopiujące, operatory przypisania?) oraz jak wywołujesz tą funkcję.

[mickolo]

ok także cała treść zadania wygląda następująco:

Napisać program zawierający:
– strukturę elipsa zawierającą cztery pola typu double: x, y, a, b; Liczby x, y będziemy
interpretować jak położenie środka elipsy w układzie współrzędnych, natomiast liczby a, b to
wielkość jej półosi,
– bezparametrową funkcję o nazwie losuj zwracającą obiekt typu elipsa wypełniony losowymi
wartościami. Pola x i y mają zawierać się w przedziale <-10.0; 10.0>. Pola a i b mają zawierać się w przedziale <0; 5.00>


a oto mój cały kod:

Kod: Zaznacz cały

#include <iostream>
#include <cstdlib>
#include <ctime>
#include <cmath>
using namespace std;

struct elipsa{
       double x;
       double y;
       double a;
       double b;
};

elipsa fun()
{
       
       elipsa o;
       o.x=rand()%(10 +10+1)-10;
       o.y=rand()%(10 +10+1)-10;
       o.a=rand()%(5-0+1)+0;
       o.b=rand()%(5-0+1)+0;
       return o;
}

int main()
{
    elipsa o;
    srand(time(0));
    fun();

cout<<o.a;
    system("pause");
    return 0;
}

i np po wyswietleniu o.a wynik to: 3.47312e-307 co jest przecież sprzeczne z założeniami programu
gdzies kryje się jakiś mały głupi błąd

[Afish]

Kod: Zaznacz cały

elipsa o;
srand(time(0));
fun();

Nigdzie nie przypisujesz wartości zwróconej przez funkcję fun, więc dane w zmiennej o są śmieciowe.

[mickolo]

czyli jak to powinno wyglądać wg Ciebie?

[wawek91]

Czyli powinno być

Kod: Zaznacz cały

elipsa o = fun();

[mickolo]

rzeczywiscie... ok dzięki wielkie chłopaki za pomoc, wszystko juz gra