一个数是3的倍数，输出 Fizz , 是5的倍数， 输出Buzz ， 同时是3和5的倍数，输出FizzBuzz

             一个数是3的倍数，输出 Fizz , 是5的倍数， 输出Buzz ， 同时是3和5的倍数，输出FizzBuzz

 

一、前言

最近在看订阅号的上的一个问题： 一个数是3的倍数，输出 Fizz , 是5的倍数， 输出Buzz ， 同时是3和5的倍数，输出FizzBuzz 。

二、代码实现

1、按照正常思路，顺序往下写 （错误）

@Test
public void calOne(){
    for (int i = 1; i < 100; i++) {
        if(i % 3 == 0){
            System.out.println("num:"+ i + " Fizz");
        }else if(i % 5 == 0){
            System.out.println("num:"+ i + " Buzz");
        }else if(i % 3 == 0 && i % 5 == 0){
            System.out.println("num:"+ i + " FizzBuzz");
        }
    }
}

1.2： 当 i = 15时，应当输出 "FizzBuzz" ，结果是： “Fizz” 。 因为 i=15时，满足条件 i % 3 == 0 ，后面的else if 就不再执行了。故以上代码不满足，程序要求。

2、对代码进行修正，如下：

@Test
public void calTwo(){
    for (int i = 1; i < 100; i++) {
        if(i % 3 == 0 && i % 5 != 0){
            System.out.println("num:"+ i + " Fizz");
        }else if(i % 5 == 0 && i % 3 != 0){
            System.out.println("num:"+ i + " Buzz");
        }else if(i % 3 == 0 && i % 5 == 0){
            System.out.println("num:"+ i + " FizzBuzz");
        }
    }
}

2.2： 当 i = 15时，输出 "FizzBuzz” ，达到预期结果，满足要求。 但是这样代码太冗余，执行效率不高，于是继续思考。

3、对代码进行优化，最终如下：

@Test
public void calThree(){
    for (int i = 1; i < 100; i++) {
        if(i % 3 == 0 && i % 5 == 0){
            System.out.println("num:"+ i + " FizzBuzz");
        }else if(i % 3 == 0){
            System.out.println("num:"+ i + " Fizz");
        }else if(i % 5 == 0){
            System.out.println("num:"+ i + " Buzz");
        }
    }
}

三、总结

一个看似简单的问题，若考虑不周的话，也是容易出错的。特别是对于程序，失之毫厘差以千里，任何问题，都要尽可能的充分思考，尽量做到面面俱到。但bug缺陷也是不可避免的，遇到了，多总结，反思如何避免下次再发生同样的问题。 日积月累，吃的草多了，也会称为大牛的！