c# - FizzBuzzを書く

Question

コーディングの恐怖を読んで、私はちょうど別の時にFizzBuzzに出くわしました。

元の投稿はここにあります：コーディングホラー：なぜプログラマーは..プログラムできないのですか？

知らない人のために：FizzBuzzは非常に人気のある子供向けゲームです。1から100まで数えて、数値が3で割り切れるたびに文字列「Fizz」が呼び出され、数値が5で割り切れるたびに文字列「Buzz」が呼び出され、数値が3と5で割り切れるたびに両方数字の代わりに「FizzBuzz」という文字列が一緒に呼び出されます。

今回はコードを書いて1分かかりましたが、気に入らないことがいくつかあります。

これが私のコードです：

public void DoFizzBuzz()
{
    var combinations = new Tuple<int, string>[] 
    { 
        new Tuple<int, string> (3, "Fizz"), 
        new Tuple<int, string> (5, "Buzz"), 
    };

    for (int i = 1; i <= 100; ++i)
    {
        bool found = false;

        foreach (var comb in combinations)
        {
            if (i % comb.Item1 == 0)
            {
                found = true;
                Console.Write(comb.Item2);
            }
        }

        if (!found)
        {
            Console.Write(i);
        }

        Console.Write(Environment.NewLine);
    }
}

だから私の質問は：

見つかったブール値を取り除くにはどうすればよいですか？
foreachよりも優れたテスト方法はありますか？

score 195 · Accepted Answer

あなたの実装は不必要に複雑だと思います。これは仕事をし、理解しやすいです：

public void DoFizzBuzz()
{
    for (int i = 1; i <= 100; i++)
    {
        bool fizz = i % 3 == 0;
        bool buzz = i % 5 == 0;
        if (fizz && buzz)
            Console.WriteLine ("FizzBuzz");
        else if (fizz)
            Console.WriteLine ("Fizz");
        else if (buzz)
            Console.WriteLine ("Buzz");
        else
            Console.WriteLine (i);
    }
}

score 27 · Accepted Answer

条件付き書式指定子を利用して、洗練されたバージョンを取得します。

public void DoFizzBuzz()
{
    for(int i=1;i<101;i++)Console.WriteLine("{0:#;}{1:;;Fizz}{2:;;Buzz}",i%3*i%5==0?0:i,i%3,i%5);
}

score 26 · Accepted Answer

最大限の効率を得るために広げます。このプログラムは、他のすべてを打ち負かすことができます。

public void FizzBuzz()
{
    const string FIZZ = "Fizz";
    const string BUZZ = "Buzz";
    const string FIZZBUZZ = "FizzBuzz";

    int i = 0;
    while (i < 150)
    {
        Console.WriteLine(++i);
        Console.WriteLine(++i);
        Console.WriteLine(FIZZ); ++i;
        Console.WriteLine(++i);
        Console.WriteLine(BUZZ); ++i;
        Console.WriteLine(FIZZ); ++i;
        Console.WriteLine(++i);
        Console.WriteLine(++i);
        Console.WriteLine(FIZZ); ++i;
        Console.WriteLine(BUZZ); ++i;
        Console.WriteLine(++i);
        Console.WriteLine(FIZZ); ++i;
        Console.WriteLine(++i);
        Console.WriteLine(++i);
        Console.WriteLine(FIZZBUZZ); ++i;
    }
}

score 22 · Accepted Answer

あなたが達成しようとしているのは、FizzBuzz の一般的なソリューションであり、任意の数の単語の組み合わせで機能すると思います。

あなたは良いスタートを切っています - 私はこの例であなたの質問に答えることができると思います:

public void DoFizzBuzz()
{
    var combinations = new List<Tuple<int, string>>
    { 
        new Tuple<int, string> (3, "Fizz"), 
        new Tuple<int, string> (5, "Buzz"), 
    };

    Func<int, int, bool> isMatch = (i, comb) => i % comb == 0;
    for (int i = 1; i <= 100; i++)
    {
        Console.Write(i);

        var matchingCombs = combinations.Where(c => isMatch(i, c.Item1)).ToList();
        if (matchingCombs.Any())
        {
            Console.Write(string.Join("", matchingCombs.Select(c => c.Item2)));
        }
        else
        {
            Console.Write(i);
        }
        Console.Write(Environment.NewLine);
    }
}

実際には、メソッドに渡しますがcombinations、簡潔にするために内部に含めました。

score 20 · Accepted Answer

3番目の編集：

これがあなたのバージョンから「ブール値を取り除く」ための1つの方法です（つまり、元の質問のforループをこれに置き換えます）：

for (int i = 1; i <= 100; i++)
{
  var x = combinations.Where(n => i % n.Item1 == 0);

  if (x.Count() == 0)
    Console.Write(i);
  else
    Console.Write(string.Join("",x.Select(e => e.Item2)));

  Console.Write(Environment.NewLine);
}

以前の回答：

純粋なC＃ソリューションについては、KeithThompsonのソリューションを確認してください。

using System;
class FizzBuzz {
    static void Main() {
        for (int n = 1; n <= 100; n ++) {
            if (n % 15 == 0) {
                Console.WriteLine("FizzBuzz");
            }
            else if (n % 3 == 0) {
                Console.WriteLine("Fizz");
            }
            else if (n % 5 == 0) {
                Console.WriteLine("Buzz");
            }
            else {
                Console.WriteLine(n);
            }
        }
    }
}

linqを使用してFixBuzzに少し取り組みました。これらは私が思いついた解決策です-Linqを使用してこの問題の解決策を表現するための最良の方法を表していると思います。（GitHub）

using System;
using System.Linq;

class FizzBuzz {
  static void Main() {
    var list = Enumerable.Range(1,100)
                .Select(n => {
                      if (n % 15 == 0) {
                        return "FizzBuzz";
                      }
                      if (n % 3 == 0) {
                        return "Fizz";
                      }
                      if (n % 5 == 0) {
                        return "Buzz";
                      }
                      return n.ToString();
                    });

    foreach(string item in list)
      Console.WriteLine(item);
  }
}

そしてクレイジーな1行バージョン：

using System;
using System.Linq;

class FizzBuzz {
    static void Main() {
      Console.WriteLine(
      String.Join(
        Environment.NewLine,
        Enumerable.Range(1, 100)
          .Select(n => n % 15 == 0 ? "FizzBuzz" 
                     : n % 3 == 0 ? "Fizz" 
                     : n % 5 == 0 ? "Buzz" 
                     : n.ToString())
      ));
    }
}

score 15 · Accepted Answer

public void DoFizzBuzz()
{
    for (int i = 1; i <= 100; i++)
    {
        if (i % 3 == 0)
            Console.Write("Fizz");
        if (i % 5 == 0)
            Console.Write("Buzz");
        if (!(i % 3 == 0 || i % 5 == 0))
            Console.Write(i);

        Console.Write(Environment.NewLine);
    }
}

これにより bool が取り除かfoundれますが、重複した評価を行う必要があります。i % 15 == 0FizzBuzz 資格に使用する他のいくつかの回答とは少し異なります。これが良いかどうかは議論の余地があります。しかし、それは別の方法です。

score 13 · Accepted Answer

誰かがすでにこれをしましたか？

Enumerable.Range(1, 100).Select(x =>
                (x % 15 == 0) ? "FIZZBUZZ"
                : (x % 5 == 0) ? "BUZZ"
                : (x % 3 == 0) ? "FIZZ"
                : x.ToString()
                )
                .ToList()
                .ForEach(console.WriteLine);

score 4 · Accepted Answer

最も効率的ではありませんが、C#-6文字列補間を使用したものを次に示します。

void Main()
{
    for (int i = 1; i <= 100; i++)
    {
       Console.WriteLine($"{(i % 15 == 0 ? "FizzBuzz" : 
                             i % 3 == 0 ? "Fizz" : 
                             i % 5 == 0 ? "Buzz" : i.ToString())}");
    }
}

score 4 · Accepted Answer

Enumerable.Range(1, 100).ToList().ForEach(i=>Console.WriteLine($"{(i%3*i%5==0?0:i):#;}{i%3:;;Fizz}{i%5:;;Buzz}"));

この答えにはすべてがあります：

リンク
条件付き書式
文字列補間
すべて 1 行で

勝利！

score 3 · Accepted Answer

答えを見ずにこの問題を解こうとしました。成功するのに3時間かかりました。(ちなみに、私は単なる趣味のプログラマーですので、私を激しく非難しないでください:)) これは私のC#バージョンのソリューションです:

        static void Main(string[] args)
    {

        for (int i = 1; i <= 100; i++)
        {
            if(  ((i % 3) != 0) && ((i % 5) != 0))
            {
                WriteLine($"{i}");
            }
            else
            {
                if ((i % 15) == 0)
                {
                    WriteLine("FizzBuzz");
                }
                else if ((i % 3) == 0)
                {
                    WriteLine("Fizz");
                }
                else if ((i % 5) == 0)
                {
                    WriteLine("Buzz");
                }
            }                 
        }
    }

score 3 · Accepted Answer

FizzBuzz の質問は、優れた面接の質問です。面接プロセスで使用し始めました。このような単純な問題を解決できない人がいかに多いかは驚くべきことです。

元のブログ投稿は、他のソリューションを投稿する人が殺到したため、最終的にロックされたことを覚えておいてください. ハハハ。

いずれにせよ、これは私の C++ です。^_^

#include <iostream>
using namespace std;

int main(int argc, char** argv)
{
    for (int i = 1; i <= 100; ++i)
    {
        bool isMultipleOfThree = (i % 3) == 0;
        bool isMultipleOfFive = (i % 5) == 0;

        if (isMultipleOfThree) cout << "Fizz";
        if (isMultipleOfFive) cout << "Buzz";
        if (!isMultipleOfThree && !isMultipleOfFive) cout << i;

        cout << '\n';
    }

    return 0;
}

score 3 · Accepted Answer

私は初心者です、ここに私の試みがあります：

public void DoFizzBuzz()
   {
       for (int i = 1; i < 101; i++)
       {

           if ((i % 3 == 0) && (i % 5 == 0))
           {
               Console.WriteLine("{0} FizzBuzz", i);
           }
           else if (i % 3 == 0)
           {
               Console.WriteLine("{0} Fizz", i);
           }
           else if (i % 5 == 0)
           {
               Console.WriteLine("{0} Buzz", i);
           }
           else
           {
               Console.WriteLine(i);
           }

       }
       Console.ReadLine();
   }

私のアプローチに何か問題がありますか？私のアプローチは他の人のアプローチよりもはるかに単純に見えるので、間違っているに違いありません。

score 3 · Accepted Answer

わかりました、一体、これが私が好きになった解決策です:)

public void DoFizzBuzz()
{
    for (int i = 1; i <= 100; ++i)
    {
        bool isDivisibleByThree = i % 3 == 0;
        bool isDivisibleByFive = i % 5 == 0;

        if (isDivisibleByThree || isDivisibleByFive)
        {
            if (isDivisibleByThree)
                cout << "Fizz";

            if (isDivisibleByFive)
                cout << "Buzz";
        }
        else
        {
            cout << i;
        }
        cout << endl;
    }
}

明らかに、これは最速の解決策ではありませんが、読みやすさを強調し、「FizzBuzz」のケースを特別なケースではなく、コードパスを通じて自然に発生するものにするので、私は気に入っています。

結局のところ、この質問が出てくるたびに私が最も気に入っているのは、人々がどれだけ多くの異なる解決策を思い付くことができるかを見ることができるということです.

score 3 · Accepted Answer

おそらく構成可能にしたいでしょうが、問題は何を構成可能にするかです-それはわかりません。たぶん、すべてのサイクルを構成可能にする必要があります (FizzBuzz にはサイクルがあります)。これは、構成可能なサイクルを備えた非常に小さくて楽しいバージョンです。

string[] fizzBuzzCycle = 
    "FizzBuzz,{0},{0},Fizz,{0},Buzz,Fizz,{0},{0},Fizz,Buzz,{0},Fizz,{0},{0}"
    .Split(',');

for (int i = 1; i <= 100; i++)
    Console.WriteLine(fizzBuzzCycle[i%fizzBuzzCycle.Length], i);

したがって、ストリングまたはサイクル全体を変更する必要がある場合は、簡単に変更できます。しかし、何を構成可能にするかがわかりません。条件が変更される可能性があります。「素数の場合は Pizz を出力します」。この変更では、@ThomasLevesque によるソリューションの方が変更しやすいため、より優れています。

score 2 · Accepted Answer

このコンパクトなコードを、以前のシンプルで優れたバージョンに追加することをお勧めします。

for (int i = 1; i <= 100; i++) // i++ but not ++i as in your example, be careful here
{
    bool fizz = i % 3 == 0;
    bool buzz = i % 5 == 0;
    string output = fizz && buzz ? "FizzBuzz" :
                            fizz ? "Fizz" :
                            buzz ? "Buzz" :
                            i.ToString();

    Console.WriteLn(output);
}

score 2 · Accepted Answer

機能的なアプローチ...

Console.WriteLine(Enumerable
    .Range(1,100)
    .Aggregate("", 
        (a,i) => a + "\n" + (i%15==0 ? "fizzbuzz" : 
                                (i%5==0 ? "buzz" :
                                    (i%3==0 ? "fizz" : i.ToString())))));

score 2 · Accepted Answer

Rob H と Jacob Krall の意見により、現時点で私が持っているものを以下に示します。おそらく私は将来それをいじるでしょう...それを提供したかっただけです。

public void DoFizzBuzz()
{
    // expect this to come in as parameter
    var combinations = new Tuple<int, string>[] 
    { 
        new Tuple<int, string> (3, "Fizz"), 
        new Tuple<int, string> (5, "Buzz"), 
    };

    Func<int, int, bool> isMatch = (i, comb) => i % comb == 0;

    // expect the borders 1, 100 to come in as parameters
    for (int i = 1; i <= 100; ++i)
    {
        var matchingCombs = combinations.Where(c => isMatch(i, c.Item1)).DefaultIfEmpty(new Tuple<int, string>(i, i.ToString())).Aggregate((v, w) => new Tuple<int, string>(v.Item1, v.Item2 + w.Item2)).Item2;
        Console.WriteLine(matchingCombs);
    }
}

score 2 · Accepted Answer

i++ ではコピーを作成する必要があるため、for ループでは i++ の代わりに ++i を使用することをお勧めします;)

public void DoFizzBuzz()
{
    for (int i = 1; i < 101; ++i)
    {
        if (i % 15 == 0)
            Console.WriteLine ("FizzBuzz");
        else if (i % 3 == 0)
            Console.WriteLine ("Fizz");
        else if (i % 5 == 0)
            Console.WriteLine ("Buzz");
        else
            Console.WriteLine (i);
    }
}

score 2 · Accepted Answer

for ループを使用した比較的単純なソリューション。

Linq などはありません - 基本的な省略形の if ステートメントのみ

for(int x=1;x<101;x++)
    Console.WriteLine(x%3==0?"Fizz"+(x%5==0?"Buzz":""):x%5==0?"Buzz":x+"");

csells (文字列補間なし) によく似ていて、1 行に収まる Linq ソリューションは次のようになります。

Enumerable.Range(1,100).ToList().ForEach(x=>Console.WriteLine(x%3==0?"Fizz"+(x%5==0?"Buzz":""):x%5==0?"Buzz":x+""));

score 2 · Accepted Answer

明らかに、これは FizzBuzz チャレンジの精神から少し外れています。しかし、私のベンチマークでは、これはシングルスレッドでありながら 100 で終了しているときに作成できる最速のものでしたStringBuilder。これは、標準的なアプローチよりも約 3 倍高速です。

const string FIZZ = " Fizz\n";
const string BUZZ = " Buzz\n";
const string FIZZBUZZ = " FizzBuzz\n";
    ...
var sb = new StringBuilder();
int i = 0;

while(true)
{       
    sb.Append(i+3);
    sb.Append(FIZZ);        
    sb.Append(i+5);
    sb.Append(BUZZ);        
    sb.Append(i+6);
    sb.Append(FIZZ);        
    sb.Append(i+9);
    sb.Append(FIZZ);        
    sb.Append(i+10);
    sb.Append(BUZZ);        
    if(i+12 > 100)
        break;
    sb.Append(i+12);
    sb.Append(FIZZ);    
    i+=15;
    sb.Append(i);
    sb.Append(FIZZBUZZ);
}

Console.Write(sb.ToString());

score 1 · Accepted Answer

これを使用して、必要な量だけ取ることができます

static void Main(string[] args)
{
    GetFizzBuzz().Take(100).ToList().ForEach(Console.WriteLine);
}

private static IEnumerable<string> GetFizzBuzz()
{
    for (var i = 0; i < int.MaxValue; i++)
    {
        if (i % 3 == 0 && i % 5 == 0) yield return "FizzBuzz";
        if (i % 3 == 0) yield return "Fizz";
        yield return i % 5 == 0 ? "Buzz" : i.ToString(CultureInfo.InvariantCulture);
    }
}

または単にこれを使用します：

Enumerable.Range(1, 100).Select(s => {
    if (s % 3 == 0 && s % 5 == 0) return "FizzBuzz";
    if (s % 3 == 0) return "Fizz";
    return s%5 == 0 ? "Buzz" : s.ToString(CultureInfo.InvariantCulture);
}).ToList().ForEach(Console.WriteLine);

score 1 · Accepted Answer

元の質問は次のとおりです。 1.見つかった bool を取り除く方法は? 2. foreach よりも良いテスト方法はありますか?

これにより bool と foreach が取り除かれ、まだ読みやすいと思います。

public static void DoFizzBuzz()
{
    var combinations = new Tuple<int, string>[]  
    {  
        new Tuple<int, string> (3, "Fizz"),  
        new Tuple<int, string> (5, "Buzz"),  
    };

    for (int i = 1; i <= 100; i++)
    {
        var fb = combinations.Where(t => {
            if (i % t.Item1 == 0)
            {
                Console.Write(t.Item2);
                return true;
            }
            return false;
        }).ToList();

        if (!fb.Any())
        {
            Console.Write(i);
        }

        Console.Write(Environment.NewLine);
    }
}

シンプルな子供向けゲームでこんなに興奮するなんて誰が思いつくでしょうか? :)

score 1 · Accepted Answer

if 条件がなく、三項演算子が 1 つだけです。

string[] s = new string[6]{"Fizz", "Buzz", "", "", "", ""};
for (int i = 1; i <= 100; i++)
{
    string output = s[(i%3)*2] + s[(i%5)+1];
    Console.WriteLine(string.IsNullOrEmpty(output)? "" + i : output);
}

score 0 · Accepted Answer

JavaScript での簡単なソリューション

var i = 1;
while (i <= 100){
    console.log((i % 3 === 0 && i % 5 === 0) ? "FizzBuzz" : (i % 3 === 0) ? "Fizz" : (i % 5 === 0 ? "Buzz" : i));
    i++;
}

score 0 · Accepted Answer

わかりました - かみます。最近の最初のコーディングインタビューで思いついた LINQ の重いソリューション。この単純な問題に対して、驚くほど多くの異なる解決策があります。

public static class FizzBuzzUtils
{
    private static List<KeyValuePair<int, string>> map = new List<KeyValuePair<int, string>> {
        new KeyValuePair<int, string>(3, "Fizz"),
        new KeyValuePair<int, string>(5, "Buzz")
    };

    public static string GetValue(int i)
    {
        var matches = map.Where(kvp => i % kvp.Key == 0)
            .Select(kvp => kvp.Value)
            .ToArray();
        return matches.Length > 0 ? string.Join(string.Empty, matches) : i.ToString();
    }

    public static IEnumerable<string> Range(int start, int count)
    {
        return Enumerable.Range(start, count)
            .Select(i => GetValue(i));
    }
}

score 0 · Accepted Answer

一般的な FizzBuzz を使用するため (私はこれをRaindropsと呼んでいます)、一般的で読みやすくするためにいくつかの簡単な変更を行います。

「combinations」の名前を「primeFactors」に変更すると、もう少し説明的になります
タプルフィールドへのアクセスをより読みやすくするためのタプルフィールド名
必要な表示文字列の素因数のリストを照会するLinqforeach (を削除)
- Whereリストを現在の整数の因数でフィルタリングします
- Select一致する要素を表示文字列に変換します
- DefaultIfEmpty「一致が見つからない」ケースを処理して、そのブール値を削除できるようにします
- Aggregate一致するすべての表示文字列を連結します

オンラインでお試しください！

public void DoFizzBuzz()
{
    var primeFactors = new (int Factor, string Display)[]
    {
        (3, "Fizz"),
        (5, "Buzz")
    };
    for (var integer = 1; integer <= 100; ++integer)
        Console.WriteLine(primeFactors
            .Where(tuple => integer % tuple.Factor == 0)
            .Select(tuple => tuple.Display)
            .DefaultIfEmpty($"{integer}")
            .Aggregate((first, second) => $"{first}{second}")
        );
}

score 0 · Accepted Answer

パイソンで……

 for i in range(0, 100) :
        a = i % 3 == 0
        b = i % 5 == 0

        if(a & b):
            print("FizzBuzz")
        elif(a):
            print("Fizz")
        elif(b):
            print("Buzz")
        else:
            print(i)

score 0 · Accepted Answer

If を使用せずに、C# コードを使用します。

 //False = 0, True = 1.
    private void DivisibilityByFiveThreeTest(int num)
    {
        string[,] values = new string [2,2]{
                             {"None","Fizz"},
                             {"Buzz","FizzBuzz"}
                             };
        for(int i=1;i< num;i++)
        Console.WriteLine(values[Convert.ToInt32(i % 5 == 0), Convert.ToInt32(i%3==0)]);

    }

score 0 · Accepted Answer

C++ 実装

vector<string> FizzBuzz::execute(int n) {

    if(n == 0)
        return vector<string>();

    auto push = n % 15 == 0 ? "fizzbuzz" :
                n % 3 == 0 ? "fizz" :
                n % 5 == 0 ? "buzz" :
                to_string(n);

    auto execution = execute(n-1);
    execution.push_back(push);
    return execution;
}

c# - FizzBu​​zzを書く

43 に答える 43

Related

Reference

c# - FizzBuzzを書く