Guter Code vs schlechter Code

Finde das mit der Formatierung wichtig, da ansonsten immer wieder Warnings kommen.
Recht ordentlich erklärt, top !

Wohl eher für Neulinge, da Fortgeschrittene Skripter das ganze schon beachten, denke ich zumindestens. Aber trotzdem gut erklärt.

Schönes Tutorial ich glaube so eins gab es bisher auch noch nicht.
sollten sich aber auch einige mal kompett durchlesen haha

Ich persönlich finde keinen Sinn bei jedem scheiß Kommentare zu setzen.
Im Falle der Veröffentlichung ist es doch nicht mein Problem ob es jemand versteht, immerhin wird es "kostenlos" angeboten.
Ebenso mit Funktionen, Ich habe mir mal sagen lassen, dass viieele Funktionen viele Ressourcen fressen, bestes bsp: Spielername.
Ob es stimmt ist mir nie aufgefallen aber naja.
Ansonsten finde ich es relativ unnötig dafür einen Thread zu machen, jeder der mit Scripten anfängt, wird sich aufregen das sein Code unübersichtlich ist,
Somit wird er es selber korrigieren und brauch nicht extra ein tutorial.

Aber naja meine Meinung.

Wo ich das gesehen hab hab ich mir nur gedacht was das bringen soll...
GetPlayerName(playerid){
new pName[MAX_PLAYER_NAME];
return GetPlayerName(playerid, pName, sizeof(pName));
}
Ich hoffe, das dir bewusst ist was GetPlayerName returnt...
Desweiteren würd ne definierung da besser aufgehoben sein als ne extra funktion

Zu den Kommentaren fällt mir nix ein denn ich find es Sinnlos
Das was wichtig ist schreibe ich ganz oben rein aber jedesmal hinter nen Command was schreiben find ich Zeitverschwendung.

dcmd_setlevel(playerid,params[])
{
new pID, Level;
if(sscanf(params, "ud",pID,Level))return SendClientMessage(playerid,0xFF0000FF,"Benutze: /setlevel [ID][Level]!"); //Falls nur /setlevel eingegeben wurde
if (pID == INVALID_PLAYER_ID)return SendClientMessage(playerid,0xFF0000FF,"Spielerid ist nicht vorhanden!");
// Falls eine falsche ID angegeben wurde, wird 0 returnt ;)
if(Spieler[playerid][AdminLevel]< 3)return SendClient..
new adminstring[128],levelsetter[MAX_PLAYER_NAME], playername[MAX_PLAYER_NAME];
GetPlayerName(pID,playername,sizeof playername);
format(adminstring,sizeof(adminstring),"Admins/%s.sav",playername);
GetPlayerName(playerid,levelsetter,sizeof(levelsetter));
Spieler[pID][AdminLevel] = Level; //Hier wird die Levelvariable mit dem neu eingegebenen Level überschrieben
dini_IntSet(adminstring,"AdminLevel",Level); // Und hier das neue Level in der Datei gespeichert.
format(adminstring,sizeof(adminstring),"%s hat dein Level auf %d gesetzt!",levelsetter, Level);
SendClientMessage(pID,0x33FF33FF, adminstring);
SendClientMessage(playerid, 0xFF0000FF, "Dein Level reicht nicht aus!");
return 1;
}
So spart man sich das Problem mit dem Einrücken.

CMD:test(playerid, params[])
{
if(IsPlayerAdmin(playerid){
if(IsPlayerInAnyVehicle(playerid){
if(IsPlayerNPC(playerid){
}
}
}
return 0;
}
Ist auch nicht sehr schön.
CMD:..
if(IsPlayerAdmin(playerid) && IsPlayerInAnyVehicle(playerid) && IsPlayerNPC(playerid))//wenn diese Bedingungen erfüllt sind, dann..
So spart man sich die Unübersichtlichkeit.

GetPlayerName(playerid){
new pName[MAX_PLAYER_NAME];
return GetPlayerName(playerid, pName, sizeof(pName));
}
Wurde bereits 100x besprochen, warum das Müll ist.

Zitat von Slash™

Ich persönlich finde keinen Sinn bei jedem scheiß Kommentare zu setzen.

Mir persönlich helfen Kommentare immens, falls ich nach einer langen Pause mal wieder an meinem Projekt weiterarbeiten möchte. In dem Fall sind die Kommentare nur für mich und ich setze diese auch an bestimmten Passagen im Code, um mich schnell wieder zurechtzufinden.

Zitat von Twonky

Mir persönlich helfen Kommentare immens, falls ich nach einer langen Pause mal wieder an meinem Projekt weiterarbeiten möchte. In dem Fall sind die Kommentare nur für mich und ich setze diese auch an bestimmten Passagen im Code, um mich schnell wieder zurechtzufinden.

Wie gesagt jedem seine meinung, aber..
Wenn du ein Grooßes Script mit sagen wir mal 40k Zeilen hast und bei jedem Zeug n Kommentar dabei ist, bezweifel ich, dass es übersichtlich ist.

Ich nutze ebenfalls kaum Kommentare.
Ich strukturiere mein Code so, dass ich es auch ohne Kommentare hinbekommen. Es gibt nur eine Sache, wo ich Kommentare verwende(n würde),
nähmlich in der Kopfzeile, wo ich reinschreiben würde, "wann" ich "was" gemacht habe. Denn wenn ich zu lange nicht gescriptet habe, weiß ich
dann immer direkt, was ich zuletzt gemacht habe, und was ich weitermachen muss.

Zum Tutorial:
Gut für Anfänger, aber für Fortgeschrittene eher nicht, denn sowas beachten sie (eigentlich). Ich würde niemals scriptet können, ohne alles ordentlich zu strukturieren
oder einzurücken. Da kommt man ja mit seinem eigenen Code nichtmehr klar, wenn man nichtmal weiß, welche Klammer wo geöffnet / geschlossen wurde. (Siehe TT: "Klammern").
Aber auf jeden Fall gutes Tutorial für Anfänger. Ich kenne / kannte selber schon Leute, die nichts eingerückt haben, und dann die Übersicht verloren