[Louris]

Код:

if(dialogid == 998)
{
if(response)
{
if(listitem == 0)
{
new vehicleid = GetPlayerVehicleID(playerid);
new vehiclemodel = GetVehicleModel( vehicleid );
if( vehiclemodel == 596 || vehiclemodel == 597 || vehiclemodel == 427 || vehiclemodel == 523 || vehiclemodel == 599 || vehiclemodel == 601 )
{
SetPVarInt(playerid, "Status", 1);
SetPVarInt(playerid, "sviturelis2", CreateObject(18646,0,0,0,0,0,0));
AttachObjectToVehicle(GetPVarInt(playerid, "sviturelis2"), GetPlayerVehicleID(playerid), -0.5, 0.0, 0.9, 0.0, 0.0, 0.0);
SendClientMessage(playerid, 0xFFFFFFAA, "Љvyturėliai uћdėti! Tai jums kainavo 500 LT.");
GivePlayerMoney(playerid,-500);
}
if(listitem == 1)
{
DeletePVar(playerid, "Status");
DestroyObject(GetPVarInt(playerid, "sviturelis2"));
SendClientMessage(playerid, 0xFFFFFFAA, "Nusiimėte."); // list item == 1 doesn't works.
}
}
}
}

Код:

CMD:svytureliai(playerid, params[])
{
ShowPlayerDialog(playerid, 998, DIALOG_STYLE_LIST, "Љvyturėliai", "Uћsidėti\nNusiimti", "Rinktis", "");
    return 1;
}

[zde5]

You placed if(listitem == 1) inside if(listitem == 0)
This should work:

Код:

if(dialogid == 998)
{
	if(response)
	{
		if(listitem == 0)
		{
		new vehicleid = GetPlayerVehicleID(playerid);
		new vehiclemodel = GetVehicleModel( vehicleid );
			if( vehiclemodel == 596 || vehiclemodel == 597 || vehiclemodel == 427 || vehiclemodel == 523 || vehiclemodel == 599 || vehiclemodel == 601 )
			{
			SetPVarInt(playerid, "Status", 1);
			SetPVarInt(playerid, "sviturelis2", CreateObject(18646,0,0,0,0,0,0));
			AttachObjectToVehicle(GetPVarInt(playerid, "sviturelis2"), GetPlayerVehicleID(playerid), -0.5, 0.0, 0.9, 0.0, 0.0, 0.0);
			SendClientMessage(playerid, 0xFFFFFFAA, "Љvytureliai uћdeti! Tai jums kainavo 500 LT.");
			GivePlayerMoney(playerid,-500);
			}
		}
		if(listitem == 1)
		{
		DeletePVar(playerid, "Status");
		DestroyObject(GetPVarInt(playerid, "sviturelis2"));
		SendClientMessage(playerid, 0xFFFFFFAA, "Nusiimete."); // list item == 1 doesn't works.
		}
	}
}

[teqskater]

pawn Код:

if(dialogid == 998)
{
    if(!response) return 1;
    if(listitem == 0)
    {
        new vehicleid = GetPlayerVehicleID(playerid), vehiclemodel = GetVehicleModel( vehicleid );
        if( vehiclemodel == 596 || vehiclemodel == 597 || vehiclemodel == 427 || vehiclemodel == 523 || vehiclemodel == 599 || vehiclemodel == 601 )
        {
            SetPVarInt(playerid, "Status", 1);
            SetPVarInt(playerid, "sviturelis2", CreateObject(18646,0,0,0,0,0,0));
            AttachObjectToVehicle(GetPVarInt(playerid, "sviturelis2"), GetPlayerVehicleID(playerid), -0.5, 0.0, 0.9, 0.0, 0.0, 0.0);
            SendClientMessage(playerid, 0xFFFFFFAA, "Љvyture.liai uћde.ti! Tai jums kainavo 500 LT.");
            GivePlayerMoney(playerid, -500);
        }
    }
    if(listitem == 1)
    {
        DeletePVar(playerid, "Status");
        DestroyObject(GetPVarInt(playerid, "sviturelis2"));
        SendClientMessage(playerid, 0xFFFFFFAA, "Nusiime.te."); // list item == 1 doesn't works.
    }
}

[zde5]

Quote:

Originally Posted by teqskater pawn Код: if(dialogid == 998) {     if(!response) return 1;     if(listitem == 0)     {         new vehicleid = GetPlayerVehicleID(playerid), vehiclemodel = GetVehicleModel( vehicleid );         if( vehiclemodel == 596 || vehiclemodel == 597 || vehiclemodel == 427 || vehiclemodel == 523 || vehiclemodel == 599 || vehiclemodel == 601 )         {             SetPVarInt(playerid, "Status", 1);             SetPVarInt(playerid, "sviturelis2", CreateObject(18646,0,0,0,0,0,0));             AttachObjectToVehicle(GetPVarInt(playerid, "sviturelis2"), GetPlayerVehicleID(playerid), -0.5, 0.0, 0.9, 0.0, 0.0, 0.0);             SendClientMessage(playerid, 0xFFFFFFAA, "Љvyture.liai uћde.ti! Tai jums kainavo 500 LT.");             GivePlayerMoney(playerid, -500);         }     }     if(listitem == 1)     {         DeletePVar(playerid, "Status");         DestroyObject(GetPVarInt(playerid, "sviturelis2"));         SendClientMessage(playerid, 0xFFFFFFAA, "Nusiime.te."); // list item == 1 doesn't works.     } }

what is point of this? Its same code as mine except you added "if(!response) return 1;" what is wrong because it implements that it was completed successfully by placing in return value 1 (truth)

[Da_Noob]

It's not completely the same. He added a '}', because you forgot one. And yes, you are correct, if(!response) should return 0.

[zde5]

Quote:

Originally Posted by Da_Noob It's not completely the same. He added a '}', because you forgot one. And yes, you are correct, if(!response) should return 0.

i didn't forgot '}' in my code, i always try to compile my code before posting.