phpfizzbuzz

Another FizzBuzz solution


I was in a job interview and was asked to solve FizzBuzz with PHP.

Write a program that prints the numbers from 1 to 100. But for multiples of three print “Fizz” instead of the number and for the multiples of five print “Buzz”. For numbers which are multiples of both three and five print “FizzBuzz”.

I never heard of FizzBuzz before but here is how I solved it because I didn't know modulo or how to use it.:

for ($i = 1; $i <= 100; $i++){
   if($i / 3 == round($i / 3) && $i / 5 == round($i / 5)){
      echo $i . " is FizzBuzz<br />";
   }
   else if($i / 3 == round($i / 3)){
      echo $i . " is Fizz<br />";
   }
   else if($i / 5 == round($i / 5)){
      echo $i . " is Buzz<br />";
   }
   else {
      echo $i."<br />";
   }
}

I googled and didn't find any solution with round and that got me thinking that maybe there is something wrong with it, this is one of the solutions I found that is close to mine:

for ($i = 1; $i <= 100; $i++){
   if($i % 3 == 0 && $i % 5 ==0){
      echo "FizzBuzz<br />";
   }
   else if($i % 3 == 0){
      echo "Fizz<br />";
   }
   else if($i % 5 == 0){
      echo "Buzz<br />";
   }
   else {
      echo $i."<br />";
   }
}

My code is working fine but is there anything wrong with it that I don't see?


Solution

  • Actually they are testing how you will solve such simple task. It should be increadibly optimized, the code shouldbe clean and easy readable.

    Your version of code is not good. The version you've found in the internet is better, but it's not ideal from the point of the optimization.

    Try to think how to get the goal with less actions.

    Some tips:

    My example of code:

    for ($i = 1; $i <= 100; $i++) {
        if ($i % 15 == 0) {
            echo 'FizzBuzz<br>';
        } elseif ($i % 3 == 0) {
            echo 'Fizz<br>';
        } elseif ($i % 5 == 0) {
            echo 'Buzz<br>';
        } else {
            echo $i . '<br>';
        }
    }