Originally Posted by CrazY.
I like more with return values, but that's doesn't matter if you do not do wrong.
if (Recv == Sender)
if (!is_user_connected(Recv) || !is_user_connected(Sender))
if (get_user_team(Recv) != get_user_team(Sender))
I believe this is considered the best practice - check your cases early and return, makes it easy to follow the "main" path of the code and keeps the indentation low.