error on dialog style list
#1

as the tittle say , i got error after adding new code for a /adminhelp CMD.
i make /adminhelp is like Dialog not message

this the code
Код:
	
if(dialogid == 11123)
	{
		if(response)
		{
			switch(listitem)
			{
				case 0:
				{
					ShowPlayerDialog(playerid,11124,DIALOG_STYLE_MSGBOX,"*** SERVER MODERATOR ***","/mjail /kick /c /togadvisor /togjoin","Close", "");
				}
				case 1:
				{
				    ShowPlayerDialog(playerid,11212,DIALOG_STYLE_MSGBOX,"*** JUNIOR STAFF ***","/kick /ban /jail /freeze /unfreeze /spec /slap /warn /gethere /gotoid /hospital /goto /n/aduty /jetpack /check /hlmute /nrn /listguns /setvw /setint /sendtols","Close", "");
				}
				case 2:
				{
					ShowPlayerDialog(playerid,11251,DIALOG_STYLE_MSGBOX,"*** GENERAL STAFF ***","/noooc /prison /fine /pfine /takeadminweapons /sendtols /prisonaccount /gotocar /entercar /getcar /n/mole /countdown /release /rto(reset) /mark /n(un)mute /ad(un)mute /leaders /ocheck /owarn /ofine /okills /revive /sendtoid /oocpower","Close", "");
				}
				case 3:
				{
					ShowPlayerDialog(playerid,12116,DIALOG_STYLE_MSGBOX,"*** SENIOR STAFF ***","/edit /setarmorall /fixvehall /blowup /veh /savechars /n /fixveh /sethp /setarmor /setfightstyle /n/destroycar /destroycars /eventhelp /contracts /sprison /sjail /sendtoip /n/ /unban /unbanip /deletehit /clearall /setinsurance /givelicense /adestroyplant /n  /vto /vtoreset /forceskin","Close", "");
				}
				case 4:
				{
				    ShowPlayerDialog(playerid,11217,DIALOG_STYLE_MSGBOX,"*** SUPERVISOR ***","/cnn /requestevent /ipcheck /ip /jetpack /vehname /watch /god /check /clearchat /nonewbie /setskin /force(death/hospital) /reloadpvehicles /dm /wepreset /respawncar(s) /givenos /setname /startlotto /givegun /cmotd /vipparty /admingatepw","Close", "");
				}
				case 5:
				{
					ShowPlayerDialog(playerid,11211,DIALOG_STYLE_MSGBOX,"*** SHIFT MANAGER ***","/famedit /refund /jailaccount /(o)clearflag /switchfac /switchfam /facpay /adivorce /facban /faccsfban /facunban /faccsfunban /listfacpay /gangwarn /gangunban /setcapping /banaccount /permaban /dedit /pedit /clearallreports /eventreset /banip ","Close", "");
				}
				case 6:
				{
                    ShowPlayerDialog(playerid,11218,DIALOG_STYLE_MSGBOX,"*** OPERATIONS MANAGER ***","/spec /specreset /pausespec /random /vrandom /giftreset /vipgifts /createdealership /destroydealership /createcdveh /destroycdveh /setcolor /payday /tod /giftall /weather /weatherall /edit /asellhouse /asellbiz /setchamp /givemoney /setmoney /setstat","Close", "");
				}
				case 7:
				{
                    ShowPlayerDialog(playerid,11219,DIALOG_STYLE_MSGBOX,"*** OPERATIONS MANAGER ***","/fcreate /fdelete /gotoco /createpvehicle /destroypvehicle /creategvehicle /gotogate /hnext /dmpnext /g(status/near/next/edit) /(goto/goin)door /(goto/goin)house /(create/delete/goto)point /hedit /dd(edit/next/name/pass) /dmpedit /dmpnear /gotomapicon","Close", "");
				}
				case 8:
				{
    				ShowPlayerDialog(playerid,12231,DIALOG_STYLE_MSGBOX,"*** DEPUTY SERVER DIRECTOR ***","/makebanappealer /makefacmod /makegangmod /motd /amotd /vipmotd /clearallreports /suspend /osuspend /rmute /makeleader /makehelper /makejhelper /makemoderator /deletetikis /deletetiki /setvip /vmute /vsuspend /gifts /givetoken","Close", "");
				)
				case 9:
				{
					ShowPlayerDialog(playerid,13324,DIALOG_STYLE_MSGBOX,"*** SERVER MODERATOR ***","/makeshoptech /announcem /clearallreports /makeadmin /fakeban /maketempadmin","Close", "");
				}
			}
			return 1;
		}
	}
and this the error
Код:
D:\Server\LARP.pwn(75379) : warning 217: loose indentation
D:\Server\LARP.pwn(75379) : error 029: invalid expression, assumed zero
D:\Server\LARP.pwn(75379 -- 75380) : warning 215: expression has no effect
D:\Server\LARP.pwn(75380) : error 001: expected token: ";", but found "case"
D:\Server\LARP.pwn(75380) : error 014: invalid statement; not in switch
D:\Server\LARP.pwn(75380) : fatal error 107: too many error messages on one line

Compilation aborted.Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


4 Errors.
Im confused , i have checked all of line but didnt found any problem / warning / error on it.
Can someone help me ?
Reply
#2

On line 75379, You have ")" instead of "}"
Here's the new script
pawn Код:
if(dialogid == 11123)
    {
        if(response)
        {
            switch(listitem)
            {
                case 0:
                {
                    ShowPlayerDialog(playerid,11124,DIALOG_STYLE_MSGBOX,"*** SERVER MODERATOR ***","/mjail /kick /c /togadvisor /togjoin","Close", "");
                }
                case 1:
                {
                    ShowPlayerDialog(playerid,11212,DIALOG_STYLE_MSGBOX,"*** JUNIOR STAFF ***","/kick /ban /jail /freeze /unfreeze /spec /slap /warn /gethere /gotoid /hospital /goto /n/aduty /jetpack /check /hlmute /nrn /listguns /setvw /setint /sendtols","Close", "");
                }
                case 2:
                {
                    ShowPlayerDialog(playerid,11251,DIALOG_STYLE_MSGBOX,"*** GENERAL STAFF ***","/noooc /prison /fine /pfine /takeadminweapons /sendtols /prisonaccount /gotocar /entercar /getcar /n/mole /countdown /release /rto(reset) /mark /n(un)mute /ad(un)mute /leaders /ocheck /owarn /ofine /okills /revive /sendtoid /oocpower","Close", "");
                }
                case 3:
                {
                    ShowPlayerDialog(playerid,12116,DIALOG_STYLE_MSGBOX,"*** SENIOR STAFF ***","/edit /setarmorall /fixvehall /blowup /veh /savechars /n /fixveh /sethp /setarmor /setfightstyle /n/destroycar /destroycars /eventhelp /contracts /sprison /sjail /sendtoip /n/ /unban /unbanip /deletehit /clearall /setinsurance /givelicense /adestroyplant /n  /vto /vtoreset /forceskin","Close", "");
                }
                case 4:
                {
                    ShowPlayerDialog(playerid,11217,DIALOG_STYLE_MSGBOX,"*** SUPERVISOR ***","/cnn /requestevent /ipcheck /ip /jetpack /vehname /watch /god /check /clearchat /nonewbie /setskin /force(death/hospital) /reloadpvehicles /dm /wepreset /respawncar(s) /givenos /setname /startlotto /givegun /cmotd /vipparty /admingatepw","Close", "");
                }
                case 5:
                {
                    ShowPlayerDialog(playerid,11211,DIALOG_STYLE_MSGBOX,"*** SHIFT MANAGER ***","/famedit /refund /jailaccount /(o)clearflag /switchfac /switchfam /facpay /adivorce /facban /faccsfban /facunban /faccsfunban /listfacpay /gangwarn /gangunban /setcapping /banaccount /permaban /dedit /pedit /clearallreports /eventreset /banip ","Close", "");
                }
                case 6:
                {
                    ShowPlayerDialog(playerid,11218,DIALOG_STYLE_MSGBOX,"*** OPERATIONS MANAGER ***","/spec /specreset /pausespec /random /vrandom /giftreset /vipgifts /createdealership /destroydealership /createcdveh /destroycdveh /setcolor /payday /tod /giftall /weather /weatherall /edit /asellhouse /asellbiz /setchamp /givemoney /setmoney /setstat","Close", "");
                }
                case 7:
                {
                    ShowPlayerDialog(playerid,11219,DIALOG_STYLE_MSGBOX,"*** OPERATIONS MANAGER ***","/fcreate /fdelete /gotoco /createpvehicle /destroypvehicle /creategvehicle /gotogate /hnext /dmpnext /g(status/near/next/edit) /(goto/goin)door /(goto/goin)house /(create/delete/goto)point /hedit /dd(edit/next/name/pass) /dmpedit /dmpnear /gotomapicon","Close", "");
                }
                case 8:
                {
                    ShowPlayerDialog(playerid,12231,DIALOG_STYLE_MSGBOX,"*** DEPUTY SERVER DIRECTOR ***","/makebanappealer /makefacmod /makegangmod /motd /amotd /vipmotd /clearallreports /suspend /osuspend /rmute /makeleader /makehelper /makejhelper /makemoderator /deletetikis /deletetiki /setvip /vmute /vsuspend /gifts /givetoken","Close", "");
                }
                case 9:
                {
                    ShowPlayerDialog(playerid,13324,DIALOG_STYLE_MSGBOX,"*** SERVER MODERATOR ***","/makeshoptech /announcem /clearallreports /makeadmin /fakeban /maketempadmin","Close", "");
                }
            }
            return 1;
        }
    }
Reply
#3

I noticed the if statement about checking the dialogid. If there are more cases to compare with, switch is much faster for doing it.
pawn Код:
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    switch(dialogid)
    {
        case 11123:
        {
            if(response)
            {
                switch(listitem)
                {
                    case 0: ShowPlayerDialog(playerid,11124,DIALOG_STYLE_MSGBOX,"*** SERVER MODERATOR ***","/mjail /kick /c /togadvisor /togjoin","Close", "");
                    case 1: ShowPlayerDialog(playerid,11212,DIALOG_STYLE_MSGBOX,"*** JUNIOR STAFF ***","/kick /ban /jail /freeze /unfreeze /spec /slap /warn /gethere /gotoid /hospital /goto /n/aduty /jetpack /check /hlmute /nrn /listguns /setvw /setint /sendtols","Close", "");
                    case 2: ShowPlayerDialog(playerid,11251,DIALOG_STYLE_MSGBOX,"*** GENERAL STAFF ***","/noooc /prison /fine /pfine /takeadminweapons /sendtols /prisonaccount /gotocar /entercar /getcar /n/mole /countdown /release /rto(reset) /mark /n(un)mute /ad(un)mute /leaders /ocheck /owarn /ofine /okills /revive /sendtoid /oocpower","Close", "");
                    case 3: ShowPlayerDialog(playerid,12116,DIALOG_STYLE_MSGBOX,"*** SENIOR STAFF ***","/edit /setarmorall /fixvehall /blowup /veh /savechars /n /fixveh /sethp /setarmor /setfightstyle /n/destroycar /destroycars /eventhelp /contracts /sprison /sjail /sendtoip /n/ /unban /unbanip /deletehit /clearall /setinsurance /givelicense /adestroyplant /n  /vto /vtoreset /forceskin","Close", "");
                    case 4: ShowPlayerDialog(playerid,11217,DIALOG_STYLE_MSGBOX,"*** SUPERVISOR ***","/cnn /requestevent /ipcheck /ip /jetpack /vehname /watch /god /check /clearchat /nonewbie /setskin /force(death/hospital) /reloadpvehicles /dm /wepreset /respawncar(s) /givenos /setname /startlotto /givegun /cmotd /vipparty /admingatepw","Close", "");
                    case 5: ShowPlayerDialog(playerid,11211,DIALOG_STYLE_MSGBOX,"*** SHIFT MANAGER ***","/famedit /refund /jailaccount /(o)clearflag /switchfac /switchfam /facpay /adivorce /facban /faccsfban /facunban /faccsfunban /listfacpay /gangwarn /gangunban /setcapping /banaccount /permaban /dedit /pedit /clearallreports /eventreset /banip ","Close", "");
                    case 6: ShowPlayerDialog(playerid,11218,DIALOG_STYLE_MSGBOX,"*** OPERATIONS MANAGER ***","/spec /specreset /pausespec /random /vrandom /giftreset /vipgifts /createdealership /destroydealership /createcdveh /destroycdveh /setcolor /payday /tod /giftall /weather /weatherall /edit /asellhouse /asellbiz /setchamp /givemoney /setmoney /setstat","Close", "");
                    case 7: ShowPlayerDialog(playerid,11219,DIALOG_STYLE_MSGBOX,"*** OPERATIONS MANAGER ***","/fcreate /fdelete /gotoco /createpvehicle /destroypvehicle /creategvehicle /gotogate /hnext /dmpnext /g(status/near/next/edit) /(goto/goin)door /(goto/goin)house /(create/delete/goto)point /hedit /dd(edit/next/name/pass) /dmpedit /dmpnear /gotomapicon","Close", "");
                    case 8: ShowPlayerDialog(playerid,12231,DIALOG_STYLE_MSGBOX,"*** DEPUTY SERVER DIRECTOR ***","/makebanappealer /makefacmod /makegangmod /motd /amotd /vipmotd /clearallreports /suspend /osuspend /rmute /makeleader /makehelper /makejhelper /makemoderator /deletetikis /deletetiki /setvip /vmute /vsuspend /gifts /givetoken","Close", "");
                    case 9: ShowPlayerDialog(playerid,13324,DIALOG_STYLE_MSGBOX,"*** SERVER MODERATOR ***","/makeshoptech /announcem /clearallreports /makeadmin /fakeban /maketempadmin","Close", "");
                }
            }
            return 1;
        }
        // other dialogids
    }
    return 0;
}
Reply
#4

Quote:
Originally Posted by Konstantinos
Посмотреть сообщение
I noticed the if statement about checking the dialogid. If there are more cases to compare with, switch is much faster for doing it.
pawn Код:
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    switch(dialogid)
    {
        case 11123:
        {
            if(response)
            {
                switch(listitem)
                {
                    case 0: ShowPlayerDialog(playerid,11124,DIALOG_STYLE_MSGBOX,"*** SERVER MODERATOR ***","/mjail /kick /c /togadvisor /togjoin","Close", "");
                    case 1: ShowPlayerDialog(playerid,11212,DIALOG_STYLE_MSGBOX,"*** JUNIOR STAFF ***","/kick /ban /jail /freeze /unfreeze /spec /slap /warn /gethere /gotoid /hospital /goto /n/aduty /jetpack /check /hlmute /nrn /listguns /setvw /setint /sendtols","Close", "");
                    case 2: ShowPlayerDialog(playerid,11251,DIALOG_STYLE_MSGBOX,"*** GENERAL STAFF ***","/noooc /prison /fine /pfine /takeadminweapons /sendtols /prisonaccount /gotocar /entercar /getcar /n/mole /countdown /release /rto(reset) /mark /n(un)mute /ad(un)mute /leaders /ocheck /owarn /ofine /okills /revive /sendtoid /oocpower","Close", "");
                    case 3: ShowPlayerDialog(playerid,12116,DIALOG_STYLE_MSGBOX,"*** SENIOR STAFF ***","/edit /setarmorall /fixvehall /blowup /veh /savechars /n /fixveh /sethp /setarmor /setfightstyle /n/destroycar /destroycars /eventhelp /contracts /sprison /sjail /sendtoip /n/ /unban /unbanip /deletehit /clearall /setinsurance /givelicense /adestroyplant /n  /vto /vtoreset /forceskin","Close", "");
                    case 4: ShowPlayerDialog(playerid,11217,DIALOG_STYLE_MSGBOX,"*** SUPERVISOR ***","/cnn /requestevent /ipcheck /ip /jetpack /vehname /watch /god /check /clearchat /nonewbie /setskin /force(death/hospital) /reloadpvehicles /dm /wepreset /respawncar(s) /givenos /setname /startlotto /givegun /cmotd /vipparty /admingatepw","Close", "");
                    case 5: ShowPlayerDialog(playerid,11211,DIALOG_STYLE_MSGBOX,"*** SHIFT MANAGER ***","/famedit /refund /jailaccount /(o)clearflag /switchfac /switchfam /facpay /adivorce /facban /faccsfban /facunban /faccsfunban /listfacpay /gangwarn /gangunban /setcapping /banaccount /permaban /dedit /pedit /clearallreports /eventreset /banip ","Close", "");
                    case 6: ShowPlayerDialog(playerid,11218,DIALOG_STYLE_MSGBOX,"*** OPERATIONS MANAGER ***","/spec /specreset /pausespec /random /vrandom /giftreset /vipgifts /createdealership /destroydealership /createcdveh /destroycdveh /setcolor /payday /tod /giftall /weather /weatherall /edit /asellhouse /asellbiz /setchamp /givemoney /setmoney /setstat","Close", "");
                    case 7: ShowPlayerDialog(playerid,11219,DIALOG_STYLE_MSGBOX,"*** OPERATIONS MANAGER ***","/fcreate /fdelete /gotoco /createpvehicle /destroypvehicle /creategvehicle /gotogate /hnext /dmpnext /g(status/near/next/edit) /(goto/goin)door /(goto/goin)house /(create/delete/goto)point /hedit /dd(edit/next/name/pass) /dmpedit /dmpnear /gotomapicon","Close", "");
                    case 8: ShowPlayerDialog(playerid,12231,DIALOG_STYLE_MSGBOX,"*** DEPUTY SERVER DIRECTOR ***","/makebanappealer /makefacmod /makegangmod /motd /amotd /vipmotd /clearallreports /suspend /osuspend /rmute /makeleader /makehelper /makejhelper /makemoderator /deletetikis /deletetiki /setvip /vmute /vsuspend /gifts /givetoken","Close", "");
                    case 9: ShowPlayerDialog(playerid,13324,DIALOG_STYLE_MSGBOX,"*** SERVER MODERATOR ***","/makeshoptech /announcem /clearallreports /makeadmin /fakeban /maketempadmin","Close", "");
                }
            }
            return 1;
        }
        // other dialogids
    }
    return 0;
}
Can i ask ? if i want to protect Higher CMD so if someone Junior Admin and want to see Senior Admin CMD's it will got ClientMessage You are not autho bla bla bla . How to do that ?

Btw nice
Reply
#5

Put some boolean as isSenadmin = true / false;
then and check it with if
.its simple
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)