Werbung Fehlerhaft

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;
}

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.

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.

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;
}

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;
}

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 ?

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);

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.