[Ajuda] Й Possнvel Otimizar ainda mais este comando /membros ?

[Ajuda] Й Possнvel Otimizar ainda mais este comando /membros ? - 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: Português/Portuguese (https://sampforum.blast.hk/forumdisplay.php?fid=34)
+---- Thread: [Ajuda] Й Possнvel Otimizar ainda mais este comando /membros ? (/showthread.php?tid=651690)

Й Possнvel Otimizar ainda mais este comando /membros ? - [BOPE]Seu._.Madruga - 25.03.2018

Primeiramente boa tarde a todos, seguinte eu estava ajudando o membro brunogusava aqui do fуrum com um comando /membros

Este comando abaixo й o feito por ele, porйm ele utiliza o processador de comandos zcmd e nгo quer mudar.

Eu queria saber se й possнvel deixar mais Otimizado do que o Comando 2 que foi o que eu criei.

• COMANDO 1 - Criado pelo brunogusava

PHP код:



CMD:membros(playerid, params[])

{

    if(Player[playerid][Organizacao] == 0)

    {

        SendClientMessage(playerid, 0xFF0000FF, "[BLC] |Erro| Vocк nгo й de uma organizaзгo.");

    }

    else

    {

        for(new i = 0, i < MAX_PLAYERS, i++)

        {

            if(Player[i][Organizacao] == Player[playerid][Organizacao])

            {

                SendClientMessage(playerid, -1, "/-----Membros Online-----/\n");

                new nome[MAX_PLAYER_NAME], string[MAX_PLAYER_NAME];

                GetPlayerName(i, nome, sizeof(nome));

                format(string, sizeof(string), "%s", nome);

                SendClientMessage(playerid, -1, string);

                SendClientMessage(playerid, -1, "\n/-----Membros Online-----/");

            }

        }

    }

}

• COMANDO 2 - Criado por mim (Leonardo Bradoks)

PHP код:



CMD:membros(playerid) {

    if(!Player[playerid][Organizacao]) return SendClientMessage(playerid, 0xFF0000FF, "[BLC] Vocк nгo й de uma organizaзгo.");

    new CountMembers, nome[MAX_PLAYER_NAME];

    foreach(Player, i) {

        if(Player[i][Organizacao] == Player[playerid][Organizacao] && i != playerid) {

            if(!CountMembers) SendClientMessage(playerid, -1, "/----- Membros da Organizaзгo Online -----/");

            GetPlayerName(i, nome, sizeof(nome));

            SendClientMessage(playerid, -1, nome);

            CountMembers++;

        }

    }

    SendClientMessage(playerid, CountMembers >= 1 ? (-1) : (0xFF0000FF), CountMembers >= 1 ? ("/----- Membros da Organizaзгo Online -----/") : ("[BLC] Fora vocк nгo hб nenhum(a) outro(a) membro(a) da sua organizaзгo estб online."));

    return true;

}

• Lembrando a todos que toda critica construtiva й bem vinda TMJ! •

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - KoloradO - 25.03.2018

Acho que nгo da pra otimizar mais que isso. Sу mudei o return true pra return 1. E CountMembers = 0 pra CountMembers.

Код:

CMD:membros(playerid) {
    if(!Player[playerid][Organizacao]) return SendClientMessage(playerid, 0xFF0000FF, "[BLC] Vocк nгo й de uma organizaзгo.");
    static CountMembers, nome[MAX_PLAYER_NAME], string[MAX_PLAYER_NAME];
    foreach(Player, i) {
        if(Player[i][Organizacao] == Player[playerid][Organizacao] && i != playerid) {
            if(!CountMembers) SendClientMessage(playerid, -1, "/-----Membros da Organizaзгo Online -----/");
            GetPlayerName(i, nome, sizeof(nome));
            format(string, sizeof(string), "%s", nome);
            SendClientMessage(playerid, -1, string);
            CountMembers++;
        }
    }
    SendClientMessage(playerid, CountMembers >= 1 ? (-1) : (0xFF0000FF), CountMembers >= 1 ? ("/----- Membros da Organizaзгo Online -----/") : ("[BLC] Fora vocк nгo hб nenhum(a) outro(a) membro(a) da sua organizaзгo estб online."));
    return 1;
}

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - [BOPE]Seu._.Madruga - 25.03.2018

Quote:

Originally Posted by KoloradO

Acho que nгo da pra otimizar mais que isso. Sу mudei o return true pra return 1. E CountMembers = 0 pra CountMembers.

Код:

CMD:membros(playerid) {
    if(!Player[playerid][Organizacao]) return SendClientMessage(playerid, 0xFF0000FF, "[BLC] Vocк nгo й de uma organizaзгo.");
    static CountMembers, nome[MAX_PLAYER_NAME], string[MAX_PLAYER_NAME];
    foreach(Player, i) {
        if(Player[i][Organizacao] == Player[playerid][Organizacao] && i != playerid) {
            if(!CountMembers) SendClientMessage(playerid, -1, "/-----Membros da Organizaзгo Online -----/");
            GetPlayerName(i, nome, sizeof(nome));
            format(string, sizeof(string), "%s", nome);
            SendClientMessage(playerid, -1, string);
            CountMembers++;
        }
    }
    SendClientMessage(playerid, CountMembers >= 1 ? (-1) : (0xFF0000FF), CountMembers >= 1 ? ("/----- Membros da Organizaзгo Online -----/") : ("[BLC] Fora vocк nгo hб nenhum(a) outro(a) membro(a) da sua organizaзгo estб online."));
    return 1;
}

Certo porйm a finalidade de trocar o true por 1 й meio que nula pois de qualquer forma true retorna 1

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - Dayvison_ - 25.03.2018

Sу removendo a operaзгo ternбria no SendClientMessage.

Menos linhas nгo quer dizer que estб mais otimizado.
"Medir o progresso de um programa por linhas de cуdigo й como medir o processo de montagem de um aviгo pelo peso." - Bill Gates

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - rjjj - 25.03.2018

No seu cуdigo de comando, troque static por new para que a variбvel CountMembers sempre comece zerada em execuзхes sucessivas do mesmo

.

Opcionalmente, vocк pode manter o static e apenas zerar essa variбvel logo abaixo de sua declaraзгo.

Espero ter ajudado

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - [BOPE]Seu._.Madruga - 25.03.2018

Quote:

Originally Posted by Dayvison_

Sу removendo a operaзгo ternбria no SendClientMessage.

Menos linhas nгo quer dizer que estб mais otimizado.
"Medir o progresso de um programa por linhas de cуdigo й como medir o processo de montagem de um aviгo pelo peso." - Bill Gates

Nгo vejo necessidade de retirar a operaзгo ternбria.

Quote:

Originally Posted by rjjj

No seu cуdigo de comando, troque static por new para que a variбvel CountMembers sempre comece zerada em execuзхes sucessivas do mesmo

.

Opcionalmente, vocк pode manter o static e apenas zerar essa variбvel logo abaixo de sua declaraзгo.

Espero ter ajudado

Certo anteriormente eu tinha utilizado como new porйm havia trocado para static mas vocк me fez lembrar de algo que eu havia esquecido, voltei para new novamente obrigado

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - Gguiz - 25.03.2018

Explique a sua lуgica de raciocнnio em utilizar a operaзгo ternбria. Lembrando que o "?" tem som de "й" e ":" tem som de "ou".

Ъnica coisa que vocк fez foi encher linguiзa na sua "otimizaзгo", e tornar seu cуdigo com uma pйssima legibilidade.

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - Dayvison_ - 25.03.2018

Com operaзгo ternбria vocк faz duas verificaзхes enquanto usando if seria apenas uma

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - [BOPE]Seu._.Madruga - 25.03.2018

Operaзгo Ternбria foi feita para ser utilizada encher de linguiзa й oque vocк estб fazendo aqui na sua publicaзгo nгo ajudando em nada no tуpico TKS e dispensбvel seu comentбrio

Quote:

Originally Posted by Gguiz

Explique a sua lуgica de raciocнnio em utilizar a operaзгo ternбria. Lembrando que o "?" tem som de "й" e ":" tem som de "ou".

Ъnica coisa que vocк fez foi encher linguiзa na sua "otimizaзгo", e tornar seu cуdigo com uma pйssima legibilidade.

Certo porйm com if seria duas linhas para fazer a mesma coisa correto ? Mostre-me uma outra forma melhor utilizando if

Quote:

Originally Posted by Dayvison_

Com operaзгo ternбria vocк faz duas verificaзхes enquanto usando if seria apenas uma

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - BrunoBM23 - 25.03.2018

Quote:

Originally Posted by [BOPE]Seu._.Madruga

Certo porйm com if seria duas linhas para fazer a mesma coisa correto ? Mostre-me uma outra forma melhor utilizando if

Otimizaзгo != nъmero de linhas

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - brunogusava - 25.03.2018

Agora atй me senti importante kkkkkk, faz tempo que estou sem PC, nem vejo a hora de voltar para continuar meus aprendizados em PAWN.

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - brunogusava - 25.03.2018

Aproveitando que estou no celular do meu tio, gostaria de mandar um salve ao meu mestre, @RazorGuigo, futuramente estarei voltando, sу esperando meu PC novo chegar.

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - JohnBlack - 25.03.2018

Alйm da recomendaзгo do Dayvison, eu tambйm modificaria a parte em que hб

Код:

if(... && i != playerid)

para

Код:

if(...) { if(i != playerid)

Com todas as adaptaзхes, claramente.

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - Whoo - 25.03.2018

Acredito que nesse caso operaзгo ternбria perder tanto para switch quanto para if, nesse caso if ganha em velocidade:

PHP код:



switch(CountMembers)

{

    case 0: SendClientMessage(0, -1, "[BLC] Fora vocк nгo hб nenhum(a) outro(a) membro(a) da sua organizaзгo estб online.");

    default: SendClientMessage(0, -1, "/----- Membros da Organizaзгo Online -----/");

}


if(CountMembers != 0) SendClientMessage(0, -1, "/----- Membros da Organizaзгo Online -----/");

else SendClientMessage(0, -1, "[BLC] Fora vocк nгo hб nenhum(a) outro(a) membro(a) da sua organizaзгo estб online.");

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - RogerCosta - 25.03.2018

Код:

(CountMembers != 0) && SendClientMessage(0, -1, "/----- Membros da Organizaзгo Online -----/") || SendClientMessage(0, -1, "[BLC] Fora vocк nгo hб nenhum(a) outro(a) membro(a) da sua organizaзгo estб online.");

Pra quem tб preocupado com a quantidade de linha, ai esta outra maneira de fazer a mesma merda. Й engraзado nego querendo corrigir o outro e nem saber programar direito. Como dito no discord: й o famoso professor que nгo entende do assunto UHAUHAU.

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - [BOPE]Seu._.Madruga - 25.03.2018

Certo entгo de fato era do jeito que eu tinha colocado anteriormente no tуpico dele, obrigado pela dica tmj

Quote:

Originally Posted by Whoo

Acredito que nesse caso operaзгo ternбria perder tanto para switch quanto para if, nesse caso if ganha em velocidade:

PHP код:



switch(CountMembers)

{

    case 0: SendClientMessage(0, -1, "[BLC] Fora vocк nгo hб nenhum(a) outro(a) membro(a) da sua organizaзгo estб online.");

    default: SendClientMessage(0, -1, "/----- Membros da Organizaзгo Online -----/");

}


if(CountMembers != 0) SendClientMessage(0, -1, "/----- Membros da Organizaзгo Online -----/");

else SendClientMessage(0, -1, "[BLC] Fora vocк nгo hб nenhum(a) outro(a) membro(a) da sua organizaзгo estб online.");

Editei pois nгo vou dar ibope para vocк e nгo vou relacionar coisas da minha vida pessoal com pessoas do seu tipo, vai que rola macumba

Quote:

Originally Posted by RogerCosta

Код:

(CountMembers != 0) && SendClientMessage(0, -1, "/----- Membros da Organizaзгo Online -----/") || SendClientMessage(0, -1, "[BLC] Fora vocк nгo hб nenhum(a) outro(a) membro(a) da sua organizaзгo estб online.");

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - renatog - 25.03.2018

O primeiro erro aн й usar uma sintaxe de foreach deprecated hб anos. A "nova" sintaxe й foreach (new i : Player)

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - Nizarh - 25.03.2018

Quote:

Originally Posted by brunogusava

Aproveitando que estou no celular do meu tio, gostaria de mandar um salve ao meu mestre, @RazorGuigo, futuramente estarei voltando, sу esperando meu PC novo chegar.

Jб agora, aproveita e vai ler as regras.

Код:

Nгo й permitido "Double Post" (Postagem dupla) antes de 24 horas. Existe um botгo chamado "Editar" .

Regras

Re: Й Possнvel Otimizar ainda mais este comando /membros ? - Adiiti - 26.03.2018

Como falei no meu tуpico e volto a falar aqui. Otimizaзгo != Organizaзгo. Isso й uma diminuiзгo de linhas e nгo otimizaзгo