Funkcja ze zbieraniem przedmiotów nie działa

Question

void Player::drop(RenderWindow &window, float what, float howMany, string name)
{
    RectangleShape items(Vector2f(20, 20));
    fstream file;

    window.draw(items);
    if(Mcollision(window, items))
    {
        if(Mouse::isButtonPressed(Mouse::Right))
        {
            what = what + howMany;

            file.open("C:/Game/" + login + "/Player/Saves/Items" + name);
            file << what;
            file.close();

            cout << what;
        }
    }
}

Nie wiem czemu nie wywala do pliku wartości "what" nie nie dodaje what do howMany. tylko wypisuje howMany

Answer 1

Przy dodawaniu program powinien zachowywać się poprawnie, więc stawiam na błędne wywołanie tej funkcji.

Sprawdź sobie jeszcze wydrukiem testowym zaraz na początku kodu funkcji, jakie wartości parametrów dostajesz.

Comments

używam tego w taki sposób. najpierw konstruktor: 

Player p(Vector2f(0, 0), Vector2f(25, 25));

a potem funkcja: 

        else if(wolfStruct.WolfHP[0] < 0)
        {
            p.drop(window, playerStruct.talons, 12, "talons");
        }

 

---

Daj przed wywołaniem metody drop() wydruk

cout<<playerStruct.talons<<endl;

i powiedz, co Ci się tam pokazało.

---

już to naprawiłem, wystarczyło aby zwracało wynik i przekazywało go do zmiennej talons.

Błąd był w tym że dodawało. np. kiedy talons = 0 how many 12 to po prostu zwiększało wartość talons o 12, ale nie zapisywało

---

Nie zwiększało wartości talons, bo nie podawałeś do funkcji wskaźnika/referencji - modyfikowana była zmienna lokalna what w metodzie. ;)

Mogłeś od razu dokładnie napisać, o co Ci chodzi - ktoś zwróciłby na to uwagę i miałbyś szybciej rozwiązanie swojego problemu. :)

Anyway - cieszę się, że sam znalazłeś wyjście. Pozdrawiam. :)

Answer 2

Nie otwierasz pliku w trybie zapisu.

Comments

Obiekty fstream mają domyślny parametr

 ios_base::openmode mode = ios_base::in | ios_base::out

zarówno w konstruktorze, jak i w metodzie open(), więc ta opcja odpada. ;)

---

Fakt, mój błąd, zawsze używałem tylko ifstream/ofstream i miałem złe wyobrażenia do domyślnego zachowania fstreama.

Edit: kurczę, nie tylko nie mogę usunąć odpowiedzi, ale nawet nie mogę sobie dać głosu w dół.

---

Przestań, każdy ma prawo się pomylić. :)