Usuwanie z vectora.

Question

Mam

std::vector<bullet> bullets;

I chcę usunąć z vectora element podany przez argument.

void tower::removeBullet(bullet * removeBullet)
{
	std::vector<bullet>::iterator it;
	it = std::find_if(bullets.begin(), bullets.end(), removeBullet);
	bullets.erase(it);
}

Narazie to wygląda tak lecz mam błąd: term does not evaluate to a function taking 1 arguments

Chcę wyszukać w vectorze ten element co jest podany w argumencie metody removeBullet i go usunąć.

Nie wiem jak to zmienić tak aby działało, help.

 

Answer 1

błąd związany jest z tym że metoda nazywa się tak samo jak zmienna, spróbuj tak:

void tower::removeBullet(bullet * rm)
{
    std::vector<bullet>::iterator it;
    it = std::find_if(bullets.begin(), bullets.end(), &rm);
    bullets.erase(it);
}

 

Comments

Faktycznie, nazwy się mieszały ale po zmianie nadal jest ten sam błąd :/

---

a mógłbyś wkleić kod pliku .h

---

Już wiem użyj funkcji std::find() nie std::find_if()

---

binary '==': no operator found which takes a left-hand operand of type 'bullet' (or there is no acceptable conversion)

Coś takiego jak zamieniam na find()

---

do klasy bullet musisz przeciążyć operator==
tak żeby można było porównywać ze sobą obiekty

 

bool operator ==( const bullet  &bull)
{
    if() //kiedy pociski mają być takie same (u ciebie będą to wszystkie zmienne klasy)
    return true;
     else
return false;
}

 

---

Dodałem
 

bool bullet::operator==(const bullet & bull)
{
    
        if (*this == bull)
            return true;
        else
            return false;    
}

Nadal ten sam błąd :/

---

Chyba, że to nie tak ma być xD

---

nie możesz tego tak zrobić, kompilator nie wie jak ma porównać twoje obiekty
musiz to zrobić np. tak:

bool bullet::operator==(const bullet & bull)
{
     
        if (*this.position == bull.position) //jeśli składowe są publiczne 
            return true;
        else
            return false;    
}

 

---

Rozumiem

---

@Munvik nie możesz zdefiniować operatora= wykorzystując operator== XD

@Michał Troche krócej

bool bullet::operator==(const bullet & bull)
{
        return position == bull.position;
}

I nie muszą być publiczne. Klasa bullet ma dostęp do składowych klasy bullet (wszystkich obiektów).

Answer 2

A czytałeś dokumentacje std::find_if? Jako ostatni argument przyjmuje wskaźnik na funkcje która zwróci true gdy dostanie element którego szukasz.

Comments

Tak, nie wiedziałem jak użyć tego :/

---

Albo możesz zrobić tak jak ci napisał Michał - zdefiniować operator== dla bullet i użyć std::find. Albo użyć stf::find_if  i skorzystać z lambdy:

std::find_if(bullets.begin(), bullets.end(), [](const bullet & b) { return b.position == { 100, 100 }; });

To tylko przykład. Poczytaj o lambdach jeśli myślisz, że to może ci się przydać.