Dinamicni rent sistem (MySQL)

Tip: FS
Slike ili Video: https://youtu.be/QY_0SMDYOIU
Opis i dodatne informacije:

/rentcreate - Kreirate rent pickup
/rentremove - Brisete rent pickup
/unrent - Vracate iznajmljeno vozilo
Slovo F - Iznajmljujete vozilo

Download link: https://pastebin.com/riR35JAh - FS: https://www.sendspace.com/file/t8wt3q  SQL file: https://www.sendspace.com/file/q6uo25Link
Scan link: https://www.virustotal.com/gui/file/46d792222cc37eb41bd6dbd4b5eda94441c6e49122cca55425c36dfbdd968aa5/detection
Credits: __vedran.xyz77

#bokenzi wrote on July 26, 2021, 1:28 pm:

Tip: FS
Slike ili Video: https://youtu.be/QY_0SMDYOIU
Opis i dodatne informacije:

/rentcreate - Kreirate rent pickup
/rentremove - Brisete rent pickup
/unrent - Vracate iznajmljeno vozilo
Slovo F - Iznajmljujete vozilo

Download link: https://pastebin.com/riR35JAh - FS: https://www.sendspace.com/file/t8wt3q  SQL file: https://www.sendspace.com/file/q6uo25Link
Scan link: https://www.virustotal.com/gui/file/46d792222cc37eb41bd6dbd4b5eda94441c6e49122cca55425c36dfbdd968aa5/detection
Credits: __vedran.xyz77

Cool je ja sam uradio slicno samo sto ja imam /createrent /removerent /unrent i /rent sa text drawovima i /editrent da promenim kordinate tog renta ili ti kratko receno da ga Premestim na novu lokaciju. svidja mi se samo sto bi mogao malo dizajna urednije da bude dosta 6-7

Goku ^ wrote on July 26, 2021, 1:31 pm:

Cool je ja sam uradio slicno samo sto ja imam /createrent /removerent /unrent i /rent sa text drawovima i /editrent da promenim kordinate tog renta ili ti kratko receno da ga Premestim na novu lokaciju. svidja mi se samo sto bi mogao malo dizajna urednije da bude dosta 6-7

Ma vec sam radio sa textdrawovima da se bira pomocu strijelica i tak neke gluposti ali stvarno nemam vise volje ovo sam odradio za 4 sata jedva cekam da zadje sunce da izadjem van 😄

#bokenzi wrote on July 26, 2021, 1:32 pm:

Ma vec sam radio sa textdrawovima da se bira pomocu strijelica i tak neke gluposti ali stvarno nemam vise volje ovo sam odradio za 4 sata jedva cekam da zadje sunce da izadjem van 😄

Ju buraz meni treba da napravim tdove i komandu /createrent prodjose 3 sata. Meni je trebalo za 4 vrstu (Motori, bicikle i auta u 1 rent) mi je trebalo sat vremena mozda sat ipo

dve varijable ti imaju istu vrednost, 'rentVehID' i 'rentedVehicle'

linija 578,598,622 umesto switch-a uradis samo 'rentChoose = listitem + 1;'

dalje nisam ni gledao

Yeah za rentChoose = listitem + 1; sam kasnije saznao da je laksi nacin al dobro hvala sto si uocio budem si popravio

Može bolje naravno. Ne treba da koristiš Pawn.CMD i dodatni plugin/inc kada imas y_commands, također y_iterate i y_timers. Ova define sranja na pocetku ukloni, bezveze.

Sve u svemu, neloše.

Ne vidim potrebu koriscenja komadnog procesora y_commands kad je pawncmd puno brzi, takodjer za taj mali sistemcic uopce mi nije potrebno ukljucivati ysi i sve silne potrebne fajlove za iteratore i timere kad sve u svemu imam samo jedan timer koji se poziva.

A ti defineovi i te boje sam izvuko usput iz moda, kome smeta sklonit ce

Top, svaka cast za trud  😎

#bokenzi wrote on July 26, 2021, 5:54 pm:

Ne vidim potrebu koriscenja komadnog procesora y_commands kad je pawncmd puno brzi, takodjer za taj mali sistemcic uopce mi nije potrebno ukljucivati ysi i sve silne potrebne fajlove za iteratore i timere kad sve u svemu imam samo jedan timer koji se poziva.

A ti defineovi i te boje sam izvuko usput iz moda, kome smeta sklonit ce

Da ti skripta ima tisuću pa na više čistih CMD linija onda se ne bih smijao na to da ga koristiš . Svi misle da je pawn.cmd brzi samo zato jer je netko prikazao neki grafikon gdje pise da se sve izvrsava za 21ms, to nije dokazano nikakvim kodom a ni primjerom na koliko se komanda izvrsavalo. Ne koristiš nikakve feature od Pawn.CMD i to je još jedna besmislenost.

Hydra. wrote on July 27, 2021, 2:51 pm:

Da ti skripta ima tisuću pa na više čistih CMD linija onda se ne bih smijao na to da ga koristiš . Svi misle da je pawn.cmd brzi samo zato jer je netko prikazao neki grafikon gdje pise da se sve izvrsava za 21ms, to nije dokazano nikakvim kodom a ni primjerom na koliko se komanda izvrsavalo. Ne koristiš nikakve feature od Pawn.CMD i to je još jedna besmislenost.

"Da ti skripta ima tisuću" - Ali nema, ja sam navikao i koristim vec dugo pawncmd, sistem je dostupan svima i svako ga moze preuzeti i svako s njime moze raditi sta zeli, neko tko voli pawncmd ostavit ce tako, neko ko vise voli ycmd prepravit ce u ycmd ili zcmd nebitno.

Pa i dosta njih sto koriste y_commands ne koriste sve njegove mogucnosti nego iskljucivo procesovanje komandi.
Ne vidim razlog u cepidlacenju ko sta koristi, a pogotovo kada je u pitanju komandni procesor - apsolutno je nebitno.

Hydra. wrote on July 27, 2021, 2:51 pm:

Da ti skripta ima tisu%u0107u pa na vi%u0161e %u010Distih CMD linija onda se ne bih smijao na to da ga koristi%u0161 . Svi misle da je pawn.cmd brzi samo zato jer je netko prikazao neki grafikon gdje pise da se sve izvrsava za 21ms, to nije dokazano nikakvim kodom a ni primjerom na koliko se komanda izvrsavalo. Ne koristi%u0161 nikakve feature od Pawn.CMD i to je jo%u0161 jedna besmislenost.

A sta je mogao da iskoristi od ycmd-a objasni nam zivo me zanima 

Bolesno je sto forsirate i tjerate ljude da koriste YSI ako ne zele.
YSI ne implementira nove funkcije sampa, koriste default funkcije od sampa, samo na neki drugi nacin.

Neću ići kroz ceo kod, samo bejzik greške ima da izdvojim:

---

Imaš

rentLabel

tako da je ovo bespotrebno:

rentVehID

---

rentTime

Tek ovo što nema smisla, zašto ti treba varijabla ovde i to globalna sa MAX_PLAYERS mesta? 😕
Bukvalno samo uradiš

new val = strval(inputtext);

umesto

rentTime = strval(inputtext);

---

nextRentID()
{
    new id = -1;
 
    for(new i = 0; i < maxRents; i++)
    {
        if(rentInfo == 0.0 && rentInfo == 0.0)
        {
            id = i;
            break;
        }
    }
    return id;
}

Da ne bude da te silujem da koristiš YSI - imaš standalone verziju foreach-a pa koristi..

---

findNearestRent(const playerid)
{
    for(new i = 0; i < maxRents; i++)
    {
        if(IsPlayerInRangeOfPoint(playerid, 1.5, rentInfo, rentInfo, rentInfo))
            return i;
    }
 
    return -1;
}

Ovo nije greška, samo savet: Koristi dinamične aree, mislim da ti je lakše tako 🙂

---

new i = getNearestRentStyle(playerid);

Ovde imaš grešku, nemaš proveru u slučaju da igrač nije blizu ni jednog renta te će varijabla vratiti -1
Znači

new i = getNearestRentStyle(playerid);
if(i == -1)
    return 0;

---

mysql_format(sql, query, sizeof query, "INSERT INTO rents (rent_id, rent_style, rent_x, rent_y, rent_z)\
                                            VALUES ('%d', '%d', '%f', '%f', '%f')",\
                                            rentInfo, rentInfo, rentInfo, rentInfo, rentInfo);

Nemoj da unosiš ID nečega ručno u bazu ako već imaš AUTO INCREMENT.

---

rentVehID = CreateVehicle(510, x, y, z, 0, 0, 0, 0, 0);
 
PutPlayerInVehicle(playerid, rentVehID, 0);
rentedVehicle = GetPlayerVehicleID(playerid);

Tek ovo Što nema smisla, čemu 2 potpuno iste varijable?

---

Ne bih išao dalje, i ovo je previše. oči mi krvare

Hydra. wrote on July 27, 2021, 2:51 pm:

Da ti skripta ima tisuću pa na više čistih CMD linija onda se ne bih smijao na to da ga koristiš . Svi misle da je pawn.cmd brzi samo zato jer je netko prikazao neki grafikon gdje pise da se sve izvrsava za 21ms, to nije dokazano nikakvim kodom a ni primjerom na koliko se komanda izvrsavalo. Ne koristiš nikakve feature od Pawn.CMD i to je još jedna besmislenost.

Sta autor plugina ima od toga da fakeuje grafikon? Grafikon je potpuno tacan. Jedini feature koji PawnCMD nudi je flag, za razliku od YSI kojeg mozes uvezati sa grupama (koje mozes uvezati dalje sa ostalim includeovima).
Ako ces se voditi logikom - ne koristis neki feature plugina - ne koristi plugin onda neka koristi momak OnPlayerCommandText sasvim lijepo ce raditi.

@Topic Sage ti je naveo dosta toga sto mozes da popravis. Ovo mijesanje lower camel case i upper camel case je ruzno, pokusavaj da to izbjegavas i da funkcije pises velikim prvim slovom.