[SAMMELTHREAD] Kleine Scripting Fragen

Xytronix · 24. Juni 2017

Abend alle zusammen.

Ich habe folgendes Problem entdeckt, dass ich wohl damals irgendwie erstellt habe...
Nur finde ich den Fehler nicht.

Es wäre nett wenn jmd von Euch al rüber schauen könnte und mir den Fehler, sowie evtl die Lösung mitteilen könnte.

Code: Fehlermeldung

C:\Users\michi\Desktop\[IVL] Investival Life\MY Server\Investival Life\gamemodes\IVL.pwn(6293) : warning 225: unreachable code
C:\Users\michi\Desktop\[IVL] Investival Life\MY Server\Investival Life\gamemodes\IVL.pwn(17674) : warning 215: expression has no effect
C:\Users\michi\Desktop\[IVL] Investival Life\MY Server\Investival Life\gamemodes\IVL.pwn(25249) : warning 213: tag mismatch
C:\Users\michi\Desktop\[IVL] Investival Life\MY Server\Investival Life\gamemodes\IVL.pwn(50025) : error 029: invalid expression, assumed zero
C:\Users\michi\Desktop\[IVL] Investival Life\MY Server\Investival Life\gamemodes\IVL.pwn(54267) : warning 225: unreachable code
C:\Users\michi\Desktop\[IVL] Investival Life\MY Server\Investival Life\gamemodes\IVL.pwn(64391) : warning 203: symbol is never used: "ShowMeAllowed"
C:\Users\michi\Desktop\[IVL] Investival Life\MY Server\Investival Life\gamemodes\IVL.pwn(64391) : warning 203: symbol is never used: "dcmd_kofferraum"
C:\Users\michi\Desktop\[IVL] Investival Life\MY Server\Investival Life\gamemodes\IVL.pwn(64391) : warning 203: symbol is never used: "dcmd_licht"
C:\Users\michi\Desktop\[IVL] Investival Life\MY Server\Investival Life\gamemodes\IVL.pwn(64391) : warning 203: symbol is never used: "dcmd_unban"
Pawn compiler 3.2.3664	 	 	Copyright (c) 1997-2006, ITB CompuPhase




1 Error.

Alles anzeigen

Code: rund um die zeile herum Zeile: 50025

if(strcmp(cmd, "/news", true) == 0 || strcmp(cmd, "/news", true) == 0)
	{
	    if(IsPlayerConnected(playerid))
	    {
			if(PlayerInfo[playerid][pMember] == 9 || PlayerInfo[playerid][pLeader] == 9)
			{
		        if(PlayerInfo[playerid][pMuted] == 1)
				{
					SendClientMessage(playerid, TEAM_CYAN_COLOR, "Du bist gemutet!");
					return 1;
				}
				{
					GetPlayerName(playerid, sendername, sizeof(sendername));
					new length = strlen(cmdtext);
					while ((idx < length) && (cmdtext[idx] <= ' '))
					{
						idx++;
					}
					new offset = idx;
					new result[64];
					while ((idx < length) && ((idx - offset) < (sizeof(result) - 1)))
					{
						result[idx - offset] = cmdtext[idx];
						idx++;
					}
					result[idx - offset] = EOS;
					if(!strlen(result))
					{
						SendClientMessage(playerid, COLOR_GRAD2, "Benutze: /news [newstext] oder /n [newstext]");
						return 1;
					}
					format(string, sizeof(string), "NR %s: %s", sendername, result);
					OOCNews(COLOR_NEWS,string);
				}
				else
				{
				    SendClientMessage(playerid, COLOR_GREY, "   Du bist nicht im News Van oder Chopper !");
				    return 1;
				}
			}
			else
			{
			    SendClientMessage(playerid, COLOR_GREY, "   Du bist kein Reporter !");
			}
		}//not connected
		return 1;

Alles anzeigen

Ich bedanke mich schon mal für die ganzen Helfer.

Mit Freundlichen Grüßen

XtremeGamer_HD

Dr. Frauenarzt · 24. Juni 2017

Code

public OnPlayerTimer()
{
	new string[128];
	for(new i=0;i<GetMaxPlayers();i++)
	{
		if(IsPlayerConnected(i))
		{
			if(pIsRobbingBank[i] == 1)
			{
				if(!IsPlayerInRangeOfPoint(i, 50.0, 1424.7828,-1003.0624,1639.7843))
				{
					pIsRobbingBank[i] = 0;
					pRobbingTimer[i] = 0;
					format(string, sizeof(string), "Bankräuber %s hat die Bank vorzeitig verlassen!", Playername(i));
					SendClientMessageToAll(COLOR_LIGHTBLUE, string);
					SendClientMessage(i, COLOR_LIGHTBLUE, "Der Bank-Raub wurde abgebrochen.");
				}
				if(pRobbingTimer[i])
				{
					if(pRobbingTimer[i] <= gettime() + 180)
						SendClientMessage(i, COLOR_LIGHTBLUE, "Noch 3 Minuten bis die Bank erfolgreich ausgeraubt wurde!");


					else if(pRobbingTimer[i] <= gettime() + 120)
						SendClientMessage(i, COLOR_LIGHTBLUE, "Noch 2 Minuten bis die Bank erfolgreich ausgeraubt wurde!");


                    else if(pRobbingTimer[i] <= gettime() + 60)
						SendClientMessage(i, COLOR_LIGHTBLUE, "Noch 1 Minute bis die Bank erfolgreich ausgeraubt wurde!");


					if(pRobbingTimer[i] <= 0)
					{
						new preis = randomex(20000, 80000);
						format(string, sizeof(string), "%s hat erfolgreich die Bank ausgeraubt. ($%d)", Playername(i), preis);
						SendClientMessageToAll(COLOR_LIGHTBLUE, string);
						GivePlayerMoney(i, preis);
						pIsRobbingBank[i] = 0;
						pRobbingTimer[i] = 0;
					}
				}
			}
		}
    }
	return 1;
}

Alles anzeigen

Hallo,

Ich habe eine frage warum wird Noch 3 Minuten bis die Bank erfolgreich ausgeraubt wurde gespamt?

Madd92 · 24. Juni 2017

Weil <= gettime() + 180 immer zutrifft, wenn die Zeit unter 180 ist

Xytronix · 24. Juni 2017

@Madd92 könntest du mir vl auch helfen?

Wäre nett...

Madd92 · 24. Juni 2017

Schreib bitte mal die betreffende Zeile raus @XtremeGamer_HD

Dr. Frauenarzt · 24. Juni 2017

Zitat von Madd92

Weil <= gettime() + 180 immer zutrifft, wenn die Zeit unter 180 ist

<= gettime() + 180

Wie soll man das den sonst einstellen,

Danke für die hilfe

Madd92 · 24. Juni 2017

Wie oft wird der Timer denn aufgerufen? Wenn <= 1 Sekunden, kannst du einfach nur == gettime() + 180 benutzen Ansonsten kannst du auch eine Variable hochzählen, die bei 0 wenn (pRobbingTimer[i] <= gettime() + 180) zutrifft auf 1 gesetzt wird, dann bei (pRobbingTimer[i] <= gettime() + 120) auf 2 etc.

Dr. Frauenarzt · 24. Juni 2017

Zitat von Madd92

Wie oft wird der Timer denn aufgerufen? Wenn <= 1 Sekunden, kannst du einfach nur == gettime() + 180 benutzen Ansonsten kannst du auch eine Variable hochzählen, die bei 0 wenn (pRobbingTimer[i] <= gettime() + 180) zutrifft auf 1 gesetzt wird, dann bei (pRobbingTimer[i] <= gettime() + 120) auf 2 etc

Code

if(pRobbingTimer[i])
				{
					if(pRobbingTimer[i] == gettime() + 180)
						SendClientMessage(i, COLOR_LIGHTBLUE, "Noch 3 Minuten bis die Bank erfolgreich ausgeraubt wurde!");


					else if(pRobbingTimer[i] == gettime() + 120)
						SendClientMessage(i, COLOR_LIGHTBLUE, "Noch 2 Minuten bis die Bank erfolgreich ausgeraubt wurde!");


                    else if(pRobbingTimer[i] == gettime() + 60)
						SendClientMessage(i, COLOR_LIGHTBLUE, "Noch 1 Minute bis die Bank erfolgreich ausgeraubt wurde!");

Alles anzeigen

so wa

Madd92 · 24. Juni 2017

Jo aber dann musst du sicher sein, dass der Timer nur einmal und auch wirklich jede Sekunde aufgerufen wird. Mit der Zähler-Variable würdest du sichergehen, dann jede Nachricht auf jeden Fall kommt und auch nur einmal.

Xytronix · 24. Juni 2017

Mein script lädt aus irgend einem Grund nicht. Es steht in der Hosted tap Liste dann Unknown.

Zu vorherigem probem:

error 029: invalid expression, assumed Zero

Da steht nur else drinne

mfg

Madd92 · 24. Juni 2017

Ist der Fehler gelöst oder nicht? Das mit dem Script laden kann viele Gründe haben

Dr. Frauenarzt · 24. Juni 2017

Code

mysql_format(Handle, strQuery, 1024, "UPDATE users SET respectpoints='%i', level='%i', payday='%i', paydaymoney='%i', team='%i', wanted='%i', jailtime='%i' , leader='%i', rang='%i' , fraksperre='%i', Messages='%i' WHERE username='%e'",
	    pInfo[playerid][pRespectPoints],
	    pInfo[playerid][pLevel],
	    pInfo[playerid][pPayday],
	    pInfo[playerid][pPaydayMoney],
	    pInfo[playerid][pTeam],
	    pInfo[playerid][pWanteds],
	    pInfo[playerid][pKnastZeit],
	    pInfo[playerid][pLeader],
	    pInfo[playerid][pRank],
	    pInfo[playerid][pFraksperre],
	    pInfo[playerid][pMessages],
		Playername(playerid));
		mysql_pquery(Handle, strQuery);

Alles anzeigen

Hey kurze frage warum wird Messages nicht gespeichert?

Alles korrekt in der Datenbank Eingetragen

tayson87 · 25. Juni 2017

Ne frage wie kann ich son format in die länge ziehen? Wollte das hier so alles untereinander in der Liste des Dialogs haben, doch das alles passt nicht rein und zeigt mir einen error..

Code

format(string,sizeof(string),""#HTML_RED"MuellmannSkill: "#HTML_WHITE"%d\n"#HTML_RED"StadtreinigungSkill: "#HTML_WHITE"%d\n"#HTML_RED"FluglieferantSkill: "#HTML_WHITE"%d\n"#HTML_RED"PostboteSkill: "#HTML_WHITE"%d\n"#HTML_RED"DetektivSkill: "#HTML_WHITE"%d\n"#HTML_RED"LandwirtSkill: "#HTML_WHITE"%d
	\n"#HTML_RED"EisverkauferSkill: "#HTML_WHITE"%d\n"#HTML_RED"BestatterSkill: "#HTML_WHITE"%d\n"#HTML_RED"GeldtransporterSkill: "#HTML_WHITE"%d\n"#HTML_RED"TruckerSkill: "#HTML_WHITE"%d\n"#HTML_RED"BusfahrerSkill: "#HTML_WHITE"%d\n"#HTML_RED"MechanikerSkill: "#HTML_WHITE"%d\n"#HTML_RED"WaffendealerSkill: "#HTML_WHITE"%d
	\n"#HTML_RED"DrogendealerSkill: "#HTML_WHITE"%d\n"#HTML_RED"AnwaltSkill: "#HTML_WHITE"%d\n"#HTML_RED"TaxifahrerSkill: "#HTML_WHITE"%d\n"#HTML_RED"ImmobilienmaklerSkill: "#HTML_WHITE"%d\n"#HTML_RED"ZigarenhandlerSkill: "#HTML_WHITE"%d\n"#HTML_RED"MinenarbeiterSkill: "#HTML_WHITE"%d\n"#HTML_RED"HohlenforscherSkill: "#HTML_WHITE"%d",
ShowPlayerDialog(playerid,DIALOG_4ALL_SONSTIGES,DIALOG_STYLE_MSGBOX,"Deine Waffen Skills",string,"ok","");

x Sky.NET x · 25. Juni 2017

Zitat von tayson87

Ne frage wie kann ich son format in die länge ziehen? Wollte das hier so alles untereinander in der Liste des Dialogs haben, doch das alles passt nicht rein und zeigt mir einen error..

Code

format(string,sizeof(string),""#HTML_RED"MuellmannSkill: "#HTML_WHITE"%d\n"#HTML_RED"StadtreinigungSkill: "#HTML_WHITE"%d\n"#HTML_RED"FluglieferantSkill: "#HTML_WHITE"%d\n"#HTML_RED"PostboteSkill: "#HTML_WHITE"%d\n"#HTML_RED"DetektivSkill: "#HTML_WHITE"%d\n"#HTML_RED"LandwirtSkill: "#HTML_WHITE"%d
	\n"#HTML_RED"EisverkauferSkill: "#HTML_WHITE"%d\n"#HTML_RED"BestatterSkill: "#HTML_WHITE"%d\n"#HTML_RED"GeldtransporterSkill: "#HTML_WHITE"%d\n"#HTML_RED"TruckerSkill: "#HTML_WHITE"%d\n"#HTML_RED"BusfahrerSkill: "#HTML_WHITE"%d\n"#HTML_RED"MechanikerSkill: "#HTML_WHITE"%d\n"#HTML_RED"WaffendealerSkill: "#HTML_WHITE"%d
	\n"#HTML_RED"DrogendealerSkill: "#HTML_WHITE"%d\n"#HTML_RED"AnwaltSkill: "#HTML_WHITE"%d\n"#HTML_RED"TaxifahrerSkill: "#HTML_WHITE"%d\n"#HTML_RED"ImmobilienmaklerSkill: "#HTML_WHITE"%d\n"#HTML_RED"ZigarenhandlerSkill: "#HTML_WHITE"%d\n"#HTML_RED"MinenarbeiterSkill: "#HTML_WHITE"%d\n"#HTML_RED"HohlenforscherSkill: "#HTML_WHITE"%d",
ShowPlayerDialog(playerid,DIALOG_4ALL_SONSTIGES,DIALOG_STYLE_MSGBOX,"Deine Waffen Skills",string,"ok","");

Benutzt die Funktion "strcat" und "strdel "

tayson87 · 25. Juni 2017

Ne frage wie kann ich son format in die länge ziehen? Wollte das hier so alles untereinander in der Liste des Dialogs haben, doch das alles passt nicht rein und zeigt mir einen error..

Code

format(string,sizeof(string),""#HTML_RED"MuellmannSkill: "#HTML_WHITE"%d\n"#HTML_RED"StadtreinigungSkill: "#HTML_WHITE"%d\n"#HTML_RED"FluglieferantSkill: "#HTML_WHITE"%d\n"#HTML_RED"PostboteSkill: "#HTML_WHITE"%d\n"#HTML_RED"DetektivSkill: "#HTML_WHITE"%d\n"#HTML_RED"LandwirtSkill: "#HTML_WHITE"%d
	\n"#HTML_RED"EisverkauferSkill: "#HTML_WHITE"%d\n"#HTML_RED"BestatterSkill: "#HTML_WHITE"%d\n"#HTML_RED"GeldtransporterSkill: "#HTML_WHITE"%d\n"#HTML_RED"TruckerSkill: "#HTML_WHITE"%d\n"#HTML_RED"BusfahrerSkill: "#HTML_WHITE"%d\n"#HTML_RED"MechanikerSkill: "#HTML_WHITE"%d\n"#HTML_RED"WaffendealerSkill: "#HTML_WHITE"%d
	\n"#HTML_RED"DrogendealerSkill: "#HTML_WHITE"%d\n"#HTML_RED"AnwaltSkill: "#HTML_WHITE"%d\n"#HTML_RED"TaxifahrerSkill: "#HTML_WHITE"%d\n"#HTML_RED"ImmobilienmaklerSkill: "#HTML_WHITE"%d\n"#HTML_RED"ZigarenhandlerSkill: "#HTML_WHITE"%d\n"#HTML_RED"MinenarbeiterSkill: "#HTML_WHITE"%d\n"#HTML_RED"HohlenforscherSkill: "#HTML_WHITE"%d",
ShowPlayerDialog(playerid,DIALOG_4ALL_SONSTIGES,DIALOG_STYLE_MSGBOX,"Deine Waffen Skills",string,"ok","");

Eventueel dann so?

strcat(string,"#HTML_RED"MuellmannSkill: "#HTML_WHITE"%d,Spieler[playerid][pMuellmannSkill]),strcat(string, "\n");

Für jeden Beruf ?

Mr.Pawn · 25. Juni 2017

Kann mir einer sagen bzw erläutern, wieso ich das Textdraw nicht anklicken kann ?

Rename[14] = TextDrawCreate(184.000000, 138.000000, "Auswahl");
TextDrawBackgroundColor(Rename[14], 255);
TextDrawFont(Rename[14], 1);
TextDrawLetterSize(Rename[14], 0.379999, 1.000000);
TextDrawColor(Rename[14], -1);
TextDrawSetOutline(Rename[14], 0);
TextDrawSetProportional(Rename[14], 1);
TextDrawSetShadow(Rename[14], 1);
TextDrawUseBox(Rename[14], 1);
TextDrawBoxColor(Rename[14], 255);
TextDrawTextSize(Rename[14], 240.000000, 0.000000);
TextDrawSetSelectable(Rename[14], 1);

Madd92 · 25. Juni 2017

TextDrawTextSize definiert die anklickbare Fläche. Daher führt 0 als Y-Parameter dazu, dass man es nicht anklicken kann.

Mr.Pawn · 25. Juni 2017

Was sollte ich dann da angeben ? Ich dachte immer, sofern man die Box anmacht, ist dies dann die Anklickbare Fläche ?

Madd92 · 25. Juni 2017

"Diese Funktion definiert den anklickbaren Bereich für die Verwendung von TextDrawSetSelectable, egal ob eine Box angezeigt wird, oder nicht." aus http://wiki.sa-mp.com/wiki/TextDrawTextSize_DE
Ich denke du kannst die gleiche Größe wie die Box nehmen, aber probier es einfach mal aus.

Beitrag von Dr. Frauenarzt (24. Juni 2017)