Was ist besser? switch oder if ?

GeForce8800GTX · 2. Januar 2013

Guten Morgen Jungs und Mädels.

Ich hätte mal eine etwas allgemeinere Frage vorallem an die PAWN-Experten unter uns.
Mich würde interessieren ob es ein großartigen unterschied zwischen einem switch und einer if Anweisung?

z.B
Was währe besser?

if (PlayerInfo[playerid][pMember] == 1 || PlayerInfo[playerid][pLeader] == 1)// LS Police spawn
{
SetPlayerToTeamColor(playerid);
GivePlayerWeapons(playerid);
rand = random(sizeof(gCopPlayerSpawns));
SetPlayerPos(playerid, gCopPlayerSpawns[rand][0], gCopPlayerSpawns[rand][1], gCopPlayerSpawns[rand][2]); // Warp the player
SetPlayerFacingAngle(playerid,random(360));
SetPlayerInterior(playerid,3);
PlayerInfo[playerid][pInt] = 0;
SetPlayerSkin(playerid,PlayerInfo[playerid][pChar]);
return 1;
}

oder

switch(PlayerInfo[playerid][pMember])
{
case 1:
{
SetPlayerToTeamColor(playerid);
GivePlayerWeapons(playerid);
rand = random(sizeof(gCopPlayerSpawns));
SetPlayerPos(playerid, gCopPlayerSpawns[rand][0], gCopPlayerSpawns[rand][1], gCopPlayerSpawns[rand][2]); // Warp the player
SetPlayerFacingAngle(playerid,random(360));
SetPlayerInterior(playerid,3);
PlayerInfo[playerid][pInt] = 0;
SetPlayerSkin(playerid,PlayerInfo[playerid][pChar]);
return 1;
}
default:
{
switch(PlayerInfo[playerid][pLeader])
{
case 1:
{
SetPlayerToTeamColor(playerid);
GivePlayerWeapons(playerid);
rand = random(sizeof(gCopPlayerSpawns));
SetPlayerPos(playerid, gCopPlayerSpawns[rand][0], gCopPlayerSpawns[rand][1], gCopPlayerSpawns[rand][2]); // Warp the player
SetPlayerFacingAngle(playerid,random(360));
SetPlayerInterior(playerid,3);
PlayerInfo[playerid][pInt] = 0;
SetPlayerSkin(playerid,PlayerInfo[playerid][pChar]);
return 1;
}
}
}
}

Abgesehen davon das die 2te Methode mehr Zeilen in Anspruch nimmt
Ich habe mir Gedanken gemacht und bin der Auffassung das die 2te Methode mit switch besser währe
den da macht er doch nur 1 Abfrage in dem "PlayerInfo[playerid][pMember]"
und spring dann in den Case

bei dem Beispiel währe natürlich die erste Methode einfacher
aber wenn man es sieht auf 24 Abfragen
if(member == 1 leader == 1 )
...
...
..
if(member == 24 leader == 24)
..

dann ist doch ein switch obtimaler oder nicht?
Ich hoffe ich habe die Frage deutlich und verständlich ausgedrückt .

Und würde mich über zahlreiche Meinungen freuen.

Mit freundlichstem Gruß
GeForce

SenTiZz · 2. Januar 2013

Ich selber bin für if da dass etwas leichter ist zu schreiben finde ich...

Aber das es je nach angewohnheit

LogispireMode · 2. Januar 2013

ich habe von if-Abfragen besseres gehört^^ mal sehen was profis wie kaliber oder blackace oder mysteriöser unbekannter oder who ever dazu sagen^^

[COE]CodeX · 2. Januar 2013

Ich denke das kommt auf die Anwendung an

Wenn du nur eine Abfrage hast z.B
if(pInfo[playerid][Level] >= 5)
{
...
}
klappt es mit if wunderbar
aber wenn du 20 oder mehr Möglichkeiten/Abfragen hast ist switch denke ich besser
z.B
switch(pInfo[playerid][checkpoint])
{
case 0:
case 1:
.
.
.
}
besser als
if(pInfo[playerid][checkpoint] == 1)
{

}
...
else if(pInfo[playerid][checkpoint] == 2)
{

}
.
.
.

Ich hoffe es ist halbwegs verständlich
Hatte außerdem lange nichts mehr mit Pawn am Hut

Jerome (: · 2. Januar 2013

Switch ist schneller als IF und wirk Profisioneller

Schakon · 2. Januar 2013

ich benutze meistens if

Shine · 2. Januar 2013

Deutlich an Zeilen würdest du alleine sparen, wenn du die Member-Variable eines Leaders ebenfalls auf 1 stellst.
D.h. beim LSPD-Leader trifft auch die Abfrage ''PlayerInfo[playerid][pMember] == 1'' zu.
So könntest du beim Spawn die ganzen Leaderabfragen weglassen, dann wäre ein Switch mit Sicherheit einfacher. Beim Spawn macht es ja keinen Unterschied ob derjenige Leader oder Member ist.

Ob jemand Leader ist brauchst du ja eig. nur bei den Leader-Befehlen abfragen...und da würde die Member-Abfrage keinen Sinn machen.

FrameWork · 2. Januar 2013

Also generell benutzt man ein Switch wenn es mehrere Fallunterscheidungen gibt. So kenne ich aus meinem Beruf und so wird es auch angewendet. Ich empfinde ein switch auch besser wartbar für die Arbeiten in der Zukunft.

Jerome (: · 2. Januar 2013

Bringt aber nichts, wenn er Member nicht auf Fraktions Variable setzt. Dazu könnte er einfach auch ein Stock/Public machen und dort eine Fraktion abfrage erstellen.

stock/Public IsPlayerInFraktion(playerid, frakid)
{
if(getmemberabfrage... == frakid || getleaderabfrage == frakid) return 1;
}

Tround · 2. Januar 2013

hier wird alles recht gut erklärt
http://wiki.sa-mp.com/wiki/Control_Structures#switch

[quote]An important thing to note here is the different ways in which ifs and switches are processed:

switch (SomeFunction())
{
case 1: {}
case 2: {}
case 3: {}
}

That will call SomeFunction() ONCE and compare it's result 3 times.

if (SomeFunction() == 1) {}
else if (SomeFunction() == 2) {}
else if (SomeFunction() == 3) {}

That will call SomeFunction() three times, which is very inefficient

Sharpadox · 2. Januar 2013

ich finde switch besser, denn das ist schneller und sieht professioneller aus (by BlackAce).
Ausserdem finde ich, sieht der auch ordentlicher aus
Hier zB ein kleiner Vergleich if/switch (wegen Ordentlichkeit):

Spoiler anzeigen

Akino Kiritani · 2. Januar 2013

switch hat das Problem,
da es sehr statisch ist und keinerlei flexiblität bietet,
deswegen ist es auch relativ schnell.

Es wirkt ein wenig Professioneller, ist es aber nicht umbedingt
Man muss immer einen festen bezug haben, deswegen hat man bei den Dialogen nen recht guten Vorteil, wenn man switch case nutzt

[DT]midomen · 2. Januar 2013

Ganz ehrlich mach es so wie du es gelernt hast ich habe z.b von anfang an mit if abfragen gescriptet und das mache ich immer noch wenn du es mit switch & case gelernt hast dann benutzt es.

Acrio · 2. Januar 2013

Blackace also wenn man z.B. die Adminränge durchgeht kann man doch auch besser switch nutzen oder nicht? Dann wäre nämlich case 0: normaler Member und 1 halt Admin usw. Da hat man den festen bezug.

Jerome (: · 2. Januar 2013

So sieht es aus, wenn du Switch hast musst die switch() ein festen bezug geben. Und bei einer Abfrage kannst du ja noch andere Sachen hinzufügen.

Akino Kiritani · 2. Januar 2013

Zitat von Acrio

Blackace also wenn man z.B. die Adminränge durchgeht kann man doch auch besser switch nutzen oder nicht? Dann wäre nämlich case 0: normaler Member und 1 halt Admin usw. Da hat man den festen bezug.

Das geht nicht, aber wenn man z.B. mit wechselnden IDs arbeitet,
Beispiel TextDraws

public OnPlayerClickTextDraw(Text:clickedid)
{
switch(_:clickid)
{
case Textdraw1: //tu was
case Textdraw2: //tu was
}
return;
}
Hier hast du keinen besten Bezuge, daher ist das da unbrauchbar
machst du das aber so

public OnPlayerClickTextDraw(Text:clickedid)
{
switch(_:clickid)
{
case 1: //tu was
case 2: //tu was
}
return;
}
Funktionert das weil du ihm so vorgibst, welche ID er zu prüfen hat.
Im allgemeinen Gebrauch ist aber "Lösung" ziehmlich schlecht,
da sich die IDs auch mal ändern können.
Da bleibt also nur das gute alte if

Sharpadox · 2. Januar 2013

Also sowas z.B. auf switch umzuschreiben wäre nicht so gut

if(playerid == 0) {
//Funktion
}
else if(pID == 0) {
//Funktion
}
else if(xID == 0) {
//Funktion
}

Weil es keinen festen Bezug eben hat. Weil es heißt ja "switch(variable)"
Dann bezieht sich das switch nur auf diese eine variable, weshalb es beim
obrigen Beispiel schwachsinnig ist, da es 3 verschiedene Variablen sind.

Also ist switch nicht immer sinnvoll.

@BlackAce:
Das geht doch
Was er meint sind die Adminränge, also sowas:

Spoiler anzeigen

Acrio · 2. Januar 2013

Blackace wieso soll es nicht gehen?
So ist es doch richtig und funktioniert. (Mal ein Beispiel aus meinem Script).

switch (pInfo[playerid][pAdmin])
{
case 0:
{
format(str, sizeof(str), "[%d:%d:%02d] %s: %s", Stunde, Minute, Sekunde, Name, text);
}
case 1:
{
format(str, sizeof(str), "[%d:%d:%02d] Supporter %s: %s", Stunde, Minute, Sekunde, Name, text);
}
case 2:
{
format(str, sizeof(str), "[%d:%d:%02d] Admin %s: %s", Stunde, Minute, Sekunde, Name, text);
}
case 3:
{
format(str, sizeof(str), "[%d:%d:%02d] Moderator %s: %s", Stunde, Minute, Sekunde, Name, text);
}
case 4:
{
format(str, sizeof(str), "[%d:%d:%02d] Serverleiter %s: %s", Stunde, Minute, Sekunde, Name, text);
}
}

Akino Kiritani · 2. Januar 2013

Zitat von Sh12

Also sowas z.B. auf switch umzuschreiben wäre nicht so gut

was du da hast, wäre nicht möglich, zumindest in dem Kontext.

Nimmt man das aus dem Kontext ist das wiederrum möglich

Acrio
So wie du es machst,
hab ich doch gesagt geht es.
Es wäre nur bei veränderlichen Dingen, nicht möglich.
Siehe mein Beispiel

Sharpadox · 2. Januar 2013

Zitat von BlackAce

was du da hast, wäre nicht möglich, zumindest in dem Kontext.

Nimmt man das aus dem Kontext ist das wiederrum möglich

Acrio
So wie du es machst,
hab ich doch gesagt geht es.
Es wäre nur bei veränderlichen Dingen, nicht möglich.
Siehe mein Beispiel

Alles anzeigen

Möglich wäre das schon
Aber das wäre total unnötig, das zu machen:

switch(playerid)
{
case 0:{
//Funktion
}
}
switch(pID)
{
case 0:{
//Funktion
}
}
switch(xID)
{
case 0:{
//Funktion
}
}