Anfänger Gamemode MySQL basierend

Ich habe das Script erstmal nur überflogen. Es ist natürlich sehr minimalistisch gehalten und hat somit kaum Umfang aber ich möchte vor allem da es sich an Script Anfänger richten soll und du, so scheint es mir, selbst einer bist mir zwei grundlegende Dinge aufgefallen sind, die ich nennen möchte. Es kann natürlich sein, dass ich hierbei nennenswertes übersehen habe, aber ich nenne einfach mal was mir auf den ersten Blick aufgefallen ist.

1.
Fangen wir mit den Befehlen an. Hier verwendest du ocmd was an sich nichts schlimmes ist.
Wo liegt jetzt das Problem? Der Fakt dass du den Befehl immer nach diesem Schema aufbaust "ocmd:befehlsname(playerid,params[])"
Unabhängig davon ob für den Befehl Paramater benötigt werden und man sich somit eigentlich params[] sparen könnte oder nicht.

2.
Dein Aufbau deiner Textdraws. Wenn ich mir das anschaue erstellst du Spielerbezogene Textdraws beim Connecten unabhängig davon ob man so viele Textdraws braucht und ob man es nicht hätte anders, Textdrawsparender mit gleichem Ergebnis gestalten können, möchte ich hier anmerken dass du temporäre Textdraws "createn" und wieder "destroyen" also löschen solltest und nicht "createn" und dann einfach Hiden solltest denn so ist immer noch ein "Slot" belegt und diese sind auch nur begrenzt. An dieser Stelle verlinke ich mal eine Liste zu allen Limits

Das Problem mit den Params kannst du auch ganz einfach lösen.

#pragma unused params

Außerdem finde ich das alles etwas sehr übertrieben, hätte man kleiner packen können.
Wozu eigentlich 200 TextDraws? Habe es lediglich überflogen; das Script soll jedoch für Anfänger sein. Wie soll dort einer damit klarkommen? Das ist ja kein Grundwissen.

Gruß

@Jeff_Hardy finde diese gerade für einen Anfänger unnütz, da man am Anfang i.d.R. nur mit den süßen Dialogen arbeitet.

Aber ist ja Ansichtssache.

Zitat von Jeff_Hardy

Normalerweise ist alles sehr Anfänger freundlich, denn wenn man sich alles anschaut ist es selbsterklärend.

Dann erklär mal bitte einem Anfänger folgendes System:

• //Nicht ändern
• #define cmd:%1(%2) forward cmd_%1(%2);\
• public cmd_%1(%2)
• public OnPlayerCommandText(playerid,cmdtext[])
• {
• cmdtext[0]='_';
• new cmdStr[128],cmdParams[128],bool:cmdAffe,cmdFu;
• for(new i;i<128;i++)
• {
• if(cmdtext[i]=='\0')break;
• if((cmdtext[i]==' ')&&(!cmdAffe))
• {
• cmdAffe=true;
• continue;
• }
• if(!cmdAffe)cmdStr[i]=cmdtext[i];
• else
• {
• cmdParams[cmdFu]=cmdtext[i];
• cmdFu++;
• }
• }
• format(cmdStr,128,"cmd%s",cmdStr);
• if(!strlen(cmdParams))format(cmdParams,128," ");
• new i,cmdStr2[128];
• while(cmdStr[++i])
• {
• if ('A' <= cmdStr[i] <= 'Z') cmdStr[i] |= 0x20;
• if(cmdStr[i] == 'ä' cmdStr[i] == 'Ä')
• {
• format(cmdStr2,128,"%s",cmdStr);
• cmdStr2[i] = 'a';
• cmdStr2[i+1] = 'e';
• for(new o = i+2; o < 128; o++)
• {
• cmdStr2[o] = cmdStr[o-1];
• }
• format(cmdStr,128,"%s",cmdStr2);
• }
• else if(cmdStr[i] == 'ö' cmdStr[i] == 'Ö')
• {
• format(cmdStr2,128,"%s",cmdStr);
• cmdStr2[i] = 'o';
• cmdStr2[i+1] = 'e';
• for(new o = i+2; o < 128; o++)
• {
• cmdStr2[o] = cmdStr[o-1];
• }
• format(cmdStr,128,"%s",cmdStr2);
• }
• else if(cmdStr[i] == 'ü' cmdStr[i] == 'Ü')
• {
• format(cmdStr2,128,"%s",cmdStr);
• cmdStr2[i] = 'u';
• cmdStr2[i+1] = 'e';
• for(new o = i+2; o < 128; o++)
• {
• cmdStr2[o] = cmdStr[o-1];
• }
• format(cmdStr,128,"%s",cmdStr2);
• }
• else if(cmdStr[i] == 'ß')
• {
• format(cmdStr2,128,"%s",cmdStr);
• cmdStr2[i] = 's';
• cmdStr2[i+1] = 's';
• for(new o = i+2; o < 128; o++)
• {
• cmdStr2[o] = cmdStr[o-1];
• }
• format(cmdStr,128,"%s",cmdStr2);
• }
• }
• if(CallLocalFunction(cmdStr,"ds",playerid,cmdParams)) return 1;
• cmdtext[0]='/';
• new string[100];
• format(string,sizeof string,"Diesen Befehl gibt es leider nicht! (%s)",cmdtext);
• return SendClientMessage(playerid,0xFF0000FF,string);
• }
• #define OnPlayerCommandText cmdOnPlayerCommandText

Da bin ich ja mal auf eine Erklärung gespannt was es sein soll und wo der Sinn des Systems ist.
Ich weiß wozu das System dient deswegen macht es auch absolut keinen Sinn da dieses Problem mittlerweile soweit ich weiß sowieso behoben wurde und selbst wenn nicht, wenn ich mich recht entsinne kümmert sich die ocmd Include sowieso darum und da eh alle Befehle mit ocmd geschrieben wurde klärt sich das von selbst.

Falls du noch mehr Feedback möchtest:
- Zeile 162: Netter Hinweis, es schadet aber nicht es dann auch zu benutzen in Zeile 169
- Die Textdraws die du erstellst, sind nicht Spielerbezogen (LoginDraw1 - LoginDraw9,LoginDraw14-15). Du überschreibst die Variablen der Textdraws, sobald mehr als 1 Spieler auf dem Server ist (Hast du also nur alleine getestet). Selbst die PlayerTextdraws sind nicht wirklich Spielerbezogen. Du überschreibst auch hier immer die Variablen (LoginDraw10-LoginDraw13)
- Du erstellst pro PlayerConnect etwa 50 Objekte (Zeile 449 bis 502). Da du die DynamicObjects nicht für den Spieler erstellst (playerid Parameter), erstellst du dir damit unnötige Objekte. Das erstellen gehört einmalig, zum Beispiel OnGameModeInit. (Hast du ebenfalls nur alleine getestest)
- /restart: Eventuell alle Spieler speichern?
- /gotoxyz: Wie soll Zeile 854 und 855 gemeinsam funktionieren?
- /gethere: Der Spieler sitzt in einem Fahrzeug. Was nun?
- /veh: Wie soll Zeile 947 und 948 gemeinsam funktionieren?
- SavePlayer(): Totaler Schrott. Eine WHERE Klausel fehlt, ohne die macht das Speichern überhaupt keinen Sinn. Du würdest immer die Daten fremder Spieler überschreiben. (Hast du auch nur alleine getestet,*wenn überhaupt). Die Platzhalter sind falsch bei Geld und Neuling.

Das sollte für den Anfang reichen

Zitat von Jeff_Hardy

Ja gut das ist einfach das man cmd: benutzen kann.
Du hast vollkommen Recht, dass Neulinge das nicht verstehen können.
Das ist aber das einzige was unverständlich ist

Daran habe ich gar nicht gedacht

Sorry :o

Nein, die Hauptfunktion ist das ersetzen von Ä durch ae. Wenn ein Befehl /hälp heißen würde würde er durch dieses System

• if(cmdStr[i] == 'ä' cmdStr[i] == 'Ä')
  
• {
  
• format(cmdStr2,128,"%s",cmdStr);
  
• cmdStr2[i] = 'a';
  
• cmdStr2[i+1] = 'e';

Daraus ein /haelp machen.

Was merkt man? Absolut unnötig und selbst du hast es nicht mal richtig verstanden. Keine Ahnung woher du den Code hast aber selbst erstellt bezweifel ich doch sehr.

Ich finde das Script mega gut für Anfänger ist das richtig gut geeignet sich da einzufinden.
Ich komme damit auch ganz gut klar

MFG. Ray

Warum muss ich mich dort nicht Registern? im Login-System... steht nur gebe dein Passwort ein dann gebe ich irgendwas ein und es funkt hilfe

Zitat von Jeff_Hardy

Bist du auch richtig mit der MySQL Datenbank verbunden usw?

wollte das lokal teste ohne MYSQL^^

Zitat von Jeff_Hardy

Du brauchst trotzdem eine Datenbank, ohne MySQL wirst du es nicht ordentlich zum laufen kriegen, da für den Login auch MySQL benötigt wird.
Downloade einfach XAMPP, starte die MySQL Server und tippe im Browser 127.0.0.1/phpmyadmin ein, dort fügst du die Datenbank ein die im Scriptpaket dabei war.
Dann im Script nur deine MySQL Daten anpassen und fertig sollte der Salat sein.

Aber dort steht kein Register