helypazarlásra fókuszálj, ha kérhetem, mert az else függvény fölös.. mindet lefuttatod, ha pl az utolsó frakcióban van, valamint ezzel nem mutatsz modot amit megnézhetnek, h \"valóban létezik\". (AMX fájl demonstrálóra gondolok)
ezt meg úgy is ki tudod íratni, hogy
if(FrakcioInfo[playerid][jatekosfrakcio] == 0)
Csak azért mondom ezt neked, mert ha ezekre odafigyelsz, a szervert fölös számításoktól kíméled meg. (apróságnak tűnik és szőrszálhasogatásnak, de sok kicsi felgyűlik egyszer)
valamint a hatékonyabbság érdekében a switch függvényt ajánlom ilyen helyzetekben.
Nem kukacoskodásképpen, de.
Az else az nem egy függvény, hanem egy elágazás. A felhasználó PAWN ismeretei valóban végesek, megoldások és megvalósítások gyanánt, egy bölcs Senior fejlesztő mondta nekem egyszer, ha egy if-hez nem tartozik else ág, akkor nincs minden esetre gondolva. Nyilván érdemesebb lett volna, performancia gyanánt switch, case-be wrappolni az if halmazt, és ott default-ként kezelni az else ágban megtalálható visszatérést, de ahány \" fejlesztő \", annyi megoldás.
Ha adhatok egy pár tippet, dolgozd át az enumerátorodban megtalálható struktúrát, legyenek azonos, maguktól értetendő változó neveid, és a magyar változó / metódus neveket felejtsd el, senki nem csinál ilyet nagyon, főleg céges környezetben nem. (Csak egy tipp ha a jövőben esetleg programozásban szeretnél elhelyezkedni)
Amúgy korrekt kis kód.
