Guter Stil der Java-Programmierung

[javaKid]

Hallo alle zusammen,

ich bin gerade dabei, mit Java laufen zu lernen

Nun habe ich folgenden Java-Pseudo-Code:

class XXX{    
    int[] array;
    
    XXX(int[] array){
        this.array=array
        }
    function(){
        //Tutirgendwas mit dem Attribut
    }

    psvm(...){
        float[] a = new float[5];
        XXX bla = new FloatArray(a);
        int count = 1;
        for (int i=0; i< a.length;i++){
            a[i]=count++;
        }  
        bla.function();
        for (int i=0; i< a.length;i++){
            a[i]=count--;
        }  
        bla.function();
        for (int i=0; i< a.length;i++){
            a[i]=1;
        }  
        bla.function();


        //oder wäre folgenes besser?
        XXX bla = new FloatArray(new float[5]);
        int count = 1;
        for (int i=0; i< bla.array.length;i++){
            bla.array[i]=count++;
        }  
        bla.function();
        for (int i=0; i< bla.array.length;i++){
            bla.array[i]=10;
        }  
        bla.function();
        for (int i=0; i< bla.array.length;i++){
            bla.array[i]=count--;
        }  
        bla.function();

    }
}

Welche der beiden Möglichkeiten in der Main-Methode haltet Ihr für die Sauberere oder habt Ihr sogar einen vil besseren Vorschlag?

Herzlichen Dank Euch allen!!!

 

[Guest]

P.S. Oder sollte ich für jeden der drei "Versuche" ein eigenes XXX Objekt erzeugen...?

 

[madboy]

Ist vielleicht ne doofe Antwort: aber es kommt drauf an, was du tun willst. Ich sehe irgendwie aus deinem Pseudocode nicht, was du vor hast.

Welche der beiden Möglichkeiten in der Main-Methode...

Welche "Main-Methode"?

Außerdem wird das mit

XXX bla = new FloatArray(a);

nix. "XXX" ist ja kein "FloatArray", daher kannst du das nicht so zuweisen.

 

[javaKid]

Sorry, die Zeile sollte folgende sein:

XXX bla = new XXX(a);

Im Prinzip handelt es sich um eine Klasse die diverse Funktionen auf das Attributarray ausführt, wie Durschnitt berechnen usw.
Da ich mehrere verschiedenen Fälle implementieren möchte, muss ich die Zahlen im Array mehrmals ändern.

 

[Marco13]

@madboy: "psvm" = "public static void main" :wink:

Schwierig, so eine Designentscheidung zu treffen, wenn man nicht weiß, worauf das rauslaufen soll. Auf Basis dessen, was du geschrieben hast, könnte "funktion" statisch sein und immer den Array übergeben bekommen

class XXX
{  
    main()
    {
        int array[] = new int[3];

        XXX.funktion(array);

        array[1] = 123; // verändere den Array

        XXX.funktion(array);

    }

    public static void funktion(int array[])
    {
         // mach was mit dem übergebenen array
    }
}

allerdings kann es gut sein, dass das NICHT das ist, was du willst.

Vermutlich (Kristallkugel-Antwort) wäre die zweite Lösung besser: Das array liegt NUR im XXX-Objekt, und wird auch NUR dort verändert. Allerdings sollte es dan unbedingt private sein, und eine Methode zum verändern des Array-Inhaltes angeboten werden. Und vermutlich sollte der Array dann nichtmal im Konstruktor übergeben werden, damit niemand auf dumme Gedanken kommt (d.h. damit sich niemand eine Referenz behält, wie in deinem ersten Ansatz).

Wie wär's z.B. mit sowas:

class XXX
{
    private int array[];

    public XXX(int size)
    {
        array = new int[size];
    }

    public int getSize()
    {
        return array.length;
    }

    public void setValue(int index, int value)
    {
        array[index] = value;
    }

    // EDIT: Das fehlte noch:
    public void funktion()
    {
        // mach was mit dem array
    }

}

und in der "psvm" würdest du dann sowas machen wie

psvm(...){

        XXX bla = new XXX(5);
        int count = 1;
        for (int i=0; i< bla.getSize();i++){
            bla.setValue(i, count++);
        } 
        bla.function();
...

Arrays sind ziemlich "low-level", und man sollte sich genau überlegen, wo man einen Array nach außen "preisgibt". In bezug auf die Information, die man in einer Klasse preisgibt (mit "public") gibt es eine Regel:
So wenig wie möglich, und so viel, wie nötig.
Das nennt man machmal "Geheimnisprinzip" (auf Englisch glaubich "Information hiding") (Das gilt aber bei ALLEN objektorientierten Sprachen, nicht nur bei Java)

In der obigen Klasse ist dieses Prinzip erfüllt: Man könnte den Array einfach ersetzen, z.B. durch eine ArrayList oder LinkedList (wenn's denn sein müßte - das ist vielleicht kein optimales Beispiel, um das zu verdeutlichen). Trozdem kann man alle nötigen Aktionen durchführen.

EDIT: "funktion" in XXX eingefügt!
EDIT2: Hach, heut hab ich's aber letzter code-Block noch geändert

 

[madboy]

@madboy: "psvm" = "public static void main" :wink:

Ah jetzt ja! Und schon wieder was gelernt