more problems calling methods

[andrew cassato]

I am getting an error when I try to pass an array (score) to a method in another class. i am not sure i have them set up right - I am trying to have the contestant's score calculated and displayed when i press the calculate scores button but get the error
getScores() in Judging cannot be applied to (double[])
here is all the code. error is on line 78 of the GUI class
Judging class

import java.util.Scanner;
import javax.swing.*;
import javax.swing.JOptionPane;
import java.awt.*;
import java.awt.event.*;
import java.applet.Applet;

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;
    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()
    {
        int n=8;
        double[] score = new double[n];
        //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));
        //Judges calcScore = new Judges(scores);
    }
    public double[] getScores()
    {
        return score;
    }
    public double getResult()
    {
        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 = 0;
    private final int APPLET_WIDTH = 450;
    private final int APPLET_HEIGHT = 200;
    private JLabel contestantName,scoreLabel,resultL;
    private JTextField name;
    private JTextField scores[] = new JTextField[n];
    private JLabel judges[] = new JLabel[n];
    private double[] score;
    
    JButton scoreB;
    JButton resetB;
    
    Judging judge = new Judging(9);
    
    public JudgingGUI(int n)
    {
        setSize(APPLET_WIDTH,APPLET_HEIGHT);
        setLayout(new FlowLayout());
        
        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);
        judges = new JLabel[n];
        
        resultL = new JLabel("2");

        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(resetB);
    }
    private class JudgeListener implements ActionListener
    {
       public void actionPerformed(ActionEvent e)
       {  
            int n=8;
            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());
                }
                judge.setScores(score);
                judge.getScores(score);
                judge.getResult(score);
                judge.calculateScore(score);
                resultL = finalScore;
                add(resultL);
            }     
            
            if(e.getSource()==resetB)
            {
                for(int i=0; i<n; i++)
                {
                    score[i]=(0);
                    scores[i].setText("");
                } 
            }
       }
     }
}

Test class

public class TestJudging
{
    public static void main(String[] args)
    {
        int n=8;
        JudgingGUI example = new JudgingGUI(n);
        example.setVisible(true);
    }
}

i dream of the day this project is done... thanks in advance.

[Christophe Verré]

judge.getScores(score);  

You are calling a method called getScores, have one parameter of type double[]. There is no such method in the Judging class. Same for getResult. What are you trying to do. Your methods have no parameter, but they return something. So you are expected to call them like :

double[] scores = judge.getScores();
double result = judge.getResult();

But still, I don't understand why you are calling these methods.

(it would help next time to post the full stack trace of your compilation error)

[andrew cassato]

im trying to pass this array of scores thru these other methods to get the final score and display it in the GUI... am i on the right track?

private class JudgeListener implements ActionListener
    {
       public void actionPerformed(ActionEvent e)
       {  
            int n=8;
            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());
                }
                double[] sScores = judge.setScores(score);
                double[] gScore = judge.getScores(score);
                double result = judge.getResult(gScore);
                double finalScore = judge.calculateScore(result);
                resultL = finalScore;
                add(resultL);
            }     
            
            if(e.getSource()==resetB)
            {
                for(int i=0; i<n; i++)
                {
                    score[i]=(0);
                    scores[i].setText("");
                } 
            }
       }
     }

[Christophe Verré]

double[] sScores = judge.setScores(score);  
double[] gScore = judge.getScores(score);

You're still passing an array to the getScores() method, that won't work. Look at the method :

public double[] getScores()

No parameter, a returned value of type double[].

Now, can you tell me in plain English what you are trying to achieve here ? Try to put some comment next to the method calls.

double[] sScores = judge.setScores(score);  
double[] gScore = judge.getScores(score);

Why do I want you to explain the method calls ? Because the setScores method set the scores you have passed it, and getScores get those scores. It doesn't make sense here to call them like this.

[andrew cassato]

I'm actually not even sure what the setScores and getScores methods do, I just have the outline of the program with those methods:
_______________________________________________
| Judging |
|_______________________________________________|
| - int n //number of judges |
| - double [] scores //scores of n judges |
| - double result;//the score from judges |
|______________________________________________ |
| + Judging(int) |
| + setScores(double []) :void |
| + inputScores() :void |
| + calculateScore() : void |
| + getScores(): double [] |
| + getResult(): double |
|_______________________________________________|

using the UML i know these methods are supposed to be used i just dont really know how... To me it seems like I would just have to send the array of scores to calculateScores and have it return the finalScore but then I dont know what the other stuff is for.

[Campbell Ritchie]

That UML tells you the Judging class has three private (= "-") fields, n, scores and result; it also tells you what types they are.
It tells you the class has a public (= "+") constructor taking an int as parameter, a setScores method taking double[] and returning void, an inputScore() method with no parameters and void return type and three other methods which I am sure you can now work out from the diagram.
There ought to be a javadoc document made from that class; it will be in html format and you open it with a web browser. That should tell you what the constructor and 4 methods do.