[Mister0]

what iswrong here?

Код HTML:

enum
{
DIALOG_LOGIN,
DIALOG_REGISTER,
DIALOG_SEX,
DIALOG_AGE,
DIALOG_EMAIL
}

this enum have not name because i use on dialogresponse

ondialogresponse

switch(dialogid)
case DIALOG_LOGIN etc...


error 001: expected token: "-identifier-", but found "-integer value-"

[Vince]

Nothing wrong with the enum itself. Show the surrounding code for OnDialogResponse.

[Mister0]

PHP код:

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    static string[500],strong[1500],string2[120];
    static named[30];
    GetPlayerName(playerid,named,sizeof(named));
    if(    strfind(inputtext, "%", true) != -1  ||
           strfind(inputtext, "'", true) != -1 ||
           strfind(inputtext, "`", true) != -1 ||
           strfind(inputtext, ";", true) != -1 )
     {
        SCM(playerid,COLOR_RED,"don't use this characters!");
        return 0;
    }
     switch(dialogid)
    {
        case DIALOG_LOGIN:
        { 


[Mister0]

so what is the problem?

[K0P]

I think its because you are not setting the value of the dialogs (dialogids)

Try doing it like this:

Код:

enum dialogs
{
DIALOG_LOGIN,
DIALOG_REGISTER,
DIALOG_SEX,
DIALOG_AGE,
DIALOG_EMAIL
}

new DialogID[dialogs];

DialogID[DIALOG_LOGIN] = 1 /*Any unique id*/;
DialogID[DIALOG_REGISTER] = 2 /*Any unique id*/;
DialogID[DIALOG_SEX] = 3 /*Any unique id*/;
DialogID[DIALOG_AGE] = 4 /*Any unique id*/;
DialogID[DIALOG_EMAIL] = 5 /*Any unique id*/;

[Deadpoop]

Quote:

Originally Posted by K0P I think its because you are not setting the value of the dialogs (dialogids) Try doing it like this: Код: enum dialogs { DIALOG_LOGIN, DIALOG_REGISTER, DIALOG_SEX, DIALOG_AGE, DIALOG_EMAIL } new DialogID[dialogs]; DialogID[DIALOG_LOGIN] = 1 /*Any unique id*/; DialogID[DIALOG_REGISTER] = 2 /*Any unique id*/; DialogID[DIALOG_SEX] = 3 /*Any unique id*/; DialogID[DIALOG_AGE] = 4 /*Any unique id*/; DialogID[DIALOG_EMAIL] = 5 /*Any unique id*/;

No need to set id''s because enum allready does that

DIALOG_LOGIN //= 0
DIALOG_REGISTER //= 1
...

[K0P]

Quote:

Originally Posted by Deadpoop No need to set id''s because enum allready does that DIALOG_LOGIN //= 0 DIALOG_REGISTER //= 1 ...

What if you want to set the id to 100 or any other unique id to prevent dialog ids collision?
Read the post before commenting.Or take English learning classes if you can't understand basic sentences.

Quote:

Originally Posted by K0P /*Any unique id*/;

[J0sh...]

Quote:

Originally Posted by K0P What if you want to set the id to 100 or any other unique id to prevent dialog ids collision? Read the post before commenting.Or take English learning classes if you can't understand basic sentences.

You could just do enum { DIALOG_REGISTER = 100 }; then it'll increment.

[Konstantinos]

Search for "#define DIALOG_LOGIN .." in your script and remove it (where .. is a number).

Quote:

Originally Posted by K0P What if you want to set the id to 100 or any other unique id to prevent dialog ids collision? Read the post before commenting.Or take English learning classes if you can't understand basic sentences.

Collisions have nothing to do with compiling and the error he gets.

[K0P]

Quote:

Originally Posted by Konstantinos Search for "#define DIALOG_LOGIN .." in your script and remove it (where .. is a number). Collisions have nothing to do with compiling and the error he gets.

But it has a bad effect on dialogs that have responses.