Nein. Das ist eine Anregung, den Code ordentlich zu formatieren, damit man Fehler besser erkennt. Und die Warnung, dass vielleicht einer passiert sein koennte.

_________________
There are no stupid questions, only stupid people.

ja da versucht er immer deine festplatte zu formatieren

aber da steht ja nur warning also is das nich so schlimm

_________________
Geht nich, gibs nich: Suchen, FAQ
www.cola-clan.de - Whiskey@cola-clan.de

??? ich hab keine warnings beim compilieren ???

Smudo, diese Warnings müssen nicht notwendigerweise von deinem Code ausgehend sein.

Es wird ja nur die Formatierung bemängelt, und die kann durchaus Ergebnis eines unglücklichen Copy & Paste - Vorgangs sein.

No need to worry about

naja solange das nichts ist wo ich angst bekommen muss
is ja egal

OK, ich weiss, Warnings sind nicht unbedingt schlimm aber ich versuche trotzdem eigentlich immer diese Meldung zu vermeiden, was mir mir bisher auch immer gelungen ist. Deshalb wundert es micht etwas, dass er diese Warnings hat.
Aber das Plugin läuft jetzt seit einer Woche ohne das mir ein Fehler aufgefallen ist. Aehhhh, doch, einmal hatte jemand einen Frag weniger als das Plugin angezeigt hat. Also wird der Spieler irgendwann einen Punkt abgezogen bekommen haben. Die Frage ist nur wann? Wann bekommt man einen Punkt abgezogen ausser beim TK?

Diese Code"verzerrung" entsteht nur beim Posten.

Zu der internen Anzeige der Kills in CS kann ich nur sagen, dass es mir manchmal so vorkommt, als vergisst CS einfach zu zählen.
Das passiert dann, wenn gerade viel auf dem Server los ist . Da kann man nix machen.
Aber das ist jetzt auch nur eine Theorie von mir.

_________________
Fehleranalyse: Poste den Inhalt Deiner liblist.gam, (listen)server.cfg, adminmod.cfg, users.ini, vault.ini, plugin.ini von adminmod und plugins.ini von metamod. Benutze auch die Such-Funktion

Da ich mit CStrike mit Beta 4 aufgehört habe, kann ich das nicht direkt bestätigen.

Aber bei TFC ist es definitiv so, daß seit Ewigkeiten dort die Punkteübersicht buggy ist. Auf dem Server scheint sie korrekt zu sein, nur wird die Information nicht korrekt an die Clients geschickt und/oder von diesen falsch interpretiert.

So bekommt z.B. das blaue Team, wenn das rote gerade mehrere Flags ergattert hat, die Punkte zugeschrieben.

Daß die Information auf dem Server jedoch korrekt ist, schließe ich daraus, daß sich die Punkteanzeige nach ein paar Minuten öfters mal wieder korrigiert.

Ich schaue mir deinen Code trotzdem nochmal an, vielleicht hat sich ja doch noch irgendwo ein Käfer versteckt.

So, noch mal komplett durchgelesen. Käfer hab ich eigentlich nicht gefunden.

Was mir aufgefallen ist (Zeilenangaben wg. Copy&Paste nur ungefähr):

<57-80>
Kann Data[0] noch andere Werte annehmen als '0'..'2'?
Wenn nicht, die if's entfernen und durch Zuweisungen der Form variable = Data[0] - '0'; ersetzen.

<120>
Doppelaufruf von playerinfo ist mir unverständlich. Wieso wird das einmal mit MAX_DATA_LENGTH und einmal mit MAX_NAME_LENGTH aufgerufen?
Prüfen, ob man den ersten playerinfo-Aufruf nicht weglassen kann.

<141>
Verschachtelung der 2 if's durch && auflösen

<150>
iTeamV == iTeamA unnötig, if durch else ersetzen

<159-236>
In diversen if's werden mehrfach Variablen auf gleichen Wert geprüft:

 if (HSCount[ iIDA ]>highScore && ...
 ...
 else if(HSCount[ iIDA ]>highScore && ...

Ebenso werden für teilgleiche Fälle teilgleiche Aktionen durchgeführt.
-> Verschachtelte if-Struktur könnte hier Mehrfachprüfungen & doppelte Anweisungen einsparen.

<280>
Unnötig kompliziertes if. Ersetzen durch

if(strcmp("Round_Start", Data) == 0 && FBSwitch == 1)
{
    fbStatus = 1;
}

<300>
Hm, da sinds mal wieder meine mangelnden Small-spezifischen Kenntnisse... Kann man an so einer Stelle nicht array = { 0, ... } schreiben, statt das in einer Schleife zu nullen? Oder geht das nur einmalig bei der Initialisierung?


Wenn du einen Punkt als Stilfrage empfindest, einfach ignorieren...

Also...
hmmm, ich glaube das geht nicht da Data[0] ein string enthält und die Variable ein Integer ist. Oder sehe ich das falsch?

Also es muesste eigentlich gerade der erste stehen bleiben. Was mit dem 2ten Aufruf geprueft werden soll weiss ich auch nicht. Habs aus dem Originalcode übernommen (den ich eigentlich nur mal gerade ein wenig ändern wollte) und mir gedacht ich lass es erstmal so.

Könnte man machen aber es funzt so genaus so gut und ist übersichtlicher, da die && Verknüpfung recht lang werden würde. Ausserdem ebenfalls aus dem Original übernommen

Jo, hab ich mal gemacht.

Ja, aber immer in Verbindung mit anderen && Verknüpfungen. Hat lange genug gedauert bis die Logik so funktioniertie. Jetzt wo sie fertig ist könnte man sie aber bestimmt noch verbessern. Werde ich mir noch mal ansehen. Das kommt dabei heraus wenn mann einfach so ohne Konzept drauf los programmiert

Keine Ahnung!

Auf jeden Fall läuft es dank deiner Hilfe schon mal super und wenn mir nicht noch ein ganz krasser Fehler auffällt werde ich es wohl so lassen.

Ich meine, dass das nur bei der Initialisierung geht. Weiss ich jetzt nicht genau, aber ich denke das ist so. Muesste man mal testen.

_________________
There are no stupid questions, only stupid people.

Also wenn Data ein String ist, dann ist Data[0] ein Integer, genauso wie '0', 'a', '?' und '^n' auch Integer sind. Wenn es keine Integer wären, könntest du das ja gar nicht mit == vergleichen.

Ich vergleiche ja auch nicht die Variable mit dem String.
Data[0] enthält einen String (mit einem Zeichen), den ich mit dem String '0' ... '2' vergleiche.
Anhand des Ergebnisses wird ein Integer in eine Variable (z.B. FBSwitch) geschrieben oder auch nicht.

Wenn das nicht richtig ist, verbesser mich. Ich habs halt nie gelernt sondern schau mir das halt mehr oder weniger ab.

Ich bin ja auch ein Small-Neuling, ich profitiere wenn dann nur von der Ähnlichkeit zu C.

Ein String ist eine Zeichenkette. Das heißt, es ist ein Array aus Zeichen.

Beispiel:

"Hallo" == { 'H', 'a', 'l', 'l', 'o', '^0' } == { 72, 97, 108, 108, 111, 0 }

Ein einzelnes Zeichen (z.B. 'A' {nicht "A"}) ist kein String mehr, sondern eine Zahl. In der ASCII-Tabelle ist jedes Zeichen einer Zahl zugeordnet.

Wenn bei dir Data nun nur "0", "1" oder "2" sein kann, erhälst du mit Data[0] den Wert des ersten Zeichens, also '0' bzw. 48. '1' ist 49 und '2' ist 50 usw.

D.h. man könnte da praktisch:

variable = Data[0] - '0';

rechnen.

Ist Data[0] == '0', dann ist Data[0] - '0' == 0.
Ist Data[0] == '1', dann ist Data[0] - '0' == 1.
usw.

Ich hoffe das trifft so auf Small zu.

Trifft auf Small zu.

Nochwas, wenn Data wirklich nur "0", "1" oder "2" enhaelt, dann ist
new Data[MAX_DATA_LENGTH];
enorme Platzverschwendung.
new Data[2];
wuerde reichen.
Wer fuer die Zukunft baut ist mit
new Data[MAX_NUMBER_LENGTH];
immernoch besser bedient als mit MAX_DATA_LENGTH (Faktor 10).

_________________
There are no stupid questions, only stupid people.

Ich habs vorher probiert mit "variable = Data[0];". Das war der Fehler. Habs jetzt mal deinen Vorschlag ausprobiert: variable = Data[0] - '0'; und siehe da, es funzt und ist damit um einiges kürzer geworden .

public vaultdata()
{
	new Data[2];
	get_vaultdata("admin_mk_fb",Data,MAX_DATA_LENGTH);
	FBSwitch = Data[0] - '0';
	get_vaultdata("admin_mk_kit",Data,MAX_DATA_LENGTH);
	KITSwitch = Data[0] - '0';
	get_vaultdata("admin_mk_hs",Data,MAX_DATA_LENGTH);
	HSSwitch = Data[0] - '0';
	return PLUGIN_HANDLED;
}

Also Data[0] ist das erst Zeichen aus Data. Soweit so gut, bis dahin wars klar. Aber was macht " - '0' " mit diesem zeichen damit da dann tatsächlich die Zahl rauskommt?

@daRope
In diesem Fall konnte die Variable nur den Wert 0,1,2 bekommen. Aber ich wollte die Funktion auch für andere Plugins (mit mehreren Zeichen) nutzen, was allerings nicht funktioniert wie wir hier festgestellt haben. Also könnte man das in diesem Fall ändern in "new Data[2];" wie du es gesagt hast. Das macht sicherlich Sinn.

Data[0] ist das erste Zeichen aus Data.
Ein Zeichen ist eine Zahl. (siehe http://www.asciitable.com )

Weil ein Zeichen eine Zahl ist, kannst du damit rechnen. '1' ist '0' + 1, '2' ist '0' + 2. 0 ist '0' - '0'. Du kommst daher bei '2' - '0' auf 2.

Aber eigentlich hab ich das alles obendran ja schon geschrieben.