[Pomoc] Oglas

Skripta koju koristim: Od 0
Detaljan opis problema: Ima dva problema, Kada dam oglas nikako ga ne izbaci, i kada jednom dam oglas odma mi izbrise mob -.-.
Dio skripte: Dole
Neke slike/video za lakse dobivanje pomoci(neobavezno): http://prntscr.com/22jiip , http://prntscr.com/22jizc

COMMAND:oglas(playerid, params[])
{
    new result,string;
	if(sscanf(params, "s", result)) return SCM(playerid,-1,"{009DFF}----- {FFFFFF}Koristite /oglas za davanje oglasa");
	if(GetPlayerMoney(playerid) <  20) return SCM(playerid,-1,"{009DFF}------ {FFFFFF}Nemas dovoljno Novca");
    if(PlayerInfo == 0) return SCM(playerid, COLOR_WHITE, "{009DFF}------ {FFFFFF}Nemas telefon");
    if(gettime() < toglas) return SCM(playerid, COLOR_WHITE, "{009DFF}------- {FFFFFF}Oglas se moze koristiti svakih 40 sekundi!");
    GetPlayerMoney(playerid, -13); toglas = gettime() + 40;
	format(string, sizeof(string), "{48E31C}Oglasavanje | Autor: %s{48E31C} | {F3FF02}Kontakt broj: %d",GetPlayerName(playerid),PlayerInfo);
	SendClientMessageToAll(COLOR_WHITE,string);
	format(string, sizeof(string), "%s.",  result);
	SendClientMessageToAll(COLOR_WHITE,string);
	GameTextForPlayer(playerid,"~w~Uspjesno ste dali oglas za ~r~13$!",5000,3);
    return 1;
}

Quote from November 7, 2013, 5:34 pm:

COMMAND:oglas(playerid, params[])
{
    new result,string;
	if(sscanf(params, "s", result)) return SCM(playerid,-1,"{009DFF}----- {FFFFFF}Koristite /oglas za davanje oglasa");
	if(GetPlayerMoney(playerid) <  20) return SCM(playerid,-1,"{009DFF}------ {FFFFFF}Nemas dovoljno Novca");
    if(PlayerInfo == 0) return SCM(playerid, COLOR_WHITE, "{009DFF}------ {FFFFFF}Nemas telefon");
    if(gettime() < toglas) return SCM(playerid, COLOR_WHITE, "{009DFF}------- {FFFFFF}Oglas se moze koristiti svakih 40 sekundi!");
    GetPlayerMoney(playerid, -13); toglas = gettime() + 40;
	format(string, sizeof(string), "{48E31C}Oglasavanje | Autor: %s{48E31C} | {F3FF02}Kontakt broj: %d",GetPlayerName(playerid),PlayerInfo);
	SendClientMessageToAll(COLOR_WHITE,string);
	GameTextForPlayer(playerid,"~w~Uspjesno ste dali oglas za ~r~13$!",5000,3);
    return 1;
}

Ja kako vidim ti nisi nista uradio u vezi telefona, a izbrisao si samo ono gdje ide text oglasa -.-

Hitmanâ„¢ wrote on November 7, 2013, 5:44 pm:

Probaj moju,ali nisam siguran.

Sta ti je ovo O.o

Quote

SendClientToMessage

COMMAND:oglas(playerid, params[])
{
    new string;
	if(isnull(params)) return SCM(playerid,-1,"{009DFF}----- {FFFFFF}Koristite /oglas za davanje oglasa");
	if(GetPlayerMoney(playerid) <  20) return SCM(playerid,-1,"{009DFF}------ {FFFFFF}Nemas dovoljno Novca");
    if(PlayerInfo == 0) return SCM(playerid, COLOR_WHITE, "{009DFF}------ {FFFFFF}Nemas telefon");
    if(gettime() < toglas) return SCM(playerid, COLOR_WHITE, "{009DFF}------- {FFFFFF}Oglas se moze koristiti svakih 40 sekundi!");
    GetPlayerMoney(playerid, -13); 
	new ime;
	GetPlayerName(playerid,ime,24);
	toglas = gettime() + 40;
	format(string, sizeof(string), "{48E31C}Oglasavanje | Autor: %s{48E31C} | {F3FF02}Kontakt broj: %d",ime,PlayerInfo);
	SendClientMessageToAll(COLOR_WHITE,string);
	format(string, sizeof(string), "%s.",  params);
	SendClientMessageToAll(COLOR_WHITE,string);
	GameTextForPlayer(playerid,"~w~Uspjesno ste dali oglas za ~r~13$!",5000,3);
    return 1;
}

Probaj nesto tako ako nista od onog ne bude radilo, iako ovo moje dodje slican fazon ko od nekoga gore ali probaj...

Quote from November 7, 2013, 5:34 pm:

COMMAND:oglas(playerid, params[])
{
    new result,string;
	if(sscanf(params, "s", result)) return SCM(playerid,-1,"{009DFF}----- {FFFFFF}Koristite /oglas za davanje oglasa");
	if(GetPlayerMoney(playerid) <  20) return SCM(playerid,-1,"{009DFF}------ {FFFFFF}Nemas dovoljno Novca");
    if(PlayerInfo == 0) return SCM(playerid, COLOR_WHITE, "{009DFF}------ {FFFFFF}Nemas telefon");
    if(gettime() < toglas) return SCM(playerid, COLOR_WHITE, "{009DFF}------- {FFFFFF}Oglas se moze koristiti svakih 40 sekundi!");
    GetPlayerMoney(playerid, -13); toglas = gettime() + 40;
	format(string, sizeof(string), "{48E31C}Oglasavanje | Autor: %s{48E31C} | {F3FF02}Kontakt broj: %d",GetPlayerName(playerid),PlayerInfo);
	SendClientMessageToAll(COLOR_WHITE,string);
	GameTextForPlayer(playerid,"~w~Uspjesno ste dali oglas za ~r~13$!",5000,3);
    return 1;
}

Ja kako vidim ti nisi nista uradio u vezi telefona, a izbrisao si samo ono gdje ide text oglasa -.-
Ja kolko vidim netrebam nista uradit jel ti se tu ne gubi telefon joj

Dobro a zasto si izbrisao ono O.o

Hitmanâ„¢ wrote on November 7, 2013, 5:53 pm:

ma zajebao se,to bi trebalo biti SendClientMessageToAll  😳

Znam 😄

Sry za spamm, Gagi radi ova tvoja komanda 😄 hvala ti.
Samo bih da pitam kako mogu dodati timer ovdje ? npr da odbrojava koliko vremena je ostalo do sledeceg oglasa ? npr kada upisem /oglas ... vrijeme do sledeceg oglasa 15 sec, i opet kad napisem da bude 12 sec i tako 😄 ?

Umesto:

if(gettime() < toglas) return SCM(playerid, COLOR_WHITE, "{009DFF}------- {FFFFFF}Oglas se moze koristiti svakih 40 sekundi!");

Ide:

if(gettime() < toglas) 
{
	SCM(playerid, COLOR_WHITE, "{009DFF}------- {FFFFFF}Oglas se moze koristiti svakih 40 sekundi!");
	new time = tooglas - gettime();//Mislim da tako ide nisam siguran mozda gresim oduzimam vreme koje treba da bude od trenutnog i time mislim da ce dobiti odredjeno vreme koje ostaje do isticanja
        //Ako je to iznad ok nisam siguran kako radi gettime ali videces vec...
	format(string,sizeof(string),"Vreme do sledeceg oglasa: %d",time);
	SCM(playerid,-1,string);
	return 1;
}

Ahhh problem 😄 http://prntscr.com/22k37d

BIRSA wrote on November 7, 2013, 6:34 pm:

Ahhh problem 😄 http://prntscr.com/22k37d

E jbg, kao sto sam i ocekivao ne radi kako treba 😄

//Promenljive
new VremeO, novoglas;
//Kada das oglas:

VremeO = SetTimer("VremeOglas",1000,true);
novoglas = 40;

//Ona provera:
if(gettime() < toglas) 
{
	SCM(playerid, COLOR_WHITE, "{009DFF}------- {FFFFFF}Oglas se moze koristiti svakih 40 sekundi!");
	format(string,sizeof(string),"Vreme do sledeceg oglasa: %d",novoglas);
	SCM(playerid,-1,string);
	return 1;
}

//Timer
forward VremeOglas();
public VremeOglas()
{
	novoglas--;
	if(novoglas == 0) KillTimer(VremeO);
	return 1;
}

Gangula a.k.a Mr.Gagi wrote on November 7, 2013, 6:34 pm:

Ahhh problem 😄 http://prntscr.com/22k37d

E jbg, kao sto sam i ocekivao ne radi kako treba 😄

//Promenljive
new VremeO, novoglas;
//Kada das oglas:

VremeO = SetTimer("VremeOglas",1000,true);
novoglas = 40;

//Ona provera:
if(gettime() < toglas) 
{
	SCM(playerid, COLOR_WHITE, "{009DFF}------- {FFFFFF}Oglas se moze koristiti svakih 40 sekundi!");
	format(string,sizeof(string),"Vreme do sledeceg oglasa: %d",novoglas);
	SCM(playerid,-1,string);
	return 1;
}

//Timer
forward VremeOglas();
public VremeOglas()
{
	novoglas--;
	if(novoglas == 0) KillTimer(VremeO);
	return 1;
}

Koliko me oči služe:

if(gettime() < toglas) 
{
	SCM(playerid, COLOR_WHITE, "{009DFF}------- {FFFFFF}Oglas se moze koristiti svakih 40 sekundi!");
	new time = tooglas - gettime();//Mislim da tako ide nisam siguran mozda gresim oduzimam vreme koje treba da bude od trenutnog i time mislim da ce dobiti odredjeno vreme koje ostaje do isticanja
        //Ako je to iznad ok nisam siguran kako radi gettime ali videces vec...
	format(string,sizeof(string),"Vreme do sledeceg oglasa: %d",time);
	SCM(playerid,-1,string);
	return 1;
}

u provjeri koristiÅ¡  "toglas"

if(gettime() < toglas) 

dok na mjestu gddje poruku izbacuje "tooglas"

new time = tooglas - gettime();

što u suštini izbacuje "-gettime()" u poruci jer "tooglas" ne koristiš nigdje u kodu i tako je tooglas = -1/0

Cekaj jel ovo trebam staviti pod OnGameModeInit😕
Ja stavio i izbaci mi error http://prntscr.com/22kbv5

Moja preporuka jeste da koristiš prvu soluciju gagija sa fixom (rekao sam gdje je prob i evo fix na ovom mjestu)

if(gettime() < toglas) 
{
	SCM(playerid, COLOR_WHITE, "{009DFF}------- {FFFFFF}Oglas se moze koristiti svakih 40 sekundi!");
	new time = toglas - gettime();
	format(string,sizeof(string),"Vreme do sledeceg oglasa: %d",time);
	SCM(playerid,-1,string);
	return 1;
}

Hvala, sad radi.
btw. sto mi ovdje izbacuje number of arguments does not match definition.

    GetPlayerMoney(playerid, -13);

Predpostavljam da si mislio koristiti

GivePlayerMoney(playerid,-13);

dok GetPlayerMoney(playerid) radiš provjere dal ima igrač dovoljno novca NPR:

if(GetPlayerMoney(playerid) >= 13) GivePlayerMoney(playerid,-13);
else return SendClientMessage(playerid,-1,"Nemate dovolčjno novca!");
dok ovaj dole primjer isto obavlja funkciju a mnogo ljepše izgleda
if(GetPlayerMoney(playerid) < 13) return SendClientMessage(playerid,-1,"Nemate dovolčjno novca!");
GivePlayerMoney(playerid,-13);

Hvala sve ok sad, moze lock  😉