C# コンパイラ警告 CS1717 について解説

using System;
public class TestIfAssignment
{
    public static void Main()
    {
        int a = 5;
        // 本来は if(a == 5) とする意図でしたが、誤って代入演算子を使用しているため、
        // aに自分自身を代入する処理が実行され、警告が生成されます。
        if ((a = a) == 5)
        {
            Console.WriteLine("a は 5 と等しいです。");
        }
        else
        {
            Console.WriteLine("a は 5 と等しくありません。");
        }
    }
}

a は 5 と等しいです。

using System;
public class TestBooleanAssignment
{
    public static void Main()
    {
        bool flag = false;
        // 本来は比較演算子(==)を使って flag を評価すべきところ、誤って代入してしまっています。
        // その結果、flagには true が代入され、if文の条件は常に true になります。
        if (flag = true)
        {
            Console.WriteLine("flag に true が代入され、常に true として判定されます。");
        }
        else
        {
            Console.WriteLine("このブロックは実行されません。");
        }
    }
}

flag に true が代入され、常に true として判定されます。

using System;
public class SampleClass
{
    private int number;
    public SampleClass(int number)
    {
        // 本来は this.number = number; とすべきですが、誤って number = number; と記述すると
        // クラスフィールドには代入されず、警告CS1717が発生します。
        number = number;
        // クラスフィールド number は初期化されず、既定値の 0 となります
        Console.WriteLine("number: " + this.number);
    }
    public static void Main()
    {
        SampleClass instance = new SampleClass(10);
    }
}

number: 0

using System;
public class Person
{
    private string name;
    public Person(string name)
    {
        // この場合、左辺の name はローカル変数(パラメーター)を指してしまい、
        // クラスフィールドへの代入が行われません。
        name = name;
        Console.WriteLine("Person name: " + this.name);
    }
    public static void Main()
    {
        Person p = new Person("Taro");
    }
}

Person name:

using System;
public class TestOperatorConfusion
{
    public static void Main()
    {
        bool condition = false;
        // 本来は if(condition == true) とすべきところ、誤って代入演算子を使用してしまっています。
        if (condition = true)
        {
            Console.WriteLine("条件に誤った自己代入が発生しています。");
        }
    }
}

条件に誤った自己代入が発生しています。

using System;
public class TestSpellingMistake
{
    public static void Main()
    {
        int counter = 10;
        // 本来は if(counter == 10) と記述すべきところ、誤って counter = 10 と記述してしまい、
        // 自己代入が発生しています。結果として counter の値は変更されませんが、警告が出ます。
        if ((counter = 10) == 10)
        {
            Console.WriteLine("自己代入により counter の値は 10 に固定されます。");
        }
    }
}

自己代入により counter の値は 10 に固定されます。

using System;
public class CodeReviewExample
{
    private int value;
    public CodeReviewExample(int value)
    {
        // フィールドへの代入を意図しているが、this キーワードを省略したためパラメーターへの自己代入となっています。
        value = value;
    }
    public static void Main()
    {
        CodeReviewExample obj = new CodeReviewExample(5);
        Console.WriteLine("value: " + obj.value);
    }
}

value: 0

using System;
public class CorrectIfComparison
{
    public static void Main()
    {
        int a = 5;
        // 正しくは == 演算子を使用して比較します。
        if (a == 5)
        {
            Console.WriteLine("a は 5 と等しいです。");
        }
        else
        {
            Console.WriteLine("a は 5 と等しくありません。");
        }
    }
}

a は 5 と等しいです。

using System;
public class CorrectConstructorAssignment
{
    private int number;
    public CorrectConstructorAssignment(int number)
    {
        // this キーワードを使用して、フィールドにパラメーターの値を正しく代入します。
        this.number = number;
        Console.WriteLine("number: " + this.number);
    }
    public static void Main()
    {
        CorrectConstructorAssignment obj = new CorrectConstructorAssignment(15);
    }
}

number: 15

using System;
public class StaticAnalysisExample
{
    public static void Main()
    {
        int b = 0;
        // この自己代入は、設定された静的解析ツールによって事前に警告を検出できる設定となっている場合があります。
        b = b;
        Console.WriteLine("b: " + b);
    }
}

b: 0

using System;
public class MSResourceDemo
{
    public static void Main()
    {
        // 詳細情報については、Microsoft Learn で「CS1717 コンパイラの警告」を検索してください。
        Console.WriteLine("CS1717 についての詳細は、Microsoft Learn の公式ページを参照してください。");
    }
}

CS1717 についての詳細は、Microsoft Learn の公式ページを参照してください。