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

Threaded Mode
Help ty.
kevi11
Huge Clucker
****
Posts: 280
Threads: 79
Joined: Sep 2014
Reputation: 0
#1

08.06.2016, 16:05
I start by saying that I'm learning to script and I can not understand some things like these errors, if you could explain to me how you solved this problem thank you.

Script:
PHP код:
if(dialogid == DROGA)
{
   switch(dialogid)
   {
       case DIALOG_DROGA:
       {
           if(response)
           {
                switch(listitem)
                {
                     case 0:
                     {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la Marijuana!");
                        GivePlayerMoney(playerid, -1000);
                     }
                     case 1:
                     {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la metanfetamina!");
                        GivePlayerMoney(playerid, -1000);
                     }
                     case 2:
                     {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la cocaina!");
                        GivePlayerMoney(playerid, -1000);
                     }
               }
          }
       }
    }
    return 0;
} 
Error:
PHP код:
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(158) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(160) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(162) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(164) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(166) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(168) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(170) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(174) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(176) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(180) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(182) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(190) : error 010: invalid function or declaration 
Find
Reply
N0FeaR
High-roller
*****
Posts: 2,308
Threads: 337
Joined: Nov 2011
Reputation: 0
#2

08.06.2016, 16:09
Try this


PHP код:
if(dialogid == DROGA)
{
   switch(dialogid)
   {
       case DIALOG_DROGA:
       {
           if(response)
           {
                switch(listitem)
                {
                     case 0:
                     {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la Marijuana!");
                        GivePlayerMoney(playerid, -1000);
                     }
                     case 1:
                     {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la metanfetamina!");
                        GivePlayerMoney(playerid, -1000);
                     }
                     case 2:
                     {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la cocaina!");
                        GivePlayerMoney(playerid, -1000);
                    }
                }
              }
           }
   }
    return 1;
} 
Find
Reply
kevi11
Huge Clucker
****
Posts: 280
Threads: 79
Joined: Sep 2014
Reputation: 0
#3

08.06.2016, 16:15
Nope. again the errors.
Find
Reply
N0FeaR
High-roller
*****
Posts: 2,308
Threads: 337
Joined: Nov 2011
Reputation: 0
#4

08.06.2016, 16:18
Quote:
Originally Posted by kevi11
Посмотреть сообщение
Nope. again the errors.
Show me the error lines.
Find
Reply
kevi11
Huge Clucker
****
Posts: 280
Threads: 79
Joined: Sep 2014
Reputation: 0
#5

08.06.2016, 16:21
I think I was wrong to do right after another look at case.
PHP код:
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    switch(dialogid)
    {
        case DIALOG_ARMI:
        {
            if(response)
            {
                switch(listitem)
                {
                    case 0:
                    {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato un'AK47!");
                        GivePlayerMoney(playerid, -1000);
                        GivePlayerWeapon(playerid, 30, 100);
                    }
                    case 1:
                    {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato una M4!");
                        GivePlayerMoney(playerid, -1000);
                        GivePlayerWeapon(playerid, 31, 100);
                    }
                    case 2:
                    {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato una desert eagle!");
                        GivePlayerMoney(playerid, -1000);
                        GivePlayerWeapon(playerid, 24, 100);
                    }
                }
            }
        }
    }
    return 0;
}
if(dialogid == DROGA)
{
   switch(dialogid)
   {
       case DIALOG_DROGA:
       {
           if(response)
           {
                switch(listitem)
                {
                     case 0:
                     {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la Marijuana!");
                        GivePlayerMoney(playerid, -1000);
                     }
                     case 1:
                     {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la metanfetamina!");
                        GivePlayerMoney(playerid, -1000);
                     }
                     case 2:
                     {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la cocaina!");
                        GivePlayerMoney(playerid, -1000);
                     }
                 }
               }
            }
    }
    return 1;
} 
Find
Reply
N0FeaR
High-roller
*****
Posts: 2,308
Threads: 337
Joined: Nov 2011
Reputation: 0
#6

08.06.2016, 16:25
This should work


PHP код:
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    switch(dialogid)
    {
        case DIALOG_ARMI:
        {
            if(response)
            {
                switch(listitem)
                {
                    case 0:
                    {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato un'AK47!");
                        GivePlayerMoney(playerid, -1000);
                        GivePlayerWeapon(playerid, 30, 100);
                    }
                    case 1:
                    {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato una M4!");
                        GivePlayerMoney(playerid, -1000);
                        GivePlayerWeapon(playerid, 31, 100);
                    }
                    case 2:
                    {
                        if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato una desert eagle!");
                        GivePlayerMoney(playerid, -1000);
                        GivePlayerWeapon(playerid, 24, 100);
                    }
                }
            }
        }
    }
}
case DIALOG_DROGA:
{
    if(response)
    {
        switch(listitem)
        {
            case 0:
             {
                  if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la Marijuana!");
                  GivePlayerMoney(playerid, -1000);
            }
             case 1:
              {
                if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                   SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la metanfetamina!");
                   GivePlayerMoney(playerid, -1000);
              }
               case 2:
            {
                   if(GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la cocaina!");
                GivePlayerMoney(playerid, -1000);
            }
        }
    }
    return 1;
} 
Find
Reply
kevi11
Huge Clucker
****
Posts: 280
Threads: 79
Joined: Sep 2014
Reputation: 0
#7

08.06.2016, 16:32
PHP код:
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(156) : warning 209: function "OnDialogResponse" should return a value
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(157) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(159) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(161) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(163) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(165) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(169) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(171) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(175) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(177) : error 010: invalid function or declaration
C:\Users\Eclipse\Desktop\Freeroam da 0\gamemodes\Freeroam.pwn(183) : error 010: invalid function or declaration 
Find
Reply
Konstantinos
Spam Machine
*****
Posts: 11,827
Threads: 33
Joined: Dec 2011
Reputation: 0
#8

08.06.2016, 16:38
As you can see before DROGA dialogid checking, there are extra closed brackets so it's like checking dialog and doing stuff outside of the callback.

Keep each "case X:" at the same column (2 tabs (8 spaces) right):
PHP код:
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    switch (dialogid)
    {
        case DIALOG_ARMI:
        {
            if (response)
            {
                switch (listitem)
                {
                    case 0:
                    {
                        if (GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato un'AK47!");
                        GivePlayerMoney(playerid, -1000);
                        GivePlayerWeapon(playerid, 30, 100);
                    }
                    case 1:
                    {
                        if (GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato una M4!");
                        GivePlayerMoney(playerid, -1000);
                        GivePlayerWeapon(playerid, 31, 100);
                    }
                    case 2:
                    {
                        if (GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato una desert eagle!");
                        GivePlayerMoney(playerid, -1000);
                        GivePlayerWeapon(playerid, 24, 100);
                    }
                }
            }
        }
        case DIALOG_DROGA:
        {
            if (response)
            {
                switch (listitem)
                {
                    case 0:
                    {
                        if (GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la Marijuana!");
                                GivePlayerMoney(playerid, -1000);
                    }
                    case 1:
                    {
                        if (GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la metanfetamina!");
                        GivePlayerMoney(playerid, -1000);
                    }
                    case 2:
                    {
                        if (GetPlayerMoney(playerid) < 1000) return SendClientMessage(playerid, 0xAA3333AA, "Non hai abbastanza denaro.!");
                        SendClientMessage(playerid, 0xAA3333AA, "Hai comprato la cocaina!");
                        GivePlayerMoney(playerid, -1000);
                    }
                }
            }
        }
        default: return 0; // dialogid not found, search in other scripts
    }
    return 1; // dialogid was found
} 
Find
Reply
kevi11
Huge Clucker
****
Posts: 280
Threads: 79
Joined: Sep 2014
Reputation: 0
#9

08.06.2016, 16:42
Ty u so much Konstantinos, I have 1 error only.
PHP код:
error 040: duplicate "case" label (value 1) 
Find
Reply
kevi11
Huge Clucker
****
Posts: 280
Threads: 79
Joined: Sep 2014
Reputation: 0
#10

08.06.2016, 16:45
Konstantinos would like to understand how did you explain it to me?
Find
Reply
« Next Oldest | Next Newest »


Forum Jump:


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