Ein wichtiger Teil beim Entwurf großer Softwaresysteme ist die Minimierung der Kopplung zwischen Komponenten und Klassen. Dies kann durch die Verkapselung der Implementierung verschiedener Komponenten erfolgen. Je mehr Komponenten voneinander wissen, desto höher ist die Abhängigkeit zwischen ihnen.
Die Verwendung des Singleton-Patterns bedeutet, dass es an eine Implementierungsklasse gekoppelt wird (nicht an eine Schnittstelle oder an eine Abstraktion). Es wäre besser, wenn jede Komponente, die den GameManager verwendet, stattdessen ein GameManagerInterface verwendet, so dass die Implementierung leicht geändert oder erweitert werden kann. Es ist schwierig, die Abhängigkeit von der konkreten Singleton-Klasse zu verbergen, denn irgendwo muss man die statische Methode aufrufen.
...
IGameManager iGameManager = GameManager.getInstance();
...
das Muster Singleton verwenden wie das ist ...:
GameManager.instance.someOtherClass.someVariable
... ist nie eine gute Idee. Dies würde die aufrufende Klasse mit der Implementierung jeder der aufgerufenen Klassen verbinden. Es ist am besten, nur mit Ihren direkten Nachbarn zu kommunizieren, wie im "Gesetz von Demeter" (a.ka.a "Tell Do not Ask" oder "Sprechen Sie nicht mit Fremden").
Vielleicht ist es für Sie an der Zeit, über den Zweck des Singleton-Pattern nachzudenken: Das Singleton-Pattern wird verwendet, um sicherzustellen, dass es nur eine Instanz einer Klasse gibt. Nichts mehr.
Die Verwendung der statischen "getInstance" -Methode überall in Ihrem Code führt zu versteckten Abhängigkeiten, wie bereits erwähnt. Sie verwenden das Singleton-Muster als Ersatz für eine globale Variable. Dies macht Ihr System fragil, da jedes Bit Ihres Codes von der Implementierung der Singleton-Klasse abhängig ist. Wenn Sie diese Klasse ändern, kann Ihr gesamtes System zum Absturz gebracht werden.
Obwohl es kein riesiges Programm ist, ist es in der Tat am besten, so früh wie möglich zu lernen. Also sage ich, ich habe eine Klasse namens 'Charakter'. Und diese Zeichenklasse möchte ein Detail wissen, das im Singleton gespeichert ist, sagen wir 'levelUpBoundary' für wie hoch der Charakter sein kann. Im Moment würde ich das von 'GameManager.instance.levelUpBoundary' bekommen. Was du sagst ist, dass ich eine Variable in meinem 'Charakter'-Skript wie folgt erstellen soll: ' IGameManager iGameManager = GameManager.getInstance(); 'genau wie du gesagt hast? Wenn ja, wie genau? –
Erstens: es gibt eine Sache, die Kohäsion genannt wird, was bedeutet, dass Methoden nah an den Daten sein sollten, an denen sie arbeiten ... Sie sollten über die Struktur nachdenken. Ist es wirklich notwendig, dass der GameManager etwas über die levelUpBoundaries eines Charakters weiß? Meiner Meinung nach ist es ein Implementierungsdetail der Klasse Character. Zweitens: Wie bereits erwähnt, ist das Singleton-Muster kein Ersatz für eine globale Variable, was Sie hier verwenden. Sie sollten den GameManager als normales Objekt in der Hauptmethode erstellen und als Abstraktion in die anderen Objekte einfügen. – Janis