使用goto时如何解决此编译错误“使用未分配的局部变量'hak'”？

问题描述

我正在尝试以下代码。它给了我一个警告“使用未分配的局部变量‘hak’”。我想我在这里遗漏了一些东西。

我希望它在局部变量“hak”等于“0”时显示“3 defa yanlış giriş yaptınız.Müşteri hizmetleri temsilcisiyle görüşünüz. İyi günler dileriz.”。但它总是显示“Hatalı Giriş Yaptınız.Lütfen tekrar deneyiniz。2 hakkınız kalmıştır.”

                   Console.Write("Seçmek istediğiniz 'Kullanıcı Adı'nı belirtiniz: ");
        string kullanici_adi = Console.ReadLine();
        FARKLISIFRE:
        Console.Write("Lütfen Şifrenizi giriniz: ");
        string sifre = Console.ReadLine();
        Console.Write("Girmiş olduğunuz şifreyi tekrar giriniz: ");
        string sifre2 = Console.ReadLine();

        int karsilastirma = String.Compare(sifre,sifre2);
        if (karsilastirma==0)
        {
            Console.WriteLine("Tebrikler! Kaydınız başarılı bir şekilde oluşturulmuştur.");
        }
        else
        {
            Console.WriteLine("Girmiş olduğunuz şifreler birbirinden farklıdır. Lütfen tekrar deneyiniz.");
            goto FARKLISIFRE;
        }

        Console.Write("Giriş yapmak için lütfen '1' seçeneğini giriniz: ");
        char komut = Convert.tochar(Console.ReadLine());
        Console.Clear();
        if (komut=='1')
        {
            goto GIRIS;
        }

        else
        {
            Console.Write("Çıkış yaptınız. İyi günler dileriz.");
            goto END;
        }
        
        int hak = 3;

        GIRIS:
        Console.Write("Lütfen Kullanıcı Adınızı Giriniz: ");
        string kullanici_adi_giris = Console.ReadLine();
        Console.Write("Lütfen belirlemiş olduğunuz Şifrenizi giriniz: ");
        string sifre_giris = Console.ReadLine();

        int karsilastirma_k_adi = String.Compare(kullanici_adi,kullanici_adi_giris);
        int karsilastirma_sifre = String.Compare(sifre,sifre_giris);
        
        if (karsilastirma_k_adi == 0 && karsilastirma_sifre == 0)
        {
            Console.Write("Başarıyla Giriş Yaptınız. Hoşgeldiniz.");
        }
        else if (hak == 0)
        {
            Console.Write("3 defa yanlış giriş yaptınız. Müşteri hizmetleri temsilcisiyle görüşünüz. İyi günler dileriz.");
        }
        else
        {
            hak--;
            Console.WriteLine("Hatalı Giriş Yaptınız. Lütfen tekrar deneyiniz. {0} hakkınız kalmıştır.",hak);
            goto GIRIS;
        }
        
        END:
        Console.ReadKey();
    }

解决方法

你得到这个编译错误，因为这一行从来没有被执行过：

int hak = 3;

这是因为 GOTO、LABELS 的位置以及此行上方的测试条件 if...else：您永远不会通过此行。

为了解决这个编译错误以及由此产生的设计问题，您需要重构所有这些以删除所有的 GOTO。

确实，在编译时，在谈论执行之前，if ( komut == '1' ) ... else ...、hak 的创建从未达到过，会转到 GIRIS 或 END。

因此，在您尝试使用 hak 之后，它从未被分配过，并且编译器知道并告诉您。

为了编译，你需要写：

GIRIS:
int hak = 3;

但是使用 GOTO 既无用又不干净，是一个非常糟糕的 smell，并且是错误和奇怪行为的根源。

每个世纪避免多次跳转，请改用 refactoring。

GOTO still considered harmful?

Spaghetti code

使用本地方法的最小解决方案示例

if ( komut == '1' )
{
  int hak = 3;
  GIRIS(ref hak);
}
else
{
  Console.Write("Çıkış yaptınız. İyi günler dileriz.");
}

Console.ReadKey();

void GIRIS(ref int hak)
{
  Console.Write("Lütfen Kullanıcı Adınızı Giriniz: ");
  string kullanici_adi_giris = Console.ReadLine();
  Console.Write("Lütfen belirlemiş olduğunuz Şifrenizi giriniz: ");
  string sifre_giris = Console.ReadLine();

  int karsilastirma_k_adi = String.Compare(kullanici_adi,kullanici_adi_giris);
  int karsilastirma_sifre = String.Compare(sifre,sifre_giris);

  if ( karsilastirma_k_adi == 0 && karsilastirma_sifre == 0 )
  {
    Console.Write("Başarıyla Giriş Yaptınız. Hoşgeldiniz.");
  }
  else if ( hak == 0 )
  {
    Console.Write("3 defa yanlış giriş yaptınız. Müşteri hizmetleri temsilcisiyle görüşünüz. İyi günler dileriz.");
  }
  else
  {
    hak--;
    Console.WriteLine("Hatalı Giriş Yaptınız. Lütfen tekrar deneyiniz. {0} hakkınız kalmıştır.",hak);
    GIRIS(ref hak);
  }
}

,

编译器看到为变量 hak 定义值的代码行在标签 GIRIS 之前。不能保证该行会被执行，因为代码行 goto GIRIS 可以到达 GIRIS。

尝试改变这个：

int hak = 3;
GIRIS:

为此：

GIRIS:
int hak = 3;