Mutator methods

[toterpunkt]

i have created this mutator in my class:

    public void set(double newLength, int newVolume)
    {
        bottleLength = newLength;
        bottleVolume = newVolume;
    }

in my tester i am using JOptionPane to get user input and would like to have the user see a pop up that says "Enter new length" "enter new volume." how can i get those inputs transferred through so when the user inputs them it automatically sees it as part of the mutator?

import javax.swing.JOptionPane;

public class BottledWaterCalculatorTester
{

    public static void main(String[] args)
    {
        String country;
        int population;
        int aroundEarth;
        double bottleLength;
        int bottleVolume;
        String input;

        country = JOptionPane.showInputDialog("Please enter your country: ");

        input = JOptionPane.showInputDialog("Enter population amount:");
        population = Integer.parseInt (input);

        input = JOptionPane.showInputDialog("Number of circumfrences:");
        aroundEarth = Integer.parseInt (input);

        input = JOptionPane.showInputDialog("Length of bottle:");
        bottleLength = Double.parseDouble (input);

        input = JOptionPane.showInputDialog("Volume of bottle:");
        bottleVolume = Integer.parseInt(input);
        
        BottledWaterCalculator myWater = new BottledWaterCalculator(country, population, aroundEarth, bottleLength, bottleVolume);

        System.out.println("Country: " + myWater.getCountry()
        + "\nPopulation: " + myWater.getPopulation()
        + "\nNumber of Circumferences: " + myWater.getAroundEarth()
        + "\nLength of bottle: " + myWater.getBottleLength()
        + "\nVolume of bottle: " + myWater.getBottleVolume()
        +"\n\nNumber of bottles used: " + myWater.bottlesUsed()
        +"\nWater consumed: " + myWater.waterConsumed());
        
[B]        input = JOptionPane.showInputDialog("Enter New Length:");
        double newLength = Integer.parseInt (input);

        input = JOptionPane.showInputDialog("Enter New volume:");
        int newVolume = Integer.parseInt (input);

        System.out.println("\n\nModified Data:");
        myWater.set(9,16);[/B]


        System.out.println("\n\n NEW Length of bottle: " + myWater.getBottleLength()
        + "\n NEW Volume of bottle: " + myWater.getBottleVolume()
        + "\n\n NEW Number of bottles used: " + myWater.bottlesUsed()
        +"\n NEW Water consumed: " + myWater.waterConsumed());

    }
}

i would like for whatever the user puts in for "enter new length" to be printed out in place of the myWater.set(9,16).... as it is now i have to manually go into my file and change (9,16) to change it all. :confused:

[gcalvin]

        myWater.set(newLength, newVolume);

-Gary-

[toterpunkt]

        myWater.set(newLength, newVolume);

-Gary-

That throws me an error each time.

Do i have to change something in my class first in order for that to work in my tester?

[gcalvin]

Except:

        double newLength = Integer.parseInt (input);

Don't you mean

[COLOR="Blue"]        double newLength = Double.parseDouble(input);
[/COLOR]

?

-Gary-

[gcalvin]

That throws me an error each time.

Do i have to change something in my class first in order for that to work in my tester?

Have you read the error message? Are you going to make us guess what it is? Are you claiming that you don't get the error when you run the code as you posted it?

-Gary-

[toterpunkt]

Have you read the error message? Are you going to make us guess what it is? Are you claiming that you don't get the error when you run the code as you posted it?

-Gary-

I'm an idiot. I keep forgetting how the slightest typo will ruin the outcome of everything. Thank you for catching that you are awesome!

[gcalvin]

How would i get the modified data to "print out" ?

        System.out.println("\n\nModified Data:" + newLength + newVolume);
        myWater.set(newLength, newVolume);

i know that isn't correct but am i even close?

The code you posted originally calls myWater.set(), and then has a long System.out.println() that calls various getter methods. That seems like the right thing to do. Doesn't it work? What does it do that you don't expect? What about the code you're posting now?

-Gary-

[gcalvin]

I should amend that. When I said "the right thing" I should have said "a reasonable thing". Really, there's no reason to put all of that into one println() statement. Break it up into multiple println() statements.

-Gary-

[toterpunkt]

The code you posted originally calls myWater.set(), and then has a long System.out.println() that calls various getter methods. That seems like the right thing to do. Doesn't it work? What does it do that you don't expect? What about the code you're posting now?

-Gary-

I figured it out after having just posted my question by doing the following: (before it was adding newLength and newVolume because of how i wrote it in)

        System.out.println("Modified Data:");
        myWater.set(newLength, newVolume);
        System.out.println("length: " + newLength + "volume: " + newVolume);

But i really didn't need to do that since i had already made println statements for "NEW length of bottle" that would have done the same thing:

        System.out.println("\n\nModified Data:");
        myWater.set(newLength, newVolume);

        System.out.println("NEW Length of bottle: " + myWater.getBottleLength()
        + "NEW Volume of bottle: " + myWater.getBottleVolume()
        + "NEW Number of bottles used: " + myWater.bottlesUsed()
        +"NEW Water consumed: " + myWater.waterConsumed());