probleme mit paintComponent

[gabriel]

erstmal hallo; cooles forum
ich hab vor ein paar wochen mit java angefangen;

komm jetzt an einer stelle nicht weiter;
undzwar hab ich ein kleines programm geschrieben das mir aus einer datei text ausliest und die buchstaben von a-z
zusammenzählt...
jetzt wollte ich die werte in einem kleinen diagramm ausgeben, aber irgendwie kann ich der protected void paintComponent den string s nicht übergeben;
ich hab irgendwie nen denkfehler, kann mir jemand helfen?

import java.io.*;
import java.awt.*; 
import javax.swing.*; 


public class Go extends JComponent{

	public static void main(String[] argv) {
		if (argv.length == 0) {
			System.err.println("Bitte einen Dateinamen angeben");
			return;
		}
		String s = argv[0];
		
		int[] out=counter(s);
		//System.out.println(out[0]);
		
		JFrame aJFrame = new JFrame("SHOW");
		aJFrame.getContentPane().add(new Go(), BorderLayout.CENTER);

		aJFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		aJFrame.pack();
		aJFrame.setVisible(true);
	}
	
	public static int[] counter(String s){	
		int[] counter = new int[26];
		try{
			BufferedReader buffReader = new BufferedReader(new FileReader(s));		
			String fullFile = ""; 
			String input = ""; 			
			
			while((input = buffReader.readLine()) != null) { 
				fullFile += input; 
			}
			buffReader.close();
			
			fullFile = fullFile.toLowerCase();
			for(int i=0;i<fullFile.length();i++){
			
				char test = fullFile.charAt(i);
				int count = (int) fullFile.charAt(i);
				if(count>=97&&count<=122){
					counter[count-97]++;
				}
			}
		}
		catch (IOException e) {		
			e.printStackTrace();
		}
		return counter;
		}
						
	public Dimension getPreferredSize() {
		return new Dimension(820,400);
	}

	protected void paintComponent(Graphics g) {
	//int[] out=counter(s);
	
		for (int x=0;x<26;x++){
			g.setColor(Color.black);
			g.fillRect(x*30, 200, 10, 50);    
		}
	}	
}

gruß
gabriel

 

[Wildcard]

Wie du ja schon bemerkt hast kannst du paintComponent nichts übergeben.
Darin kannst du nur ein bestehendes model abbilden.
In deinem Fall müsstest du also aus 's' eine Instanzvariable machen(wenn du nur den String brauchst).

EDIT:
nachdem ich mir deinen Code gerade mal angesehen hab währe es wohl geschickter 'count' im Konstruktor aufzurufen,
und das int[] zur Instanzvariablen zu machen...

 

[gabriel]

ohh cool danke für sie schnelle antwort;

also wenn ich das jetzt richtig verstanden habe einfach
"public int[] counter(String s)" oder wie?
aber warum im Konstruktor ?

aber

   protected void paintComponent(Graphics g) { 
   int[] out=counter(s); 
    
      for (int x=0;x<26;x++){ 
         g.setColor(Color.black); 
         g.fillRect(x*30, 200, 10, out[x]);    
      }

ich wollte sowas in der art basteln;
wie würde das ungefähr aussehen?

gruß
gab

 

[Wildcard]

mach am anfang deiner Klasse

private int[] bla;

counter machst du dann void, weil du direkt die Instanzvariable verwenden kannst.
counter rufst du dann am besten in einem Konstruktor auf.
Anschließend kannst du auf des array in der paintComponent zugreifen.

 

[gabriel]

also ich hab das jetzt so verstanden...

private int[] out;

public void counter(String s){
.
.
.
}


protected void paintComponent(Graphics g) {
    int out[]= counter(s);

    for (int x=0;x<26;x++){
    g.setColor(Color.black);
    g.fillRect(x*30, 200, 10, out[0]);    
    }
}

aber ich kann trotzdem nicht in der paintComponent mit out[x] drauf zugreifen...
weil ich ja nicht an string s komme!?!?!
oder???


gruß
gab

 

[Wildcard]

Wenn du's so machst wie ich vorgeschlagen brauchst du das

int out[]= counter(s);

in der paintComponent gar nicht, weil du out schon hast.

 

[gabriel]

ok ist mir glaub ich schon klar aber wo übergebe ich denn
int out[] dem counter... (und brauch ich nicht auch string s?)
ich peil nichts mehr, sorry...

könntest du mir nicht vielleicht formal zeigen wie sowas aussehen würde?
danke nochmal für deine zeit bzw. hilfe!

gruß
gabriel

 

[Wildcard]

So in etwa:

public class Go extends JComponent{

   private int[] counter;

   //hier einen Konstruktor hin:
   public Go(String s)
   {
       counter(s);
   }

   public static void main(String[] argv) {
      if (argv.length == 0) {
         System.err.println("Bitte einen Dateinamen angeben");
         return;
      }
      String s = argv[0];
      
      //System.out.println(out[0]);
      
      JFrame aJFrame = new JFrame("SHOW");
      aJFrame.getContentPane().add(new Go(s), BorderLayout.CENTER);

      aJFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      aJFrame.pack();
      aJFrame.setVisible(true);
   }
   
   public static void counter(String s){   
      counter = new int[26];
      try{
         BufferedReader buffReader = new BufferedReader(new FileReader(s));      
         String fullFile = "";
         String input = "";          
         
         while((input = buffReader.readLine()) != null) {
            fullFile += input;
         }
         buffReader.close();
         
         fullFile = fullFile.toLowerCase();
         for(int i=0;i<fullFile.length();i++){
         
            char test = fullFile.charAt(i);
            int count = (int) fullFile.charAt(i);
            if(count>=97&&count<=122){
               counter[count-97]++;
            }
         }
      }
      catch (IOException e) {      
         e.printStackTrace();
      }
      }
                  
   public Dimension getPreferredSize() {
      return new Dimension(820,400);
   }

   protected void paintComponent(Graphics g) {
   //hier kann counter benutzt werden
    counter[blubb]=5;
   
      for (int x=0;x<26;x++){
         g.setColor(Color.black);
         g.fillRect(x*30, 200, 10, 50);   
      }
   }   
}

 

[gabriel]

hmmmm okay danke,
so in der art hatte ich es glaub ich auch schonmal :roll:
ich hatte den Konstruktor immer unten!
big thx;
nun nun nochwas

ich möchte ja nun den arrayinhalt als höhe meines rechtecks verwenden;
so in der art...

	protected void paintComponent(Graphics g) {
		//int out[]= counter(s);
	
		for (int x=0;x<26;x++){
			g.setColor(Color.black);
			g.fillRect(x*30, 200, 10, counter[x]);    
		}
	}

aber wenn ich das tuhe bekomm ich diese


Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at Go.paintComponent(Go.java:68)
at javax.swing.JComponent.paint(JComponent.java:1003)
at javax.swing.JComponent.paintChildren(JComponent.java:840)
at javax.swing.JComponent.paint(JComponent.java:1012)
.
.
.

kann es sein das ich irgendwas nicht initialisiert habe??
oder was könnte das sein

gruß
gab

 

[Wildcard]

Zeig mal her wie du das gemacht hast

 

[gabriel]

hmmm so??

ich glaub kann es sein das counter leer ist??
ich kann schon hier nicht aufs array zugreifen...

also ich dachte mir das so:

	protected void paintComponent(Graphics g) {
		//int out[]= counter(s);
	
		for (int x=0;x<26;x++){
			g.setColor(Color.black);
			g.fillRect(x*30, 200, 10, counter[x]);    
		}
	}

import java.io.*;
import java.awt.*; 
import javax.swing.*; 


public class Go extends JComponent{

	public static void main(String[] argv) {
		if (argv.length == 0) {
			System.err.println("Bitte einen Dateinamen angeben");
			return;
		}
		String s = argv[0];
		JFrame aJFrame = new JFrame("SHOW");
		aJFrame.getContentPane().add(new Go(s), BorderLayout.CENTER);
		aJFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		aJFrame.pack();
		aJFrame.setVisible(true);
	}

	private int[] counter;
	
	public Go(String s){
		counter(s);
	}	
	
	public void counter(String s){	
	
		int[] counter = new int[26];
		try{
			BufferedReader buffReader = new BufferedReader(new FileReader(s));		
			String fullFile = ""; 
			String input = ""; 			
			
			while((input = buffReader.readLine()) != null) { 
				fullFile += input; 
			}
			buffReader.close();
			
			fullFile = fullFile.toLowerCase();
			for(int i=0;i<fullFile.length();i++){
			
				char test = fullFile.charAt(i);
				int count = (int) fullFile.charAt(i);
				if(count>=97&&count<=122){
					counter[count-97]++;
				}
			}
		}
		catch (IOException e) {		
			e.printStackTrace();
		}
	}
			
	public Dimension getPreferredSize() {
		return new Dimension(820,400);
	}

	protected void paintComponent(Graphics g) {
		//int out[]= counter(s);
	
		for (int x=0;x<26;x++){
			g.setColor(Color.black);
			g.fillRect(x*30, 200, 10, counter[x]);    
		}
	}	
}

 

[Wildcard]

Du hättest das schon so übernehmen müssen!

   public void counter(String s){   
   
      counter = new int[26];
      try{
         BufferedReader buffReader = new BufferedReader(new FileReader(s));      
         String fullFile = "";
         String input = "";          
         
         while((input = buffReader.readLine()) != null) {
            fullFile += input;
         }
         buffReader.close();
         
         fullFile = fullFile.toLowerCase();
         for(int i=0;i<fullFile.length();i++){
         
            char test = fullFile.charAt(i);
            int count = (int) fullFile.charAt(i);
            if(count>=97&&count<=122){
               counter[count-97]++;
            }
         }
      }
      catch (IOException e) {      
         e.printStackTrace();
      }
   }

 

[gabriel]

WOW es geht aber was ist dort anders?
danke, danke, danke

gruß
gabriel

 

[Wildcard]

int[] counter = new int[26];

Wenn du das in einer Methode aufrufst erzeugst du eine lokale Variable counter, die zufällig genauso heißt wie die
Instanzvariable counter. Ansonsten haben die beiden NICHTS miteinander zu tun.

 

[gabriel]

ach ja "counter ist void" jetzt versteh ich was du gemeint hast!
danke für deine gedult!

gruß
gabriel