مشکل کوچک یک تازه کار

سلام دوستان
من تازه چند روزه که دارم C پلاس پلاس یاد می گیرم.
یک برنامه ساده نوشتم برای اینکه نمره قبولی یا ردی 10 دانش آموز را بگیرد به طوری که 1 به معنی قبول شده و 2 به معنی رد شده در امتحان است.تا اینجا مشکلی پیش نیامد.
بعد خواستم کاری کنم که اگه کاربر عددی غیر از یک یا دو را بزند اخطار بدهد و از اون سوال کند که مایل به ادامه هست یا نه . اگر y را زد دوباره ادامه کار را سر بگیرد و اگر n را زد از برنامه خارج شود.
ولی وقتی run می کنمش برنامه توی یک حلقه بی پایان می افتد و بعد از 10 بار وارد کردن 1 یا 2 دوباره شروع به دریافت نمره می کند.
لطفا بگید مشکل از کجاست.
سورس برنامه قبل از اضافه کردن ویژگی تشخیص عدد غیر از 1 و 2 که درست کار میکند:

کد:

---

#include <iostream.h>
#include <conio.h>

void main()
        {
          /////////
      int n,i,pass=0,fail=0;

              for(i=10;i>=1;i--)
              {
                cout <<"Enter Result("<<i<<" Remaining) : ";
                      cin >>n;
                if (n==1 || n==2)
                  {
                        if (n==1)
                            pass = pass + 1;
                    if (n==2)
                            fail = fail + 1;
                }
                              else
                {
                        cout <<"Wrong Number\n";
              i=0;
                  }
                }
                cout <<pass<<" Passed.\n";
          cout <<fail<<" failed.\n";
      getch();
        }

---

سورس بعد از اضافه کردن این ویژگی که درست کار نمی کند:

کد:

---

#include <iostream.h>
#include <conio.h>

void main()
        {
          /////////
      int n,i,pass=0,fail=0;
      char ch='n';
      // do-while loop start
      do
      {
              for(i=10;i>=1;i--)
              {
                cout <<"Enter Result("<<i<<" Remaining) : ";
                      cin >>n;
                if (n==1 || n==2)
                  {
                        if (n==1)
                            pass = pass + 1;
                    if (n==2)
                            fail = fail + 1;
                }
                              else
                {
                        cout <<"Wrong Number\n";
                        cout <<"Do you want to continue? Press 'y' if YES / 'n' if NO : ";
                                cin >>ch;
                  }
                }

      }
      while(ch='y');
      // do-while loop end
                cout <<pass<<" Passed.\n";
          cout <<fail<<" failed.\n";
         
      getch();
        }

---

نقل قول:

---

while(ch='y');

---

به نظرم ( یعنی قطعا! ) این شرط رو اشتباه به جای == یک علامت مساوی گذاشتی.

سلام
ممنون
ولی درست نشد.
من اون دستور انتساب رو به == تغییر دادم ولی مشکل کماکان هست.
به نظرتون ایراد از کجاست؟
کد اصلاح شده:

کد:

---

#include <iostream.h>
#include <conio.h>

void main()
        {
          /////////
      int n,i,pass=0,fail=0;
      char ch='n';
      // do-while loop start
      do
      {
              for(i=10;i>=1;i--)
              {
                cout <<"Enter Result("<<i<<" Remaining) : ";
                      cin >>n;
                if (n==1 || n==2)
                  {
                        if (n==1)
                            pass = pass + 1;
                    if (n==2)
                            fail = fail + 1;
                }
                              else
                {
                        cout <<"Wrong Number\n";
                        cout <<"Do you want to continue? Press 'y' if YES / 'n' if NO : ";
                                cin >>ch;
                  }
                }

      }
      while(ch == 'y');
      // do-while loop end
                cout <<pass<<" Passed.\n";
          cout <<fail<<" failed.\n";

      getch();
        }

---

یک عکس از برنامه بعد از کامپایل و اصلاح :
[ برای مشاهده لینک ، با نام کاربری خود وارد شوید یا ثبت نام کنید ]

شرط رو به while ch == 'n' تغییر بده.

سلام

برنامه شما چند ايراد كوچك دارد.اولا"محل پرانتزها اشتباه است .ثانيا"محل سوال تكرار نيز صحيح نيست.ثالثا"يادتان باشد هرگاه سوال تكرار داشتيد بايد تمام متغيرها رااينشيالايز كنيد يعني به مقدار اوليه برگردانيد.اينها مهمترين ايراد برنامه شما بود.در ضمن اشكالات كوچك ديگري نيز دارد، مثلا" با وارد كردن عددي اشتباه (بزرگتر از 2)پس از اعلام خطا موقعيت تصحيح وجود نداردو...من با اجازه شما تغييراتي در برنامه اتان دادم وباز نويسي كردم.بررسي آن خالي از فايده نيست

کد:

---

#include <iostream.h>
#include <conio.h>

void main()
    {int n,i,pass,fail;
    char ch='n';
    do
      {pass=0,fail=0;
      for(i=10;i>=1;i--)
      {cout <<"Enter Result("<<i<<" Remaining) : ";
          cin >>n;
          switch(n)
        {case 1:pass = pass + 1;
          break;
          case 2:fail = fail + 1;
          break;
          default:{cout <<"Wrong Number\n"; 
                  i++;
                  }
        } 
      }
      cout <<pass<<" Passed.\n";
      cout <<fail<<" failed.\n";
      cout <<"Do you want to continue(y/n)?";
      cin >>ch;
      }while(ch == 'y');
    }

---

موفق باشيد

سلام
دوست عزیز
ممنون از برنامه تون ولی این برنامه یک کم با مال من فرق دارد.
در واقع برنامه شما تا وقتی ch=='y' است از کاربر عدد می گیرد.
ولی من می خواهم که عمل گرفتن عدد از کاربر که شامل 10 مرحله است فقط یک بار انجام شود و اگر در بین آن کاربر عددی غیر از 1 و 2 وارد کرد پیغام wrong pass بدهد و از او بپرسد آیا مایل به ادامه است یا نه .اگر فرد n را وارد کرد عدد گرفتن را متوقف کرده و متغیر pass و fail را چاپ کند هر چند مجموع آنها 10 نباشد.
و اگر فرد y را وارد کرد یه عدد گرفتن ادامه دهد(تا وقتی i صفر شود)

من فکر می کنم مشکل برنامه من این هست که تا وقتی کاربر عددی غیر از 1 و 2 وارد نکرده برنامه درست کار می کند.وقتی عددی غیر از 1 و 2 وارد کند کامپایلر وارد قسمت else می شود که پیغام ها را چاپ کرده و متغیر ch را می گیرد.
ولی مشکل اینجاست که اگر ch=='n' بشود برنامه در این حالت باید از حلقه for خارج شود ولی این کار انجام نمی شود.در نتیجه برنامه به گرفتن عدد ادامه می دهد!
×××من همین الان تونستم درستش کنم ×××
ولی مشکل این است که اولا یک کم پیچیده شد.
دوما فکر کنم استفاده از دستور break در حلقه برخلاف اصول برنامه نویسی ساخت یافته باشد.
اینم کدش :

کد:

---

#include <iostream.h>
#include <conio.h>

void main()
        {
          /////////
      int n,i,pass=0,fail=0;
      char ch='n';
      // do-while loop start
      do
      {
              for(i=10;i>=1;i--)
              {
                cout <<"Enter Result("<<i<<" Remaining) : ";
                      cin >>n;
                if (n==1 || n==2)
                  {
                        if (n==1)
                            pass = pass + 1;
                    if (n==2)
                            fail = fail + 1;
                }
                              else
                {
                        cout <<"Wrong Number\n";
                          cout <<"Do you want to correct it? Press 'y' if YES / 'n' if NO : ";
                                cin >>ch;
              if (ch=='n')
                      {
                  break;
                  }
              if(ch=='y')
                      {
                  i+=1;
                  }
                  }
                }
        if(i==0)
        {
                break;
        }

      }
      while(ch == 'y');
      // do-while loop end
                cout <<pass<<" Passed.\n";
          cout <<fail<<" failed.\n";

      getch();
        }

---

حالا خواهشم این است که این را یک جوری تغییر بدهید که از break استفاده نکرده باشد.
مرسی