[brunogusava]

Galera estou fazendo um sistema de ORGs para o BETA do servidor em que estou ajudando a codar, porйm sou iniciante no Pawn e o erro/warning que estб dando aqui, nгo em vem na cabeзa uma soluзгo.

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-----/");
            }
        }
    }
} 


PHP код:

..\modulos\LoginRegistro.pwn(227) : error 021: symbol already defined: "i"
..\modulos\LoginRegistro.pwn(227) : warning 204: symbol is assigned a value that is never used: "i" 


Para vocкs se localizarem melhor estб aн uma imagem do cуdigo:

[Cycle]

PHP код:

CMD:membros(playerid) 
{ 
    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-----/"); 
            } 
        } 
    }
    return true;
} 


[FerrariL]

Uma observaзгo,isso aqui nгo vai funcionar como esperado :

PHP код:

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


Modo correto :

PHP код:

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


[n0minal]

Provavelmente vocк ou algum outro novato do seu time criaram uma variбvel global chamada "i", vai dar esse erro em todas as funзхes que vocк criar uma variбvel com nome "i", sugiro que revise seu cуdigo e elimine se possнvel essa variбvel global, ou renomeie a variбvel do loop (nгo й uma boa prбtica deixar uma variбvel i global, й pйssimo na verdade).

[brunogusava]

Quote:

Originally Posted by FerrariL Uma observaзгo,isso aqui nгo vai funcionar como esperado : PHP код:   if(Player[playerid][Organizacao] == 0)      {          SendClientMessage(playerid, 0xFF0000FF, "[BLC] |Erro| Vocк nгo й de uma organizaзгo.");      }  Modo correto : PHP код:   if(Player[playerid][Organizacao] == 0)    return SendClientMessage(playerid, 0xFF0000FF, "[BLC] |Erro| Vocк nгo й de uma organizaзгo.");

Tipo assim?

PHP код:

CMD:membros(playerid) 
{ 
    if(Player[playerid][Organizacao] == 0) 
     return SendClientMessage(playerid, 0xFF0000FF, "[BLC] |Erro| Vocк nгo й de uma organizaзгo."); 
        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-----/"); 
            } 
        } 
    }
    return true;
} 


---------------------------------------------------------------------------------

Quote:

Originally Posted by n0minal Provavelmente vocк ou algum outro novato do seu time criaram uma variбvel global chamada "i", vai dar esse erro em todas as funзхes que vocк criar uma variбvel com nome "i", sugiro que revise seu cуdigo e elimine se possнvel essa variбvel global, ou renomeie a variбvel do loop (nгo й uma boa prбtica deixar uma variбvel i global, й pйssimo na verdade).

Pior que nгo n0minal, nгo existe nenhuma variбvel "i" global, inclusive й a primeira vez que declaro ela, tentei mudar para outras letras, mas persistiu o erro e o warning, mas assim que chegar em casa vou tentar botar o "return true" antes da ъltima chave, igual o mano Cycle mandou ali em cima no post.

Mas realmente й algo que faria sentido, valeu pela ajuda, mas acredito que seja sу a falta de um "return" mesmo.

[n0minal]

Ah sim, jб vi o erro, vocк tб usando vнrgulas no loop ao invйs de ponto e vнrgula, o correto seria:

Код:

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

[brunogusava]

Quote:

Originally Posted by n0minal Ah sim, jб vi o erro, vocк tб usando vнrgulas no loop ao invйs de ponto e vнrgula, o correto seria: Код: for(new i = 0; i < MAX_PLAYERS; i++)

Nossaaaaaa mano, como que eu nгo vi KKKKKKKKK, na programaзгo tem dessas, por um simples ";" da erro na metade do cуdigo kkkkkkkkk, valeu mano, vc e o Cycle ganharam Rep

[[BOPE]Seu._.Madruga]

Adicionei a funзгo para informar se nгo tem nenhum membro da organizaзгo online e jб aproveitei e dei uma otimizada no cуdigo e na forma que estava sendo chamada as mensagens.

PHP код:

CMD:membros(playerid) {
    if(Player[playerid][Organizacao] == 0) return SendClientMessage(playerid, 0xFF0000FF, "[BLC] |Erro| Vocк nгo й de uma organizaзгo.");
    new CountMembers = 0;
    SendClientMessage(playerid, -1, "/-----Membros da Organizaзгo Online -----/");
    for(new i = 0; i < MAX_PLAYERS; i++) {
        if(Player[i][Organizacao] == Player[playerid][Organizacao]) {
            new nome[MAX_PLAYER_NAME], string[MAX_PLAYER_NAME];
            GetPlayerName(i, nome, sizeof(nome));
            format(string, sizeof(string), "%s", nome);
            SendClientMessage(playerid, -1, string);
            CountMembers++;
        }
    }
    if(CountMembers >= 1) return SendClientMessage(playerid, -1, "/-----Membros da Organizaзгo Online -----/");
    if(CountMembers == 0) return SendClientMessage(playerid, 0xFF0000FF, "[BLC] Nenhum membro da sua organizaзгo estб online.");
    return true;
} 


[brunogusava]

Quote:

Originally Posted by [BOPE]Seu._.Madruga Adicionei a funзгo para informar se nгo tem nenhum membro da organizaзгo online e jб aproveitei e dei uma otimizada no cуdigo e na forma que estava sendo chamada as mensagens. PHP код: CMD:membros(playerid) {     if(Player[playerid][Organizacao] == 0) return SendClientMessage(playerid, 0xFF0000FF, "[BLC] |Erro| Vocк nгo й de uma organizaзгo.");     new CountMembers = 0;     SendClientMessage(playerid, -1, "/-----Membros da Organizaзгo Online -----/");     for(new i = 0; i < MAX_PLAYERS; i++) {         if(Player[i][Organizacao] == Player[playerid][Organizacao]) {             new nome[MAX_PLAYER_NAME], string[MAX_PLAYER_NAME];             GetPlayerName(i, nome, sizeof(nome));             format(string, sizeof(string), "%s", nome);             SendClientMessage(playerid, -1, string);             CountMembers++;         }     }     if(CountMembers >= 1) return SendClientMessage(playerid, -1, "/-----Membros da Organizaзгo Online -----/");     if(CountMembers == 0) return SendClientMessage(playerid, 0xFF0000FF, "[BLC] Nenhum membro da sua organizaзгo estб online.");     return true; }

Caraca, valeu manoo, +REP !!!!

[[BOPE]Seu._.Madruga]

Quote:

Originally Posted by [BOPE]Seu._.Madruga Adicionei a funзгo para informar se nгo tem nenhum membro da organizaзгo online e jб aproveitei e dei uma otimizada no cуdigo e na forma que estava sendo chamada as mensagens. PHP код: CMD:membros(playerid) {     if(Player[playerid][Organizacao] == 0) return SendClientMessage(playerid, 0xFF0000FF, "[BLC] |Erro| Vocк nгo й de uma organizaзгo.");     new CountMembers = 0;     SendClientMessage(playerid, -1, "/-----Membros da Organizaзгo Online -----/");     for(new i = 0; i < MAX_PLAYERS; i++) {         if(Player[i][Organizacao] == Player[playerid][Organizacao]) {             new nome[MAX_PLAYER_NAME], string[MAX_PLAYER_NAME];             GetPlayerName(i, nome, sizeof(nome));             format(string, sizeof(string), "%s", nome);             SendClientMessage(playerid, -1, string);             CountMembers++;         }     }     if(CountMembers >= 1) return SendClientMessage(playerid, -1, "/-----Membros da Organizaзгo Online -----/");     if(CountMembers == 0) return SendClientMessage(playerid, 0xFF0000FF, "[BLC] Nenhum membro da sua organizaзгo estб online.");     return true; }

Apenas mais uma otimizada bбsica

PHP код:

CMD:membros(playerid) {
    if(!Player[playerid][Organizacao]) return SendClientMessage(playerid, 0xFF0000FF, "[BLC] Vocк nгo й de uma organizaзгo.");
    static CountMembers = 0, 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 true;
}