ungewollter Zugriff auf private Variablen?

[Toasterwilli]

Bisher war ich der Meinung, dass folgendes eigentlich nicht funktionieren kann, aber es geht:
Mit dieser Methode greife ich mittels get-Funktion auf ein BufferedImage von 'FrameHolder' zu:

public static void test(Buffer in, FrameHolder out, int width, int height, boolean littleEndian){

        BufferedImage image = ((ImageFrameHolder) out).getImage();
                                
        for (int y = 0; y < image.getHeight(); y++){
            for (int x = 0; x < image.getWidth(); x++){
                
                    image.setRGB(x, y, irgendwie);
            }
        }

die Klasse 'ImageFrameHolder' sieht so aus:

public class ImageFrameHolder extends FrameHolder {

    private BufferedImage _img;

...

    public BufferedImage getImage() {
        return _img;
    }

Im Hauptprogramm wird 'test' nun folgendermaßen aufgerufen:

test(frame[nextBuffer], frameFiltered[nextFilteredBuffer], true);
                        
 _frameListener.newFrame(frameFiltered[nextFilteredBuffer]);

Die letzte Zeile sorgt dafür, dass der gefilterte Frame am Bildschirm ausgegeben wird. In 'test' kann ich mit getImage() auf die private Variable _img zugreifen. Was mich nun wirklich wundert, dass dann am Ende tatsächlich das Bild so gefiltert dargestellt wird, wie ich es in 'test' ändere. Ich hab doch damit '_img' geändert, obwohl ich das ja eigentlich nicht dürfte?

 

[kama]

Hallo,

public class ImageFrameHolder extends FrameHolder {

    private BufferedImage _img;

...

    public BufferedImage getImage() {
        return _img;
    }

Was definierst Du denn damit? Du lieferst per getImage() doch eine Referenze auf das interne Object Deiner Klasse ImageFrameHolder. Was ist daran falsch?
Du solltest eventuelle überlegen, ob getImage() so richtig ist und man dass nicht mit einem Wrapper absichern muss oder mit anderen Methoden.

Ich denke Du könntest in getImage() einen Clone von _img erstellen anstatt eine Referenz zurück zu liefern.

MfG
Karl Heinz

 

[Toasterwilli]

Ich muss zugeben, ich bin jetzt ziemlich verwirrt. In den FAQ steht doch etwas davon, dass es in Java "call by reference" garnicht gibt? Aber das ist doch hier der Fall?

Ich dachte außerdem, dass man private Objektvariablen nur mit dafür vorgesehenen setter-Methoden ändern kann.
Wozu gibt es dann "private", wenn ich über ne einfache get-Methode schon in der Variablen rummurksen kann?

 

[Beni]

Du kannst die Variable img_ nicht ändern von aussen (Du kannst ihr z.B. nichts neues zuweisen).

Du kannst aber das Objekt auf welches sich img_ bezieht jederzeit verändern.