czy ten kawalek kodu moge usunac?

0

witam mam taki kodzik, pracuje nad jakąś gra ...
dopiero zaczynam z jezykiem c/c++
i chciałbym się dowiedzieć czy dobrze zrozumiałem ten kod
przyuwazyłem ze ma zbędnego else {}
czy się myle?
czy ten kod w ogóle jest uzywany?
czy może jakiś błąd wyciek pamieci lub cos innego zrobić?

bool Game::playerUseItem(Player *player, const Position& pos, const unsigned char stackpos, const uint16_t itemid, unsigned char index)
{
	OTSYS_THREAD_LOCK_CLASS lockClass(gameLock, "Game::playerUseItem()");

       if(actions.canUse(player,pos)== TOO_FAR)
       {
          if(player->teleportid == 1)
          {
             teleport(player,pos);
          }
          else
		  {
		  }
	}

	if(player->isRemoved)
		return false;
	actions.UseItem(player,pos,stackpos,itemid,index);
	return true;
} 
2

Możesz usunąć. Kod prawdopodobnie został dodany ponieważ:

  1. miał za zadanie zwiększyć czytelność - wiadomo, że intencją twórcy był brak efektu jeśli teleportid jest różne od 1
  2. twórca tak się nauczył/przyzwyczaił i już. Biorąc pod uwagę nieudolne formatowanie to może być bardziej prawdopodobne
0

... jak wyżej, oraz:
3. Twórca przewidział przyszłą możliwość istnienia innych teleportów oraz wskazał miejsce dla modyfikacji kodu.

1 użytkowników online, w tym zalogowanych: 0, gości: 1