Void type not allowed

[Lisa Beglaw]

public void runTest()
{
    // Testing how many movies, should be 0 since non have been added yet.
System.out.println("The MovieCollection has " + movieCollection.numberOfMovies() + " movies.");
}

Not sure why I am getting a void type not allowed error here. All I want to do is print off a line pulling into from my other class.

[Kristin Stromberg]

Hi again. What is the return type of your numberOfMovies() method? My guess is it's void, when it should be a String. Hope this helps...

[Lisa Beglaw]

But I don't want anything to return, I just want it to print out.

[Lisa Beglaw]

Just looked back at my code for my class MovieCollection and numberOfMovies is a void type.

[Kristin Stromberg]

The problem is you're calling the numberOfMovies() method in your print statement, and unless the numberOfMovies() method returns a string, you'll get the error you're seeing. It's okay if your runTest() method returns void.

Alternatively, maybe you don't have a numberOfMovies() method at all but a numberOfMovies instance variable instead? In that case, you just need to remove the parentheses like this:

System.out.println("The MovieCollection has " + movieCollection.numberOfMovies + " movies.");  

[Lisa Beglaw]

I double checked my methods and it is definately a method with a void data type.

Is there anyway I can call a method that has void return?

Here is the method I am trying to call...

    public void numberOfMovies()
    {
        movieCollection.size();
        
        int count = 0;
        int rentedResults = 0;
        Iterator it = movieCollection.iterator();
        while (it.hasNext()) {
            Movie currentMovie = (Movie) it.next();
            if (currentMovie.getCurrentlyRented() == true) {
                rentedResults = rentedResults + 1;
                count++;
            }
        }
        System.out.println("Total number of movies = " + movieCollection.size());
        System.out.println("Total number of movies rented out = " + rentedResults);
        System.out.println();
    }

[Kristin Stromberg]

In this case, you just call the method but don't try to embed the call in a print statememnt. The method handles the printing for you. So instead of this

System.out.println("The MovieCollection has " + movieCollection.numberOfMovies + " movies.");

you would do this:

movieCollection.numberOfMovies();

[Arul Prasad]

hi kristin

The problem is you're calling the numberOfMovies() method in your print statement, and unless the numberOfMovies() method returns a string, you'll get the error you're seeing. It's okay if your runTest() method returns void.

for ur information numerOfMovies() can return any data type not only String
it can return Float,Integer...etc, Object or any primitive data type
println() method will work since this method inturn will call the Object.toString()

[Saeed Amer]

Lisa, As suggested by Kristin - following line of code will print required message:

movieCollection.numberOfMovies();

[/qb]<hr></blockquote>

You can only call a method within the "println" method if it is returning anything. The value being returned is then concatenated to the rest of the string (if any) in the "println" method.

For example, if your numberOfMovies() method was returning the count, your original line of code would have copiled with no errors.

Saeed
[ August 10, 2005: Message edited by: Saeed Amer ]

[Ulf Dittmer]

Something like the following should do:

System.out.print("The MovieCollection has ");
movieCollection.numberOfMovies();
System.out.println(" movies.");

[Sathya Srinivasan]

As per your code, the best approach would be

 
public void runTest()
{
    // Testing how many movies, should be 0 since non have been added yet.
movieCollection.numberOfMovies();
System.out.println("The MovieCollection has " + movieCollection.size() + " movies.");
}

Since your code really is updating the number of movies, you might want to change your method name to updateMovieCollection() to cause less confusion.

[Lisa Beglaw]

Everyone, Thank you!

As suggested, I will use Kristin's advice. Thanks Kristin!