Problem mit BubbleSort und Array

[willi2409]

Hallo,

ich habe da ein kleines Problem:

import java.util.*;

class bubble{

  static InteractiveIO io = new InteractiveIO();

  static int[] zahlen;
  public bubble()
  {
  	zahlen = new int[20];
  }

  public static void main(String[] vArgs) throws Exception
  {
  
  int s = io.readInt("Zahlen :");
  
  while (s !=0)
  {
  	int e = 0;
  	
  	zahlen[e] = 7;
  	s = io.readInt("Zahlen :");
  	e++;
  }
    bubble b = new bubble();
    zahlen=b.sort(zahlen);
    System.out.println(b.toString());
  }

  public int[] sort(int[] z){
    for(int i=0;i<z.length-1;i++){
      for(int j=1;j<z.length;j++){
        if(z[j-1]>z[j]){
          int temp=z[j-1];
          z[j-1]=z[j];
          z[j]=temp;
        }
      }
    }
    return z;
  }

  public String toString(){
    String s="";
    for(int i=0;i<zahlen.length;i++){
      s=s+zahlen[i]+",";
    }
    s=s+"\n";
    return s;
  }

}

Compilieren geht, und die erste Zahl kann ich auch eingeben aber dann schmierts ab

Zahlen :5
Exception in thread "main" java.lang.NullPointerException
        at bubble.main(bubble.java:22)

Währe super wenn mir jemand einen Tip geben könnte

mfg

 

[SlaterB]

die statische Variable zahlen wird nur im Konstruktor bubble() angelegt,
was an sich schonmal falsch ist,

und da du nie ein bubble-Objekt erzeugst, bleibt zahlen immer null -> NullPointerException

falls du dachtest, dass zahlen nicht null sein sollte, ist das ja einfach zu prüfen:

public bubble()
{
zahlen = new int[20];
System.out.println("zahlen gesetzt: "+zahlen);
}

keine Meldung -> nix passiert

 

[willi2409]

Hallo,

ich habe doch ein bubble objekt erzeugt

bubble b = new bubble();
    zahlen=b.sort(zahlen);

 

[SlaterB]

in Zeile 26,
was bringt dir das, wenn du schon in Zeile 22 auf zahlen zugreifen möchtest?

 

[willi2409]

Ok,

hab ich jetzt gecheckt. jetzt schreibt er auch was ins array, allerdings nur den letzten eingegeben wert.

mfg

 

[willi2409]

hallo

korrektur er schreibt alle werte ins array nur krieg ich se jetzt net sortiert.

mfg

 

[SlaterB]

falls das eine Frage was:

e = 0 zu Beginn jeder Schleife zusammen mit e++;
macht wenig Sinn oder?

die while-Schleife sollte auch die Größe des Array berücksichtigen und gegebenfalls selber abbrechen,
usw., viele kleine Dinge, einfach mal genau nachschauen


edit:

> korrektur er schreibt alle werte ins array nur krieg ich se jetzt net sortiert.

hmm, sortieren sieht ganz gut aus, poste doch mal deinen aktuellen Code

 

[willi2409]

import java.util.*;

class bubble{

  static InteractiveIO io = new InteractiveIO();
  
  static int[] zahlen;
  static bubble b = new bubble();

  public bubble()
  {
  	
  	zahlen = new int[20];
  	
  	System.out.println("zahlen gesetzt: "+zahlen); 
  }

  public static void main(String[] vArgs) throws Exception
  {
  zahlen = b.sort(zahlen);
  int s = io.readInt("Zahlen :");
  int e = 0;
  while (s !=0 || e> 20)
  {
  	zahlen[e] = s;
  	s = io.readInt("Zahlen :");
  	e++;
  	
  }
   
    System.out.println(b.toString());
  }

  public int[] sort(int[] z){
    for(int i=0;i<z.length-1;i++){
      for(int j=1;j<z.length;j++){
        if(z[j-1]>z[j]){
          int temp=z[j-1];
          z[j-1]=z[j];
          z[j]=temp;
        }
      }
    }
    return z;
  }

  public String toString(){
    String s="";
    for(int i=0;i<zahlen.length;i++){
      s=s+zahlen[i]+",";
    }
    s=s+"\n";
    return s;
  }

}

 

[Guest]

hallo,

sortieren tut er jetzt auch, nur wird das array von hinten ausgegeben.

mfg

 

[SlaterB]

der Sortieren-Aufruf steht VOR dem Einlesen der Zahlen,
so kann doch nix sortiert werden,

wenn dir die Ausgabereienfolge nicht gefällt, dann ändere entweder die Reihenfolge bei der Ausgabe (so eine Schleife läßt sich doch bearbeiten..)
oder die Reihenfolge der Sortierung,

--

die Schleifenbedingung muss
> while (s != 0 && e < 20)
heißen,

wenn du nicht ständig Zahlen eintippen willst, dann nimm
s = 1 + (int)(Math.random() * 17);

das statische Zahlenarray und nun sogar
> static bubble b = new bubble();
ist ganz schön schlimm

naja usw., bastel dir mal irgendwas zusammen

 

[Guest]

danke für die hilfe, ich muss jetzt erstmal ne pause machen

mfg,