Fraktions Farbe

xSoldier · 1. September 2014

Hallo liebes Breadfish Forum,
Undzwar möchte ich Fraktions farben erstellen z.B. LSPD -> Blau oder Hitman -> Braun aber bei mir funktioniert das irgendwie nicht.
Ich besitzt zurzeit das Godfather World of SanAndreas (WOS) von K3VIN.
Ich habe das so versucht.

defindet hab ich sie auch mit den farben die ich gerne haben möchte.

//Fraktion Farben

#define COLOR_LSPD 0x009BFFFF
#define COLOR_FBI 0x0041FFFF
#define COLOR_ARMY 0x4AA76AFF
#define COLOR_MEDIC 0x9B0000FF
#define COLOR_LCN 0x00FF78FF
#define COLOR_YAKUZA 0xFF00A5FF
#define COLOR_BM 0x646464FF
#define COLOR_HITMAN 0x730000FF
#define COLOR_CNN 0x37FF00FF
#define COLOR_TAXI 0xF5FF00FF
#define COLOR_FAHRLEHRER 0x00FFA0FF
#define COLOR_CRIPTZ 0x505050FF
#define COLOR_GS 0x05FF00FF
#define COLOR_BALLAS 0xFF0055FF
#define COLOR_TRANSPORT 0x00AFFFFF
#define COLOR_VAGOS 0x9BFF00FF
#define COLOR_AZTECAS 0xAAAAAAFF
#define COLOR_RIFAS 0x646464FF
#define COLOR_GHOSTRIDER 0xFF3C00FF
#define COLOR_ALQAIDA 0x0A0A0AFF
#define COLOR_WHEELMAN 0xFF1400FF
#define COLOR_SAPD 0x0041FFFF
#define COLOR_Brigada 0x323232FF
#define COLOR_O-AMT 0xE1FF00FF

//<----------------Fraktions-Farben---------------->//
Function SetPlayerToTeamColor(playerid)
{
if(IsPlayerConnected(playerid))
{
if(PlayerInfo[playerid][pMember] == 1 || PlayerInfo[playerid][pLeader] == 1) //LSPD Farbe
{
SetPlayerColor(playerid,COLOR_LSPD);
}
else if(PlayerInfo[playerid][pMember] == 0 || PlayerInfo[playerid][pLeader] == 0) //Zivi Farbe
{
SetPlayerColor(playerid,COLOR_WHITE);
}
else if(PlayerInfo[playerid][pMember] == 2 || PlayerInfo[playerid][pLeader] == 2)
{
SetPlayerColor(playerid,COLOR_FBI);
}
else if(PlayerInfo[playerid][pMember] == 3 || PlayerInfo[playerid][pLeader] == 3)
{
SetPlayerColor(playerid,COLOR_ARMY);
}
else if(PlayerInfo[playerid][pMember] == 4 || PlayerInfo[playerid][pLeader] == 4)
{
SetPlayerColor(playerid,COLOR_MEDIC);
}
else if(PlayerInfo[playerid][pMember] == 5 || PlayerInfo[playerid][pLeader] == 5)
{
SetPlayerColor(playerid,COLOR_LCN);
}
else if(PlayerInfo[playerid][pMember] == 6 || PlayerInfo[playerid][pLeader] == 6)
{
SetPlayerColor(playerid,COLOR_YAKUZA);
}
else if(PlayerInfo[playerid][pMember] == 7 || PlayerInfo[playerid][pLeader] == 7)
{
SetPlayerColor(playerid,COLOR_BM);
}
else if(PlayerInfo[playerid][pMember] == 8 || PlayerInfo[playerid][pLeader] == 8)
{
SetPlayerColor(playerid,COLOR_HITMAN);
}
else if(PlayerInfo[playerid][pMember] == 9 || PlayerInfo[playerid][pLeader] == 9)
{
SetPlayerColor(playerid,COLOR_CNN);
}
else if(PlayerInfo[playerid][pMember] == 10 || PlayerInfo[playerid][pLeader] == 10)
{
SetPlayerColor(playerid,COLOR_TAXI);
}
else if(PlayerInfo[playerid][pMember] == 11 || PlayerInfo[playerid][pLeader] == 11)
{
SetPlayerColor(playerid,COLOR_FAHRLEHRER);
}
else if(PlayerInfo[playerid][pMember] == 12 || PlayerInfo[playerid][pLeader] == 12)
{
SetPlayerColor(playerid,COLOR_CRIPTZ);
}
else if(PlayerInfo[playerid][pMember] == 13 || PlayerInfo[playerid][pLeader] == 13)
{
SetPlayerColor(playerid,COLOR_GS);
}
else if(PlayerInfo[playerid][pMember] == 14 || PlayerInfo[playerid][pLeader] == 14)
{
SetPlayerColor(playerid,COLOR_BALLAS);
}
else if(PlayerInfo[playerid][pMember] == 15 || PlayerInfo[playerid][pLeader] == 15)
{
SetPlayerColor(playerid,COLOR_TRANSPORT);
}
else if(PlayerInfo[playerid][pMember] == 16 || PlayerInfo[playerid][pLeader] == 16)
{
SetPlayerColor(playerid,COLOR_VAGOS);
}
else if(PlayerInfo[playerid][pMember] == 17 || PlayerInfo[playerid][pLeader] == 17)
{
SetPlayerColor(playerid,COLOR_AZTECAS);
}
else if(PlayerInfo[playerid][pMember] == 18 || PlayerInfo[playerid][pLeader] == 18)
{
SetPlayerColor(playerid,COLOR_RIFAS);
}
else if(PlayerInfo[playerid][pMember] == 19 || PlayerInfo[playerid][pLeader] == 19)
{
SetPlayerColor(playerid,COLOR_GHOSTRIDER);
}
else if(PlayerInfo[playerid][pMember] == 20 || PlayerInfo[playerid][pLeader] == 20)
{
SetPlayerColor(playerid,COLOR_ALQAIDA);
}
else if(PlayerInfo[playerid][pMember] == 22 || PlayerInfo[playerid][pLeader] == 22)
{
SetPlayerColor(playerid,COLOR_SAPD);
}
else if(PlayerInfo[playerid][pMember] == 23 || PlayerInfo[playerid][pLeader] == 23)
{
SetPlayerColor(playerid,COLOR_Brigada);
}
else if(PlayerInfo[playerid][pMember] == 24 || PlayerInfo[playerid][pLeader] == 24)
{
SetPlayerColor(playerid,COLOR_O-AMT);
}
else
{

}
}
}

Aber das ist irgendwie komisch denn nur die LSPD Farbe funktioniert aber der rest nicht z.B Hitman, CNN, Army...

Ich würde mich sehr um eine hilfreichende Rückmeldung freuen.
Mit freundlichen Grüßen,
xChicagoBulls

Jeffry · 1. September 2014

Fange oben mit der höchsten Zahl an, nicht mit der kleinsten.

24
23
22
.
.
.
1
0

Du kannst auch versuchen nur die Abfrage auf Zivi (0) nach ganz unten zu setzen, sollte auch gehen, falls du die Werte nicht selbst undefined setzt.

xSoldier · 1. September 2014

Meinst du das so?

//<----------------Fraktions-Farben---------------->//
Function SetPlayerToTeamColor(playerid)
{
if(IsPlayerConnected(playerid))
{
if(PlayerInfo[playerid][pMember] == 24 || PlayerInfo[playerid][pLeader] == 24) //SAPD Farbe
{
SetPlayerColor(playerid,COLOR_SAPD);
}
else if(PlayerInfo[playerid][pMember] == 23 || PlayerInfo[playerid][pLeader] == 23) //Frak farbe..
{
SetPlayerColor(playerid,COLOR_blablabla);
}

und so weiter....
...
..
.

Pall. · 1. September 2014

Ich frage mich was das bringen soll Oo. Also wenns klappt ist ja gut, aber ist ja kein wirklicher Unterschied außer die Reihenfolge.
Versuchs mal ohne das "else", hab ich früher auch so gemacht Bin aber auch kein Pro

xSoldier · 1. September 2014

haha krass daran hätte ich jetzt nicht gedacht. Ich bedanke mich an dir Pall.: großes Lob hihi
und danke trz Jeffry für deine Hilfe.

Pall. · 1. September 2014

Zitat von xChicagoBulls

haha krass daran hätte ich jetzt nicht gedacht. Ich bedanke mich an dir Pall.: großes Lob hihi
und danke trz Jeffry für deine Hilfe.

Ahh gut dass es funktioniert hat, kein Problem

Jeffry · 1. September 2014

@xChicagoBulls:
Der Grund warum du es andersherum schreiben sollst ist Folgender:

Du fragst oben ab:
Member=0 oder Leader=0
Dort hört er immer auf, wenn du mit else arbeitest, da er ja entweder in der Fraktion Member ist, oder Leader ist, also ist eines von beidem immer 0, wegen der Standardinitialisierung von 0.

Wenn du, so wie du es jetzt machst die else alle weg machst, dann setzt er dir erst die Farbe zu Zivil und anschließend zu der Farbe die du eigentlich hast. Funktioniert zwar, ist aber kein schöner Stil, vor allem wenn du mal mit etwas arbeitest was man sieht (zum Beispiel Nachrichten) klappt das so nicht mehr.

Daher solltest du es so machen wie du in deiner Antwort auf meinen ersten Post bereits angefangen hast.

AirM4X · 1. September 2014

Jeffry: Würde das nicht auch so funktionieren?
new TeamFarbenArray[][] =//Global oben im Script
{
0xFFFFFFFF,
0x009BFFFF,
0x0041FFFF,
0x4AA76AFF,
0x9B0000FF,
0x00FF78FF,
0xFF00A5FF,
0x646464FF,
0x730000FF,
0x37FF00FF,
0xF5FF00FF,
0x00FFA0FF,
0x505050FF,
0x05FF00FF,
0xFF0055FF,
0x00AFFFFF,
0x9BFF00FF,
0xAAAAAAFF,
0x646464FF,
0xFF3C00FF,
0x0A0A0AFF,
0xFF1400FF,
0x0041FFFF,
0x323232FF,
0xE1FF00FF
};

Function SetPlayerToTeamColor(playerid)
{
if(PlayerInfo[playerid][pMember] > 0 || PlayerInfo[playerid][pLeader] > 0)return SetPlayerColor(playerid,TeamFarbenArray[PlayerInfo[playerid][pMember]]);
return 1;
}
Nur mal eine Frage, eigentlich doch schon, oder?

Jeffry · 1. September 2014

Sofern ich das am Handy beurteilen kann, nicht immer ganz. So wäre es allgemein sicherer (keine Bugs):
Function SetPlayerToTeamColor(playerid)
{
if(PlayerInfo[playerid][pMember] > 0)return SetPlayerColor(playerid,TeamFarbenArray[PlayerInfo[playerid][pMember]]);
if(PlayerInfo[playerid][pLeader] > 0)return SetPlayerColor(playerid,TeamFarbenArray[PlayerInfo[playerid][pLeader]]);
return SetPlayerColor(playerid,-1);//Falls Zivi
}

Außer du hast Member und Leader auf dem gleichen Wert, viele vergessen das aber und setzen nur eines davon auf den Wert der Fraktion.

Aber generell natürlich, ja.

breadfish.de 12. März 2022