Retur typen

hvad bruger du parameteren til i den metoder du viser os?

retur typen skal være er typen "Sol"

fx.

Sol tmp;

tmp = printSol(Node nn)

hvad en din Node er til:)

Jeg bruger værdierne til at sætte til nogle faste variable som jeg skal bruge til senere udregninger

okay men det du var ude efter var det mit forslag eller noget helt andet?

jo det var helt korrekt, men har du tid til en lille mere ?

jeg får en fejl på P "The local variable P may not have been initialized"

i den samlede kode af denne funktion:

        Punkt P;
        Color C;
       
        if(nn.getNodeType()!=Node.ELEMENT_NODE)
            throw new XMLParseError("Kunne ikke omdanne til Element");

        NamedNodeMap nnm=nn.getAttributes();
        String id=nnm.getNamedItem("id").getNodeValue();
   
        NodeList nl=((Element)nn).getElementsByTagName("farve");
        C = printColorFromNNM(nl.item(0).getAttributes());
   
        nl=((Element)nn).getElementsByTagName("punkt");
        NamedNodeMap punktnnm=nl.item(0).getAttributes();
        if(punktnnm.getNamedItem("id").getNodeValue().equals("position")){
            P = printPunktFromNNM(punktnnm);
        }
        return new Sol(P,C);

hele funktionen ;O)

    private static Sol printSol(Node nn) throws XMLParseError{//læser og skriver en sol givet en xml-knude
        Punkt P;
        Color C;
       
        if(nn.getNodeType()!=Node.ELEMENT_NODE)
            throw new XMLParseError("Kunne ikke omdanne til Element");

        NamedNodeMap nnm=nn.getAttributes();
        String id=nnm.getNamedItem("id").getNodeValue();
   
        NodeList nl=((Element)nn).getElementsByTagName("farve");
        C = printColorFromNNM(nl.item(0).getAttributes());
   
        nl=((Element)nn).getElementsByTagName("punkt");
        NamedNodeMap punktnnm=nl.item(0).getAttributes();
        if(punktnnm.getNamedItem("id").getNodeValue().equals("position")){
            P = printPunktFromNNM(punktnnm);
        }
        return new Sol(P,C);

Det ser ud til at være fordi du laver en if sætning.

if(punktnnm.getNamedItem("id").getNodeValue().equals("position")){
            P = printPunktFromNNM(punktnnm);

og den bliver ikke kørt åbenbart.. så P bliver aldrig initialiseret.. så du må enten lave en else sætning eller sætte p = noget øverst

Punkt P = ?;

prøv og se om ikke det skulle være det eller må jeg lige kigge din kode ordenligt igennem.. skimmede kun hehe

tusinde tak for den hurtige hjælp og her dine vel fortjente point ;O)

Det er ikke fordi den ikke bliver kørt, den if-sætning. Java analyserer metoden
for at se om P med sikkerhed får en værdi inden den bliver brugt. En if-sætning
betyder jo at den ved kørsel måske udføres, måske ikke, og derfor er det ikke
sikkert at P får en værdi der.

Hvis det ellers passer til problemet, kan man blot sætte P til null, allerøverst i metoden.

Thanks for point:)

og til mit forsvar så kiggede jeg ikke rigtig på koden men gik mere efter hvad fejlmeddelelsen lød på:) men ramte da rigtigt hvad angå løsningsforslag:)