[xRandomGuy]

Hey, I'm quite new to pawn so please don't laugh.
I'm developing a training gamemode for a roleplay server and I'm trying to stock the names of the faction's ranks.
In this case the police department, and I'm having some problems with it.
I want it so when I type "rankname[playerid]" it will display the rank's name instead of the number.

PHP код:

stock rankname(playerid)
{
    switch(PlayerInfo[playerid][pRank])
    {
        case 0: rankname(playerid) = "None";
        case 1: rankname(playerid) = "Recruit";
        case 2: rankname(playerid) = "Police Officer I";
        case 3: rankname(playerid) = "Police Officer II";
        case 4: rankname(playerid) = "Lead Officer";
        case 5: rankname(playerid) = "Senior Lead Officer";
        case 6: rankname(playerid) = "Sergeant";
        case 7: rankname(playerid) = "Staff Sergeant";
        case 8: rankname(playerid) = "Lieutenant";
        case 9: rankname(playerid) = "Captain";
        case 10: rankname(playerid) = "Commander";
        case 11: rankname(playerid) = "Deputy Chief";
        case 12: rankname(playerid) = "Chief of Police";
    }
} 


Please tell me what's wrong with it and how can I fix this, thanks.

[LetsOWN[PL]]

Hello.

Can you show us that error, which appears?

Greetz, LetsOWN

[Wesley221]

pawn Код:

stock rankname(playerid) 
{ 
    switch(PlayerInfo[playerid][pRank]) 
    { 
        case 0: return "None"; 
        case 1: return "Recruit"; 
        case 2: rankname(playerid) = "Police Officer I"; 
        case 3: rankname(playerid) = "Police Officer II"; 
        case 4: rankname(playerid) = "Lead Officer"; 
        case 5: rankname(playerid) = "Senior Lead Officer"; 
        case 6: rankname(playerid) = "Sergeant"; 
        case 7: rankname(playerid) = "Staff Sergeant"; 
        case 8: rankname(playerid) = "Lieutenant"; 
        case 9: rankname(playerid) = "Captain"; 
        case 10: rankname(playerid) = "Commander"; 
        case 11: rankname(playerid) = "Deputy Chief"; 
        case 12: rankname(playerid) = "Chief of Police"; 
    } 
}

This should work, just change the last ones with 'return' aswell. Though, im not really sure if it works like this

[MP2]

pawn Код:

new ranknames[][] = {
"None",
"Recruit",
"Police Officer I",
"Police Officer II",
"Lead Officer",
"Senior Lead Officer",
"Sergeant",
"Staff Sergeant",
"Lieutenant",
"Captain",
"Commander",
"Deputy Chief",
"Chief of Police"
};

Then use this as a string for the rank names:

ranknames[RANK NUMBER GOES HERE]

[Ash.]

Quote:

Originally Posted by Wesley221 pawn Код: stock rankname(playerid) {     switch(PlayerInfo[playerid][pRank])     {         case 0: return "None";         case 1: return "Recruit";         case 2: rankname(playerid) = "Police Officer I";         case 3: rankname(playerid) = "Police Officer II";         case 4: rankname(playerid) = "Lead Officer";         case 5: rankname(playerid) = "Senior Lead Officer";         case 6: rankname(playerid) = "Sergeant";         case 7: rankname(playerid) = "Staff Sergeant";         case 8: rankname(playerid) = "Lieutenant";         case 9: rankname(playerid) = "Captain";         case 10: rankname(playerid) = "Commander";         case 11: rankname(playerid) = "Deputy Chief";         case 12: rankname(playerid) = "Chief of Police";     } } This should work, just change the last ones with 'return' aswell. Though, im not really sure if it works like this

As far as I know, returning strings like that in PAWN crashes the compiler. I personally do what you're (the original poster) trying to do like this:

pawn Код:

stock MyFunc()
{
    new szRet[64]; //Replace 64 with your longest return string length.
    switch(NUMBER) //Replace NUMBER with your array
    {
        case 0: strcat(szRet, "Case Zero");
        case 1: strcat(szRet, "Case One");
        //etc etc
    }
    return szRet;
}

[MP2]

But why use cases? It's such an inefficient way to do it. Look at my code.

[Ash.]

Quote:

Originally Posted by MP2 But why use cases? It's such an inefficient way to do it. Look at my code.

You posted at the same time as me, but yes this would work in a much better way - however the original poster requested a function, not an alternate way to do it.

[yanir3]

PHP код:

stock rankname(playerid)
{
new restring[128];
            switch(PlayerInfo[playerid][pRank])
            {
                        case 0: restring = "None";
                        case 1: restring = "Recruit";
                        case 2: restring = "Police Officer I";
                        case 3: restring = "Police Officer II";
                        case 4: restring = "Lead Officer";
                        case 5: restring = "Senior Lead Officer";
                        case 6: restring = "Sergeant";
                        case 7: restring = "Staff Sergeant";
                        case 8: restring = "Lieutenant";
                        case 9: restring = "Captain";
                        case 10: restring = "Commander";
                        case 11: restring = "Deputy Chief";
                        case 12: restring = "Chief of Police";
            }
return restring;
} 


[xRandomGuy]

Lock and archive.
You really confused me, but yanir3's post is working great, thank you!
[rep+]