Java probleem

[Crooker]

Hallo,

Ik ben net begonnen met java te programmeren en heb hieronder een scherm gemaakt waarmee het mogelijk zou moeten zijn om in het vak "optelling" het getal dat je bovenaan invult op te tellen en in het vak "vermenigvuldiging" het bovenstaande getal mee te vermenigvuldigen. Ik heb het geprogrammeerd in Eclipse welke de melding geeft: The field debugpaneel.optel is never read locally
Dezelfde melding krijg ik voor debugpaneel.verm.

Kan iemand mij helpen?

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

public class debugpaneel extends JPanel implements ActionListener
{
private JTextField v1, v2, v3; private JButton knop; private int getal, optel, verm;

public debugpaneel()
{
v1 = new JTextField( 10 );
v2 = new JTextField( 10 );
v3 = new JTextField( 10 );
v2.setEditable( false );
v3.setEditable( false );
knop = new JButton( "druk hier" );
knop.addActionListener( this );
add( v1 );
add( knop );
add( new JLabel( "optelling: " ) );
add( v3 ); add( new JLabel( "vermenigvuldiging: " ) );
add( v2 );
}

public void actionPerformed( ActionEvent e )
{
int optel = getal ; int verm = getal;
int getal = Integer.parseInt( v1.getText() );
optel += getal; verm *= getal;
v2.setText(""+ optel );
v3.setText(""+ verm );
}
}

[Avalaxy]

Dat betekent dat je niks met je JButton optel doet.

[Matis]

Avalaxy schreef op dinsdag 28 september 2010 @ 21:00:
Dat betekent dat je niks met je JButton optel doet.

optel is een private int

Maar het is nogal rottig weergegeven moet ik idd bekennen.

[Herko_ter_Horst]

Je declareert de variabele optel twee keer: een keer als field, en een keer als lokale variable binnen de method actionPerformed. En gebruik in het vervolg even [code] tags.

[Avalaxy]

Matis schreef op dinsdag 28 september 2010 @ 21:01:
[...]

optel is een private int

Maar het is nogal rottig weergegeven moet ik idd bekennen.

Ah sorry, is inderdaad onduidelijk neegezet.

Gebruik voortaan eventjes [code = java] (zonder spaties).

Zoals hierboven gezegd doe je dus niets met de initiële versie van optel (en ook van verm), want later creëer je een nieuwe int waarop je de bewerking(en) doet.

Zo is hij dus wel goed (althans, puur kijkend naar warnings):

Java:

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

public class debugpaneel extends JPanel implements ActionListener
{
    private JTextField v1, v2, v3; 
    private JButton knop; 
    private int getal, optel, verm;

    public debugpaneel()
    {
        v1 = new JTextField( 10 );
        v2 = new JTextField( 10 );
        v3 = new JTextField( 10 );
        v2.setEditable( false );
        v3.setEditable( false );
        knop = new JButton( "druk hier" );
        knop.addActionListener( this );
        add( v1 );
        add( knop );
        add( new JLabel( "optelling: " ) );
        add( v3 ); add( new JLabel( "vermenigvuldiging: " ) );
        add( v2 );
    }

    public void actionPerformed( ActionEvent e )
    {
        optel = getal ; 
        verm = getal;
        getal = Integer.parseInt( v1.getText() );
        optel += getal; 
        verm *= getal;
        v2.setText(""+ optel );
        v3.setText(""+ verm );
    }
}

[ Voor 48% gewijzigd door Avalaxy op 28-09-2010 21:06 ]

[Crooker]

Thnx voor de snelle reactie. De melding is zo idd weg Nou moet ik hem nog werkend krijgen. Ik moet het invoergetal kunnen optellen (dus ook onthouden) bij het getal dat er al staat. Hetzelfde geld voor vermenigvuldigen alleen dan moet het getal wat je invoert vermenigvuldigd worden met het getal dat er al staat.

Alvast bedankt

[ Voor 78% gewijzigd door Crooker op 28-09-2010 21:34 ]

[Avalaxy]

Crooker schreef op dinsdag 28 september 2010 @ 21:27:
Thnx voor de snelle reactie alleen krijg ik nog steeds dezelfde melding.

Op de regel: private int getal, optel, verm; geeft hij de melding op optel en verm

Met de code die ik net postte? Ik heb hem zelf even in eclipse gegooid en ik zie geen warnings hoor

Je kunt als het goed is op de warning klikken, dan zie je als het goed is suggesties om het probleem op te lossen, waaronder waarschijnlijk de suggestie om de integers optel en verm te verwijderen. Als je daar gewoon op klikt is je probleem opgelost.

[Crooker]

Je hebt gelijk, ik heb mij post aangepast

Avalaxy schreef op dinsdag 28 september 2010 @ 21:34:
[...]


Met de code die ik net postte? Ik heb hem zelf even in eclipse gegooid en ik zie geen warnings hoor

Je kunt als het goed is op de warning klikken, dan zie je als het goed is suggesties om het probleem op te lossen, waaronder waarschijnlijk de suggestie om de integers optel en verm te verwijderen. Als je daar gewoon op klikt is je probleem opgelost.

[Creepy]

Crooker schreef op dinsdag 28 september 2010 @ 21:27:
Thnx voor de snelle reactie. De melding is zo idd weg Nou moet ik hem nog werkend krijgen. Ik moet het invoergetal kunnen optellen (dus ook onthouden) bij het getal dat er al staat. Hetzelfde geld voor vermenigvuldigen alleen dan moet het getal wat je invoert vermenigvuldigd worden met het getal dat er al staat.

Alvast bedankt

Succes met het zelf eerst uitproberen en verder uit te zoeken hoe en wat. Je komt nu niet verder dan "hoi, dus is mijn code, dit is er mis, wie helpt me", en dat is nu net niet de bedoeling. Kan je wat aan de volgende punten doen?

- Topic titel. "java probleem" zegt natuurlijk helemaal niks. Via een topicreport kan je een andere topictitel doorgeven
- Je eigen inzet. Wat heb je nu allemaal zelf al geprobeerd? Wat lukte daar niet mee?
- Code posten. Post aub alleen de relevante code en gebruik de code tags: [code]codehier[/code]

Dit staat overigens allemaal netjes uitgelegd in PRG Beleid en ook in Het algemeen beleid waarbij Het algemeen beleid #quickstart laat zien wat je eigenlijk zou moeten doen bij een topicstart Kan je de informatie aanvullen?

[ Voor 14% gewijzigd door RobIII op 28-09-2010 22:00 . Reden: code-tags linkified :+ ]

[Davio]

Crooker schreef op dinsdag 28 september 2010 @ 21:27:
Thnx voor de snelle reactie. De melding is zo idd weg Nou moet ik hem nog werkend krijgen. Ik moet het invoergetal kunnen optellen (dus ook onthouden) bij het getal dat er al staat. Hetzelfde geld voor vermenigvuldigen alleen dan moet het getal wat je invoert vermenigvuldigd worden met het getal dat er al staat.

Alvast bedankt

Tja, op het moment dat je op de optel- of vermenigvuldigknop drukt, moet je gewoon de huidige waarde opslaan en op het moment dat je op de =-knop drukt de nieuwe huidige waarde met de vorige waarde optellen of vermenigvuldigen.