Posts: 6,129
Threads: 36
Joined: Jan 2009
Quote:
|
Originally Posted by Eric_
Quote:
|
Originally Posted by Don Correlli
Erm, not bad, but you should really look into this:
pawn Код:
if(listitem == 1)
{
SendClientMessage(playerid, 0xFFFFFFFF, "Your Character is Muslim");
OnReligonGiven(playerid, "Muslim");
}
if(listitem == 2)
{
SendClientMessage(playerid, 0xFFFFFFFF, "Your Character is Christian");
OnReligonGiven(playerid, "Christian");
}
if(listitem == 3)
{
SendClientMessage(playerid, 0xFFFFFFFF, "Your Character is Catholic");
OnReligonGiven(playerid, "Catholic");
}
if(listitem == 4)
{
SendClientMessage(playerid, 0xFFFFFFFF, "Your Character is Islam");
OnReligonGiven(playerid, "Islam");
}
It doesn't make sense for me. |
How doesn't it make sense ? |
1) You're not even using a define for colours
2) You're using if statements instead of switch statements for something that has more than 1 parameter
3) You've created a callback for religions... which will only be called, what, once to set a religion?
