En nullPointerExceptioin?

num = (int) (Math.random()*100+1 );


prøv den istedet for

num = generator.nextInt(100)+1;

3 muligheder:

rollinput er null
gui er null
gui.control er null

find ud af hvilken og ret det

+gui.control.dice()

bliver da egentlig forvirret over 2 gange punktum!!!

Hej Kalp

nej desværre, det var ikke det

hej Arne v..

øhh..hvordan gør jeg det?

Skriver dem ud !

private void rollButton_actionPerformed(ActionEvent e)
  {
    System.out.println(rollinput);
    System.out.println(gui);
    System.out.println(gui.control);
    rollinput.setText("h"+gui.control.dice());
  }

skulle også lige til at sige det kunne være smart hvis man vidste hvilke exception det er;)

+gui.control.dice() bør virke kalp

kalp>

Jeg har ligesom fået indtrykker at det er NullPointerException ...

det står i overskriften kalp

nåhh ja.. sådan er det jo når man ryger for meget :o) ---> joke

Hej Arne v

Det er gui, der er null

har du initialiseret din gui?

OK

så skal du jo have fundet ud af hvorfor den er det !

Og vi skal nok se noget mere kode hvis vi skal bidrage til det.

Men muligvis er det ret nemt at finde når nu dy ved hvad det er du skal kigge efter.

Ok min gui kode fylder en del...

Men først... er det ikke nok at i den JFrame jeg kalder fra kalder til gui, der kalder til kontrolklassen, når der i min frame står GUI gui; samt i min GUI står Kontrol control; ??

package mypackage1;

import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.ObjectInputStream;
import java.io.ObjectOutputStream;
import java.io.Serializable;
import java.util.ArrayList;
//import javax.naming.ldap.Control;
import javax.swing.*;
import java.awt.event.*;
import java.awt.*;
import javax.swing.JDesktopPane;
import javax.swing.ImageIcon;
import java.awt.Image;

public class GUI extends JFrame implements Serializable
{ 
    Kontrol control;
    CharIcon enemyIcon;
    Attacker attack;
    Defender defend;
    Dialog dia;
    App app;
    JDesktopPane desktop;
    JPanel p;
    ArrayList array;
    Object hej;
    int antal;
    ImageIcon icon;
   
    public GUI()
   
    { 
        super("Merp-HelpDesk");
        Dimension screenSize = Toolkit.getDefaultToolkit().getScreenSize();
        //Dimension frameSize = content.getSize();
        setSize(screenSize.width, screenSize.height-30);
        this.setDefaultCloseOperation(EXIT_ON_CLOSE);
       
                   
        desktop = new JDesktopPane();
        //setContentPane(desktop);
        final JPanel p = new JPanel(new GridBagLayout());
        desktop.setBackground(new Color(67, 175, 16));
        desktop.addComponentListener(new ComponentAdapter()
        {
          public void ComponentResized(ComponentEvent ev)
          {
            Dimension desktopSize = desktop.getSize();
            p.setBounds(0,0,desktopSize.width, desktopSize.height);
            p.validate();
          }
        });
        desktop.add(p);
       
               
        JPanel buttonPanel = new JPanel();       
       
        JButton addEnemy = new JButton("Add Enemy");
        addEnemy.setBackground(Color.getHSBColor(12,23,345));
        addEnemy.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent event)
            { 
              AddEnemy();
              System.out.println("add");
            }
        });
       
               
        JButton nextRound = new JButton("Next Round");
        nextRound.setBackground(Color.getHSBColor(12,23,345));
        nextRound.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent event)
            {
             
            }
        });

        JButton save = new JButton("Save");
        save.setBackground(Color.getHSBColor(12,23,345));
        save.addActionListener(new ActionListener()
       
        {
            public void actionPerformed(ActionEvent event)
            {
              array.trimToSize();
              try
              {
                FileOutputStream file = new FileOutputStream("Savings.ser");
                ObjectOutputStream outStream = new ObjectOutputStream(file);
               
                outStream.writeObject(array);
                outStream.close();
                //file.close();
              } 
              catch(Exception e)
              {}
            }
        });
        JButton load = new JButton("Load");
        load.setBackground(Color.getHSBColor(12,23,345));
       
        buttonPanel.add(addEnemy);
        buttonPanel.add(nextRound);
        buttonPanel.add(save);
        buttonPanel.add(load);

        getContentPane().add(desktop, BorderLayout.CENTER);
        getContentPane().add(buttonPanel, BorderLayout.SOUTH);
       
        control = new Kontrol();
        attack = new Attacker();
        desktop.add(attack);
        attack.setVisible(true);
        defend = new Defender();
        desktop.add(defend);
        defend.setVisible(true);
        dia = new Dialog();
        desktop.add(dia);
        dia.setVisible(true);
    }
   
    public void AddEnemy()
    { 
        /*
        int count = desktop.getAllFrames().length;
        enemyIcon = new CharIcon();
       
        desktop.add(enemyIcon);
        //charIcon.setLocation(count*8,count*8);
        //charIcon.setVisible(true);
        array.add(enemyIcon);
       
        System.out.println(array.size());
          */
      int count = desktop.getComponentCount();
         
      icon = new ImageIcon("dwarf.gif");
      JLabel charIcon = new JLabel(icon);

      charIcon.setBounds(new Rectangle(155, 40, icon.getIconWidth(), icon.getIconHeight())); 
      charIcon.setLocation(count*8,count*8);
      charIcon.addMouseListener(new java.awt.event.MouseAdapter()
      {
        public void mouseClicked(MouseEvent e)
        {
          if (e.getButton()==3 &&  e.getClickCount() == 2)
            højreDobKlik(e);
          else
         
            if ( e.getClickCount() == 2)
              dobbeltKlik(e);
        }
      });
      desktop.add(charIcon);
      charIcon.setVisible(true);
     
      add_c(charIcon);
    }
   

 
  private void dobbeltKlik(MouseEvent e)
  {
    System.out.println("Dobbeltklik");
  }
  private void højreDobKlik(MouseEvent e)
  {
    System.out.println("højre/Dobbeltklik");
  }


    public void add_c(final Component c)
      {    c.addMouseMotionListener(new MouseMotionAdapter()
      {    public void mouseDragged(MouseEvent m)
      {
          c.setLocation((c.getX()+m.getX())-(icon.getIconWidth()/2),(c.getY()+m.getY())-(icon.getIconHeight()/2));
      }});
          //this.getContentPane().add(c);
      } 

}

nej det er referencer

du skal give dine reference nogen værdier..

fx.

control = ?? ;

og det gør du i din kontruktør

det samme gælder din gui.. du har oprettet en reference til den.. men ikke initialiseret den.. altså ud fra det du skrev.. nu har jeg ikke set din kode

I hvilken klasse ligger

private void rollButton_actionPerformed(ActionEvent e)

?

i din frame.. laver du også en

gui = new GUI();

?

Hej Kalp

min App initialiserer min GUI klasse og
min gui initialiserer en frame "dialog", samt den initialiserer control klassen.

i min "dialog" har jeg skrevet GUI gui; hvor at kunne kalde en metode gennem GUI klassen til kontrol klassen.

hvis jeg har forstået det rigtigt.. så skal du i din dialog have en reference med til din gui

hej kalp

control = new Kontrol();  står lidt nede på siden

dvs. når du laver din frame.. skal du have this. med som parameter..

og i din dialog.. skal gui = den parameter

Så skal du formentlig have en konstruktion som:

GUI.java:

...
dialog d = new dialog(this);
..

dialog.java:

...
GUI gui;
...
public dialog(GUI gui) {
  this.gui = gui;
}
...

for at dialog har en reference til den rigtige GUI instans

jeghaderorganisation :)

Kommentar: kalp
12/02-2005 23:28:34

Kommentar: kalp
12/02-2005 23:29:12

og et flot eksempel på det

Kommentar: arne_v
12/02-2005 23:29:12

slut resultat.. du initialiserede altså ikke din gui.. i din frame.. der oprettede du bare en reference til et gui objekt;)

hej Arne

min Private void roll... metode ligger i en framen dialog som bliver initialiseret af GUI.

Spørgsmålet er vel at jeg kun har sat en reference tilbage fra dialog til Gui klassen

"GUI gui;" er det ikke fint, da GUI'en er blevet initialiseret én gang fra App klassen.

GUI gui;

er jo bare en reference til et gui objekt.... du har ikke angivet hvilket!!

det gør du ved at initialisere det med gui = det gui objekt som det skal rumme

GUI.java:

...
Dialog dia = new Dialog(this);
..

Dialog.java:

...
GUI gui;
...
public Dialog(GUI gui) {
  this.gui = gui;
}
...

Hej Kalp


#¤%&¤%¤ arrrhhh  selvfølgelig....

kan jeg sku ikke sige, da det var en ting jeg ikke havde helt styr på..

Men hurra det virkede....

Mange tak

og du forstod hvorfor?:)

Nå nej vent lige...det var skyu da Arne  der gav det rigtige svar

Hold da op den er svær...i får begge to point.

pis, nu gav jeg mig selv point....øjeblik

hvordan fanden giver jeg jeres point nu...ved i det?

Du bliver nød til at oprette et nyt spørgsmål i java katagorien... husk at smide et link hertil i det nye spørgsmål så folk kan se hvad det drejer sig om

http://expfaq.1go.dk/?id=30#giv_flere_point

arne_v>> hmm.. han skal da ikke give flere point.. han har afsluttet dette spørgsmål

http://www.eksperten.dk/spm/590511

titlen passer ikke men proceduren gør