Fehler im Switch

Deazl_ · 2. August 2016

Nabend,

nach zig Jahren widme ich mich nun wieder SAMP. Ich möchte nichts großartiges, nur ein bisschen herumprobieren und mir nebenbei einen kleinen Testserver für weitere Dinge zusammen basteln. Als Grundlage nehme ich das grandlarc Script. Da ich Code gerne auslagere (Und die Includes dafür ideal sind), erstellte ich mir ein Include mit folgendem Inhalt:

C

#include <a_samp>
#include "../include/dzl_vardefs.inc"


stock GetSkinGroup(skinid) 
{
    new group;
    switch(skinid)
    {
        case 280, 286: group = SKIN_GROUP_POLICE;
        case 287: group = SKIN_GROUP_ARMY;
        case 274, 276: group = SKIN_GROUP_MEDIC;
        default: group = SKIN_GROUP_CIVIL;
    }


    return group;
}

Alles anzeigen

In der Datei "dzl_vardefs.inc", welche sowohl in diesem Include als auch im Gamemode selbst geladen wird befindet sich folgendes:

C

// Colors
#define COLOR_WHITE             0xFFFFFFFF
#define COLOR_NORMAL_PLAYER     0xFFBB7777


// Cities
#define CITY_LOS_SANTOS         10001
#define CITY_SAN_FIERRO         10002
#define CITY_LAS_VENTURAS       10003


// Dialogs
#define DIALOG_VEHICLE_INFO     11001


// Skin Groups
#define SKIN_GROUP_CIVIL        12001
#define SKIN_GROUP_MEDIC        12002
#define SKIN_GROUP_POLICE       12003
#define SKIN_GROUP_ARMY         12004

Alles anzeigen

Und nun die entstehenden Fehler:

Code

../include/dzl_skinutil.inc(9) : error 002: only a single statement (or expression) can follow each "case"
../include/dzl_skinutil.inc(10) : error 014: invalid statement; not in switch
../include/dzl_skinutil.inc(10) : warning 215: expression has no effect
../include/dzl_skinutil.inc(10) : error 001: expected token: ";", but found ":"
../include/dzl_skinutil.inc(10) : error 029: invalid expression, assumed zero
../include/dzl_skinutil.inc(10) : fatal error 107: too many error messages on one line

Mir ist aufgefallen, dass diese Fehler nur entstehen, sobald ich auf die definierten Konstanten in der dzl_vardefs.inc zugreife (Sowohl im Include als auch im grandlarc Script). Sobald ich diese durch z.B. einfach 0 ersetze, funktioniert es einwandfrei. Aber unter anderem dafür definiert man sich doch globale Variablen via #define, oder nicht?

Viele Grüße,
Deazl_

AirM4X · 2. August 2016

C

// Dialogs
#define DIALOG_VEHICLE_INFO 11001;
// Skin Groups
#define SKIN_GROUP_CIVIL 	12001;
#define SKIN_GROUP_MEDIC 	12002;
#define SKIN_GROUP_POLICE 	12003;
#define SKIN_GROUP_ARMY 	12004;

Hier sind Semikolons hinter, die da nicht hingehören. Schreibe das so:

C

// Dialogs
#define DIALOG_VEHICLE_INFO 11001
// Skin Groups
#define SKIN_GROUP_CIVIL 	12001
#define SKIN_GROUP_MEDIC 	12002
#define SKIN_GROUP_POLICE 	12003
#define SKIN_GROUP_ARMY 	12004

Deazl_ · 3. August 2016

Whoups, vielen Dank @AirM4X. Da hat mir die Gewohnheit wohl einen Strich durch die Rechnung gemacht

Eine weitere, ich hoffe einfache Frage hätte ich noch: Wie siehts aus wenn ich ein zweidimensionales Float-Array habe (beispielsweise gRandomSpawns_LosSantos) und dieses in eine neue Variable klonen möchte? Möglicherweise hilft dieser Schnipsel die Frage etwas besser zu verstehen:

C

new randSpawn = 0, spawnsList[][];
    switch(GetSkinGroup(GetPlayerSkin(playerid)))
    {
        case SKIN_GROUP_MEDIC: spawnsList = gMedicalSpawns;
        case SKIN_GROUP_POLICE: spawnsList = gPoliceSpawns;
        case SKIN_GROUP_ARMY: spawnsList = gArmySpawns;
        default:
        {
            switch(gPlayerCitySelection[playerid])
            {
                case CITY_LOS_SANTOS: { spawnsList = gRandomSpawns_LosSantos; }
                case CITY_SAN_FIERRO: { spawnsList = gRandomSpawns_SanFierro; }
                case CITY_LAS_VENTURAS: { spawnsList = gRandomSpawns_LasVenturas; }
            }
        }
    }


    randSpawn = random(sizeof(spawnsList));
    SetPlayerPos(playerid,
        spawnsList[randSpawn][0],
        spawnsList[randSpawn][1],
        spawnsList[randSpawn][2]);
    SetPlayerFacingAngle(playerid, spawnsList[randSpawn][3]);

Alles anzeigen

AirM4X · 3. August 2016

Ich würde für Medic, Police, Army und Zivilisten jeweils ein eigenes Array erstellen und daraus dann jeweils eine Random-Position suchen.
Das ist am Ende zwar mehr Code, aber dann hast du es geordnet und kommst damit auch nicht durch einander.

So wie du es nämlich machst, funktioniert das nicht. Du willst spawnsList einen Wert zu weisen, obwohl das ein zweidimensionales Array ist.

Deazl_ · 3. August 2016

Hey,

jede Kategorie hat seine eigenen Spawns. Jedes Array ist gleich aufgebaut, jedoch hat die erste Dimension jeweils eine unterschiedliche Größe. Die zweite ist immer gleich. Beispiel:

Code

new Float:gMedicalSpawns[3][4] = {
	{1568.2250,-1693.5483,5.8906,177.0983},
	{-1604.8442,732.9442,-5.2344,37.6041},
	{1609.5922,1820.9641,10.8280,348.1614}
};

Ich möchte jeweils diesen Array quasi in eine neue Variable Klonen bzw. diesen Array übergeben. RAM-Technisch gesehen ist das zwar eher Kontraproduktiv, jedoch würde ich gerne das ganze Array in dieser Variable haben.

AirM4X · 3. August 2016

Ah, jetzt weiß ich was du vor hast. Das geht aber nicht so, schreibe das dann so:

C

new randSpawn = 0;
switch(GetSkinGroup(GetPlayerSkin(playerid)))
{
	case SKIN_GROUP_MEDIC:
	{
		randSpawn = random(sizeof(gMedicalSpawns));
		SetPlayerPos(playerid,gMedicalSpawns[randSpawn][0],gMedicalSpawns[randSpawn][1],gMedicalSpawns[randSpawn][2]);
		SetPlayerFacingAngle(playerid,gMedicalSpawns[randSpawn][3]);
	}
 	case SKIN_GROUP_POLICE:
	{
 		randSpawn = random(sizeof(gPoliceSpawns));
		SetPlayerPos(playerid,gPoliceSpawns[randSpawn][0],gPoliceSpawns[randSpawn][1],gPoliceSpawns[randSpawn][2]);
		SetPlayerFacingAngle(playerid,gPoliceSpawns[randSpawn][3]);
	}
 	case SKIN_GROUP_ARMY:
	{
 		randSpawn = random(sizeof(gArmySpawns));
		SetPlayerPos(playerid,gArmySpawns[randSpawn][0],gArmySpawns[randSpawn][1],gArmySpawns[randSpawn][2]);
		SetPlayerFacingAngle(playerid,gArmySpawns[randSpawn][3]);
	}
 	default:
  	{
   		switch(gPlayerCitySelection[playerid])
     	{
      		case CITY_LOS_SANTOS:
			{
   				randSpawn = random(sizeof(gRandomSpawns_LosSantos));
				SetPlayerPos(playerid,gRandomSpawns_LosSantos[randSpawn][0],gRandomSpawns_LosSantos[randSpawn][1],gRandomSpawns_LosSantos[randSpawn][2]);
				SetPlayerFacingAngle(playerid,gRandomSpawns_LosSantos[randSpawn][3]);
			}
   			case CITY_SAN_FIERRO:
			{
   				randSpawn = random(sizeof(gRandomSpawns_SanFierro));
				SetPlayerPos(playerid,gRandomSpawns_SanFierro[randSpawn][0],gRandomSpawns_SanFierro[randSpawn][1],gRandomSpawns_SanFierro[randSpawn][2]);
				SetPlayerFacingAngle(playerid,gRandomSpawns_SanFierro[randSpawn][3]);
			}
   			case CITY_LAS_VENTURAS:
			{
   				randSpawn = random(sizeof(gRandomSpawns_LasVenturas));
				SetPlayerPos(playerid,gRandomSpawns_LasVenturas[randSpawn][0],gRandomSpawns_LasVenturas[randSpawn][1],gRandomSpawns_LasVenturas[randSpawn][2]);
				SetPlayerFacingAngle(playerid,gRandomSpawns_LasVenturas[randSpawn][3]);
			}
   		}
     }
}

Alles anzeigen

Ja, das ist länger. Du musst ja immer einzeln z.B. auf gRandomSpawns_LasVenturas zu greifen.
Klonen geht nicht so ohne weiteres, da müsstest du die Werte erst alle zwischenspeichern, was aber weniger Sinn macht.

Deazl_ · 3. August 2016

Heyho @AirM4X,

ich bin gestern leider nicht mehr dazu gekommen noch zu antworten. Vielen Dank für deine Lösung. So funktionierts. Jedoch hätte ich eine kleine Frage:
Was die Programmierung, egal bei was, angeht bin ich wer der Codeverdopplung vermeidet. Also erstellte ich mir eine extra Methode (In einem Include welches ich eingebunden habe) an welchen ich eine Spieler-ID sowie ein Eindimensionales Array übergeben kann. Das Array wird korrekt übergeben. Die Daten darin sind auch korrekt, jedoch sawne ich, wenn ich diese Methode nutze, irgendwo in SF im Boden. Anschließend erscheint "Stay within the world boundaries". Nutze ich den selben Code direkt dort wo ich die Methode eigentlich aufrufen würde habe ich dieses Problem nicht. Dazu kommt, dass die Koordinaten zwar korrekt sind, ich auf der Minimap jedoch irgendwie abseits im Wasser angezeigt werde.

Hier meine Methode in meiner dzl_playerutil.inc:

C: dzl_playerutil.inc

/**
 * Spawnt den Spieler an dem angegeben Ort.
 */
stock SpawnPlayerAtLocation(playerid, location[])
{
    printf("%f, %f, %f, %f", location[0],location[1],location[2],location[3]);
    SetPlayerPos(playerid,
        location[0],
        location[1],
        location[2]);
    SetPlayerFacingAngle(playerid, location[3]);
}
/**
 * Spawnt den Spieler an einem aus der Liste zufällig gewähltem Ort.
 */
stock SpawnPlayerAtRandomLocation(playerid, locations[][4], locationAmount)
{
    new randSpawn = random(locationAmount);
    printf("%d: %d, %d, %d", playerid, sizeof(locations), locationAmount, randSpawn);
    SpawnPlayerAtLocation(playerid, locations[randSpawn]);
}

Alles anzeigen

Mein Methodenaufruf:
SpawnPlayerAtLocation(playerid, gMedicalSpawns[1]);

Testweise habe ich den switch-case Block auskommentiert:

Ebenfalls ist mir aufgefallen, dass egal welchen Skin ich habe bei GetSkinGroup immer der Wert vom Zivilisten zurückkommt:

C

#include <a_samp>
#include "../include/dzl_vardefs.inc"


stock GetSkinGroup(skinid)
{
    new group;
    switch(skinid)
    {
        case 280 .. 286, 288: group = SKIN_GROUP_POLICE;
        case 287: group = SKIN_GROUP_ARMY;
        case 274 .. 276, 308: group = SKIN_GROUP_MEDIC;
        default: group = SKIN_GROUP_CIVIL;
    }


    return group;
}


// Aufruf: GetSkinGroup(GetPlayerSkin(playerid))

Alles anzeigen

AirM4X · 3. August 2016

C

stock SpawnPlayerAtLocation(playerid, location[])
{
    printf("%f, %f, %f, %f", location[0],location[1],location[2],location[3]);
    SetPlayerPos(playerid,
        location[0],
        location[1],
        location[2]);
    SetPlayerFacingAngle(playerid, location[3]);
}

Das kann so nicht funktionieren. Du möchtest bestimmt, wenn man gMedicalSpawn[0] eingibt, das die Koordinaten aus der ersten Zeile bei location[0], location[1] und location[2] sind, richtig?
Das wird nicht funktionieren, daher mache das so:

C

stock SpawnPlayerAtLocation(playerid,Float:location1,Float:location2,Float:location3,Float:location4)
{
    printf("%f, %f, %f, %f", location1,location2,location3,location4);
    SetPlayerPos(playerid,
        location1,
        location2,
        location3);
    SetPlayerFacingAngle(playerid, location4);
}

So kannst du dann folgendes machen:

Code

SpawnPlayerAtLocation(playerid,gMedicalSpawn[0][0],gMedicalSpawn[0][1],gMedicalSpawn[0][2],gMedicalSpawn[0][3]);

Die andere Funktion sollte, wenn ich das richtig verstanden habe, so lauten:

C

stock SpawnPlayerAtRandomLocation(playerid, spawnPlatz)
{
	switch(spawnPlatz)
	{
	    case 0:
	    {
			new randSpawn = random(sizeof(gMedicalSpawns));
			SpawnPlayerAtLocation(playerid,gMedicalSpawns[randSpawn][0],gMedicalSpawns[randSpawn][1],gMedicalSpawns[randSpawn][2],gMedicalSpawns[randSpawn][3]);
	    }
	    case 1:
	    {
	        new randSpawn = random(sizeof(gPoliceSpawns));
			SpawnPlayerAtLocation(playerid,gPoliceSpawns[randSpawn][0],gPoliceSpawns[randSpawn][1],gPoliceSpawns[randSpawn][2],gPoliceSpawns[randSpawn][3]);
	    }
	    case 2:
	    {
	        new randSpawn = random(sizeof(gArmySpawns));
			SpawnPlayerAtLocation(playerid,gArmySpawns[randSpawn][0],gArmySpawns[randSpawn][1],gArmySpawns[randSpawn][2],gArmySpawns[randSpawn][3]);
	    }
	    case 3:
	    {
	        new randSpawn = random(sizeof(gRandomSpawns_LosSantos));
			SpawnPlayerAtLocation(playerid,gRandomSpawns_LosSantos[randSpawn][0],gRandomSpawns_LosSantos[randSpawn][1],gRandomSpawns_LosSantos[randSpawn][2],gRandomSpawns_LosSantos[randSpawn][3]);
	    }
	    case 4:
	    {
	        new randSpawn = random(sizeof(gRandomSpawns_SanFierro));
			SpawnPlayerAtLocation(playerid,gRandomSpawns_SanFierro[randSpawn][0],gRandomSpawns_SanFierro[randSpawn][1],gRandomSpawns_SanFierro[randSpawn][2],gRandomSpawns_SanFierro[randSpawn][3]);
	    }
	    case 5:
	    {
	        new randSpawn = random(sizeof(gRandomSpawns_LasVenturas));
			SpawnPlayerAtLocation(playerid,gRandomSpawns_LasVenturas[randSpawn][0],gRandomSpawns_LasVenturas[randSpawn][1],gRandomSpawns_LasVenturas[randSpawn][2],gRandomSpawns_LasVenturas[randSpawn][3]);
	    }
	}
}

Alles anzeigen

Wenn du bei spawnPlatz 0 einträgst, dann wird eine random Position von gMedicalSpawns gesucht. Bei 1 dann von gPoliceSpawns, usw.

Das Problem mit dem Skin:
Lasse dir deine skin-id mal printen:

C

stock GetSkinGroup(skinid)
{
    new group;
	printf("skinid: %i",skinid);
	switch(skinid)
    {
        case 280 .. 286, 288: group = SKIN_GROUP_POLICE;
        case 287: group = SKIN_GROUP_ARMY;
        case 274 .. 276, 308: group = SKIN_GROUP_MEDIC;
        default: group = SKIN_GROUP_CIVIL;
    }
    return group;
}

Alles anzeigen

Fehler im Switch

breadfish.de 12. März 2022

Ähnliche Themen

[SAMMELTHREAD] Kleine Scripting Fragen

string und switch problem

Seit Umstellung laden nur noch 2 Fraktionen