[xSeveNx]

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 ._.?

[junkbuster]

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

[xSeveNx]

No se no tengo ni idea ._.

[LuisGraph]

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

[clavador]

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;
}

[xSeveNx]

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:<

[Kurama]

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.

[xSeveNx]

Si funciona.