Übergabe von Arrays an Methoden

[Satako]

Ich habe ein Problem mit der Ausgabe.
Da es ja jetzt nicht nur eine Methode, sondern ein Array ist, gibt er die Werte nicht richtig aus.
Wie ändere ich die System.out...?

Es geht im eine Kreisberechnung:
Der Umfang eines Kreises soll berechnet werden.

Mein Programm sieht wie folgt aus:

public class Kreis {

public static void main(String[] args) {
// Kreis
int radius = 30;

// hier gibt er Der Umfang des Kreises[radius = 30]: [D@4517d9a3 aus, und das ist verkehrt.
System.out.println("Der Umfang des Kreises[radius = " + radius + "]: " + umfangKreis(radius));

}

// Berechnung des Umfangs eines Kreises
public static double[] umfangKreis(double radius)
{

double feld[] = {radius * radius * Math.PI};
return feld;

}
}

 

[SonoHimitsu]

Was geht denn nicht...

import java.util.Arrays;

public class Kreis {

	public static void main(String[] args) {
// Kreis
		int radius = 30;

// hier gibt er Der Umfang des Kreises[radius = 30]: [D@4517d9a3 aus, und das ist verkehrt.
		System.out.println("Der Umfang des Kreises[radius = " + radius + "]: " + Arrays.toString(umfangKreis(radius)));

	}

// Berechnung des Umfangs eines Kreises
	public static double[] umfangKreis(double radius) {

		double feld[] = { radius * radius * Math.PI };
		return feld;

	}
}

 

[SonoHimitsu]

Oder auch nur umfangKreis(radius)[0]

 

[Kirby.exe]

Das liegt daran, dass du die Referenz des Arrays printest Arrays sind Referenzdatentypen. Um jedes Element des Arrays zu printen musst du mit einer Schleife durch das Feld iterieren

 

[Satako]

Bei der Ausgabe des Umfangs gibt er: [D@4517d9a3 aus
Also kuddelmuddel, anstatt den Wert

 

[Kirby.exe]

Das ist die Referenz des Arrays

 

[SonoHimitsu]

So viel Kuddelmuddel ist das gar nicht "[D" steht für double array, "@" für @, und "4517d9a3" ist das Ergebnis des Aufrufs von "Integer.toHexString(hashCode())".

 

[Kirby.exe]

Du kannst das Array entweder, wie @SonoHimitsu geschrieben hat, mit Arrays.toString(deinArray) printen oder du gehst mit einer Schleife durch dein Array und printest jeden Wert einzeln

 

[Satako]

Ok, dankeschön SonoHimitsu Funktioniert

 

[SonoHimitsu]

👍

 

[kneitzel]

Warum gibst du denn ein Array mit nur einem Wert zurück? Wäre es nicht einfacher, nur den Wert zurück zu geben?

 

[SonoHimitsu]

Warum gibst du denn ein Array mit nur einem Wert zurück? Wäre es nicht einfacher, nur den Wert zurück zu geben?

Vielleicht möchte er sein Programm ja noch erweitern. 5, setzen.

 

[thecain]

Vielleicht möchte er sein Programm ja noch erweitern. 5, setzen.

So viele Umfänge hat ein Kreis auch nicht, dass es da noch viel Ausbaupotential gäbe.

 

[Kirby.exe]

So viele Umfänge hat ein Kreis auch nicht, dass es da noch viel Ausbaupotential gäbe.

Da stimme ich dir zu xD Vielleicht möchte er jedoch mehrere Kreisumfänge speichern

 

[kneitzel]

Vielleicht möchte er sein Programm ja noch erweitern. 5, setzen.

Also wenn man mehrere Daten hat, dann schafft man sich entsprechende Datenstrukturen.

Wenn Du absolut keine Ahnung von Software Entwicklung hat, dann würde ich Dir raten, Deine Klappe zu halten oder zumindest nicht so aufzutreten. Zumal Du heute erst dein Können in einem eigenen Thead unter Beweis gestellt hast....

Und da wunderst Du Dich, wenn Accounts von Dir gesperrt werden?

 

[SonoHimitsu]

dann schafft man sich entsprechende Datenstrukturen

Wozu denn der Mehraufwand?

Wenn Du absolut keine Ahnung

Und warum gleich so bissig....

 

[Kirby.exe]

Also wenn man mehrere Daten hat, dann schafft man sich entsprechende Datenstrukturen.

Da gebe ich dir Recht, aber ich vermute einfach mal dass @Satako noch relativ am Anfang steht und vermutlich noch nicht die Kenntnisse und das Wissen dazu hat

 

[kneitzel]

Hier empfehle ich einfach Clean Code von Uncle Bob. Wurde schon oft erwähnt von mir. Kann ich nur empfehlen.

Code gehört immer an eine vernünftige Stelle. Eine Methode wie hier würde einer Art Utility Klasse/Methode entsprechen. Bezüglich Objektorientierung recht fatal. Da dann lieber eine Klasse Kreis und schon hat man eh die Klasse.

Bleibt die Frage, ob man so Werte wie Umfang überhaupt speichert. Die Berechnung geht ja schnell. Damit hat man dann rein die Klasse Kreis mit holeUmfang Methode.

Das ist halt das, was ich da sehe. Aber das ist etwas viel für einen Anfänger denke ich. Daher habe ich nur einen kleinen Hinweis gegeben, dass das Array bei der Rückgabe hier unnötig ist.

Und wegen dem bissig: schau Dir einfach Deine Aussage von wegen Setzen 5 an....

 

[kneitzel]

Da gebe ich dir Recht, aber ich vermute einfach mal dass @Satako noch relativ am Anfang steht und vermutlich noch nicht die Kenntnisse und das Wissen dazu hat

Daher gab es nur den Hinweis, dass die Rückgabe eines Arrays mit immer genau einem Wert wenig Sinn macht. Das ging an den TE. Der Hinweis mit dem erweitern ging rein an Tobias, der hier irgendwelche Erweiterungen gesehen hat (die so hoffentlich nicht kommen. Ich sehe da schon tolle Lösungen a.la. Array von Object da ggf ja nicht nur double Werte zurück gegeben werden sollen ... da erscheint dann ein YT Tutorial mit Klasse Variablen und ganz vielen public static Feldern als gute Lösung ).

 

[SonoHimitsu]

da erscheint dann ein YT Tutorial mit Klasse Variablen und ganz vielen public static Feldern als gute Lösung

 

[kneitzel]

Also ein "Definitiv Nein" das Du hier bringst, ist in meinen Augen nicht zu rechtfertigen. Ich habe zwei Lösungen vorgestellt und die erste Lösung, die Du präferierst, habe ich so noch nie in einem Projekt gesehen. Das letztere ist aber ein Pattern, das durchaus in der Praxis vorkommt. C / C++ Entwickler kennen das evtl. - Stichwort "getLastError()"....

Der Vorteil der zweiten skizzierten Lösung ist die Typsicherheit. Man hat halt "globale" Variablen (public static), die gefüllt werden und dann nach dem Aufruf ausgewertet werden können.
Bei der anderen Lösung hat man dann ganz viele casts um aus dem Object[] eben String, Integer, Double, ... heraus zu lesen.
Und wenn man dann auf diese Art und Weise zugreift:
berechneKreis(radius)[UMFANG_KREIS]
dann sehe ich auch die Gefahr, dass da mehrfach aufgerufen wird. Da ist dann etwas wie:

Kreis.brechneKreis(radius);
Double umfang = Kreis.letzterUmfang;
String beschreibung = Kreis.letzteBeschreibung;
// ...

schon deutlich besser lesbar.

Aber machen würde ich das so nie - Das ist ja ein Alptraum bezüglich Multitasking....

Aber um das etwas aufzuzeigen, wie man sowas machen würde und was dann hoffentlich auch dem TE wieder helfen kann:

public class Kreis {
  private double radius;
 
  public Kreis(final double radius) {
    this.radius = radius;
  }
 
  public double berechneUmfang() {
    return radius * radius * Math.PI ;
  }
}

Und da hat man dann alles zum Kreis schön enthalten. Und wenn man da noch mehr braucht, wie z.B. Flächenberechnung oder einen Kreis aus einer gegebenen Fläche oder gegebenen Umfang zu erzeugen, dann kann das alles da rein:

public class Kreis {
  private double radius;
 
  public Kreis(final double radius) {
    this.radius = radius;
  }
 
  public double berechneUmfang() {
    return 2 * radius * Math.PI; // Formel war bei TE falsch, er hat die Fläche berechnet!
  }

  public double berechneFlaeche() {
    return radius * radius * Math.PI;
  }

  public static Kreis vonFlaeche(final double flaeche) {
    return new Kreis(....); // Berechnung des Radius weggelassen.
  }

  public static Kreis vonUmfang(final double flaeche) {
    return new Kreis(....); // Berechnung des Radius weggelassen.
  }
}

Ansonsten fällt mir gerade nichts ein, was man da noch einbauen könnte. Aber das zeigt hoffentlich recht deutlich den objektorientierten Ansatz, der auch nicht sonderlich komplex ist.