GTAONLINE.PL :: Pętla - ogarnięcie spamu wiadomościami

Zaloguj się by uzyskać pełen dostęp. Nie masz jeszcze konta? Założ je już teraz w kilka sekund.

Cze??, napisa?em sobie prosty system ?owienia ryb i chcia?em zrobi? "wykrywanie" czy gracz jest przy ?owisku, wi?c napisa?em sobie taki kodzik u?ywaj?c p?tli

Pawn

CMD:low(playerid,params[])
{
    for(new o;o!=sizeof(LowiskaPos);o++)
    {
        if(IsPlayerInRangeOfPoint(playerid, 2.0, LowiskaPos[o][0], LowiskaPos[o][1], LowiskaPos[o][2]))
        {
            if(!MaWedke[playerid]) return SendPlayerMessage(playerid,COLOR_ERROR,"Nie masz w?dki! Aby kupi? w?dke u?yj komendy {b}/wedka{/b}");
            if(IloscRyb[playerid] >= MAX_RYB) return SendPlayerMessage(playerid,COLOR_ERROR,"Nie mo?esz z?owi? wi?cej ni? %d ryb. Udaj si? do starego rybaka i sprzedaj je. {b}/rybak{/b}", MAX_RYB);
            ApplyAnimation(playerid,"SWORD","sword_block",50.0 ,0,1,0,1,1);
            SetPlayerAttachedObject(playerid, 0,18632,6,0.079376,0.037070,0.007706,181.482910,0.000000,0.000000,1.000000,1.000000,1.000000);
            SetTimerEx("GraczLowi",10000,false,"i",playerid);
        }
        else
        {
            SendPlayerMessage(playerid, COLOR_ERROR, "Nie jeste? przy ?owisku!"
        }
    }
    return 1;
}

Wszystko pi?kne dzia?a, tylko ?e gdy gracz nie jest przy ?owisku to spamuje informacj?, ?e gracz nie jest przy ?owisku. Da si? to jako? ogarn???

Gracz SA:MP - 2010-2019
Support Team - 26.11.2017 - 16.01.2018

"NitroFuN" napisał/a:

Cze??, napisa?em sobie prosty system ?owienia ryb i chcia?em zrobi? "wykrywanie" czy gracz jest przy ?owisku, wi?c napisa?em sobie taki kodzik u?ywaj?c p?tli

Pawn

else
        {
            SendPlayerMessage(playerid, COLOR_ERROR, "Nie jeste? przy ?owisku!"
        }
    }
    return 1;
}

Z tego co widz? brakuje nawiasu zamykaj?cego w funkcji SendPlayerMessage.

A wi?c:

Kod:

SendPlayerMessage(playerid, COLOR_ERROR, "Nie jeste? przy ?owisku!");

Pawn

CMD:low(playerid,params[])
{
    if(!MaWedke[playerid]) return SendPlayerMessage(playerid,COLOR_ERROR,"Nie masz w?dki! Aby kupi? w?dke u?yj komendy {b}/wedka{/b}");
    if(IloscRyb[playerid] >= MAX_RYB) return SendPlayerMessage(playerid,COLOR_ERROR,"Nie mo?esz z?owi? wi?cej ni? %d ryb. Udaj si? do starego rybaka i sprzedaj je. {b}/rybak{/b}", MAX_RYB);
    new InPos;
    for(new o = sizeof(LowiskaPos); o >= 0; o--)
    {
        if(IsPlayerInRangeOfPoint(playerid, 2.0, LowiskaPos[o][0], LowiskaPos[o][1], LowiskaPos[o][2]))
        {
            InPos = 1;
            ApplyAnimation(playerid,"SWORD","sword_block",50.0 ,0,1,0,1,1);
            SetPlayerAttachedObject(playerid, 0,18632,6,0.079376,0.037070,0.007706,181.482910,0.000000,0.000000,1.000000,1.000000,1.000000);
            SetTimerEx("GraczLowi",10000,false,"i",playerid);
            break;
        }
    }
    if(InPos == 0) return SendPlayerMessage(playerid, COLOR_ERROR, "Nie jeste? przy ?owisku!");
    return 1;
}

Przerobi?em Tw?j kod, poniewa? gdyby? da? break; w else po wiadomo?ci to p?tla przy pierwszym nie znalezieniu pozycji przerwa?aby si?.

Lepiej aby? te warunki od sprawdzenia czy ma w?dk? oraz ile ma ryb da? przed p?tl? tak nie b?dzie p?tla wykonywa?a si? po to aby tylko te warunki sprawdzi? i wy?wietli? wiadomo?? gdy kt?ry? si? spe?ni. Zawsze to jaki? tam delikatny plusik do optymalizacji.

Ups, ma?e niedopatrzenie

Ale nie o to chodzi, jak jestem przy ?owisku lub przy nim nie jestem to spamuje mi informacj? "Nie jeste? przy ?owisku!" i chcia?bym to jako? zabezpieczy?

[ Dodano: 2017-12-30, 15:42 ]
WWilk1, dzi?ki za pomoc, tw?j kod ?miga

Gracz SA:MP - 2010-2019
Support Team - 26.11.2017 - 16.01.2018