Java - eindimensionales Array zu zweidimensionales

[Stev (Gast)]

Hallo Leute,

versuche mich gerade an einer Funktion, die ein eindimensionales Array 
als Argument übergeben bekommt, als Rückgabewert aber "möglichst 
quadratisch" ein 2 dimensionales Array daraus rückgeben soll.

Leider weiß ich momentan nicht weiter.

Der Fehler liegt laut Eclipse bei "test[a]

=input[i];" da die Meldung "exceptionoutofbound:10" liegt.

Hier der code

[c]

public static int[][] convertArrayTo2D(int[] input) 
  {
    int [][]test = new int[50][50];
    int i=0;
    int seitenlaenge = (int)sqrt(input.length);
    boolean gerade;
    
    
    if((sqrt(input.length)%2) == 0)        // überprüft, ob Arraylaenge gerade oder ungerade ist.
    {
      gerade = true;
    }
    else
    {
      gerade = false;
    }
    
    if(gerade==true)
    {
      
      for (int a = 0; a < seitenlaenge; a++)
      {
        for (int b = 0; b < seitenlaenge; b++)
        {
          i++;
          test[a][c]=input[i];
          
        }
      }
    }
    else
    {
      for (int a = 0; a < seitenlaenge; a++)
      {
        for (int b = 0; b < seitenlaenge+1; b++)
        {
          i++;
          test[a][c]=input[i];
        }
      }
    }
    
     return test;
  }

Vielen Dank

[Stev (Gast)]

sorry, die Zählvariable "c" ist normalerweise "b", aber das Forum hat 
dies als Fehler ("Spam") angekreidet, deshalb habe ich es ändern müssen.

[gassit (Gast)]

Das "i++" in den Schleifen mit der jeweils darunterstehenden 
vertauschen. Im letzten Durchlauf wird auf input[10] zugegriffen. Das 
Array hat aber nur Indices von 0 bis 9.

[Stev (Gast)]

Ok,

habe das i++ nun unter die "test[a][c]=input[i];" geschrieben.

Allerdings besteht der Fehler immer noch, weiß nicht, wie er auf den 
Index 10 zugreift, obwohl doch die Abbruchbedingung in der Schleife 
stimmt oder?!


Danke

[AVR-Frickler (Gast)]

Der Fehler liegt wieder einmal beim Programmierer!

Du nimmst als Ausgangsarray ein Array mit 9 Feldern! (Ja meine Glaskugel 
habe ich gerade geputzt)

Wurzel von ist 3, 3 ist eine ungerade Zahl.

also läuft a von 0 bis 2 macht 3 Durchläufe, b läuft von 0 bis 3 das 
sind 4 Durchläufe.
Folglich läuft i von 0 bis (3 * 4) - 1 macht 12 Durchläufe!

6 setzen!

[AVR-Frickler (Gast)]

AVR-Frickler schrieb im Beitrag #1925502:
> Wurzel von ist 3, 3 ist eine ungerade Zahl.

Sollte heißen
Wurzel von 9 ist 3, 3 ist eine ungerade Zahl.

Hmm ich sollte mich mal anmelden, dann könnte man auch editieren. :P

[huhu (Gast)]

i < input.length in die Abbruchbedingung der zweiten Schleife?

[Michael (Gast)]

AVR-Frickler hat recht!

i wird viel zu hoch gedreht.
Wenn Du die B-Schleife verlässt musst Du i wieder 0 setzen.

[Karl H. (kbuchegg)]

huhu schrieb:
> i < input.length in die Abbruchbedingung der zweiten Schleife?

Du ratest rum.

Mal dir die Situation auf

Bei einem Eingangsarray mit 9 Feldern, verteilst du die 9 Elemente wie 
auf dein 2-D Array?

Wenn du es nicht durchschaust dann mal es dir auf und spiel selber 
Computer und geh deinen Code Schritt für Schritt durch.


Das ist das eine.
Das andere ist deine Berechnung der Grösse des 2D Arrays.

Ein Array mit 10 Feldern lässt sich nun mal nicht in ein 3*4 Array 
einpassen.  Denn ein 3*4 Array hat nun mal 12 Felder und nicht 10
Wenn die 10 Felder möglichst quadratich in ein 2D-Feld eingepasst werden 
sollen und das 2D Array kein Element zuviel oder zuwenig haben soll, 
dann ist das am besten passende Array ein 2*5 Array. Das mag dir 
gefallen oder nicht. Musst du entscheiden.

[Karl H. (kbuchegg)]

Michael schrieb:
> AVR-Frickler hat recht!
>
> i wird viel zu hoch gedreht.
> Wenn Du die B-Schleife verlässt musst Du i wieder 0 setzen.

Quatsch

[D. I. (Gast)]

Du solltest daran denken dass hier auch AuD-Tutoren/Dozenten lesen und 
fertig gepastete Codes im Plagiatschecker landen, ...

Das sieht nämlich sehr nach der letzten Übungsaufgabe aus die zu tun ist 
;)