[valych]

PHP код:

stock GetDivisionName(playerid)
{
     new astring[24];
    if(PlayerInfo[playerid][pFaction] == 1 && PlayerInfo[playerid][pDivision] == 1)format(astring, sizeof(astring), "None");
    else if(PlayerInfo[playerid][pFaction] == 2 && PlayerInfo[playerid][pDivision] == 3) format(astring, sizeof(astring), "FTO");
    else if(PlayerInfo[playerid][pFaction] == 3 && PlayerInfo[playerid][pDivision] == 4) format(astring, sizeof(astring), "S.W.A.T.");
    else if(PlayerInfo[playerid][pFaction] == 4 && PlayerInfo[playerid][pDivision] == 5) format(astring, sizeof(astring), "D.E.B.");
    else if(PlayerInfo[playerid][pFaction] == 5 && PlayerInfo[playerid][pDivision] == 7) format(astring, sizeof(astring), "TE");
    return astring;
} 


But I didn't get what values does pDivision stores? In case if for each faction you have several divisions this stock should be written like this:

PHP код:

stock GetDivisionName(playerid)
{
     new astring[24];
    if(PlayerInfo[playerid][pFaction] == 1)
        {
             switch(PlayerInfo[playerid][pDivision])
             {
                  case 1: format(astring, sizeof(astring), "Division1");
                  case 2: format(astring, sizeof(astring), "Division2");
                  default:  format(astring, sizeof(astring), "none");
             }
        }
        if(PlayerInfo[playerid][pFaction] == 2)
        {
             switch(PlayerInfo[playerid][pDivision])
             {
                  case 1: format(astring, sizeof(astring), "Division1");
                  case 2: format(astring, sizeof(astring), "Division2");
                  default:  format(astring, sizeof(astring), "none");
             }
        }
        // And so on for each faction
        return astring;
} 


I hope you got the idea. And btw it's better not to return arrays from functions.