FizzBuzz using Methods in Java

Question

I have solved this all inside the main method, however the professor has asked we follow very specific guidelines. Here are the instructions:

1. Create methods with the following signatures (don’t forget static):

   • private static boolean isFizz (int val) check if a multiple of 3.
   • private static boolean isBuzz(int val) check if a multiple of 5.

2. Create a class member variable (this means it’s not inside a method, but is inside a class):

   • private static int counter
   • Note: you can initialize this when you declare it, or inside the main method.
3. In the main method:
   1. Use the counter to iterate from 1 to 100.
   2. Use the two other methods you define to determine what to print.
      • Note that the methods should not print anything, they just return a boolean value.
   3. Your program should include at least one of each of the following:
      • branch control statement (like if).
      • loop.

public class Fizzy {

    //checking if a multiple of 3
    private static boolean isFizz(int i){
        if (i % 3 == 0){
            return true;
        }
        return false;
    }
    //checking if a multiple of 5
    private static boolean isFuzz(int i){
        if (i % 5 == 0){
            return true;
        }
        return false;
    }

    //professor wants a class here outside of main with a private static int.
    //But I get an error and I'm not sure what I need to do to fix it.
    //also, is this where my booleans need to be called?

    public class Counter {

       private static int counter(int x);

    }


    public static void main (String [] args){

        //I think I'm supposed to call something here?
        //I've tried Counter a = new Counter(); but it doesn't like it.
        //I've tried new booleans but also doesn't like it.

        /**
         * for loop to iterate i to 100
         */

        //counter is supposed to be iterated here. However I am not sure
        //how to exactly access counter from a separate class.

        for(counter; counter <= 100; ++counter){

             //if Statement to check if a multiple of 3 and 5.
            if (counter % 3 == 0 && counter % 5 == 0){
                System.out.println("FizzBuzz");
            }
            // else if statement to check if multiple of 3
            else if (isFizz == true){
                System.out.println("Fizz");
            }
            //else if statement to check if multiple of 5
            else if (isFuzz == true){
                System.out.println("Buzz");
            }
             //else just run the loop
            else {
                System.out.println(counter);
            }

        }
    }

    }
}

It is supposed to go like this:

1
2
Fizz
4
Buzz
Fizz
.
.
.
14
FizzBuzz
16

And so on.

Answer 1

Mostly as what others said, but with a slightly different flair: the fizzbuzz case is a composition of fizz and buzz, so there is no need to make it different: You can just extract the code to go to a new line and compose them together. See if you can understand this other approach:

for (int counter = 1; counter<=100; counter++) {
  // if it's fizz or buzz, we need to print not the number but a word
  if (isFizz(counter) || isBuzz(counter)) {
    //if it's fizz, we write fizz (note that fizzbuzz are fizz!)
    if (isFizz(counter)) {
      System.out.print("Fizz");
    }
    //if it's buzz, we write buzz (note that fizzbuzz are buzz!)
    if (isBuzz(counter)) {
      System.out.print("Buzz");
    }
  }
  //if it's neither fizz nor buzz, we need to print the number itself
  else {
    System.out.print(counter);
  }
  //once we wrote whatever we had to write, we go to a newline for the next one
  System.out.println();    
}

Answer 2

You have already written isFuzz and isFizz method, so why are you checking with % 3 and 5 again?

Instead, in the for loop, call your isFizz and isFuzz method with counter as argument. Store the return boolean value in variable.

Your code:

for(counter; counter <= 100; ++counter){

         //if Statement to check if a multiple of 3 and 5.
        if ( isFizz(counter) && isBuzz(counter) ){
            System.out.println("FizzBuzz");
        }
        // else if statement to check if multiple of 3
        else if ( isFuzz(counter) ) {
            System.out.println("Fizz");
        }
        else if( isBuzz(counter) )
            System.out.println("Buzz");
        else {
            System.out.println(counter);
        }

}

Edited: I'd left you to do it.

Answer 3

Here is a possible solution:

public class Fizzy {

    //a class *variable*, not a class
    private static int counter;

    //checking if a multiple of 3
    private static boolean isFizz(int i) {
        return i % 3 == 0; //no need for if
    }

    //checking if a multiple of 5
    private static boolean isBuzz(int i) {
        return i % 5 == 0; //no need for if
    }

    public static void main(String[] args) {
        for (counter = 1; counter <= 100; ++counter) {
            if (isFizz(counter) && isBuzz(counter)) { //reuse the methods
                System.out.println("FizzBuzz");
            } else if (isFizz(counter)) { //no need for "== true"
                System.out.println("Fizz");
            } else if (isBuzz(counter)) { //no need for "== true"
                System.out.println("Buzz");
            } else {
                System.out.println(counter);
            }
        }
    }
}

Usually I would just give pointers, but here you are so close to a solution that "giving pointers" is more or less the same as "show how it works".