C# Sonsuz döngüyü kıramıyorum..

[flowchartx]

private void checkBox1_CheckedChanged(object sender, EventArgs e)
{
   if (checkBox1.Checked == true)
   {
      while (checkBox1.Checked == true)
      {
         string[] dizi = { "1", "2", "3", "4", "5", "6" };

         for (int i = 0; i < 3; i++)
         {
            serialPort1.Write(dizi[2 * i]);
            serialPort1.Write(dizi[2 * i + 1]);
            System.Threading.Thread.Sleep(300);
            serialPort1.Write("0");
         }
         if (checkBox1.Checked == false)
            break;

      }
   }
}

Merhabalar
C# ile yeni çalışmaya başladım. Yukardaki kodda chexboxı enable yapınca sürekli seri porta veri göndermek istiyorum ve disable yapınca döngüden çıkmasını istiyorum. Fakat programı çalıştırdığımda chexboxu enable yaptığım anda arayüz kilitleniyor arayüzde hiç bir işlemm yapamıyorum, while döngüsü içinden çıkamıyorum nasıl bir yol izlemeliyim yardımcı olursanız çok sevinirim

 

[taydin]

Bir mesaja program kodu ekleyeceğin zaman, editör menüsünden "Kod" seçeneğini seçerek ekle. Daha anlaşılır olur.

Seri porta yazmada takılıyor olabilir. Mesela seri portta akış kontrolü (flow control) aktifleştirilmiş ise ve kontrol sinyalleri de portun meşgul olduğunu gösteriyorsa o yazme bekler durur.

 

[Endorfin35+]

Ben bir mantık hatası göremedim. Olması gerekir gibi sanki. Olmadı birde şöyle deneyebilirisin;

private void checkBox1_CheckedChanged(object sender, EventArgs e)
{
   if (checkBox1.Checked == true)
   {
      gonder();
   }
    return;
}

void gonder()
{

         string[] dizi = { "1", "2", "3", "4", "5", "6" };

         for (int i = 0; i < 3; i++)
         {
            serialPort1.Write(dizi[2 * i]);
            serialPort1.Write(dizi[2 * i + 1]);
            System.Threading.Thread.Sleep(300);
            serialPort1.Write("0");
         }
         if (checkBox1.Checked == true)  gonder(); // yineleme
        
    return;
}

Bu arada c# dan anlamam

 

[taydin]

Ben bir mantık hatası göremedim. Olması gerekir gibi sanki. Olmadı birde şöyle deneyebilirisin;

private void checkBox1_CheckedChanged(object sender, EventArgs e)
{
   if (checkBox1.Checked == true)
   {
      gonder();
   }
    return;
}

void gonder()
{

         string[] dizi = { "1", "2", "3", "4", "5", "6" };

         for (int i = 0; i < 3; i++)
         {
            serialPort1.Write(dizi[2 * i]);
            serialPort1.Write(dizi[2 * i + 1]);
            System.Threading.Thread.Sleep(300);
            serialPort1.Write("0");
         }
         if (checkBox1.Checked == true)  gonder(); // yineleme
      
    return;
}

Bu arada c# dan anlamam

gonder fonksiyonu kendi kendini çağırıyor burada. Kısa süre sonra program stack overflow ile sonlanır. Kendi kendini çağırma, derinlik sınırlı ise mantıklı. Burada derinliğin bir sınırı yok, adam checkbox'u bir hafta sonra da işaretleyebilir

 

[Endorfin35+]

gonder fonksiyonu kendi kendini çağırıyor burada. Kısa süre sonra program stack overflow ile sonlanır. Kendi kendini çağırma, derinlik sınırlı ise mantıklı. Burada derinliğin bir sınırı yok, adam checkbox'u bir hafta sonra da işaretleyebilir

En azından event döngüsünde mi bir sorun var onu anlarız.

 

[tekosis]

if (checkBox1.Checked == false) break;

16. ve 17. satırda bulunan bu sorguya gerek yok, zaten while döngüsü bu kontrolü yapıyor ve false ise döngüden çıkıyor. Aynı mantıkla ilk if sorugusu da gereksiz.

 

[tekosis]

      while (checkBox1.Checked == true)
      {
         string[] dizi = { "1", "2", "3", "4", "5", "6" };

Her döngüde dizi stringi yeniden tanımlanıyor, bu nasıl bir sonuç doğurur? Bakmak lazım. Bunun yerine ;

  string[] dizi = { "1", "2", "3", "4", "5", "6" };     

while (checkBox1.Checked == true)
      {       
         for (int i = 0; i < 3; i++)
         {

daha doğru olur.

 

[wagabond12]

private void checkBox1_CheckedChanged(object sender, EventArgs e)
{
   string[] dizi = { "1", "2", "3", "4", "5", "6" };
   if (checkBox1.Checked == true)
   {
      while (checkBox1.Checked == true)
      {
       
         for (int i = 0; i < 3; i++)
         {
            serialPort1.Write(dizi[2 * i]);
            serialPort1.Write(dizi[2 * i + 1]);
            System.Threading.Thread.Sleep(300);
            serialPort1.Write("0");
         }
            checkBox1.Checked = false;
      }
   }
}