Funktion nur 1 Rueckgabewert

[Dennis190]

Hallo,

das ist jetzt nur pseudo code. Aber wie ihr seht erzeugt die Funktion eine map. Sie durchlaeuft 2 for loops.
Je nachdem ob die if Bedingung greift, wird in der map ein Eintrag "success" oder "failure" einem key zugewiesen.
Wie ihr auch seht gibt es eine 2.Variable counter die zaehlt wie oft success in die map eingetragen wurde.
Da es sich aber um eine funktion handelt und ich nur 1 Wert zurueckgeben kann, geht mir der counter leider verloren.
Mir bleibt wohl nichts anderes uebrig als nochmal ueber die map mit dem Keyset zu iterieren und nochmal den counter zu berechnen.

HashMap<String,String> createMap(list1, list2 )
{
HashMap<String,String> map = new HashMap<String,String>();
int counter = 0;

for(.....) {
for(......)
{
if(condition)
{
map.put(key,success);
counter++;
}
}
map.put(key,failed);
}
return map;

}

 

[fhoffmann]

Und weshalb schreibst du nicht eine eigene Klasse, die du zurückgibst:

class MeinRueckgabewert { // sinnvolleren Klassennamen wählen
  Map<String,String> map;
  int counter;
}

 

[Dennis190]

Ja das ist eine Moeglichkeit. Aber dann kann ich die Methode nicht mehr createMap nennen
Aber danke, ist zumindest eine Moeglichkeit. Und ja wie du auch schon festgestellt hast. Eine Namen finden.(nicht so einfach)
Die Methode gibt dann letztendlich eine Map und den Counter zurueck.

ich wuerde die Klasse dann als public static Helper als inner class deklarieren.

 

[mihe7]

3

Aber dann kann ich die Methode nicht mehr createMap nennen

Die Aufgabe der Methode besteht auch nicht darin, irgendeine Map zu erstellen...

Je nachdem ob die if Bedingung greift, wird in der map ein Eintrag "success" oder "failure" einem key zugewiesen.

... was soll das Ganze also werden, wenn es fertig ist?

public static Helper

Das ist so ziemlich der schlechteste aller Namen (X wäre kürzer und würde genauso viel aussagen).

Mir bleibt wohl nichts anderes uebrig als nochmal ueber die map mit dem Keyset zu iterieren und nochmal den counter zu berechnen.

Du könntest z. B.
1. ein Objekt mit dem Ergebnis zurückgeben (s. Antwort von @flopalko)
2. die Anzahl zurückgeben und eine gegebene Map füllen
3. die erzeugte Map zurückgeben und die Anzahl an einen gegebenen Consumer (oder anderen Callback) weitergeben

 

[flopalko]

1. ein Objekt mit dem Ergebnis zurückgeben (s. Antwort von @flopalko)

Schön, dass du mich erwähnst, obwohl ich gar nix geschrieben hab

 

[Dennis190]

public static Helper

Das ist so ziemlich der schlechteste aller Namen (X wäre kürzer und würde genauso viel aussagen).

Es ist ja eine innere Klasse also wird sie immer nur in Verbindung mit der umgebenden Klasse aufgerufene. Also z.B. TestService.Helper, folglich ist das schon aussagekraeftig. Wie soll man sonst eine Klasse nennen die nur eine Map und einen counter enthaelt.


2. die Anzahl zurückgeben und eine gegebene Map füllen
3. die erzeugte Map zurückgeben und die Anzahl an einen gegebenen Consumer (oder anderen Callback) weitergeben

zu 2. Ja ich koennte die map als Parameter uebergeben und sie dann in der Methode fuellen. Ich denke das ist wohl die beste Moeglichkeit.

zu 3. das versteh ich leider nicht. Klar die map zurueckgeben, aber wie soll ich dann die Anzahl an einen Consumer weitergeben ?

 

[kneitzel]

Es ist ja eine innere Klasse also wird sie immer nur in Verbindung mit der umgebenden Klasse aufgerufene. Also z.B. TestService.Helper, folglich ist das schon aussagekraeftig. Wie soll man sonst eine Klasse nennen die nur eine Map und einen counter enthaelt.

Die Frage ist doch wirklich, was Du da überhaupt genau machst, so dass Du diese innere Klasse brauchst? Was ist darin? Was willst Du kapseln?

Wenn die Kombination aus dieser Map und Count zusammen einen Sinn machen, dann gib dem einen sinnvollen Namen und kapsel es in der inneren Klasse. Und dann könnte die innere Klasse eine Instanz von sich zurück geben. Aber wenn diese Kombination keinen Sinn macht, dann ist die Frage, wieso Du beider zurück geben musst. Dann ist es evtl. etwas, das anders aufgebaut werden sollte, sprich das OO Design müsste überdacht werden. Da könnte es also durchaus sinnvoll sein, das noch einmal zu beleuchten.

Und bezüglich Consumer / Callback: Du übergibst dann einfach eine Instanz, in der Du dann eine Methode aufrufst. Also hast Du z.B. ein Interface, in dem Du nur setNumberOfResults(int) definierst und davon übergibst Du dann z.B. eine Instanz an die Methode. Dann kann vorm Ende einmal darauf setNumberOfResults(counter) aufgerufen werden.

 

[mihe7]

Schön, dass du mich erwähnst, obwohl ich gar nix geschrieben hab

Ooops. Da war wohl nicht ganz auf der Höhe: sollte natürlich @fhoffmann heißen.

 

[mihe7]

Es ist ja eine innere Klasse also wird sie immer nur in Verbindung mit der umgebenden Klasse aufgerufene. Also z.B. TestService.Helper, folglich ist das schon aussagekraeftig.

Zusätzlich zur Antwort von @kneitzel: Namen wie Helper und Util sind Ausdruck dafür, dass einem nichts anderes eingefallen ist, was wiederum ein starkes Indiz ist, dass man irgendein Konzept des Problembereichs noch nicht erkannt oder noch nicht ganz verstanden hat. Ein TestService.Helper sagt mir genauso viel wie TestService.X -> nix.

Nun ist es schwierig, etwas einem Namen zu geben, dessen Sinn man nicht kennt. Beispiel: Array und Zähler. Sagt erst mal nichts. Kennt man den Sinn dahinter, könnte sich daraus eine ArrayList ergeben. Daher die Frage, was das Ganze werden soll.

In Deinem letzten Kommentar hast Du TestService erwähnt. Auch das ist ein sehr allgemeiner Name, der vermuten lässt, dass ganz allgemein Tests durchgeführt werden. Deinem Pseudocode nach zu urteilen, ist das aber nicht der Fall, denn scheinbar wird hier eine ganz bestimmte Bedingung abgeprüft. Insofern wäre der Name TestService auch nicht optimal (btw: Service/Manager sind auch Namen, die gerne unbedarft verwendet werden). Unabhängig davon: selbst wenn diese Klasse ganz allgemein Tests durchführt, dann könnte man die innere Klasse wenigstens Result nennen. Das macht zumindest klar, dass es sich um ein (irgendein) Ergebnis des TestService handelt.

 

[White_Fox]

zu 3. das versteh ich leider nicht. Klar die map zurueckgeben, aber wie soll ich dann die Anzahl an einen Consumer weitergeben ?

Ich denke er meint, daß der Consumer deiner Berechnungsklasse vor der Berechnung mitteilt "Ich will das Ergebnis". Etwas umständlich, wie ich finde.

Liefere doch sowas als Ergebnis zurück:

public class IrgendwasMitMapUndCounterMacher{
    private int counter;
    private Map map;
    
    public IrgendwasMitMapUndCounterMacher(Object berechungsparameter){
        //...führe Berechnung aus
    }
    
    public int getCounter(){return counter;}
    public Map getMap(){return map;}
}