C# hata aliyorum

AE_ren_Tr

Selamlar, .net console da kendimi gelistirmek icin bir proje yapiyorum ama anlamlandiramadigim bir hata aliyorum kodu okuyup yardimci olursaniz sevinirim.Birde kodda method veya daha ust duzey seyler kullanmak istiyorum.

kodun bir kismi diger kisimlari henuz gelistirmedim

int masaS = 7;
string[] masalar = new string[masaS];
for (int i = 0; i < masalar.Length; i++)
{

masalar[i] = "bos";

}
x:
Console.Clear();
int masaY = 0;
bool masaAcma=false;
Console.Write("MasaAc(1)\nMasaIslem(2)\nMasaHesap(3)\nKasaIslemleri(4)\nCikisYap(0)\nSeciminiz: ");
int secim=int.Parse(Console.ReadLine());
if (secim==1)
{
Console.WriteLine("--------------------");
Console.WriteLine("2 MASA AC");
Console.WriteLine("--------------------");
for (int i = 0; i < masalar.Length; i++)
{

    masalar[i] = "bos";
    Console.WriteLine($"{i+1}.masa {masalar[i]} dur");
}                
y:
Console.WriteLine("ana menuye(1) gitmek veya cikis(2) veya masa acma(3) icin numaralari yaziniz ");
int devam=int.Parse(Console.ReadLine());
if (devam == 1)
{
    goto x;
}
else if (devam == 2)
{
    Console.Clear();
    Console.WriteLine("cikis yapildi");
}
else if (devam == 3)
{
a:
    Console.WriteLine("masa acmak icin lutfen id giriniz(ornek 8 (sadece sayi olabilir)):");
    masaY = int.Parse(Console.ReadLine());
    if (masaY-1 == masalar.Length)
    {
        
        masaS = masaY;
        masalar = new string[masaS];
        for (int i = 0; i < masalar.Length; i++)
        {
            masalar[i] = "bos";
        }
        Console.WriteLine($"{masaS}.masa basariyle acildi");
        masaAcma=true;
    }
    else
    {
        Console.WriteLine("lutfen son masa kacinci masaysa ondan bir sonraki rakami giriniz");                        
        goto a;
    }
}
else
{
    Console.WriteLine("yanlis bir sey girdiniz lutfen talimatlara uyunuz");
    Console.ReadLine();
    goto y;
}
if (masaAcma == true)
{
    o:
    Console.WriteLine("lutfen actiginiz masa dolu mu bos mu onu yaziniz");
    string bosDolu = Console.ReadLine();
    if (bosDolu == "bos" )
    {
        masalar[masaS] ="bos" ;
    }
    else if (bosDolu == "dolu")
    {
        masalar[masaS] = "dolu";
    }
    else
    {
        Console.WriteLine("lutfen sadece bos veya dolu yaziniz");
        goto o;
    }
    Console.Clear();
    for (int i = 0; i < masalar.Length; i++)
    {
        Console.WriteLine($"{i + 1}.masa {masalar[i]} ");
    }
}

}

Console.ReadLine();

İşlenmeyen Özel Durum: System.IndexOutOfRangeException: Dizin, dizi sınırlarının dışındaydı.
konum: restUyg.Program.Main(String[] args) C:\Users\DELL\Desktop.NET\restUyg\restUyg\Program.cs içinde: satır 90
Press any key to continue . . .
aldigim hatada bu

By_Surmeneli

AE_ren_Tr

Merhaba! Kodu okuduktan sonra, hatanın muhtemelen dizinin sınırlarının dışına çıkılmasından kaynaklandığını görülüyor. Kodun daha temiz ve okunabilir olması için metotlar kullanarak yeniden düzenlendi.

Hatanın olduğu kısmı kontrol ederken, dizinin son elemanına erişmeye çalıştığınızda, dizinin sınırlarının dışına çıkılmıştır. Bu tür hataların önüne geçmek için kontroller ekleyeceğiz.

Aşağıda, hatayı düzelttikten sonra ve daha iyi bir yapıya kavuşturduktan sonra kodu bulabilirsiniz:

using System;

namespace restUyg
{
    class Program
    {
        static void Main(string[] args)
        {
            int masaS = 7;
            string[] masalar = new string[masaS];

            for (int i = 0; i < masalar.Length; i++)
            {
                masalar[i] = "bos";
            }

            AnaMenu(masalar);
        }

        static void AnaMenu(string[] masalar)
        {
            while (true)
            {
                Console.Clear();
                Console.Write("MasaAc(1)\nMasaIslem(2)\nMasaHesap(3)\nKasaIslemleri(4)\nCikisYap(0)\nSeciminiz: ");
                int secim = int.Parse(Console.ReadLine());

                switch (secim)
                {
                    case 1:
                        MasaAc(masalar);
                        break;
                    case 0:
                        Console.Clear();
                        Console.WriteLine("Çıkış yapıldı.");
                        return;
                    default:
                        Console.WriteLine("Geçersiz seçim, tekrar deneyin.");
                        break;
                }
            }
        }

        static void MasaAc(string[] masalar)
        {
            Console.Clear();
            Console.WriteLine("--------------------");
            Console.WriteLine("2 MASA AC");
            Console.WriteLine("--------------------");

            for (int i = 0; i < masalar.Length; i++)
            {
                Console.WriteLine($"{i + 1}.masa {masalar[i]} dur");
            }

            while (true)
            {
                Console.WriteLine("Ana menuye(1) gitmek veya çıkış(2) veya masa açma(3) için numaraları yazınız ");
                int devam = int.Parse(Console.ReadLine());

                if (devam == 1)
                {
                    return; // Ana menüye dön
                }
                else if (devam == 2)
                {
                    Console.Clear();
                    Console.WriteLine("Çıkış yapıldı.");
                    Environment.Exit(0);
                }
                else if (devam == 3)
                {
                    MasaEkle(ref masalar);
                }
                else
                {
                    Console.WriteLine("Yanlış bir şey girdiniz, lütfen talimatlara uyunuz.");
                }
            }
        }

        static void MasaEkle(ref string[] masalar)
        {
            while (true)
            {
                Console.WriteLine("Masa açmak için lütfen id giriniz (örnek 8 (sadece sayı olabilir)):");
                int masaY = int.Parse(Console.ReadLine());

                if (masaY - 1 >= masalar.Length)
                {
                    Array.Resize(ref masalar, masaY);
                    masalar[masaY - 1] = "bos";
                    Console.WriteLine($"{masaY}.masa başarıyla açıldı");
                    MasaDurumGuncelle(masalar, masaY - 1);
                    return;
                }
                else
                {
                    Console.WriteLine("Lütfen son masa kaçıncı masaysa ondan bir sonraki rakamı giriniz");
                }
            }
        }

        static void MasaDurumGuncelle(string[] masalar, int masaIndex)
        {
            while (true)
            {
                Console.WriteLine("Lütfen açtığınız masa dolu mu boş mu onu yazınız:");
                string bosDolu = Console.ReadLine();

                if (bosDolu == "bos" || bosDolu == "dolu")
                {
                    masalar[masaIndex] = bosDolu;
                    Console.Clear();
                    for (int i = 0; i < masalar.Length; i++)
                    {
                        Console.WriteLine($"{i + 1}.masa {masalar[i]} ");
                    }
                    return;
                }
                else
                {
                    Console.WriteLine("Lütfen sadece 'bos' veya 'dolu' yazınız.");
                }
            }
        }
    }
}

Yapılan Değişiklikler:

Kod metotlar halinde düzenlendi.
Dizi sınırlarının dışına çıkmayı engellemek için gerekli kontroller eklendi.
Array.Resize kullanılarak dizi boyutu gerektiğinde artırıldı.
goto ifadeleri kaldırıldı ve yerlerine döngüler ve şart blokları konuldu.

Bu yapıyla kodun daha okunabilir ve yönetilebilir olmasını sağlandı.

AE_ren_Tr

tessekurler ancak goto ifadelerinin ne gibi sikintilari var

By_Surmeneli

AE_ren_Tr

Gönderdiğiniz koddaki "goto" ifadelerinin özel olarak neden sıkıntılı olduğunu açıklayayım.

Gönderdiğiniz kodda, "goto" ifadeleri şu şekillerde kullanılmıştı:

Ana Menüye Dönüş:

   goto x;

Masa Açma İşlemi Tekrarı:

   goto a;

Masa Durumu Belirleme Tekrarı:

   goto o;

Bu "goto" ifadeleri, aşağıdaki sorunları yaratabilir:

Kodun Takip Edilmesi Zorlaşır:
- "Goto" ifadeleri kodun belirli yerlerine sıçramalar yaptığından, kodun akışını takip etmek zorlaşır. Hangi koşulda hangi kodun çalışacağını anlamak, "goto" ifadeleri ile daha karmaşık hale gelir.
Kodun Okunabilirliği Azalır:
- "Goto" ifadeleri, kodun doğal akışını bozar ve okuyucunun hangi durumda ne olacağını anlamasını zorlaştırır. Bu da kodun anlaşılmasını zorlaştırır.
Hataların Bulunması ve Düzeltilmesi Zorlaşır:
- Hataları bulmak ve düzeltmek, "goto" ifadeleri ile daha zor olabilir çünkü kodun her yerine sıçrayabilme potansiyeli vardır. Bu da hata ayıklamayı ve kodun bakımını zorlaştırır.
Yapısal Programlama Prensiplerine Aykırıdır:
- Yapısal programlama, kodun daha modüler, okunabilir ve bakımı kolay olmasını savunur. "Goto" ifadeleri, bu yapısallığı bozar ve programın rastgele yerlerinde sıçramalara neden olur.

Gönderdiğiniz kodda bu sorunları çözmek için "goto" ifadelerini kaldırarak, döngüler ve fonksiyonlar kullanarak daha yapılandırılmış ve okunabilir bir kod haline getirdik.

Düzenlenmiş Kodun Avantajları:

Daha Temiz ve Okunabilir Kod:
- Fonksiyonlar ve döngüler kullanarak kodun akışını daha temiz ve okunabilir hale getirdik.
Hataların Daha Kolay Bulunması:
- Yapısal programlama teknikleri kullanarak hataların daha kolay bulunmasını sağladık. Her işlev kendi fonksiyonunda yer alıyor ve bu da hata ayıklamayı kolaylaştırıyor.
Yeniden Kullanılabilirlik:
- Fonksiyonlar kullanarak kodun belirli kısımlarını tekrar tekrar kullanabilir hale getirdik. Bu da kodun daha modüler olmasını sağladı.

Örnek:

İşte "goto" ifadeleri yerine döngü ve fonksiyonlar kullanılarak yapılan bir örnek:

Eski Kod:

y:
Console.WriteLine("ana menuye(1) gitmek veya cikis(2) veya masa acma(3) icin numaralari yaziniz ");
int devam=int.Parse(Console.ReadLine());
if (devam == 1)
{
    goto x;
}
else if (devam == 2)
{
    Console.Clear();
    Console.WriteLine("cikis yapildi");
}
else if (devam == 3)
{
    goto a;
}
else
{
    Console.WriteLine("yanlis bir sey girdiniz lutfen talimatlara uyunuz");
    Console.ReadLine();
    goto y;
}

Yeni Kod:

while (true)
{
    Console.WriteLine("Ana menuye(1) gitmek veya çıkış(2) veya masa açma(3) için numaraları yazınız ");
    int devam = int.Parse(Console.ReadLine());

    if (devam == 1)
    {
        return; // Ana menüye dön
    }
    else if (devam == 2)
    {
        Console.Clear();
        Console.WriteLine("Çıkış yapıldı.");
        Environment.Exit(0);
    }
    else if (devam == 3)
    {
        MasaEkle(ref masalar);
    }
    else
    {
        Console.WriteLine("Yanlış bir şey girdiniz, lütfen talimatlara uyunuz.");
    }
}

Bu şekilde, "goto" ifadelerinden kaçınarak daha yapılandırılmış ve okunabilir bir kod elde etmiş olduk.