[ross8839]

Quote:

Originally Posted by Vince Because you have a return right there. Return stops the function immediately. Also use constant arrays to store the rank instead of repeating "set your rank to" a hundred times. If you ever wanted to change that message you'd have to change it in hundred different places which is far from ideal. Your current setup is also quite weird. Why not switch through factions first, then through ranks so you at least have them more or less together? Something like this might work. It gets rid of the switch structure entirely. Not tested. PHP код: static const ranks[][][] = {     /* 0 */ { "Cadet", "Trooper", "Lieutenant", "Deputy Chief of Police", "Chief of Police" },     /* 1 */ { "Associate", "Soldier",    "Capo",    "Underboss", "Boss"    },         /* 2 */ { "Associate", "Soldier", "Capo", "Underboss", "Boss" },         /* 3 */ { "Raw Recruit", "Street Soldier", "Gang Fighter", "Henchman", "Gang Leader" },         /* 4 */ { "Raw Recruit", "Street Soldier", "Gang Fighter", "Henchman", "Gang Leader"    }     }; format(string, sizeof(string), "%s set your rank to %s!", pName, ranks[PlayerInfo[playerid][pFac] - 1][rank - 1]);

I'm going to look in to this actually. Thank you!

+rep