Stock IstSpielerinFraktion

Zeig uns doch mal den Fehlercode und benutze bitte den PAWN -Syntax.

stock IstSpielerInFraktion(playerid, fraktionsid)

{

if(sSpieler[playerid][Fraktion] == fraktionsid || sSpieler[playerid][FMember] == fraktionsid) return 1;
return 0;

}

So, so müsste es eig. richtig sein. Probier es mal aus.

stock IstSpielerInFraktion(playerid, fraktionsid)
{
if(sSpieler[playerid][Fraktion] == fraktionsid || sSpieler[playerid][FMember] == fraktionsid) return 1;
else
return 0;
}

So vielleicht? wie sind denn die Errors?

Zitat

stock IstSpielerInFraktion(playerid, fraktionsid)
{
if(sSpieler[playerid][Fraktion] == fraktionsid || sSpieler[playerid][FMember] == fraktionsid) return 1;
else
return 0;
}

So vielleicht? wie sind denn die Errors?

Alles anzeigen

Bin kein Skripting Profi aber wieso das "else" ?! Das gibt doch nur Errors. Und macht dann was ganz anderes...

Er soll nur die fraktionsid wiedergeben. Den stock ansich 0 - returnen lassen. Und nicht Else?!

Zitat von $this->getUser();

So vielleicht? wie sind denn die Errors?

Nun, das else sollte eigentlich nicht notwendig sein.
Ich hab zwar nichts mehr mit dem Skripting am Hut - daher könnte ich falsch liegen - aber wenn er in der Fraktion ist bricht er die Funktion mit dem Rückgabewert 1 ab, oder?`
Dann sollte das return 0 garnicht mehr Relevant sein

Zitat

Nun, das else sollte eigentlich nicht notwendig sein.

Ich hab zwar nichts mehr mit dem Skripting am Hut - daher könnte ich
falsch liegen - aber wenn er in der Fraktion ist bricht er die Funktion
mit dem Rückgabewert 1 ab, oder?`

Dann sollte das return 0 garnicht mehr Relevant sein

Ob das return 0; Unrelevant ist, weiss ich auch nicht. Ich habe es so im Skript drinne und es Funktioniert. Natürlich ohne "else"

Leute, das ist richtig, er hat nur ne Klammer zu viel reingemacht

stock IstSpielerInFraktion(playerid, fraktionsid)
{
if(sSpieler[playerid][Fraktion] == fraktionsid) /*FEHLER*/ || sSpieler[playerid][FMember] == fraktionsid) return 1;
return 0;
}

HIer das richtige:

stock IstSpielerInFraktion(playerid, fraktionsid)
{
if(sSpieler[playerid][Fraktion] == fraktionsid || sSpieler[playerid][FMember] == fraktionsid) return 1;
return 0;
}

Zitat von Famacertech

Ob das return 0; Unrelevant ist, weiss ich auch nicht. Ich habe es so im Skript drinne und es Funktioniert. Natürlich ohne "else"

Nene so mein ich das nicht.
Er muss ja schon false als Wert zurückgeben.
Aber wenn er in ner 'FRaktion ist, ist das return 0 uninteressant, weil ja schon zuvor abgebrochen wurde.

Zitat von firestormZ

Leute, das ist richtig, er hat nur ne Klammer zu viel reingemacht

stock IstSpielerInFraktion(playerid, fraktionsid)
{
if(sSpieler[playerid][Fraktion] == fraktionsid) /*FEHLER*/ sSpieler[playerid][FMember] == fraktionsid) return 1;
return 0;
}

HIer das richtige:

stock IstSpielerInFraktion(playerid, fraktionsid)
{
if(sSpieler[playerid][Fraktion] == fraktionsid sSpieler[playerid][FMember] == fraktionsid) return 1;
return 0;
}

Alles anzeigen

Ich will ja nicht frech wirken, aber wer Lesen kann ist klar im Vorteil... Ich habe es oben bereits gepostet

Zitat

Nene so mein ich das nicht.

Er muss ja schon false als Wert zurückgeben.

Aber wenn er in ner 'FRaktion ist, ist das return 0 uninteressant, weil ja schon zuvor abgebrochen wurde.

Achso, ja... Bin halt neu aber habe wohl was Missverstanden