[Jstylezzz]

Hello everybody!

I have a piece of code, payday code, and the problem is that it doesn't work, but doesn't give any errors.

It worked before, when i had 3 factions, but now i have much more, and nothing happens.

Code:

if (PlayerOrg[playerid] ==  0)
	{
		GivePlayerMoney(playerid,1200);
		SendClientMessage(playerid,COLOR_WHITE,"You've received 1200$ from your unsuitable employment insurance.");
        
	}
	else if (PlayerOrg[playerid] ==  2 && GetPVarInt( playerid, "Rank" ) == 1  )
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,3200);
		

	}
	else if (PlayerOrg[playerid] ==  2 && GetPVarInt( playerid, "Rank" ) == 2)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,3500);
		
	}
	else if (PlayerOrg[playerid] ==  2 && GetPVarInt( playerid, "Rank" ) == 3)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4000);
	
	}
	else if (PlayerOrg[playerid] ==  2 && GetPVarInt( playerid, "Rank" ) == 4)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4600);
        format(string3, sizeof(string3), "Paycheck: 4600$");
	}
	else if (PlayerOrg[playerid] ==  2 && GetPVarInt( playerid, "Rank" ) == 5)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,5000);
        
	}
	else if (PlayerOrg[playerid] ==  2 && GetPVarInt( playerid, "Rank" ) == 6)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,9500);
	
	}
	else if (PlayerOrg[playerid] ==  3 && GetPVarInt( playerid, "Rank" ) == 1)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,3200);
	
	}
	else if (PlayerOrg[playerid] ==  3 && GetPVarInt( playerid, "Rank" ) == 2)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,3800);
	
	}
	else if (PlayerOrg[playerid] ==  3 && GetPVarInt( playerid, "Rank" ) == 3)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4300);
	
	}
	else if (PlayerOrg[playerid] ==  4 && GetPVarInt( playerid, "Rank" ) == 1  )
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,3200);
	
	}
	else if (PlayerOrg[playerid] ==  4 && GetPVarInt( playerid, "Rank" ) == 2)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,3500);
		
	}
	else if (PlayerOrg[playerid] ==  4 && GetPVarInt( playerid, "Rank" ) == 3)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4000);
	
	}
	else if (PlayerOrg[playerid] ==  4 && GetPVarInt( playerid, "Rank" ) == 4)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4600);
	
	}
	else if (PlayerOrg[playerid] ==  4 && GetPVarInt( playerid, "Rank" ) == 5)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,5000);
		
	}
	else if (PlayerOrg[playerid] ==  4 && GetPVarInt( playerid, "Rank" ) == 6)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,9500);
	
	}
	else if (PlayerOrg[playerid] ==  5 && GetPVarInt( playerid, "Rank" ) == 1  )
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4000);
	
	}
	else if (PlayerOrg[playerid] ==  5 && GetPVarInt( playerid, "Rank" ) == 2)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,5000);
		
	}
	else if (PlayerOrg[playerid] ==  5 && GetPVarInt( playerid, "Rank" ) == 3)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,6000);
	
	}
	else if (PlayerOrg[playerid] ==  5 && GetPVarInt( playerid, "Rank" ) == 4)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,7000);
	
	}
	else if (PlayerOrg[playerid] ==  5 && GetPVarInt( playerid, "Rank" ) == 5)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,8000);
	
	}
	else if (PlayerOrg[playerid] ==  5 && GetPVarInt( playerid, "Rank" ) == 6)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,20000);
	
	}
	else if (PlayerOrg[playerid] ==  6 && GetPVarInt( playerid, "Rank" ) == 1  )
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,3000);
		
	}
	else if (PlayerOrg[playerid] ==  6 && GetPVarInt( playerid, "Rank" ) == 2)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,3500);
		
	}
	else if (PlayerOrg[playerid] ==  6 && GetPVarInt( playerid, "Rank" ) == 3)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4000);
	
	}
	else if (PlayerOrg[playerid] ==  6 && GetPVarInt( playerid, "Rank" ) == 4)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4500);
		
	}
	else if (PlayerOrg[playerid] ==  6 && GetPVarInt( playerid, "Rank" ) == 5)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,5000);
		
	}
	else if (PlayerOrg[playerid] ==  6 && GetPVarInt( playerid, "Rank" ) == 6)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,7000);
	
	}
   	else if (PlayerOrg[playerid] ==  7 && GetPVarInt( playerid, "Rank" ) == 1  )
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,3000);
		
	}
	else if (PlayerOrg[playerid] ==  7 && GetPVarInt( playerid, "Rank" ) == 2)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,3500);
		
	}
	else if (PlayerOrg[playerid] ==  7 && GetPVarInt( playerid, "Rank" ) == 3)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4000);
	
	}
	else if (PlayerOrg[playerid] ==  7 && GetPVarInt( playerid, "Rank" ) == 4)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4500);
		
	}
	else if (PlayerOrg[playerid] ==  7 && GetPVarInt( playerid, "Rank" ) == 5)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,5000);
		
	}
	else if (PlayerOrg[playerid] ==  7 && GetPVarInt( playerid, "Rank" ) == 6)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,7000);
	
	}
	else if (PlayerOrg[playerid] ==  8 && GetPVarInt( playerid, "Rank" ) == 1  )
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,100);
		
	}
	else if (PlayerOrg[playerid] ==  8 && GetPVarInt( playerid, "Rank" ) == 2)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,200);
		
	}
	else if (PlayerOrg[playerid] ==  8 && GetPVarInt( playerid, "Rank" ) == 3)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,500);
		
	}
	else if (PlayerOrg[playerid] ==  8 && GetPVarInt( playerid, "Rank" ) == 4)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,700);
		
	}
	else if (PlayerOrg[playerid] ==  8 && GetPVarInt( playerid, "Rank" ) == 5)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,1000);
		
	}
	else if (PlayerOrg[playerid] ==  8 && GetPVarInt( playerid, "Rank" ) == 6)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4500);
		
	}
	else if (PlayerOrg[playerid] ==  9 && GetPVarInt( playerid, "Rank" ) == 1  )
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,100);
		
	}
	else if (PlayerOrg[playerid] ==  9 && GetPVarInt( playerid, "Rank" ) == 2)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,3333);
		
	}
	else if (PlayerOrg[playerid] ==  9 && GetPVarInt( playerid, "Rank" ) == 3)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4000);
	
	}
	else if (PlayerOrg[playerid] ==  9 && GetPVarInt( playerid, "Rank" ) == 4)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4500);
		
	}
	else if (PlayerOrg[playerid] ==  9 && GetPVarInt( playerid, "Rank" ) == 5)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,5000);
		
	}
	else if (PlayerOrg[playerid] ==  9 && GetPVarInt( playerid, "Rank" ) == 6)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,40000);
		
	}
	else if (PlayerOrg[playerid] ==  10 && GetPVarInt( playerid, "Rank" ) == 1  )
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,100);
		
	}
	else if (PlayerOrg[playerid] ==  10 && GetPVarInt( playerid, "Rank" ) == 2)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,200);
		
	}
	else if (PlayerOrg[playerid] ==  10 && GetPVarInt( playerid, "Rank" ) == 3)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,500);
		
	}
	else if (PlayerOrg[playerid] ==  10 && GetPVarInt( playerid, "Rank" ) == 4)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,700);
		
	}
	else if (PlayerOrg[playerid] ==  10 && GetPVarInt( playerid, "Rank" ) == 5)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,1000);
		
	}
	else if (PlayerOrg[playerid] ==  10 && GetPVarInt( playerid, "Rank" ) == 6)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4500);
		
	}
	else if (PlayerOrg[playerid] ==  11 && GetPVarInt( playerid, "Rank" ) == 1  )
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,100);
		
	}
	else if (PlayerOrg[playerid] ==  11 && GetPVarInt( playerid, "Rank" ) == 2)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,200);
		
	}
	else if (PlayerOrg[playerid] ==  11 && GetPVarInt( playerid, "Rank" ) == 3)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,500);
		
	}
	else if (PlayerOrg[playerid] ==  11 && GetPVarInt( playerid, "Rank" ) == 4)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,700);
		
	}
	else if (PlayerOrg[playerid] ==  11 && GetPVarInt( playerid, "Rank" ) == 5)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,1000);
		
	}
	else if (PlayerOrg[playerid] ==  11 && GetPVarInt( playerid, "Rank" ) == 6)
	{
		SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
		GivePlayerMoney(playerid,4500);
		
	}

Anyone any ideas?

[ViniBorn]

It's problem with ' else if '

[Jstylezzz]

Do i need to change it to only "if" ?

[Sphex]

Make it a switch.

[Jstylezzz]

Quote:

Originally Posted by Sphex Make it a switch.

Could you show me an example please? I'm not very good with switches

[iggy1]

This is how i'd do it.

pawn Code:

switch( GetPVarInt( playerid, "Rank" ) )
{
    case 1://rank 1
    {
        switch( PlayerOrg[playerid] )
        {
            case 2:// organisation id 2
            {
                SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");
                GivePlayerMoney(playerid,3200);
            }
        }
    }
        
    case 2://rank 2
    {
        switch( PlayerOrg[playerid] )
        {
            case 2:
            {
                //...
            }
        }
    }
}

[Jstylezzz]

Quote:

Originally Posted by iggy1 This is how i'd do it. pawn Code: switch( GetPVarInt( playerid, "Rank" ) )     {         case 1://rank 1         {             switch( PlayerOrg[playerid] )             {                 case 2:// organisation id 2                 {                     SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");                     GivePlayerMoney(playerid,3200);                 }             }         }                 case 2://rank 2         {                     switch( PlayerOrg[playerid] )             {                             case 2:                 {                     //...                 }                         }                 }     } EDIT: I don't know why indentation messed up its fine in the text editor.

Thanks, I will try this out

[iggy1]

I only started cases from 2 because that's what your code appears to do. You might want to add a default case too for safety.

EG,

pawn Code:

switch( GetPVarInt( playerid, "Rank" ) )
{
    case 1:
    {
    }
        
    case 2:
    {
    }
    
    default: //this will be executed if none of the cases match
    {
    }
}

[ViniBorn]

pawn Code:

switch(PlayerOrg[playerid])
{
    case 0:
    {
        GivePlayerMoney(playerid,1200);
        return SendClientMessage(playerid,COLOR_WHITE,"You've received 1200$ from your unsuitable employment insurance.");
    }
    case 2:
    {
        switch(GetPVarInt( playerid, "Rank" )
        {
            case 1: GivePlayerMoney(playerid,3200);
            case 2: GivePlayerMoney(playerid,3500);
            case 3: GivePlayerMoney(playerid,4000);
            case 4: GivePlayerMoney(playerid,4600);
            case 5: GivePlayerMoney(playerid,5000);
            case 6: GivePlayerMoney(playerid,9500);
        }
    }
    case 3:
    {
        switch(GetPVarInt( playerid, "Rank" )
        {
            case 1: GivePlayerMoney(playerid,3200);
            case 2: GivePlayerMoney(playerid,3800);
            case 3: GivePlayerMoney(playerid,4300);
        }
    }
    case 4:
    {
        switch(GetPVarInt( playerid, "Rank" )
        {
            case 1: GivePlayerMoney(playerid,3200);
            case 2: GivePlayerMoney(playerid,3500);
            case 3: GivePlayerMoney(playerid,4000);
            case 4: GivePlayerMoney(playerid,4600);
            case 5: GivePlayerMoney(playerid,5000);
            case 6: GivePlayerMoney(playerid,9500);
        }
    }
    case 5:
    {
        switch(GetPVarInt( playerid, "Rank" )
        {
            case 1: GivePlayerMoney(playerid,4000);
            case 2: GivePlayerMoney(playerid,5000);
            case 3: GivePlayerMoney(playerid,6000);
            case 4: GivePlayerMoney(playerid,7000);
            case 5: GivePlayerMoney(playerid,8000);
            case 6: GivePlayerMoney(playerid,20000);
        }
    }
    case 6:
    {
        switch(GetPVarInt( playerid, "Rank" )
        {
            case 1: GivePlayerMoney(playerid,3000);
            case 2: GivePlayerMoney(playerid,3500);
            case 3: GivePlayerMoney(playerid,4000);
            case 4: GivePlayerMoney(playerid,4500);
            case 5: GivePlayerMoney(playerid,5000);
            case 6: GivePlayerMoney(playerid,7000);
        }
    }
    case 7:
    {
        switch(GetPVarInt( playerid, "Rank" )
        {
            case 1: GivePlayerMoney(playerid,3000);
            case 2: GivePlayerMoney(playerid,3500);
            case 3: GivePlayerMoney(playerid,4000);
            case 4: GivePlayerMoney(playerid,4500);
            case 5: GivePlayerMoney(playerid,5000);
            case 6: GivePlayerMoney(playerid,7000);
        }
    }
    case 8:
    {
        switch(GetPVarInt( playerid, "Rank" )
        {
            case 1: GivePlayerMoney(playerid,100);
            case 2: GivePlayerMoney(playerid,200);
            case 3: GivePlayerMoney(playerid,500);
            case 4: GivePlayerMoney(playerid,700);
            case 5: GivePlayerMoney(playerid,1000);
            case 6: GivePlayerMoney(playerid,4500);
        }
    }
    case 9:
    {
        switch(GetPVarInt( playerid, "Rank" )
        {
            case 1: GivePlayerMoney(playerid,100);
            case 2: GivePlayerMoney(playerid,3333);
            case 3: GivePlayerMoney(playerid,4000);
            case 4: GivePlayerMoney(playerid,4500);
            case 5: GivePlayerMoney(playerid,5000);
            case 6: GivePlayerMoney(playerid,40000);
        }
    }
    case 10:
    {
        switch(GetPVarInt( playerid, "Rank" )
        {
            case 1: GivePlayerMoney(playerid,100);
            case 2: GivePlayerMoney(playerid,200);
            case 3: GivePlayerMoney(playerid,500);
            case 4: GivePlayerMoney(playerid,700);
            case 5: GivePlayerMoney(playerid,1000);
            case 6: GivePlayerMoney(playerid,4500);
        }
    }
    case 11:
    {
        switch(GetPVarInt( playerid, "Rank" )
        {
            case 1: GivePlayerMoney(playerid,100);
            case 2: GivePlayerMoney(playerid,200);
            case 3: GivePlayerMoney(playerid,500);
            case 4: GivePlayerMoney(playerid,700);
            case 5: GivePlayerMoney(playerid,1000);
            case 6: GivePlayerMoney(playerid,4500);
        }
    }
}
SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");

[Jstylezzz]

Quote:

Originally Posted by Viniborn pawn Code: switch(PlayerOrg[playerid]) {     case 0:     {         GivePlayerMoney(playerid,1200);         return SendClientMessage(playerid,COLOR_WHITE,"You've received 1200$ from your unsuitable employment insurance.");     }     case 2:     {         switch(GetPVarInt( playerid, "Rank" )         {             case 1: GivePlayerMoney(playerid,3200);             case 2: GivePlayerMoney(playerid,3500);             case 3: GivePlayerMoney(playerid,4000);             case 4: GivePlayerMoney(playerid,4600);             case 5: GivePlayerMoney(playerid,5000);             case 6: GivePlayerMoney(playerid,9500);         }     }     case 3:     {         switch(GetPVarInt( playerid, "Rank" )         {             case 1: GivePlayerMoney(playerid,3200);             case 2: GivePlayerMoney(playerid,3800);             case 3: GivePlayerMoney(playerid,4300);         }     }     case 4:     {         switch(GetPVarInt( playerid, "Rank" )         {             case 1: GivePlayerMoney(playerid,3200);             case 2: GivePlayerMoney(playerid,3500);             case 3: GivePlayerMoney(playerid,4000);             case 4: GivePlayerMoney(playerid,4600);             case 5: GivePlayerMoney(playerid,5000);             case 6: GivePlayerMoney(playerid,9500);         }     }     case 5:     {         switch(GetPVarInt( playerid, "Rank" )         {             case 1: GivePlayerMoney(playerid,4000);             case 2: GivePlayerMoney(playerid,5000);             case 3: GivePlayerMoney(playerid,6000);             case 4: GivePlayerMoney(playerid,7000);             case 5: GivePlayerMoney(playerid,8000);             case 6: GivePlayerMoney(playerid,20000);         }     }     case 6:     {         switch(GetPVarInt( playerid, "Rank" )         {             case 1: GivePlayerMoney(playerid,3000);             case 2: GivePlayerMoney(playerid,3500);             case 3: GivePlayerMoney(playerid,4000);             case 4: GivePlayerMoney(playerid,4500);             case 5: GivePlayerMoney(playerid,5000);             case 6: GivePlayerMoney(playerid,7000);         }     }     case 7:     {         switch(GetPVarInt( playerid, "Rank" )         {             case 1: GivePlayerMoney(playerid,3000);             case 2: GivePlayerMoney(playerid,3500);             case 3: GivePlayerMoney(playerid,4000);             case 4: GivePlayerMoney(playerid,4500);             case 5: GivePlayerMoney(playerid,5000);             case 6: GivePlayerMoney(playerid,7000);         }     }     case 8:     {         switch(GetPVarInt( playerid, "Rank" )         {             case 1: GivePlayerMoney(playerid,100);             case 2: GivePlayerMoney(playerid,200);             case 3: GivePlayerMoney(playerid,500);             case 4: GivePlayerMoney(playerid,700);             case 5: GivePlayerMoney(playerid,1000);             case 6: GivePlayerMoney(playerid,4500);         }     }     case 9:     {         switch(GetPVarInt( playerid, "Rank" )         {             case 1: GivePlayerMoney(playerid,100);             case 2: GivePlayerMoney(playerid,3333);             case 3: GivePlayerMoney(playerid,4000);             case 4: GivePlayerMoney(playerid,4500);             case 5: GivePlayerMoney(playerid,5000);             case 6: GivePlayerMoney(playerid,40000);         }     }     case 10:     {         switch(GetPVarInt( playerid, "Rank" )         {             case 1: GivePlayerMoney(playerid,100);             case 2: GivePlayerMoney(playerid,200);             case 3: GivePlayerMoney(playerid,500);             case 4: GivePlayerMoney(playerid,700);             case 5: GivePlayerMoney(playerid,1000);             case 6: GivePlayerMoney(playerid,4500);         }     }     case 11:     {         switch(GetPVarInt( playerid, "Rank" )         {             case 1: GivePlayerMoney(playerid,100);             case 2: GivePlayerMoney(playerid,200);             case 3: GivePlayerMoney(playerid,500);             case 4: GivePlayerMoney(playerid,700);             case 5: GivePlayerMoney(playerid,1000);             case 6: GivePlayerMoney(playerid,4500);         }     } } SendClientMessage(playerid,COLOR_WHITE,"You have received your paycheck!");

Ah thanks

[ViniBorn]

349 to 126 lines.

Code easier to read ....

[Jstylezzz]

hmm.. i tried this, but now i don't receive any money..

[iggy1]

Are you setting your organization to '1' somewhere? Add a default and print/send a message inside it so you know if no cases were found.

[Jstylezzz]

what do you mean?
I have something that sets the organization, like transport, delivery e.t.c....
is that the problem??

[Jstylezzz]

Anyone?
EDIT:nVm Solved