FS Trucking

---

Čestitam na trudu.

Zašto koristiš č, ć, đ, š, ž u SendClientMessage ??

Greška:

Jedna od potencijalnih grešaka je u funkciji OnPlayerEnterDynamicCP, gde se proverava da li je checkpointid jednak PlayerCheckpoint. Ova provera može biti suvišna jer se već proverava u uslovu if (checkpointid == PlayerCheckpoint && PlayerTruckJob > 0).

Čitljivost na zadovoljavajućem nivou, trebalo bi da dodaš više komentara.
Obrada grešaka je takođe minimalna kao u prethodnoj FS za gate create.

Pitanje friendly:

Znaš li šta znači static i kako funkcioniše? Ona zadržava svoju vrednost između poziva f-ja. Kod tebe će znači name zadržati. Govorim o f-ji stock GetName(playerid). Ako više igrača poziva f-ju istovremeno, svi će dobiti istu vrednost imena, što garant nije ono što želiš?? Static ti nije thread safe da znaš. Ako se GetName pozove više puta, vrednost name će biti zamenjena.

Preporučujem ti sledeće:

stock GetName(playerid, name[], maxlen)
{
    GetPlayerName(playerid, name, maxlen);
}

Vidiš da koristim niz kao argument, znači ne treba mi return name.

Korišćenje:

new playerName;
GetName(playerid, playerName, sizeof(playerName));

Ovo će ti raditi ko Singer, slušaj Dragija.
SP

P.S. Ako hoćeš na tvoju vodenicu, evo ti sličan kod tvom.

stock GetName(playerid)
{
	new name;
	GetPlayerName(playerid, name, sizeof(name));
	return name;
}

Dragi wrote on November 27, 2024, 5:06 pm:

Čestitam na trudu.

Zašto koristiš č, ć, đ, š, ž u SendClientMessage ??

Greška:

Jedna od potencijalnih grešaka je u funkciji OnPlayerEnterDynamicCP, gde se proverava da li je checkpointid jednak PlayerCheckpoint. Ova provera može biti suvišna jer se već proverava u uslovu if (checkpointid == PlayerCheckpoint && PlayerTruckJob > 0).

Čitljivost na zadovoljavajućem nivou, trebalo bi da dodaš više komentara.
Obrada grešaka je takođe minimalna kao u prethodnoj FS za gate create.

Pitanje friendly:

Znaš li šta znači static i kako funkcioniše? Ona zadržava svoju vrednost između poziva f-ja. Kod tebe će znači name zadržati. Govorim o f-ji stock GetName(playerid). Ako više igrača poziva f-ju istovremeno, svi će dobiti istu vrednost imena, što garant nije ono što želiš?? Static ti nije thread safe da znaš. Ako se GetName pozove više puta, vrednost name će biti zamenjena.

Preporučujem ti sledeće:

stock GetName(playerid, name[], maxlen)
{
    GetPlayerName(playerid, name, maxlen);
}

Vidiš da koristim niz kao argument, znači ne treba mi return name.

Korišćenje:

new playerName;
GetName(playerid, playerName, sizeof(playerName));

Ovo će ti raditi ko Singer, slušaj Dragija.
SP

Hvala na izdvojenom vremenu pregledavanjem koda, nisam testirao InGame nista, kod je pravljen na suho iz glave i koristenjem znanja.
Nisam skriptao skoro 2 godine pa sam se malo zanio.
Ko se razumi u Pawn jezik znat ce i sam to rijesiti i prilagoditi svojoj skripti.
LP

kaspersky wrote on November 27, 2024, 5:11 pm:

Hvala na izdvojenom vremenu pregledavanjem koda, nisam testirao InGame nista, kod je pravljen na suho iz glave i koristenjem znanja.
Nisam skriptao skoro 2 godine pa sam se malo zanio.
Ko se razumi u Pawn jezik znat ce i sam to rijesiti i prilagoditi svojoj skripti.
LP

Taman posla. Razumem sve. Eto, uhvatio sam malo vremena frei. 😁 Kao što rekoh, svaka čast na trudu i nemoj da shvatiš moje kritike negativno. Drago mi je što si se potrudio i respect ako si tako radio. Ali kako je moja razredna govorila: "Uvek može bolje!". Ima štofa.

Dragi wrote on November 27, 2024, 5:14 pm:

Taman posla. Razumem sve. Eto, uhvatio sam malo vremena frei. 😁 Kao što rekoh, svaka čast na trudu i nemoj da shvatiš moje kritike negativno. Drago mi je što si se potrudio i respect ako si tako radio. Ali kako je moja razredna govorila: "Uvek može bolje!". Ima štofa.

moze bolje naravno 😄 radi optimizacije, string 'message' bi trebalo da ima 27 karaktera u komandi /trucker (new message;new message;).

𐌃 𐌄 𐌍 𐌌 𐌋 𐌀 wrote on November 27, 2024, 5:23 pm:

moze bolje naravno 😄 radi optimizacije, string 'message' bi trebalo da ima 27 karaktera u komandi /trucker (new message;new message;).

Ne videh to. U pravu si, ima dosta propusta, ali ako uzmemo u obzir da nije 2 godine programirao... Ne znam.

𐌃 𐌄 𐌍 𐌌 𐌋 𐌀 wrote on November 27, 2024, 5:23 pm:

moze bolje naravno 😄 radi optimizacije, string 'message' bi trebalo da ima 27 karaktera u komandi /trucker (new message;new message;).

Da bismo optimizovali kod u kontekstu promenljivih i optimizacije memorije, možeš koristiti new message; u komandi /trucker samo ako si siguran da je dužina poruke uvek 27 karaktera. U suprotnom, ako poruka može biti duža od 27 karaktera, ne bi trebalo smanjivati veličinu niza.

Dakle, ako je tvoje pitanje da optimizuješ kod, evo nekoliko objašnjenja i mogućih rešenja:

Koristi manju veličinu niza ako znaš da je dužina poruke konstantna (27 karaktera).
Ako nije jasno koliko će poruka biti dugačka (dinamična dužina), bolje je ostaviti veću veličinu niza (npr. message😉.
Evo kako bi to izgledalo u slučaju kada želiš da ograničiš veličinu niza na 27 karaktera, pretpostavljajući da komanda /trucker uvek šalje poruku sa 27 karaktera:

Optimized kod sa porukom od 27 karaktera:

CMD:trucker(playerid, params[])
{
    new message;  // Ovdje koristimo niz sa 27 karaktera

    // Postavite poruku (pretpostavljamo da je duzina poruke tacno 27 karaktera)
    strmid(message, "Poruka sa tacno 27 karaktera", 0, 27); // Primer poruke

    // Posaljite poruku igracu
    SendClientMessage(playerid, COLOR_GREEN, message);
    return 1;
}

Za dinamične poruke:
Ako dužina poruke nije fiksna, bolje je koristiti veći niz, kao što je message;, da bi mogao da pokriješ sve moguće slučajeve.

Dakle, ako znaš da poruka nikad neće biti duža od 27 karaktera, korišćenje new message; može biti dobro za optimizaciju memorije. Međutim, ako dužina varira, bolje je da koristiš message; ili neki drugi dovoljno veliki niz da bi bio siguran da se cela poruka može smestiti.

Da li to odgovara na tvoje pitanje o optimizaciji?

kaspersky wrote on November 27, 2024, 7:08 pm:

Da bismo optimizovali kod u kontekstu promenljivih i optimizacije memorije, možeš koristiti new message; u komandi /trucker samo ako si siguran da je dužina poruke uvek 27 karaktera. U suprotnom, ako poruka može biti duža od 27 karaktera, ne bi trebalo smanjivati veličinu niza.

Dakle, ako je tvoje pitanje da optimizuješ kod, evo nekoliko objašnjenja i mogućih rešenja:

Koristi manju veličinu niza ako znaš da je dužina poruke konstantna (27 karaktera).
Ako nije jasno koliko će poruka biti dugačka (dinamična dužina), bolje je ostaviti veću veličinu niza (npr. message😉.
Evo kako bi to izgledalo u slučaju kada želiš da ograničiš veličinu niza na 27 karaktera, pretpostavljajući da komanda /trucker uvek šalje poruku sa 27 karaktera:

Optimized kod sa porukom od 27 karaktera:

CMD:trucker(playerid, params[])
{
    new message;  // Ovdje koristimo niz sa 27 karaktera

    // Postavite poruku (pretpostavljamo da je duzina poruke tacno 27 karaktera)
    strmid(message, "Poruka sa tacno 27 karaktera", 0, 27); // Primer poruke

    // Posaljite poruku igracu
    SendClientMessage(playerid, COLOR_GREEN, message);
    return 1;
}

Za dinamične poruke:
Ako dužina poruke nije fiksna, bolje je koristiti veći niz, kao što je message;, da bi mogao da pokriješ sve moguće slučajeve.

Dakle, ako znaš da poruka nikad neće biti duža od 27 karaktera, korišćenje new message; može biti dobro za optimizaciju memorije. Međutim, ako dužina varira, bolje je da koristiš message; ili neki drugi dovoljno veliki niz da bi bio siguran da se cela poruka može smestiti.

Da li to odgovara na tvoje pitanje o optimizaciji?

naravno, upravu si, ali ja bih rekao isto da treba 128 karaktera ako bi negde u funkciji bilo potrebno 128 karaktera, dok je ovde dovoljno 27.

kaspersky wrote on November 27, 2024, 7:08 pm:

Da bismo optimizovali kod u kontekstu promenljivih i optimizacije memorije, možeš koristiti new message; u komandi /trucker samo ako si siguran da je dužina poruke uvek 27 karaktera. U suprotnom, ako poruka može biti duža od 27 karaktera, ne bi trebalo smanjivati veličinu niza.

Dakle, ako je tvoje pitanje da optimizuješ kod, evo nekoliko objašnjenja i mogućih rešenja:

Koristi manju veličinu niza ako znaš da je dužina poruke konstantna (27 karaktera).
Ako nije jasno koliko će poruka biti dugačka (dinamična dužina), bolje je ostaviti veću veličinu niza (npr. message😉.
Evo kako bi to izgledalo u slučaju kada želiš da ograničiš veličinu niza na 27 karaktera, pretpostavljajući da komanda /trucker uvek šalje poruku sa 27 karaktera:

Optimized kod sa porukom od 27 karaktera:

CMD:trucker(playerid, params[])
{
    new message;  // Ovdje koristimo niz sa 27 karaktera

    // Postavite poruku (pretpostavljamo da je duzina poruke tacno 27 karaktera)
    strmid(message, "Poruka sa tacno 27 karaktera", 0, 27); // Primer poruke

    // Posaljite poruku igracu
    SendClientMessage(playerid, COLOR_GREEN, message);
    return 1;
}

Za dinamične poruke:
Ako dužina poruke nije fiksna, bolje je koristiti veći niz, kao što je message;, da bi mogao da pokriješ sve moguće slučajeve.

Dakle, ako znaš da poruka nikad neće biti duža od 27 karaktera, korišćenje new message; može biti dobro za optimizaciju memorije. Međutim, ako dužina varira, bolje je da koristiš message; ili neki drugi dovoljno veliki niz da bi bio siguran da se cela poruka može smestiti.

Da li to odgovara na tvoje pitanje o optimizaciji?

Druze, sta ti pises? To ti je covek sve i rekao? Pises sta mozemo uraditi u mnozini, a sve ti receno? Gde ti duzina varira?

Quote

Dakle, ako je tvoje pitanje da optimizuješ kod, evo nekoliko objašnjenja i mogućih rešenja:

Kakvo crno pitanje, ko ti je postavio pitanje? Boze, Petra.

Koliko ti ovaj string sadrzi karaktera? "Izaberite rutu (1-%d):"  Za ovaj string dovoljno je 25 karaktera! Ukljucujuci tvoje rute i null terminator. Kakvo crno variranje stringa do 128 karaktera? Ako se bojis variranja, metni ga na 30 i mirna kuca. Sta blebeces, svega ti? Sta  ce ti strmid ovde? Reci mi iskreno, pises li ti sam ovo ili AI?