third attempt at solving this

[andrew cassato]

hey, me again. this wont work. I want to have the final score show up once i press the calculate scores button but am losing the data because I don't know what code is missing to save the score, nor do i know where to put it. I understand why the result is 0.0, i just really really really dont know how to make it work.

this assignment is already late to the point that I won't get credit so dont worry about just telling me the answer at this point i just want it working for a reference to later assignments / exam.

any sort of help is appreciated.

its in 3 classes and i will give you all the code in case you want to try to run it, the error is on line 97 of GUI class: "cannot find symbol - method parseInt(double); maybe you meant: parseInt(String) or parseInt(String, int)"

JUDGING CLASS

import java.util.Scanner;
import javax.swing.*;
import javax.swing.JOptionPane;
import java.awt.*;
import java.awt.event.*;
import java.applet.Applet;
/**
 * Write a description of class Judging here.
 * 
 * @author (your name) 
 * @version (a version number or a date)
 */
public class Judging
{
    private int n =0;
    private double[] score = new double[n];
    private double maxScore=0, minScore=11, totalScore=0, finalScore=0;
    private double result =finalScore;
    //Judging judge = new Judging();
    public Judging(int n)
    {
        score = new double[n];
        this.n = n;
        Scanner console = new Scanner(System.in);
        String answer = "", name="";
    }
    public  void setScores(double[] score)
    {
        this.score = score;
    }
    public void inputScores()
    {
        Scanner console = new Scanner(System.in);
        int n=0;
        String name = "", scoresS="";
        double[] score = new double[n];
        for (int i = 0; i < n; i++)
        {
           score[i] = Double.parseDouble(JOptionPane.showInputDialog (null,"enter score "+(i+1)));
           if((score[i]) > 10)
           {
               JOptionPane.showMessageDialog(null,
               "You can't score higher than 10",
               "OOPS!",
               JOptionPane.ERROR_MESSAGE);
               i--;
            }
           else
           {
               i++;
            }
        }
    }
    public void calculateScore()
    {
        //max score
        for (int i = 0; i < n; i++)
        {
           if (score[i] > maxScore)
           {
              maxScore=score[i];
            }
        }
        //min score
        for (int i = 0; i < n; i++)
        {
           if (score[i] < minScore)
           {
               minScore = score[i];
            }
        }
         //total score
        for (int i = 0; i<n; i++)
        {
            totalScore = (totalScore + score[i]);
        }
        //final score
        finalScore = (totalScore - (minScore+maxScore));
    }
    public double[] getScores()
    {
        return score;
    }
    public double getResult()
    {
        finalScore = result;
        return result;
    }
}

GUI CLASS

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

/**
 * Class JudgingApplet - write a description of the class here
 * 
 * @author (your name) 
 * @version (a version number)
 */
public class JudgingGUI extends JFrame
{
    int n = 8;
    double result;
    private final int APPLET_WIDTH = 450;
    private final int APPLET_HEIGHT = 200;
    private JLabel contestantName,scoreLabel,resultL;
    private JTextField name, scoreOutput;
    private JTextField scores[] = new JTextField[n];
    private JLabel judges[] = new JLabel[n];
    private double[] score;
    //String resultString = Double.toString(result);
    
    JButton scoreB;
    JButton resetB;
    
    Judging judge;
    
    public JudgingGUI(int n)
    {
        setSize(APPLET_WIDTH,APPLET_HEIGHT);
        setLayout(new FlowLayout());
        
        judge = new Judging(n);
        
        scoreB = new JButton("Compute Score");
        JudgeListener cbHandler;
        cbHandler = new JudgeListener();
        scoreB.addActionListener(cbHandler);
        
        resetB = new JButton("Reset Scores for Next Contestant");
        JudgeListener rbHandler;
        rbHandler = new JudgeListener();
        resetB.addActionListener(rbHandler);
        
        contestantName = new JLabel("Name of contestant");
        scoreLabel = new JLabel("Contestant's score:");
        name = new JTextField(25);
        scoreOutput = new JTextField(8);
        judges = new JLabel[n];
        
        

        add(contestantName);
        add(name);        
        for(int i=0; i<n; ++i)
        {
            judges[i] = new JLabel("Judge "+(i+1));
            add(judges[i]);
            scores[i] = new JTextField(4);
            add(scores[i]);
        } 
        add(scoreB);
        add(scoreLabel);
        //add(resultL);
        add(resetB);
        
    }
    private class JudgeListener implements ActionListener
    {
       public void actionPerformed(ActionEvent e)
       {  
            int n=8;
            double result=0;
            double[] score = new double[n];
            Object source = e.getSource();
            if(e.getSource()==scoreB)
            { 
                for(int i=0; i<n; i++)
                {
                    score[i] = Double.parseDouble(scores[i].getText());
                    if((score[i]) > 10)
                    {
                        JOptionPane.showMessageDialog(null,
                        "You can't score higher than 10. Program is terminating",
                        "OOPS!",
                        JOptionPane.ERROR_MESSAGE);
                        System.exit(1);
                        i--;
                    }
                }
                judge.setScores(score);
                judge.calculateScore();
                judge.getScores();
                result = judge.getResult();
                scoreOutput = setText(Integer.parseInt(result));
                scoreOutput.setText(Integer.parseInt(result));

                System.out.println(scoreOutput);
                
                add(scoreOutput);
            }     
            
            if(e.getSource()==resetB)
            {
                for(int i=0; i<n; i++)
                {
                    score[i]=(0);
                    scores[i].setText("");
                    name.setText("");
                } 
            }
       }
     }
}

TEST

/**
 * Write a description of class TestJudging here.
 * 
 * @author (your name) 
 * @version (a version number or a date)
 */
public class TestJudging
{
    public static void main(String[] args)
    {
        int n=8;
        JudgingGUI example = new JudgingGUI(n);
        example.setVisible(true);
    }
}

[/code]

[Christophe Verré]

"cannot find symbol - method parseInt(double); maybe you meant: parseInt(String) or parseInt(String, int)"

Andrew, I'm quite disappointed with your progression so far. You're still getting the same kind of error again and again. You don't seem to be willing to study how to read a method signature and how to call a method based on that signature. You'll still be struggling in the next assignment, even if you get the answer of this one.

I'd like to advice you to read something abuot methods. For example, Thinking In Java. Read the part called "Methods, arguments, and return values". It's very important.

[andrew cassato]

thanks i read over that and have a better understanding. i thought my problem now is that the data is not getting stored, when i print the result i just get 0.0 even tho the debugger shows the array score to be full of data. once its passed thru calculate scores im not really sure what happens but by the time i get to line 96:

result = judge.getResult();

the debugger shows the value of result at 0.0. the name of the variable in calculateScores that holds the final number i want to display as finalScore, do i use this variable in the GUI class? to me that seems like its the bridge i cant really build - especially since calculateScore() returns void.

[Christophe Verré]

 public double getResult()  
     {  
         finalScore = result;  
         return result;  
     }  

This method is not right. Why are you setting the finalScore here ? The finalScore is set in calculateScore. You should just return the finalScore here.

Actually, by looking at your assignment again, there is no "finalScore" variable at all. Remove it. Remove it completely. Then, in calculateScore, set the "result" variable. This variable should hold the final score. Then, in getResult, you should just return the "result" variable.

[andrew cassato]

VICTORYYYYYYYYYYYYYYY

thanks a ton, that last bit really made a lot click

[Campbell Ritchie]

Well done, and it's good when you can see the penny drop.

But be careful with thread titles, please; somebody coming new to this forum won't know what the first and second attempts were.