Werbung Fehlerhaft

BlAckNighT. · 22. Dezember 2014

Guten abend,

Und zwar weiss ich nicht wie die Werbung schreiben soll, wovon ich auch keine ahnung habe wie ich dies mache.
Ich möchte gerne pro Buchstabe was er schreib , 7$ kostest d.h. 10 Buchstaben = 70$.
Ich hoffe es könnte mir jemand helfen.

Code:

CMD:werbung(playerid,params[])
{
if(!IsLoggedIn(playerid))return 1;
if(Spieler[playerid][pTot] == 1)return SendClientMessage(playerid,COLOR_GREY,"Mann kann im Krankenhaus keine Werbung schreiben.");
if(Spieler[playerid][pLevel] < 0)return SendClientMessage(playerid, COLOR_GREY, "Du brauchst Level 3 oder höher um Werbung schreiben zu können.");
new string[256],text;
if(sscanf(params,"s[256]",text))return SendClientMessage(playerid,COLOR_GREY,"Benutze /werbung [Nachricht]");
new payout = text * 40;
if(Spieler[playerid][pMoney] < payout)
{
format(string, sizeof(string), "Du besitzt nicht genügend Geld (%d$), um die Werbung schalten zu können.",payout);
SendClientMessage(playerid, COLOR_GREY, string);
return 1;
}
GiveMoney(playerid,-payout);
format(string, sizeof(string), "|Werbung| %s, Absender: %s, Handy(Nr.%d)",text,GetName(playerid),Spieler[playerid][pNumber]);
SendClientMessageToAll(COLOR_GREEN, string);
return 1;
}

whitetiiger · 22. Dezember 2014

new payout = text * 40; zu new payout = strval(text) * 40;

P.s. if(Spieler[playerid][pLevel] < 0)return SendClientMessage(playerid, COLOR_GREY, "Du brauchst Level 3 oder höher um Werbung schreiben zu können.");
zu
if(Spieler[playerid][pLevel] <= 2)return SendClientMessage(playerid, COLOR_GREY, "Du brauchst Level 3 oder höher um Werbung schreiben zu können.");

BlAckNighT. · 22. Dezember 2014

Zitat von [GSR]Kaito-sensei

new payout = strval(text) * 40;

C:\Users\Large City\gamemodes\MySelfmade.pwn(2473) : error 035: argument type mismatch (argument 1)
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

1 Error.

Kinimod · 22. Dezember 2014

new string[256],text;

ändern in:

new string[256],text[144];

Du hast text als Integer deklariert, da die Funktion strval() aber einen String ( Char Array ) erwartet, erscheint eben der Error.
Außerdem muss jede Zeichenkette als String definiert werden.
128 = max. Anzahl an Zeichen
Da man im Chat eh nur 144 Zeichen verwenden kann, macht mehr keinen Sinn umd belastet den Speicher unnoetig.

Douq · 22. Dezember 2014

Auch sollte man eher die Funktion strlen() anstatt strval() verwenden, da du die Länge der Zeichenkette und nicht die konvertierte Version von string zu int haben willst.

BlAckNighT. · 22. Dezember 2014

Zitat von Mr.Reese

new string[256],text;

ändern in:

new string[256],text[144];

Du hast text als Integer deklariert, da die Funktion strval() aber einen String ( Char Array ) erwartet, erscheint eben der Error.
Außerdem muss jede Zeichenkette als String definiert werden.
128 = max. Anzahl an Zeichen
Da man im Chat eh nur 144 Zeichen verwenden kann, macht mehr keinen Sinn umd belastet den Speicher unnoetig.

Alles anzeigen

Zitat von Douq

Auch sollte man eher die Funktion strlen() anstatt strval() verwenden, da du die Länge der Zeichenkette und nicht die konvertierte Version von string zu int haben willst.

Jetzt kommt nichts mehr das Er mir Geld abzieht oder das er pro Buchstabe 7$ nimmt.

Kinimod · 22. Dezember 2014

Poste uns nochmal Deinen kompletten aktuellen Code, wie Du ihn jetzt hast.

AirM4X · 22. Dezember 2014

CMD:werbung(playerid,params[])
{
if(!IsLoggedIn(playerid))return 1;
if(Spieler[playerid][pTot] == 1)return SendClientMessage(playerid,COLOR_GREY,"Mann kann im Krankenhaus keine Werbung schreiben.");#
if(Spieler[playerid][pLevel] < 0)return SendClientMessage(playerid, COLOR_GREY, "Du brauchst Level 3 oder höher um Werbung schreiben zu können.");
new string[144],text[100];
if(sscanf(params,"s[100]",text))return SendClientMessage(playerid,COLOR_GREY,"Benutze /werbung [Nachricht]");
new payout = strlen(text) * 7;
if(Spieler[playerid][pMoney] < payout)
{
format(string,sizeof string,"Du besitzt nicht genügend geld (%d$), um die Werbung schalten zu können.",payout);
return SendClientMessage(playerid,COLOR_GREY,string);
}
GiveMoney(playerid,-payout);
format(string,sizeof string,"|Werbung| %s, Absender: %s, Handy(Nr.%d)",text,GetName(playerid),Spieler[playerid][pNumber]);
return SendClientMessageToAll(COLOR_GREEN,string);
}
Schreibe das so. Das sollte so auch dann eigentlich klappen.
Außerdem hast du pro Buchstaben 40€ genommen, obwohl du nur 7 wolltest, oder?

BlAckNighT. · 22. Dezember 2014

Zitat von Icy.

CMD:werbung(playerid,params[])
{
if(!IsLoggedIn(playerid))return 1;
if(Spieler[playerid][pTot] == 1)return SendClientMessage(playerid,COLOR_GREY,"Mann kann im Krankenhaus keine Werbung schreiben.");#
if(Spieler[playerid][pLevel] < 0)return SendClientMessage(playerid, COLOR_GREY, "Du brauchst Level 3 oder höher um Werbung schreiben zu können.");
new string[144],text[100];
if(sscanf(params,"s[100]",text))return SendClientMessage(playerid,COLOR_GREY,"Benutze /werbung [Nachricht]");
new payout = strlen(text) * 7;
if(Spieler[playerid][pMoney] < payout)
{
format(string,sizeof string,"Du besitzt nicht genügend geld (%d$), um die Werbung schalten zu können.",payout);
return SendClientMessage(playerid,COLOR_GREY,string);
}
GiveMoney(playerid,-payout);
format(string,sizeof string,"|Werbung| %s, Absender: %s, Handy(Nr.%d)",text,GetName(playerid),Spieler[playerid][pNumber]);
return SendClientMessageToAll(COLOR_GREEN,string);
}
Schreibe das so. Das sollte so auch dann eigentlich klappen.
Außerdem hast du pro Buchstaben 40€ genommen, obwohl du nur 7 wolltest, oder?

Alles anzeigen

So nun geht die Zahlung.

Aber meine 60 Sekunden Timer werden nicht gezählt.

Ich kann durchgehend Werbung schreiben anstadt nach jede Werbung 60 Sek warten.

Code.

CMD:werbung(playerid,params[])
{
if(!IsLoggedIn(playerid))return 1;
if(Spieler[playerid][pTot] == 1)return SendClientMessage(playerid,COLOR_GREY,"Mann kann im Krankenhaus keine Werbung schreiben.");
if(Spieler[playerid][pLevel] <= 3)return SendClientMessage(playerid, COLOR_GREY, "Du brauchst Level 3 oder höher um Werbung schreiben zu können.");
new string[256],text[144];
if(sscanf(params,"s[256]",text))return SendClientMessage(playerid,COLOR_GREY,"Benutze /werbung [Nachricht]");
new payout = strlen(text) * 9;
if(Spieler[playerid][pMoney] <= payout) {
format(string, sizeof(string), "Du besitzt nicht genügend Geld (%d$), um die Werbung schalten zu können.",payout);
SendClientMessage(playerid, COLOR_GREY, string);
return 1;
}
if(WerbungZeit < 0) {
format(string, sizeof(string), "Bitte warte noch %d Sekunden für die Nächste Werbung!", WerbungZeit);
SendClientMessage(playerid, COLOR_GREY, string);
SetPVarInt(playerid,"adtimer",1);
WerbungZeit = 60;
SetTimer("Werbung", 60000, 0);
return 1;
}
else{
GiveMoney(playerid,-payout);
format(string, sizeof(string), "|Werbung| %s, Absender: %s, Handy(Nr.%d)",text,GetName(playerid),Spieler[playerid][pNumber]);
SendClientMessageToAll(COLOR_GREEN, string);
format(string, sizeof(string), "Werbung erfolgreich abgesendet.~n~Zahlung: ~r~-%d$",payout);
GameTextForPlayer(playerid, string, 11000, 5);
}
return 1;
}

AirM4X · 22. Dezember 2014

CMD:werbung(playerid,params[])
{
if(!IsLoggedIn(playerid))return 1;
if(Spieler[playerid][pTot] == 1)return SendClientMessage(playerid,COLOR_GREY,"Mann kann im Krankenhaus keine Werbung schreiben.");
if(Spieler[playerid][pLevel] <= 3)return SendClientMessage(playerid, COLOR_GREY, "Du brauchst Level 3 oder höher um Werbung schreiben zu können.");
new string[256],text[144];
if(sscanf(params,"s[256]",text))return SendClientMessage(playerid,COLOR_GREY,"Benutze /werbung [Nachricht]");
new payout = strlen(text) * 9;
if(Spieler[playerid][pMoney] <= payout)
{
format(string, sizeof(string), "Du besitzt nicht genügend Geld (%d$), um die Werbung schalten zu können.",payout);
SendClientMessage(playerid, COLOR_GREY, string);
return 1;
}
if(WerbungZeit > 0)
{
format(string, sizeof(string), "Bitte warte noch %d Sekunden für die Nächste Werbung!", WerbungZeit);
SendClientMessage(playerid, COLOR_GREY, string);
return 1;
}
else
{
WerbungZeit = 1;
SetTimer("Werbung", 60000, 0);
GiveMoney(playerid,-payout);
format(string, sizeof(string), "|Werbung| %s, Absender: %s, Handy(Nr.%d)",text,GetName(playerid),Spieler[playerid][pNumber]);
SendClientMessageToAll(COLOR_GREEN, string);
format(string, sizeof(string), "Werbung erfolgreich abgesendet.~n~Zahlung: ~r~-%d$",payout);
GameTextForPlayer(playerid, string, 11000, 5);
}
return 1;
}
Möchtest du, dass man alle 60 Sekunden nur eine Werbung schalten kann?
Dann musst du das so machen, und im Callback "Werbung" "WerbungZeit" einfach auf 0 setzen.

Kinimod · 22. Dezember 2014

Entfern bitte den Timer und mach es folgendermaßen, denn ein Timer ist in diesem Fall total unnötig und resourcenaufwendig.

Abfrage, ob er eine Werbung erstellen kann:
if(GetPVarInt(playerid, "lastAdvertise") < gettime()){
//Werbung kann erstellt werden
SetPVarInt(playerid, "lastAdvertise", gettime() + 60); //60 durch die Anzahl der Sekunden ersetzen
}else{
//Werbung kann NICHT erstellt werden
}

Dieser Code arbeitet mit Timestamps. Das sind die Sekunden seit dem 01.01.1970.

Du fragst einfach ab, ob der Timestamp der letzten Werbung + 60 Sekunden kleiner als der aktuelle Timestamp ist. Wenn ja, liegt es in der Vergangenheit und die Werbung darf erstellt werden, wobei der Timestamp dann wieder neu gesetzt wird auf in 60 Sekunden.

BlAckNighT. · 22. Dezember 2014

Zitat von Icy.

CMD:werbung(playerid,params[])
{
if(!IsLoggedIn(playerid))return 1;
if(Spieler[playerid][pTot] == 1)return SendClientMessage(playerid,COLOR_GREY,"Mann kann im Krankenhaus keine Werbung schreiben.");
if(Spieler[playerid][pLevel] <= 3)return SendClientMessage(playerid, COLOR_GREY, "Du brauchst Level 3 oder höher um Werbung schreiben zu können.");
new string[256],text[144];
if(sscanf(params,"s[256]",text))return SendClientMessage(playerid,COLOR_GREY,"Benutze /werbung [Nachricht]");
new payout = strlen(text) * 9;
if(Spieler[playerid][pMoney] <= payout)
{
format(string, sizeof(string), "Du besitzt nicht genügend Geld (%d$), um die Werbung schalten zu können.",payout);
SendClientMessage(playerid, COLOR_GREY, string);
return 1;
}
if(WerbungZeit > 0)
{
format(string, sizeof(string), "Bitte warte noch %d Sekunden für die Nächste Werbung!", WerbungZeit);
SendClientMessage(playerid, COLOR_GREY, string);
return 1;
}
else
{
WerbungZeit = 1;
SetTimer("Werbung", 60000, 0);
GiveMoney(playerid,-payout);
format(string, sizeof(string), "|Werbung| %s, Absender: %s, Handy(Nr.%d)",text,GetName(playerid),Spieler[playerid][pNumber]);
SendClientMessageToAll(COLOR_GREEN, string);
format(string, sizeof(string), "Werbung erfolgreich abgesendet.~n~Zahlung: ~r~-%d$",payout);
GameTextForPlayer(playerid, string, 11000, 5);
}
return 1;
}
Möchtest du, dass man alle 60 Sekunden nur eine Werbung schalten kann?
Dann musst du das so machen, und im Callback "Werbung" "WerbungZeit" einfach auf 0 setzen.

Alles anzeigen

geht immer noch nicht.

public

public Werbung(playerid)
{
if(WerbungZeit == 60)
{
WerbungZeit --;
}
if(WerbungZeit == 0)
{
WerbungZeit = 0;
}
return 1;
}

Kinimod · 22. Dezember 2014

Zitat von BlAckNighT.

geht immer noch nicht.

Siehe meinen letzten Post, hast Du scheinabr übersehen.
Und unterlass bitte die Vollzitate, das Scrollen verlängert sich dadurch immens. ^^^

BlAckNighT. · 22. Dezember 2014

Zitat von Mr.Reese

Siehe meinen letzten Post, hast Du scheinabr übersehen.
Und unterlass bitte die Vollzitate, das Scrollen verlängert sich dadurch immens. ^^^

wie mach ich das ?
Wo muss ich dies hintuhen ?

Soll ich denn public Werbung entfernen ?

AirM4X · 22. Dezember 2014

Spoiler anzeigen

So.
Den Callback "Werbung" kannst du entfernen, ja.

Kinimod · 22. Dezember 2014

In den /werbung Befehl.

BlAckNighT. · 22. Dezember 2014

Der zeigt mir hier keine Sek an wie lange die Werbung noch gesperrt ist.

format(string, sizeof(string), "Bitte warte noch %d Sekunden für die Nächste Werbung!", WerbungZeit);

AirM4X · 22. Dezember 2014

format(string, sizeof(string), "Bitte warte noch %d Sekunden für die Nächste Werbung!", gettime() - GetPVarInt(playerid,"Werbung"));
Probier das mal so.

BlAckNighT. · 22. Dezember 2014

Danke klappt alles, aber was mich stört.
"Bitte warte noch -32 Sekunden für die nächste Werbung!"
Das Minus stört mich, kann man dies auch noch Weg machen dann wäre dies perfekt.

AirM4X · 22. Dezember 2014

Ja kann man
Dafür musst du das einmal drehen.
format(string, sizeof(string), "Bitte warte noch %d Sekunden für die Nächste Werbung!", GetPVarInt(playerid,"Werbung")-gettime());
#edit:
Das war mein Fehler, du musst gettime() von dem Wert in der PVar abziehen.

breadfish.de 12. März 2022