Als Rückgabewert ode call by referenz was ist sauberer?

[Kalkulon]

Habe eine Frage, was haltet ihr für schöner? ich habe eine Methode die ein Array als Übergabeparameter bekommt. bis jetzt habe ich es so gemacht:

[JAVA=42]

class Test {
//Möglichkeit 1
public byte[] test(byte[] test) {
// tu irgendwas
}
// Möglichkeit 2
public void test(byte[] test) {
// tu irgendwas
}

public static void main(String[] args) {

// Möglichkeit 1
Test t = new Test();
byte[] data = new byte[]
data = t.test(data)

// Möglichkeit 2

t.test(data);
}
}


[/code]

 

[Firestorm87]

Ich wäre mir bei byte aktuell nichtmal sicher, ob möglichkeit 2 überhaupt funktioniere.....

 

[Kalkulon]

es handelt sich doch um ein Array und die sind in Java Objekte, welche wiederum als Referenz übergeben werden.

Sonst wären Dinge wie byte[] data = new byte[2]; System.out.println(data.length); nicht möglich!

 

[Firestorm87]

Auch wahr...
Aber arrays sind einfach insgesamt unschön
Collections haben schon was

 

[SlaterB]

@Firestorm87
gut dass du dennoch postest


> data = t.test(data)
ist recht sinnlos wenn test() dasselbe Array zurückgibt,
erstellt es ein neues sähe es ganz anders aus

denselben Parameter zurückzugeben macht nur dann Sinn, wenn das zu irgendwas nützt,
z.B. direkter Weitergabe:
t.test2(t.test(data))

 

[Kalkulon]

warum?

edit: Das warum bezog sich auf die Sinnlosigkeit von Arrays.

In der Methode mach ich schon ein neues Array, könnte aber die Referenz "umbiegen".

 

[Landei]

Ich versuche grundsätzlich, an den Methoden-Argumenten nicht herumzufummeln. Bei einem Rückgabewert "weiß man, was man hat", dass eine Methode ihre Argumente manipuliert ist weniger offensichtlich.

Natürlich können Effizienzgründe, das Fehlen eines Tupel-Typs in Java (wenn man mehrere Werte zurückgeben will) oder auch eine dummes Bibliotheksdesign dagegen sprechen, dann würde ich es aber ordentlich kommentieren.

So Sachen wie beim StringBuilder, der sich bei append() selbst manipuliert und auch noch selbst zurückgibt, halte ich für gefährlich, weil es im ersten Augenblick so aussieht, als würde er nur ein neues, verändertes Objekt zurückliefern.

Wenn keine guten Gründe dagegen sprechen, bin ich für einen eher funktionalen Stil: Unveränderliche Objekte und Methoden ohne Seiteneffekte (und Manipulation der Argumente ist ein Seiteneffekt). Damit ist man übrigens automatisch threadsafe.

 

[Kalkulon]

ok alles klar also bleib ich dabei byte[] methode(final byte[] ba)...

 

[Firestorm87]

@Firestorm87
gut dass du dennoch postest

Hier soll doch keine unterschwellige Botschaft stecken?

ok alles klar also bleib ich dabei byte[] methode(final byte[] ba)...

Solltest dann aber aufpassen, dass du das Element ba nicht direkt weiterverarbeitest, da du sonst trotzdem deine parameter veränderst, auch wenn du ihn hinten herum wieder zurück gibst...

 

[Janus]

[...]könnte aber die Referenz "umbiegen".

Nein, kannst du nicht in Java.

 

[bygones]

Ich versuche grundsätzlich, an den Methoden-Argumenten nicht herumzufummeln. Bei einem Rückgabewert "weiß man, was man hat", dass eine Methode ihre Argumente manipuliert ist weniger offensichtlich.

this.

jegliche Seiteneffekte sollten IMMER vermieden werden

Collection<String> foo = ... // wird schoen gefuellt
someFunnyName(foo); 
// ups aufeinmal ist foo leer oder so..

nicht gut...

Collection<String> foo =  someFunnyName()

so gut.

 

[bygones]

ok alles klar also bleib ich dabei byte[] methode(final byte[] ba)...

ich hoffe dein realer code ist anders als

  Test t = new Test();
  byte[] data = new byte[]
  data = t.test(data)

wenn schon dann bitte

  Test t = new Test();
  byte[] data = t.test(); // test erstellt sich den array... warum sollte es der aufrufer vorher schon tun ???

 

[Kalkulon]

doch mein realer Code sieht in etwa so aus! ich habe eine Vorwärtsfehlerkorrektur implementiert übergebe dieser ein Byte Array übertrage es über einen "unzuverlässigen" Übertragungskanal und decodiere die bytes wieder, bevor ich sie weiterverarbeite.

Das ganze sieht dann schon in etwa so aus:

byte[] data = fec.encode("Nachricht".getBytes());

...
Übertragung der Daten
...

DatagramPacket packet = new DatagramPackt.....

socket.receive(packet);
byte[] data = packet.getContent();
data = fec.decode(data);

System.out.println(new String(data));// Nachricht


So in etwa hab jetzt die genaue Java Syntax nicht im Kopf.