Kast Exceptions fra Innerclass

mangler du ikke en } ?

try
{
    andre iffer....
    if(source == jButtonValidate)
    {
        final SwingWorker worker = new SwingWorker()
        {
            public Object construct()
            {
                //denne linie smidder en masse exceptions
                jctl.ctlValidate(jTextFile.getText(), (Template)jComboTemplates.getSelectedItem());
                jButtonMakeScript.setEnabled(ctl.ctlGetValidatedStatus());
                jEditorPaneMsg.setCaretPosition(0);
                jProgressBar1.setIndeterminate(false);
       
                if(ctl.ctlIsLinks())
                {
                    ctl.ctlDrawWorkflow();
                }
       
                return "test";
            }
        };
       
        worker.start();
    }
}
catch(Exception e){System.out.println(e);}


og prøv den her



try
{
    andre iffer....
    if(source == jButtonValidate)
    {
        final SwingWorker worker = new SwingWorker()
        {
            public Object construct()
            {
                //denne linie smidder en masse exceptions
                jctl.ctlValidate(jTextFile.getText(), (Template)jComboTemplates.getSelectedItem());
                jButtonMakeScript.setEnabled(ctl.ctlGetValidatedStatus());
                jEditorPaneMsg.setCaretPosition(0);
                jProgressBar1.setIndeterminate(false);
       
                if(ctl.ctlIsLinks())
                {
                    ctl.ctlDrawWorkflow();
                }
       
                return "test";
            }
        }
       
        worker.start();
    }
}
catch(Exception e){System.out.println(e);}


(har ikke sovet så meget i nat.. så måske snakker jeg sort hehe)

Jo nok fordi jeg klippede det sammen. min compiler besked:

script/gui/Form.java [366:1] unreported exception java.io.IOException; must be caught or declared to be thrown
jEditorPaneMsg.setText(ctl.ctlValidate(jTextFile.getText(),(Template)jComboTemplates.getSelectedItem()));
                                      ^

hvis du i

public static void main(String[] args)

skriver

public static void main(String[] args) throws Exception


så får du den i hvertfald ikke hehe

kalp ->Hehe
den brokker sig dog stadig :-(

det lyder som om det er fordi der en metode som kalder på noget som kan lave exceptions.. den metoder som kalder skal lige have en throws Exception.. hvis du gør det i main som jeg skrev før så mener jeg ikke den gør det mere.. medmindre fejlen er noget andet!

ellers smid det her ind i sin egen try catch block


//denne linie smidder en masse exceptions
                jctl.ctlValidate(jTextFile.getText(), (Template)jComboTemplates.getSelectedItem());
                jButtonMakeScript.setEnabled(ctl.ctlGetValidatedStatus());
                jEditorPaneMsg.setCaretPosition(0);
                jProgressBar1.setIndeterminate(false);

Tjae men problemet er at metoden som kaster exceptions er i en indre klasse SwingWorker så hvordan kan jeg sætte den til at kaste videre? eller kan man ikke det, for så må jeg lave strukturen om.. :-(

hmm.. (prøver at ryste hjernen på plads)

hvad sker der hvis du  skriver throws Exception efter dim indre klasses navn?

script/gui/LIMSEMForm.java [363:1] ';' expected
                    final SwingWorker worker = new SwingWorker()throws Exception
                                                                ^

Det lidt ærgeligt du ikke bruger jdeveloper... kan du dobbelt klikke på fejlmeddelelser hos dig og din markør ender på punktet med fejl?

Jo og den giver denne linie
final SwingWorker worker = new SwingWorker()throws Exception

Har ikke helt styr på de her inner classes gør det kun pga. af en progress bar.

http://java.sun.com/products/jfc/tsc/articles/threads/threads2.html

script/gui/Form.java [366:1] unreported exception java.io.IOException; must be caught or declared to be thrown
jEditorPaneMsg.setText(ctl.ctlValidate(jTextFile.getText(),(Template)jComboTemplates.getSelectedItem()));

var din oprindelig fejl.. og du mangler thrown (har de med på det link jeg fandt.. kunne nemlig ikke finde klassen i API'et)

Ok nu tabte jeg tråden ??

kan bare se deres eksempel her .

Object doWork() {
  try {
      for(int i = 0; i < NUMLOOPS; i++) {
        updateStatus(i);
        if (Thread.interrupted()) {
            throw new InterruptedException();
        }
        Thread.sleep(500);
      }
  }
  catch (InterruptedException e) {
      updateStatus(0);
      return "Interrupted"; 
  }
  return "All Done";
}


men vil sgu helst ikke forvirre dig:o(

næe det blev jeg ikke meget klogere af, så skal jeg ikke bare bide i det sure æble og lave Exception handling i Construct().

hmmnjaee måske.. du kan også vente lidt og håbe på at en med 20års programmering kommer herind med lidt guldkorn;)

uha det med at venter er jeg ikke så god til ARNE hvor er du!! :-)

hehe hver toldmodig;)

Problemet er at SwingWorker construct ikke er erklæret til at throwe nogen exception.

Jeg kan se 3 mulige løsninger:

1)  du catcher exception i construct (kan være uhensigtsmæssig)

2)  du retter SwingWorker construct så den thrower en Exception + alle
    de nødvendige konsekvens rettelser i SwingWorker (en del arbejde og
    måske ikke så pænt) og lader så din construct implementation throwe en Exception

3)  du thrower en unchecked exception altså en exception som extender
    RuntimeException

og #3 virker meget fint.

Men men men ...

Den exception bliver jo ikke catchet i din tråd, fordi den kode køres i en separat tråd.

Og så er du jo lige vidt.

:-(

/*
* Created on 2005-01-13
*/

import java.io.IOException;
import java.lang.reflect.InvocationTargetException;

/**
* @author Snoop_one
*/
public class SwingWorkerExceptionTester {

    public SwingWorkerExceptionTester() {
        try {
            final SwingWorker worker = new SwingWorker() {
                public Object construct() {
                   
                    try {
                        //denne linie smidder en masse exceptions
                        exceptionMethod();
                    } catch (Exception e) {
                        return e;
                    }

                    return "test";
                }
            };

            worker.start();

            if (worker.get() instanceof Exception) {
                Throwable exp = new InvocationTargetException(
                        (Exception) worker.get()).getTargetException();
                System.out.println("The Exception was a: " + exp +
                        "\n\twith this message: "+ exp.getMessage());
            }
        }

        catch (Exception e) {
            System.out.println(e);
        }

    }

    public void exceptionMethod() throws IOException {
        throw new IOException("Testing");
    }

    public static void main(String[] args) {
        new SwingWorkerExceptionTester();
    }
}

Håber du kan følge tankegangen

snoop_one -> det ser jo lovende ud kan man ikke smide exceptionen videre her:
if (worker.get() instanceof Exception)
{
    Throwable exp = new InvocationTargetException(
      (Exception) worker.get()).getTargetException();
    System.out.println("The Exception was a: " + exp +
                        "\n\twith this message: "+ exp.getMessage());
}

Så kan man ikke sige throw (Exception) worker.get(); ?

Jo altså du kan jo lave den på denne måde hvor du tager dig af exception handling et andet sted, men ved ikke hvor god idé det er? (tegn på design fejl!):

/*
* Created on 2005-01-13
*/

import java.io.IOException;
import java.lang.reflect.InvocationTargetException;

/**
* @author Snoop_one
*/
public class SwingWorkerExceptionTester {

    public SwingWorkerExceptionTester() throws IOException {

        final SwingWorker worker = new SwingWorker() {
            public Object construct() {
                try {
                    // denne linie smidder en masse exceptions
                    exceptionMethod();
                } catch (Exception e) {
                    return e;
                }
                return "test";
            }
        };
       
        worker.start();
        if (worker.get() instanceof IOException) {
            throw (IOException) worker.get();
        }
    }

    public void exceptionMethod() throws IOException {
        throw new IOException("Testing");
    }

    public static void main(String[] args) {
        try {
            new SwingWorkerExceptionTester();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

Har lavet mit design om så jeg slipper for denne problematik så kan folk ikke smide et svar. Og tak for hjælpen.

Og undskyld det har taget så lang tid havde glemt dette spørgsmål. Dette kunne være sejt med en slags reminder funktion her på eksperten eller en 'time to live' på sine spørgsmål..

svar

Jeg synes at du skal give snoop alle pointsene for hans gode ide

Ja det var oxo hans løsning jeg næsten fik til at virke.. så værsgo