NullPointerException

[Sindbad1983]

Hi!

Ich hab da eine NullPointerException bekommen! :cry:

Die Methoden müssten passen!
In der main-Methode dürfte der Fehler liegen!
Vielleicht kann sich die irgendwer ganz schnell durchschaun bitte!

Irgendwas mit der Klasse dürft nicht stimmen, keine Ahnung!
Such schon seit ner stunde! :###
Bitte!
Danke!

import java.io.*;

class Buchstabenmix{
	

		StringBuffer sb;  //brauch ich da einen Konstruktor?

	
	public void change(StringBuffer sb){ //die Methode stimmt sicher!
		
		for(int i=0;i<sb.length()/2;i++){
			
			char tmp;
			tmp=sb.charAt(i);
			
			sb.setCharAt(i,sb.charAt(sb.length()-i-1));
			sb.setCharAt(sb.length()-i-1,tmp);
	  }
		
	}
	
	public String toString(){ //dürft auch stimmen
		
		String s="";
		StringBuffer b=new StringBuffer();
		
		for(int i=0;i<sb.length();i++){ //geht vielleicht auch kürzer! :-) nur sb.toString();
			b=b.append(sb.charAt(i));
		}
		
		s=b.toString();
	
		return s;
	}

	
	//ABER HIER MUSS FEHLER LIEGEN!!
	
	public static void main(String [] args) throws IOException{
		
		Buchstabenmix test=new Buchstabenmix(); // ??
		
		StringBuffer sb=new StringBuffer(); //??
		
		try{
		BufferedReader rb=new BufferedReader(new InputStreamReader(System.in));
		
		System.out.println("Geben Sie das zu konvertierende Wort ein:");
		String s=rb.readLine();
		
		for(int i=0;i<s.length();i++){
			sb=sb.append(s.charAt(i));
		}
		
	
		}
		catch(IOException e){};
		
	
		test.change(sb);  //oder da?
		
		System.out.println(test.toString()); //oder da?? ich weiß es nicht!
		
		
		
	}
}

 

[Reality]

Da liegt der Fehler. :lol:

StringBuffer sb;  //brauch ich da einen Konstruktor?

So muss das heissen.

StringBuffer sb = new StringBuffer();  //brauch ich da einen Konstruktor?

EDIT: Hab wohl übereilt... Wieso hast du zwei Objekte mit demselben Variablennamen?

Liebe Grüße
Reality

 

[Sindbad1983]

hm?
wie meinst du das jetzt?
Wie gehört das jetzt richtig?

class Buchstabenmix{ 
    

      StringBuffer sb;  //brauch ich da einen Konstruktor?

naja...die Deklaration brauch ich ja, weil er sonst in der toStringMethode sb nicht kennt!

public static void main(String [] args) throws IOException{ 
       
      Buchstabenmix test=new Buchstabenmix(); 
       
      StringBuffer sb=new StringBuffer();

und hier erzeug ich ihn dann! wenn ich nur sb=new StringBuffer() schreib, gibt das weiter unten einen Fehler:

	String s=rb.readLine();
		
		for(int i=0;i<s.length();i++){
			sb=sb.append(s.charAt(i)); //dann kennt er sb nicht!  :autsch: 
			
		}

Hm, weißt du, was ich mein?
Aber wahrscheinlich liegt darin eh der Fehler?
Ich deklarier bzw. erzeug irgendwas falsch!
Weißt du was?

 

[Reality]

Mach mal ganz am Anfang

class Buchstabenmix{
   

      StringBuffer sb = new StringBuffer();  //brauch ich da einen Konstruktor?

Und lösche das hier:

public static void main(String [] args) throws IOException{
       
      Buchstabenmix test=new Buchstabenmix();
       
      StringBuffer sb=new StringBuffer(); // das da löschen

Liebe Grüße
Reality

 

[Sindbad1983]

ja das hab ich auch schon versucht...geht leider nicht! :cry:

Weil dann meldet er den Fehler bei der for-Schleife!!!

String s=rb.readLine(); 
       
      for(int i=0;i<s.length();i++){ 
         sb=sb.append(s.charAt(i)); //dann kennt er sb nicht!  :autsch: 
          
      }

Ich versteh das nicht!

Normalerweise müsste er sb doch kennen!
Ist doch eine Instanzvariable oder?
*verzweifeltbin*

 

[Reality]

OK, dann mach mal

StringBuffer sb=new StringBuffer("");

Liebe Grüße
Reality

 

[Sindbad1983]

leider nein!

 

[Beni]

Du verwendest hier verschiedene StringBuffer.. du darfst aber nur einen benutzen.

Z.b. so:

import java.io.*;

class Buchstabenmix{
   

      StringBuffer sb = new StringBuffer();

   
   public void change(){ //die Methode stimmt sicher!
      
      for(int i=0;i<sb.length()/2;i++){
         
         char tmp;
         tmp=sb.charAt(i);
         
         sb.setCharAt(i,sb.charAt(sb.length()-i-1));
         sb.setCharAt(sb.length()-i-1,tmp);
     }
      
   }

   public void append( String s ){
      sb.append( s );
   }
   
   public String toString(){ //dürft auch stimmen
      
      String s="";
      StringBuffer b=new StringBuffer();
      
      for(int i=0;i<sb.length();i++){ //geht vielleicht auch kürzer! :-) nur sb.toString();
         b=b.append(sb.charAt(i));
      }
      
      s=b.toString();
   
      return s;
   }

   
   //ABER HIER MUSS FEHLER LIEGEN!!
   
   public static void main(String [] args) throws IOException{
      
      Buchstabenmix test=new Buchstabenmix(); // ??
      
      StringBuffer sb=new StringBuffer(); //??
      
      try{
      BufferedReader rb=new BufferedReader(new InputStreamReader(System.in));
      
      System.out.println("Geben Sie das zu konvertierende Wort ein:");
      String s=rb.readLine();
      
      test.append( s );
   
      }
      catch(IOException e){};
      
   
      test.change();  //oder da?
      
      System.out.println(test.toString()); //oder da?? ich weiß es nicht!
      
      
      
   }
}

 

[Reality]

Versuch mal.

sb.append(s.charAt(i));

statt

sb=sb.append(s.charAt(i));

Liebe Grüße
Reality

 

[Sindbad1983]

ok...spitze Beni!
So stimmts!
Ich hab schon gewusst, dass ich zu viele StringBuffer verwende, aber ich hab nicht gewusst, wie ich es lösen soll!
Die Idee mit dem append ist natürlich sehr gut!
Wär ich allein wohl nicht draufgekommen!
Danke!

 

[Sindbad1983]

jetzt hab ich auch noch ne Frage?

gibt es da noch eine andere Lösung?
hätt ich das auch ohne die Methode append lösen können?
Wenn ich jetzt z.B. sb in der main-Methode doch dringend gebraucht hätte (weil jetzt kommt sb ja nicht mehr vor), wie wär man da vorgegangen?
Oder geht das eh nicht?

 

[Beni]

Dann erweiterst du die Klasse BuchstabenMix um eine Methode "getStingBuffer", wo du den StringBuffer zurückgibst. (so kannst du dann "test.getStringBuffer" aufrufen, um an den Buffer heranzukommen). Noch besser wäre allerdings, wenn du BuchtabenMix so schreibst, dass dieser Buffer niemals von sonst jemandem benötigt wird.

 

[Sindbad1983]

hm, wie hast du das gemeint?
Ich habs so probiert, wie du es beschrieben hast, aber das geht leider nicht:
Der Compiler findet zwar keinen Fehler, aber ausgegeben wird auch nichts! :bahnhof:

import java.io.*;

class Buchstabenmix{
	

	StringBuffer sb=new StringBuffer();

	
	public void change(){ //die Methode stimmt sicher!
		
		for(int i=0;i<sb.length()/2;i++){
			
			
			char tmp;
			tmp=sb.charAt(i);
			
			sb.setCharAt(i,sb.charAt(sb.length()-i-1));
			sb.setCharAt(sb.length()-i-1,tmp);
	  }
		
	}
	
	public StringBuffer getStringBuffer(){
		
		return sb;
		
	}	
	
	
	public String toString(){
		
	 String s=""; 
      StringBuffer b=new StringBuffer(); 
      
      for(int i=0;i<sb.length();i++){ //geht vielleicht auch kürzer! :-) nur sb.toString(); 
         b=b.append(sb.charAt(i)); 
      } 
      
      s=b.toString(); 
      return s;

		
	}

	public static void main(String [] args) throws IOException{
		
		Buchstabenmix test=new Buchstabenmix();
		test.getStringBuffer();
		
		try{
		BufferedReader rb=new BufferedReader(new InputStreamReader(System.in));
		
		System.out.println("Geben Sie das zu konvertierende Wort ein:");
		String s=rb.readLine();
		
		for(int i=0;i<s.length();i++){
		test.sb=test.sb.append(s.charAt(i));
			
		}
		
	
		}
		catch(IOException e){};
		
		test.change(); 
		test.toString();	
	}
}

Vielleicht kannst mir da nochmal helfen!
Das wär wichtig für die Verständnis!
danke,Tommy

 

[Beni]

Du machst ja auch nie eine Ausgabe...

Du musst noch ein System.out.println( "die ausgabe, bei dir ist das test.toString()" ); einbauen :wink:

 

[Sindbad1983]

jaaaa, es geht!

Super, danke!

Also hab ich das richtig implementiert!..schön!
Ich glaub, ich checks!
Spitze!!!