Шаблон не хочет работать.

[dimaSlon]

Я написал шаблон

template <typename T>
bool IsNotActive(T gameObject)
{
	return gameObject
}

вместо этого всего:

bool IsEnemyDestroyed(const Enemy& enemy)
{
	return !enemy.mIsActive;
}
bool IsGoldCoinDestroyed(const GoldCoin& goldCoin)
{
	return !goldCoin.mIsActive;
}


bool IsPowerUpToLevel2Destroyed(const PowerUpToLevel2& powerUpToLevel2)
{
	return !powerUpToLevel2.mIsActive;
}

bool IsPowerUpToLevel3Destroyed(const PowerUpToLevel3& powerUpToLevel3)
{
	return !powerUpToLevel3.mIsActive;
}

И так передаю шаблон

auto itEnemy = std::remove_if(mEnemies.begin(), mEnemies.end(), IsNotActive(mEnemies));

но увы не работает. как тогда правильно делать?

[dimaSlon]

Тоже самое тут:
2) Вместо двох решил шаблон сделать. но ... не работает. Я что то не так с шаблоном делаю?

bool Collide(const GameUnit& gameUnit1, const GameUnit& gameUnit2)
{
	Point topLeft1 = gameUnit1.mCenter - gameUnit1.mHalfSize;
	Point bottomRight1 = gameUnit1.mCenter + gameUnit1.mHalfSize;

	Point topLeft2 = gameUnit2.mCenter - gameUnit2.mHalfSize;
	Point bottomRight2 = gameUnit2.mCenter + gameUnit2.mHalfSize;

	return (bottomRight1.mX > topLeft2.mX) && 
		(bottomRight2.mX > topLeft1.mX) &&
		(bottomRight2.mY > topLeft1.mY) &&
		(bottomRight1.mY > topLeft2.mY);
}

bool Collide(const Trigger & trigger, const GameUnit & gameUnit)
{
	Point topLeft1 = trigger.mCenter - trigger.mHalfSize;
	Point bottomRight1 = trigger.mCenter + trigger.mHalfSize;

	Point topLeft2 = gameUnit.mCenter - gameUnit.mHalfSize;
	Point bottomRight2 = gameUnit.mCenter + gameUnit.mHalfSize;

	return (bottomRight1.mX > topLeft2.mX) &&
		(bottomRight2.mX > topLeft1.mX) &&
		(bottomRight2.mY > topLeft1.mY) &&
		(bottomRight1.mY > topLeft2.mY);
}

template <typename T1, typename T2>
bool Collide(const T1& gameObject1, const T2& gameObject2)
{
	Point topLeft1 = gameObject1.mCenter - gameObject1.mHalfSize;
	Point bottomRight1 = gameObject1.mCenter + gameObject1.mHalfSize;

	Point topLeft2 = gameObject2.mCenter - gameObject2.mHalfSize;
	Point bottomRight2 = gameObject2.mCenter + gameObject2.mHalfSize;

	return (bottomRight1.mX > topLeft2.mX) && 
		(bottomRight2.mX > topLeft1.mX) &&
		(bottomRight2.mY > topLeft1.mY) &&
		(bottomRight1.mY > topLeft2.mY);
}

[Alex11223]

Так а где ".mIsActive;" там?

И код странный какой-то. Может например лучше всем объектам наследоваться от GameObject?

Ну а в remove просто что-то странное написано в конце, шаблон не причем. Смотрите пример в документации.

[dimaSlon]

Сообщение от Alex11223

Так а где ".mIsActive;" там?

И код странный какой-то. Может например лучше всем объектам наследоваться от GameObject?

Ну а в remove просто что-то странное написано в конце, шаблон не причем. Смотрите пример в документации.

Пример наведу для огня. У меня все они наследуются от

class FireBall : public GameUnit

struct GameWorld;

class FireBall : public GameUnit
{
public:
	FireBall(State state, const Point& center, const Vector& velocity, const Size& size);
	void Update(float elapsedTime, GameWorld& gameWorld);
	
	bool mIsActive;

private:
	void TestCollisionVersusEnemy(GameWorld& gameWorld);

	bool mCollidedWithGround;
};

или от тригера

#pragma once

#include "Vector.h"

struct Trigger
{
public:
	Trigger(bool isActive, const Point& center, const Size& size);

	bool mIsActive;
	Point mCenter;
	Size mHalfSize;
};

и у них и есть эта переменая mIsActive
GameObject у меня нету. В ремув ж я должен передавать функцию шаблон
если я так передаю auto itFireBall = std::remove_if(mFireBalls.begin(), mFireBalls.end(), IsFireBallDestroyed); то все ок работает

[Alex11223]

Не видите разницу между IsFireBallDestroyed и IsNotActive(mEnemies)?

[alexzk]

template <typename T>
bool IsNotActive(const T& gameObject)
{
	return !gameObject.mIsActive;
}

т.о. можно не наследоватся, в этот шаблон подойдет любая фигня с членом-переменной mIsActive

Но я бы там еще во всех классах ее заменил на геттер типа

inline bool isActive(){return mIsActive;}

Компилятор сделает все быстро, но у нас тут свободы больше в будущем.

Да, еще я ссылку в параметре сделал, иначе вы копии плодите.

Плюс наследования с общим предком на всех в том, что все объекты (или указатели на них) можно свалить в 1 общий массив "предков". А дальше простым циклом пробегать по нему и выдавать команду типа "нарисуйся".
В вашем варианте вы логику объекта выносите за пределы объекта. Т.е. это уже у вас процедурное построение, а не ООП. В ООП вернее было бы сделать метод у предка:

bool Collide(const Parent& gameObject2) const
{
	Point topLeft1 = this->mCenter - this->mHalfSize;
	Point bottomRight1 = this->mCenter + this->mHalfSize;

	Point topLeft2 = gameObject2.mCenter - gameObject2.mHalfSize;
	Point bottomRight2 = gameObject2.mCenter + gameObject2.mHalfSize;

	return (bottomRight1.mX > topLeft2.mX) && 
		(bottomRight2.mX > topLeft1.mX) &&
		(bottomRight2.mY > topLeft1.mY) &&
		(bottomRight1.mY > topLeft2.mY);
}

[dimaSlon]

Сообщение от Alex11223

Не видите разницу между IsFireBallDestroyed и IsNotActive(mEnemies)?

IsFireBallDestroyed я без аргумента передаю. а там с аргументом.
Так как правильно делать?

[dimaSlon]

Сообщение от alexzk

template <typename T>
bool IsNotActive(const T& gameObject)
{
	return !gameObject.mIsActive;
}

т.о. можно не наследоватся, в этот шаблон подойдет любая фигня с членом-переменной mIsActive

Но я бы там еще во всех классах ее заменил на геттер типа

inline bool isActive(){return mIsActive;}

Компилятор сделает все быстро, но у нас тут свободы больше в будущем.

Да, еще я ссылку в параметре сделал, иначе вы копии плодите.

Плюс наследования с общим предком на всех в том, что все объекты (или указатели на них) можно свалить в 1 общий массив "предков". А дальше простым циклом пробегать по нему и выдавать команду типа "нарисуйся".
В вашем варианте вы логику объекта выносите за пределы объекта. Т.е. это уже у вас процедурное построение, а не ООП. В ООП вернее было бы сделать метод у предка:

bool Collide(const Parent& gameObject2) const
{
	Point topLeft1 = this->mCenter - this->mHalfSize;
	Point bottomRight1 = this->mCenter + this->mHalfSize;

	Point topLeft2 = gameObject2.mCenter - gameObject2.mHalfSize;
	Point bottomRight2 = gameObject2.mCenter + gameObject2.mHalfSize;

	return (bottomRight1.mX > topLeft2.mX) && 
		(bottomRight2.mX > topLeft1.mX) &&
		(bottomRight2.mY > topLeft1.mY) &&
		(bottomRight1.mY > topLeft2.mY);
}

template <typename T>
bool IsNotActive(const T& gameObject)
{
	return !gameObject.mIsActive;
}

auto itEnemy = std::remove_if(mEnemies.begin(), mEnemies.end(), IsNotActive(mEnemies));

Severity Code Description Project File Line Suppression State
Error C2039 'mIsActive': is not a member of 'std::vector<Enemy,std::allocator<_ Ty>>'

[p51x]

Да не суйте вы шаблону вектор. Там указывается предикат, а параметр сам подставится.

[dimaSlon]

Сообщение от p51x

Да не суйте вы шаблону вектор. Там указывается предикат, а параметр сам подставится.

я не совсем понял. я же не сую. можете выделеть что не нужно и как нужно написать,буду благодарен