[darkokan]

I made a dialog for hospital but nothing happens when I choose one of the selections. Thats the part of my dialog;

Код:

if(dialogid == DIALOG_HOSPITAL)
{
	    if(response)
	    {
	        if(listitem == 0)
	        {
	        
        		if(GetPlayerHealth(playerid) < 100 && GetPlayerMoney(playerid) >= 10000 && PlayerInfo[playerid][pHealthInsurance] < 0)
	        	{
	            	SetPlayerHealth(playerid,99);
	            	GivePlayerMoney(playerid, -10000);
	            	SendClientMessage(playerid, -1, "Your health increased.");
			}
        		if(GetPlayerHealth(playerid) < 100 && GetPlayerMoney(playerid) < 10000 && PlayerInfo[playerid][pHealthInsurance] < 0)
	        	{
			SendClientMessage(playerid, -1, "Not enough money.");
			}
        		if(GetPlayerHealth(playerid) < 100 && GetPlayerMoney(playerid) >= 10000 && PlayerInfo[playerid][pHealthInsurance] > 0)
	        	{
         		SetPlayerHealth(playerid, 99);
			SendClientMessage(playerid, -1, "Your expenses was covered by health insurance.");
			}
     			if(GetPlayerHealth(playerid) > 99 && GetPlayerMoney(playerid) >= 10000)
	        	{
			SendClientMessage(playerid, -1, "You don't need this.");
			 }

	        }
	    }


    return 1;
}

[ZBits]

Try

pawn Код:

switch(listitem)
{
case 0:
{

}
case 1:
{

}
case 2:
{

}
//... and so on
}

[darkokan]

Still nothing happens and it says "number of arguments does not match definition"

Код:

   	if(dialogid == DIALOG_HOSPITAL)
	{
		if(response)
		{
			switch(listitem)
			{

				case 1:
				{

        			    if(GetPlayerHealth(playerid) < 100 && GetPlayerMoney(playerid) >= 10000 && PlayerInfo[playerid][pHealthInsurance] < 0)
	        		    {
	            		          SetPlayerHealth(playerid, 99);
	            		          GivePlayerMoney(playerid, -10000);
	            		          SendClientMessage(playerid, -1, "Your health increased.");
				     }
        			     if(GetPlayerHealth(playerid) < 100 && GetPlayerMoney(playerid) < 10000 && PlayerInfo[playerid][pHealthInsurance] < 0)
	        		     {
					   SendClientMessage(playerid, -1, "Not enough money.");//Tell them.
				      }
        			      if(GetPlayerHealth(playerid) < 100 && GetPlayerMoney(playerid) >= 10000 && PlayerInfo[playerid][pHealthInsurance] > 0)
	        		       {
         				    SetPlayerHealth(playerid, 99);
					    SendClientMessage(playerid, -1, "Your expenses was covered by health insurance.");
			    	        }
     				        if(GetPlayerHealth(playerid) > 99 && GetPlayerMoney(playerid) >= 10000)
	        		        {
					    SendClientMessage(playerid, -1, "You dont need this.");
			    	         }
				         }

				case 2:
				{
 					if(GetPlayerMoney(playerid) >= 10000 && PlayerInfo[playerid][pHealthInsurance] < 0)
	        		        {
	        	    	            GivePlayerMoney(playerid, -250);
	        	    	            PlayerInfo[playerid][pCondom] += 1;
	        	    	            SendClientMessage(playerid, -1, "Bought a condom.");
					 }
				 }
				case 4:
				{
 					if(GetPlayerMoney(playerid) >= 10000 && PlayerInfo[playerid][pHealthInsurance] < 0)
	        		        {
	        	    	              GivePlayerMoney(playerid, -10000);
	        	    	              PlayerInfo[playerid][pCondom] += 5;
	        	    	              SendClientMessage(playerid, -1, "Bought 5 condoms.");
					}
				}
				case 5:
				{
 					if(GetPlayerMoney(playerid) >= 5000 && PlayerInfo[playerid][pHealthInsurance] < 0)
	        		       {
	        	    	            GivePlayerMoney(playerid, -5000);
	        	    	            PlayerInfo[playerid][pCondom] += 1;
	        	    	            SendClientMessage(playerid, -1, "Bought viagra.");	        	    return 1;
					}
				}
			}
		}
 	return 1;
	}

[ranme15]

GetPlayerHealth shouldn't be used that way. Correct way:

PHP код:

new Float: Health;
GetPlayerHealth(playerid, Health);
if(Health < 100...) 


[darkokan]

Quote:

Originally Posted by ranme15 GetPlayerHealth shouldn't be used that way. Correct way: PHP код: new Float: Health; GetPlayerHealth(playerid, Health); if(Health < 100...)

I tried it. Result is the same. The problem is not only increasing health but also buying items. for example

Код:

PlayerInfo[playerid][pCondom] += 1;

doesnt work

[Threshold]

pawn Код:

if(dialogid == DIALOG_HOSPITAL)
    {
        if(response)
        {
            if(listitem == 0)
            {
                new Float:hp;
                GetPlayerHealth(playerid, hp);
                if(hp >= 99) return SendClientMessage(playerid, -1, "You don't need this.");
                if(PlayerInfo[playerid][pHealthInsurance])
                {
                    SetPlayerHealth(playerid, 99);
                    SendClientMessage(playerid, -1, "Your expenses were covered by health insurance.");
                }
                else
                {
                    if(GetPlayerMoney(playerid) < 10000) return SendClientMessage(playerid, -1, "Not enough money.");
                    SetPlayerHealth(playerid, 99);
                    GivePlayerMoney(playerid, -10000);
                    SendClientMessage(playerid, -1, "Your health increased.");
                }
            }
        }
        return 1;
    }

There is nothing wrong with this code. If you are experiencing other issues, it is because you are not using the variables correctly.