SA-MP Forums Archive
[DUDA]: Dialog - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: Non-English (https://sampforum.blast.hk/forumdisplay.php?fid=9)
+--- Forum: Languages (https://sampforum.blast.hk/forumdisplay.php?fid=33)
+---- Forum: Español/Spanish (https://sampforum.blast.hk/forumdisplay.php?fid=29)
+---- Thread: [DUDA]: Dialog (/showthread.php?tid=319958)


[DUDA]: Dialog - xSeveNx - 21.02.2012

A ver, me han dicho que esto estб mal hecho y todo, y la verdad no veo que tiene de malo ._. ya que me funciona perfectamente

pawn Код:
#define Musica 1313

public OnPlayerCommandText(playerid, cmdtext[])
{

    if(strcmp(cmdtext,"/linkinpark",true) == 0)
    {
        ShowPlayerDialog(playerid, Musica,DIALOG_STYLE_LIST,"LinkinPark", "Prueba\n Prueba2","Aceptar","Cancelar");
        return 1;
}

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    if(dialogid == Musica)
    {
            if(listitem ==0)
            {
            PlayAudioStreamForPlayer(playerid, "http://members.shaw.ca/meowzer/music/acdc.mp3");
            }
            if(listitem ==1)
            {
            PlayAudioStreamForPlayer(playerid, "http://pds9.egloos.com/pds/200806/20/75/Numb_Encore-Jay-Z__Linkin_Park.mp3");
            }
            }
    return 1;
}
Me pueden decir que me falta o que estб mal hecho ._.?

Respuesta: [DUDA]: Dialog - junkbuster - 21.02.2012

Si te funciona porque deberнa estar mal, es ilуgico.

Respuesta: [DUDA]: Dialog - xSeveNx - 21.02.2012

No se no tengo ni idea ._.

Respuesta: [DUDA]: Dialog - LuisGraph - 21.02.2012

-.-! estб bien cual fue el cуjonudo que te dijo que estб mal?

Re: [DUDA]: Dialog - clavador - 21.02.2012

Esta bien hecho, no tiene nada de malo.

Lo unico que te sugiero es cambiar los ifs por un switch, ya que procesa mas rapido las consultas:

Код:
public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    switch( dialogid )
    {
       case Musica:
       {
           switch(listitem)
           {
               case 0: PlayAudioStreamForPlayer(playerid, "http://members.shaw.ca/meowzer/music/acdc.mp3");
               case 1: PlayAudioStreamForPlayer(playerid, "http://pds9.egloos.com/pds/200806/20/75/Numb_Encore-Jay-Z__Linkin_Park.mp3");
           }
       }
    }
    return 1;
}


Respuesta: [DUDA]: Dialog - xSeveNx - 21.02.2012

Ya se >:0
Viendo un FS con casi lo mismo, tenia un if(response) debajo de if(dialogid ==asd)
y por eso me decнa que estaba mal hecho ._____.

PD: gracias por el consejo clavador 0:<

Respuesta: [DUDA]: Dialog - Kurama - 21.02.2012

pawn Код:
#define Musica 1313

public OnPlayerCommandText(playerid, cmdtext[])
{
    if(strcmp(cmdtext,"/linkinpark",true) == 0)
    {
        ShowPlayerDialog(playerid, Musica,DIALOG_STYLE_LIST,"LinkinPark", "Prueba\n Prueba2","Aceptar","Cancelar");
        return 1;
    }
    return 0; //o return SendClientMessage(playerid, 0xFF0000FF, "Comando invбlido"),1;
}

public OnDialogResponse(playerid, dialogid, response, listitem, inputtext[])
{
    switch(dialogid)
    {
        case Musica:
        {
            switch(listitem)
            {
                case 0: return PlayAudioStreamForPlayer(playerid, "http://members.shaw.ca/meowzer/music/acdc.mp3"),1;
                case 1: return PlayAudioStreamForPlayer(playerid, "http://pds9.egloos.com/pds/200806/20/75/Numb_Encore-Jay-Z__Linkin_Park.mp3"),1;
            }
        }
    }
    return 0;
}
Cуdigo no testeado, pero deberнa funcionar.

Respuesta: [DUDA]: Dialog - xSeveNx - 21.02.2012

Si funciona.