1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help

Threaded Mode
Error pls help me
kevi11
Huge Clucker
****
Posts: 280
Threads: 79
Joined: Sep 2014
Reputation: 0
#1

27.05.2016, 08:29
Guys this is my script: http://pastebin.com/gGLFprw7


This is the error:
PHP код:
error 030: compound statement not closed at the end of file (started at line 94) 
PS: If you can I can do this problem you explain how you did it so I learn a little
Find
Reply
luke49
Big Clucker
***
Posts: 134
Threads: 0
Joined: May 2016
Reputation: 0
#2

27.05.2016, 08:39
You didn't close the bracket after SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that."); (2 of them), did you?
Find
Reply
Slawiii
Huge Clucker
****
Posts: 293
Threads: 29
Joined: May 2015
Reputation: 0
#3

27.05.2016, 09:09
try this
PHP код:
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    if(listitem == 0)
    {
        if(Materiali[playerid] > 399)
        {
            SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a desert eagle.");
            GivePlayerWeapon(playerid, 24, 9999999);
            return 1;
        }
        else
        {
            SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
            return 1;
        }
    }
    if(listitem == 1)
    {
          if(Materiali[playerid] > 149)
        {
            SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a pump-action shotgun.");
            GivePlayerWeapon(playerid, 25, 9999999);
            return 1;
        }
        else
        {
            SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
            return 1;
        }
    }
    if(listitem == 2)
    {
        if(Materiali[playerid] > 249)
        {
            SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a MP5.");
            GivePlayerWeapon(playerid, 29, 9999999);
            return 1;
        }
        else
        {
            SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
            return 1;
        }
    }
    if(listitem == 3)
    {
        if(Materiali[playerid] > 499)
        {
            SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a M4 carbine.");
            GivePlayerWeapon(playerid, 31, 9999999);
            return 1;
        }
        else
        {
            SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
            return 1;
        }
    }
    if(listitem == 4)
    {
        if(Materiali[playerid] > 549)
        {
            SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a Spas-12.");
            GivePlayerWeapon(playerid, 27, 9999999);
            return 1;
        }
        else
        {
            SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
            return 1;
        }
    }
} 
Find
Reply
kevi11
Huge Clucker
****
Posts: 280
Threads: 79
Joined: Sep 2014
Reputation: 0
#4

27.05.2016, 09:16
This 2 Error
PHP код:
warning 209: function "OnDialogResponse" should return a value 
PHP код:
warning 217: loose indentation 
Find
Reply
Slawiii
Huge Clucker
****
Posts: 293
Threads: 29
Joined: May 2015
Reputation: 0
#5

27.05.2016, 09:21
try this

Код:
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[]) 
{ 
    if(listitem == 0) 
    { 
        if(Materiali[playerid] > 399) 
        { 
            SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a desert eagle."); 
            GivePlayerWeapon(playerid, 24, 9999999); 
            return 1; 
        } 
        else 
        { 
            SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that."); 
            return 1; 
        } 
    } 
    if(listitem == 1) 
    { 
        if(Materiali[playerid] > 149) 
        { 
            SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a pump-action shotgun."); 
            GivePlayerWeapon(playerid, 25, 9999999); 
            return 1; 
        } 
        else 
        { 
            SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that."); 
            return 1; 
        } 
    } 
    if(listitem == 2) 
    { 
        if(Materiali[playerid] > 249) 
        { 
            SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a MP5."); 
            GivePlayerWeapon(playerid, 29, 9999999); 
            return 1; 
        } 
        else 
        { 
            SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that."); 
            return 1; 
        } 
    } 
    if(listitem == 3) 
    { 
        if(Materiali[playerid] > 499) 
        { 
            SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a M4 carbine."); 
            GivePlayerWeapon(playerid, 31, 9999999); 
            return 1; 
        } 
        else 
        { 
            SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that."); 
            return 1; 
        } 
    } 
    if(listitem == 4) 
    { 
        if(Materiali[playerid] > 549) 
        { 
            SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a Spas-12."); 
            GivePlayerWeapon(playerid, 27, 9999999); 
            return 1; 
        } 
        else 
        { 
            SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that."); 
            return 1; 
        } 
    }
    return 1; 
}
Find
Reply
kevi11
Huge Clucker
****
Posts: 280
Threads: 79
Joined: Sep 2014
Reputation: 0
#6

27.05.2016, 09:26
Ty u so much
Find
Reply
kevi11
Huge Clucker
****
Posts: 280
Threads: 79
Joined: Sep 2014
Reputation: 0
#7

27.05.2016, 09:28
Guys this command don't give any Material why?
PHP код:
CMD:Compramateriali(playerid, params[])
{
    ShowPlayerDialog(playerid,7,DIALOG_STYLE_LIST,"Materiali","500 Materiali (500$)\n700 Materiali (900$)\n 1000 Materiali (2000$)","Compra","Esci");
    return 1;
} 
Find
Reply
Konstantinos
Spam Machine
*****
Posts: 11,827
Threads: 33
Joined: Dec 2011
Reputation: 0
#8

27.05.2016, 09:29
You must check the dialogid before and then its listitems otherwise will surely be bugged with other scripts.
Find
Reply
Amunra
Gangsta
****
Posts: 502
Threads: 57
Joined: Nov 2015
Reputation: 0
#9

27.05.2016, 09:34
Quote:
Originally Posted by kevi11
Посмотреть сообщение
Guys this command don't give any Material why?
PHP код:
CMD:Compramateriali(playerid, params[])
{
    ShowPlayerDialog(playerid,7,DIALOG_STYLE_LIST,"Materiali","500 Materiali (500$)\n700 Materiali (900$)\n 1000 Materiali (2000$)","Compra","Esci");
    return 1;
} 
PHP код:
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    if(dialogid == 7)
    {
        if(response)
        {
            if(listitem == 0)
            {
                if(Materiali[playerid] > 399)
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a desert eagle.");
                    GivePlayerWeapon(playerid, 24, 9999999);
                    return 1;
                }
                else
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
                    return 1;
                }
            }
            if(listitem == 1)
            {
                if(Materiali[playerid] > 149)
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a pump-action shotgun.");
                    GivePlayerWeapon(playerid, 25, 9999999);
                    return 1;
                }
                else
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
                    return 1;
                }
            }
            if(listitem == 2)
            {
                if(Materiali[playerid] > 249)
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a MP5.");
                    GivePlayerWeapon(playerid, 29, 9999999);
                    return 1;
                }
                else
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
                    return 1;
                }
            }
            if(listitem == 3)
            {
                if(Materiali[playerid] > 499)
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a M4 carbine.");
                    GivePlayerWeapon(playerid, 31, 9999999);
                    return 1;
                }
                else
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
                    return 1;
                }
            }
            if(listitem == 4)
            {
                if(Materiali[playerid] > 549)
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a Spas-12.");
                    GivePlayerWeapon(playerid, 27, 9999999);
                    return 1;
                }
                else
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
                }
            }
        }
    }
    return 0; // Some Script can't used Dialog if you use return 1 .. Lets Make Return 0 for the other script can use dialog
} 
Try this ^^
Find
Reply
Slawiii
Huge Clucker
****
Posts: 293
Threads: 29
Joined: May 2015
Reputation: 0
#10

27.05.2016, 11:40
Quote:
Originally Posted by Amunra
Посмотреть сообщение
PHP код:
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    if(dialogid == 7)
    {
        if(response)
        {
            if(listitem == 0)
            {
                if(Materiali[playerid] > 399)
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a desert eagle.");
                    GivePlayerWeapon(playerid, 24, 9999999);
                    return 1;
                }
                else
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
                    return 1;
                }
            }
            if(listitem == 1)
            {
                if(Materiali[playerid] > 149)
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a pump-action shotgun.");
                    GivePlayerWeapon(playerid, 25, 9999999);
                    return 1;
                }
                else
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
                    return 1;
                }
            }
            if(listitem == 2)
            {
                if(Materiali[playerid] > 249)
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a MP5.");
                    GivePlayerWeapon(playerid, 29, 9999999);
                    return 1;
                }
                else
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
                    return 1;
                }
            }
            if(listitem == 3)
            {
                if(Materiali[playerid] > 499)
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a M4 carbine.");
                    GivePlayerWeapon(playerid, 31, 9999999);
                    return 1;
                }
                else
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
                    return 1;
                }
            }
            if(listitem == 4)
            {
                if(Materiali[playerid] > 549)
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You have crafted yourself a Spas-12.");
                    GivePlayerWeapon(playerid, 27, 9999999);
                    return 1;
                }
                else
                {
                    SendClientMessage(playerid, COLOUR_WHITE, "You dont have enough materials to make that.");
                }
            }
        }
    }
    return 0; // Some Script can't used Dialog if you use return 1 .. Lets Make Return 0 for the other script can use dialog
} 
Try this ^^
DIALOG id 7 has only 3 listitem
Find
Reply
« Next Oldest | Next Newest »


Forum Jump:


Users browsing this thread: 3 Guest(s)
  1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help