Try catch Problem

[Lazarus]

Ich bin mir nicht sicher aber kann es sein daß ich in folgendem Codeblock eine Exception Behandlung brauche?

      StringTokenizer tokenizer1 = new StringTokenizer(text, "\n");
      while (tokenizer1.hasMoreTokens()) {
        String token = tokenizer1.nextToken();

        StringTokenizer tokenizer2 = new StringTokenizer(token, delimiter);
        i=0;
        while (tokenizer2.hasMoreTokens()) {
          String token2 = tokenizer2.nextToken();
          if(jComboBoxC1[i].getSelectedIndex()==1)
            token2 = token2.toUpperCase();
          if(jComboBoxC1[i].getSelectedIndex()==2)
            token2 = token2.toLowerCase();
          if(jTextFieldRo[i].getText()!=""){
            if(jComboBoxC2[i].getSelectedIndex()==0)
            token2 = token2.replaceFirst(jTextFieldRo[i].getText(),jTextFieldRw[i].getText());
            if(jComboBoxC2[i].getSelectedIndex()==1)
            token2 = token2.replaceAll(jTextFieldRo[i].getText(),jTextFieldRw[i].getText());
          }
            desire[i] = token2+outputdel;
            i++;
          }
       for(i=0;i<k.length;i++)
       buffer[i]=desire[index[i]];
       list = Arrays.asList(buffer);
       if(jTinstringout.getText()!="")
       list.add(Integer.parseInt(jTinstringout.getText()),jLinstring.getText());
      /* else
       list.remove(k.length+1);*/
       output += list+"\n";
      }
      exchanger.write(exchanger.getFile(),output);

Wenn ich das programm laufen lasse kommen folgende Hinweise, was aber nicht sein kann. Der String der zerlegt werden soll ist nicht null.
java.lang.NullPointerException
at java.util.StringTokenizer.<init>(StringTokenizer.java:146)
at java.util.StringTokenizer.<init>(StringTokenizer.java:162)
at datasetparser.parseFrame.parse(parseFrame.java:298)
at datasetparser.parseFrame.actionPerformed(parseFrame.java:235)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1786)
at javax.swing.AbstractButton$ForwardActionEvents.actionPerformed(AbstractButton.java:1839)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:420)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:258)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:245)
at java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:231)
at java.awt.Component.processMouseEvent(Component.java:5100)
at java.awt.Component.processEvent(Component.java:4897)
at java.awt.Container.processEvent(Container.java:1569)
at java.awt.Component.dispatchEventImpl(Component.java:3615)
at java.awt.Container.dispatchEventImpl(Container.java:1627)
at java.awt.Component.dispatchEvent(Component.java:3477)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:3483)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:3198)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:3128)
at java.awt.Container.dispatchEventImpl(Container.java:1613)
at java.awt.Window.dispatchEventImpl(Window.java:1606)
at java.awt.Component.dispatchEvent(Component.java:3477)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:456)
at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:201)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:151)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:145)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:137)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:100)

und wenn ich Debugge kommt auf einmal so ein Reiter wo drauf steht EventDispatchThread und der Cursor steht bei diesem Block

synchronized (theQueue) {
theQueue.detachDispatchThread();
/*
* Event dispatch thread dies in case of an uncaught exception.
* A new event dispatch thread for this queue will be started
* only if a new event is posted to it. In case if no more
* events are posted after this thread died all events that
* currently are in the queue will never be dispatched.
*/
/*
* Fix for 4648733. Check both the associated java event
* queue and the PostEventQueue.
*/
if (theQueue.peekEvent() != null ||
!SunToolkit.isPostEventQueueEmpty()) {
theQueue.initDispatchThread();
}
AWTAutoShutdown.getInstance().notifyThreadFree(this);
}

 

[mic_checker]

Wenn ich das recht in Erinnerung hab brauchst du für best. Exceptions keinen expliziten try...catch Block, da best. Exceptions so oft auftauchen können. Afaik zählen zu diese Art Exceptions z.B. die NullPointerException oder auch die ArrayIndexOutOfBoundsException. Das heisst natürlich nicht das du sie nicht in einen catch Block schreiben darfst.

 

[niemand]

Die Exception zu fangen kann sehr hilfreich sein, wenn man den Fehler finden möchte: Man gibt einfach alle Variablen aus, die den Fehler verursachen könnten und weiß dann auch ohne Debugger, wo es hakt. Was man nicht machen sollte, ist, die Exception zu fangen und das Programm mit undefinierten Werten weiterlaufen zu lassen.

cu

 

[mic_checker]

klar, ich mach meist auch erst zig System.out.println's und lass alles ausgeben. Nur zielte seine Frage , meiner Meinung nach, darauf ab ob er die NPE an der Stelle try...catchen muss.
Kannst sie natürlich mal catchen und StackTrace ausgeben lassen. Ist es sicher das der String != null ist?

 

[Mac Systems]

Ich würde Zeile 25 zumindest in einem try / catch laufen lassen. Ist ja nicht gesagt das ein Integer im Textfeld eingegeben wurde, hier wird zwar keine NullPointerException geworfen die Zeile fällt mir aber auf.
Oder man nimmt einen InputVerifier um sicher zu gehen.


- jens

 

[foobar]

Wenn ich das recht in Erinnerung hab brauchst du für best. Exceptions keinen expliziten try...catch Block, da best. Exceptions so oft auftauchen können. Afaik zählen zu diese Art Exceptions z.B. die NullPointerException oder auch die ArrayIndexOutOfBoundsException. Das heisst natürlich nicht das du sie nicht in einen catch Block schreiben darfst.

Exceptions die von RuntimeException abgeleitet wurden, müssen nicht aufgefangen oder weiter geworfen werden.

 

[Bleiglanz]

Klammern, schaun dass die Strings nicht == null sind...

      StringTokenizer tokenizer1 = new StringTokenizer(""+text, "\n");
      
      while (tokenizer1.hasMoreTokens()) 
      {
          String token = ""+ tokenizer1.nextToken();// eigentlich nicht nötig
          StringTokenizer tokenizer2 = new StringTokenizer(""+token,""+ delimiter);
          i=0;
          while (tokenizer2.hasMoreTokens()){
             String token2 = "" + tokenizer2.nextToken(); // nur auf nummer sicher :-)
             if(jComboBoxC1[i].getSelectedIndex()==1) token2 = token2.toUpperCase();
             if(jComboBoxC1[i].getSelectedIndex()==2) token2 = token2.toLowerCase();
             if(jTextFieldRo[i].getText()!="") {
                 if(jComboBoxC2[i].getSelectedIndex()==0) token2 = token2.replaceFirst(jTextFieldRo[i].getText(),jTextFieldRw[i].getText());
                 if(jComboBoxC2[i].getSelectedIndex()==1) token2 = token2.replaceAll(jTextFieldRo[i].getText(),jTextFieldRw[i].getText());
             }
             desire[i] = token2+outputdel;
             i++;
          }
          for(i=0;i<k.length;i++)
          buffer[i]=desire[index[i]];
          list = Arrays.asList(buffer);
          if(jTinstringout.getText()!="") list.add(Integer.parseInt(jTinstringout.getText()),jLinstring.getText());
          output += list+"\n";
      }
   
      exchanger.write(exchanger.getFile(),output);

 

[niemand]

Nur zielte seine Frage , meiner Meinung nach, darauf ab ob er die NPE an der Stelle try...catchen muss.

Die Frage beantwortet doch der Compiler schon: Wenn er eine mögliche Exception nicht auffängt, die er auffangen müsste, wird das doch gar nicht erst kompiliert. Oder habe ich schon wieder was falsch verstanden?

cu

 

[mic_checker]

Nein,da hast du natürlich recht. Da würde er wohl "Exception xxx must be caught" melden.

 

[Lazarus]

So wie du es geschrieben hast klappt es

nur bei diesem Block hat er noch Schwierigkeiten

/ if(jTinstring.getText()!= "") list.add(Integer.parseInt(jTinstringout.getText()),jLinstring.getText());

erzählt was von NumberFormat Exception. Na ich probier mal etwas rum.

Danke erst mal. Und nun weiß ich auch das es nicht an einer unabgefangenen Exception liegt.

 

[KSG9|sebastian]

numberformatexception heist, dass der text in jTinstringout kein integerwert ist! lass dir den string mal ausgeben und trim ihn:

if(jTinstring.getText()!= "") list.add(Integer.parseInt(jTinstringout.getText().trim()),jLinstring.getText());