چرا اولین برنامم کار نمی کنه ؟

سلام خدمت اساتید اینورای پی سی ورلد !!!
اولین یار هست که میام ایم محله انجمن ولی انشا... از این بعد زیاد سر میزنم ( مجبورم که بزنم !!! )
بریم سر اصل مطلب...
چون کارم فوری و فوتی هست مجبور شدم ت صفحه اصلی تاپیک بزنم شزمنده

برنامه ای که 4 گزینه برای کاربر چاپ کنه و متناسب با هر انتخاب محاسبات رو انجام بده
خودم از switch() استفاده نکردم تا کمی آب بندی شم
اول اینکه این یرنامه باید تو نرم افزار visual studio 10 ران بشه ( استاده دیگه ... چی بهش بگم ؟؟؟ )
دوم اینکه ممنون که کمک میکنید

کد:

---


// calculating area
#include <iostream>
#include <conio.h>
#include <stdio.h>

using namespace std;

int main()
{
    const int PI = 3.14
    int main_select;
    char sub_select;
    float C_area, T_area, R_area, r, b, h, l, w;

  clrscr();

    cout << "Press \" 1 \" to calculate circle area." << endl;
    cout << "Press \" 2 \" to calculate triangle area." << endl;
    cout << "Press \" 3 \" to calculate rectangle area." << endl;
    cout << "press \" 4 \" to exit.\n" << endl;

//circle

    if ( main_select == 1 ) {
        cout << "Please enter Radius: " << endl;
        cin >> r;
        C_area = PI * r * r;
        cout << "Your entered Radius is " << r << endl;
        cout << "Circle's area is " << C_area << endl;

        cout << "Do you want to start a new calculation ? (y/n). "
        cin >> sub_select;

        if ( sub_select == "y" )
            continue;
        else if ( sub_select == "n" )
            break;
        else
            cout << " Please select Yes (y) or No (n). ";

// triangle

    else if ( select == 2 ) {
        cout << "Please enter Base and Height " << endl;
        cin >> b >> h;
        T_area = ( b * h ) / 2;
        cout << "Your entered Base and Height is " << b << h << endl;
        cout << "Triangle's area is " << T_area << endl;

        cout << "Do you want to start a new calculation ? (y/n). "
        cin >> sub_select;

        if ( sub_select == y )
            continue;
        else if ( sub_select == n )
            break;
        else if
            cout << " Please select Yes (y) or No (n). ";

// recangle

    else if ( select == 3 ) {
        cout << "Please enter Length and Width " << endl;
        cin >> l >> w;
        R_area = l * w
        cout << "Your entered Length and Width is " << l << w << endl;
        cout << "Rectangle's area is " << R_area << endl;

        cout << "Do you want to start a new calculation ? (y/n). "
        cin >> sub_select;

        if ( sub_select == "y" )
            continue;
        else if ( sub_select == "n" )
            break;
        else if
            cout << " Please select Yes (y) or No (n). ";

// exit

    else if ( select == 4 )
        cout << " Good Time ! ";

// unknown entry

    else
        cout << " Please enter correct number for your selection. ";

    getch();

    return 0;
}

---

كسي نيست راهنمايي كنه ؟

با سوییچ هم اینجوری نوشتم ولی بازم گیر بیخودی میده !

کد:

---

// calculating area
#include <iostream.h>
#include <conio.h>
#include <stdio.h>

int main()
{
    const PI = 3.14;
    int main_select = 0;
//har sub_select = 'y';
    float C_area, T_area, R_area, r, b, h, l, w;

    clrscr();

    cout << "Press \" 1 \" to calculate circle area." << endl;
    cout << "Press \" 2 \" to calculate triangle area." << endl;
    cout << "Press \" 3 \" to calculate rectangle area." << endl;
    cout << "press \" 4 \" to exit.\n" << endl;

      while (  ( main_select = cin.get() ) != '4' ) {

            switch ( main_select ) {
                  case '1':
                                cout << "Please enter Radius: " << endl;
                        cin >> r;
                        C_area = PI * r * r;
                        cout << "Circle's area is " << C_area << endl;
                        break;

                  case '2':
                                cout << "Please enter Base and Height " << endl;
                            cin >> b >> h;
                        T_area = ( b * h ) / 2;
                        cout << "Triangle's area is " << T_area << endl;
                        break;

                  case '3':
                                cout << "Please enter Length and Width " << endl;
                                cin >> l >> w;
                        R_area = l * w
/* be in khat gir " ; " mide */                        cout << "Rectangle's area is " << R_area << endl;
                        break;

                  case '4':
                                cout << " Good Time ! ";
                        break;

                default:
                                cout << "Incorrect number entered. "
                                    << " Enter new number. " << endl;
        }
    }

        return 0;
}

---

نقل قول:

---

نوشته شده توسط subliz [ برای مشاهده لینک ، با نام کاربری خود وارد شوید یا ثبت نام کنید ]

با سوییچ هم اینجوری نوشتم ولی بازم گیر بیخودی میده !

کد:

---

// calculating area
#include <iostream.h>
#include <conio.h>
#include <stdio.h>

int main()
{
    const PI = 3.14;
    int main_select = 0;
//har sub_select = 'y';
    float C_area, T_area, R_area, r, b, h, l, w;

    clrscr();

    cout << "Press \" 1 \" to calculate circle area." << endl;
    cout << "Press \" 2 \" to calculate triangle area." << endl;
    cout << "Press \" 3 \" to calculate rectangle area." << endl;
    cout << "press \" 4 \" to exit.\n" << endl;

      while (  ( main_select = cin.get() ) != '4' ) {

            switch ( main_select ) {
                  case '1':
                                cout << "Please enter Radius: " << endl;
                        cin >> r;
                        C_area = PI * r * r;
                        cout << "Circle's area is " << C_area << endl;
                        break;

                  case '2':
                                cout << "Please enter Base and Height " << endl;
                            cin >> b >> h;
                        T_area = ( b * h ) / 2;
                        cout << "Triangle's area is " << T_area << endl;
                        break;

                  case '3':
                                cout << "Please enter Length and Width " << endl;
                                cin >> l >> w;
                        R_area = l * w
/* be in khat gir " ; " mide */                        cout << "Rectangle's area is " << R_area << endl;
                        break;

                  case '4':
                                cout << " Good Time ! ";
                        break;

                default:
                                cout << "Incorrect number entered. "
                                    << " Enter new number. " << endl;
        }
    }

        return 0;
}

---

---

برنامت رو من در c ران کردم فقط از

[PHP]
case '3':
cout << "Please enter Length and Width " << endl;
cin >> l >> w;
R_area = l * w[/PHP]

خط اخر که سمیکالن ; یادت رفته ارور میگرفت

چنتا توصیه دوستانه بهت میکنم چون اول راهی و اگر این مسائل کوچک رو رعایت کنی برنامه نویس بانظم تری خواهی بود

اول اینکه اگر استاد یا هر کسی سوالی بهت داد سریع نیا توی اینترنت و برو پیش دوستت و ...... بگو برام حلش کن سعی کن تا هرکجاش که میتونی خودت حل کنی من حاضرم قسم بخورم که اگر خودت تا هرکجا که تونستی حل کنی و ببری به استادت نشون بدی بهت نمره کامل رو بده و بقیه مشکلت رو حل کنه ( البته طرفت باید استاد باشه نه یه بی ش...... بماند )

سعی کن برنامه ات رو توری بنویسی که هر کس بخواهد اون رو بخونه راحت باشه وگرنه برنامه نویست فقط و فقط بدرد خودت میخوره اصلا اسمش رو نمیشه گذاشت برنامه نویس مثلا یکی از این موارد استفاده از متغییر ها با اسم و معانی قابل درک برای همه نه اینکه همه جای برنامه بنویسی i ، j ، k بعد اخرش هم خودت نفهمی چیکار کردی

سعی کن هر برنامه ای که حل میکنی بیکار ننشینی بگی خوب من زودتر از همه حل کردم دیگه اخر برنامه نویس هستم سعی کن برنامه ات رو توسعه بدهی مثلا همین برنامه بالا سعی کنم اگر فرد عدد منفی یا حرف وارد کرد برنامه از دور خارج نشه

فعلا باید برم جایی بعدا اگر دوست داشتی یکی دو مرود دیگه هم هست بهت میگم امیدوارم که بدردت بخوره

نقل قول:

---

نوشته شده توسط szh_1367 [ برای مشاهده لینک ، با نام کاربری خود وارد شوید یا ثبت نام کنید ]

چنتا توصیه دوستانه بهت میکنم چون اول راهی و اگر این مسائل کوچک رو رعایت کنی برنامه نویس بانظم تری خواهی بود

اول اینکه اگر استاد یا هر کسی سوالی بهت داد سریع نیا توی اینترنت و برو پیش دوستت و ...... بگو برام حلش کن سعی کن تا هرکجاش که میتونی خودت حل کنی من حاضرم قسم بخورم که اگر خودت تا هرکجا که تونستی حل کنی و ببری به استادت نشون بدی بهت نمره کامل رو بده و بقیه مشکلت رو حل کنه ( البته طرفت باید استاد باشه نه یه بی ش...... بماند )

سعی کن برنامه ات رو توری بنویسی که هر کس بخواهد اون رو بخونه راحت باشه وگرنه برنامه نویست فقط و فقط بدرد خودت میخوره اصلا اسمش رو نمیشه گذاشت برنامه نویس مثلا یکی از این موارد استفاده از متغییر ها با اسم و معانی قابل درک برای همه نه اینکه همه جای برنامه بنویسی i ، j ، k بعد اخرش هم خودت نفهمی چیکار کردی

سعی کن هر برنامه ای که حل میکنی بیکار ننشینی بگی خوب من زودتر از همه حل کردم دیگه اخر برنامه نویس هستم سعی کن برنامه ات رو توسعه بدهی مثلا همین برنامه بالا سعی کنم اگر فرد عدد منفی یا حرف وارد کرد برنامه از دور خارج نشه

فعلا باید برم جایی بعدا اگر دوست داشتی یکی دو مرود دیگه هم هست بهت میگم امیدوارم که بدردت بخوره

---

ممنون از راهنمايي هاتون
با وجود اينكه همش تكراري بود ولي با اين حال تو اين برهه هر چي تكرار كنن اين نكات رو بازم يادم ميره
تو اين مدتي كه اين تاپيك رو زدم هم تغييراتي دادم توش ولي اولي بازم ران نشد
من اگه تو فكر پيچوندن و راه در رو اين حرفا بودم كه از دو طريق حلش نميكردم !

اميدوارم بتونم از تجربيات شما دوستان عزيز استفاده كنم !

فقط يه سوال
برنامه كه ران نشد ولي نظرتون راجع به الگوريتم و طرز فكرم چيه ؟

سلام
چند تا اشکال کویک داشت.
if ها , else ها بعضی جاها درست تعریف نشده بود.
همچنین وقتی از break و Continue استفاده میشه توی این جور برنامه ها باید از while استفاده کنین.
این برنامه توی ویژوال استدیو کامپایل شد.

[HTML] #include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <stdio.h>

using namespace std;

int _tmain(int argc, _TCHAR* argv[])
{
const int PI = 3.14;
int main_select,done=1;
char sub_select;
float C_area, T_area, R_area, r, b, h, l, w;

cout << "Press \" 1 \" to calculate circle area." << endl;
cout << "Press \" 2 \" to calculate triangle area." << endl;
cout << "Press \" 3 \" to calculate rectangle area." << endl;
cout << "press \" 4 \" to exit.\n" << endl;
cin >> main_select;
//circle
while(done)
{

if ( main_select == 1 )
{
cout << "Please enter Radius: " << endl;
cin >> r;
C_area = PI * r * r;
cout << "Your entered Radius is " << r << endl;
cout << "Circle's area is " << C_area << endl;

cout << "Do you want to start a new calculation ? (y/n). ";
cin >> sub_select;

if ( sub_select == 'y' )
continue;
else if ( sub_select == 'n' )
{
done=0;
break;
}
else
cout << " Please select Yes (y) or No (n). ";
}
// triangle

else if ( main_select == 2 )

{
cout << "Please enter Base and Height " << endl;
cin >> b >> h;
T_area = ( b * h ) / 2;
cout << "Your entered Base and Height is " << b << h << endl;
cout << "Triangle's area is " << T_area << endl;

cout << "Do you want to start a new calculation ? (y/n). ";
cin >> sub_select;

if ( sub_select == 'y' )
continue;
else if ( sub_select == 'n' )
{
done=0;
break;
}
else
cout << " Please select Yes (y) or No (n). ";
}
// recangle

else if ( main_select== 3 )
{
cout << "Please enter Length and Width " << endl;
cin >> l >> w;
R_area = l * w;
cout << "Your entered Length and Width is " << l << w << endl;
cout << "Rectangle's area is " << R_area << endl;

cout << "Do you want to start a new calculation ? (y/n). ";
cin >> sub_select;

if ( sub_select == 'y' )
continue;
else if ( sub_select == 'n' )
{
done=0;
break;
}
else
cout << " Please select Yes (y) or No (n). ";
}
// exit

else if ( main_select == 4 )
{
cout << " Good Time ! ";
// unknown entry
}
else
cout << " Please enter correct number for your selection. ";

}// end of while

return 0;
} [/HTML]

نگفتي دقيقا چه خطايي ميده كه اجرا نميشه. اين شايد اجرا شد:

[PHP]// calculating area
#include <iostream>
#include <conio.h>
#include <stdio.h>

using namespace std;

int main()
{
const int PI = 3.14;
int main_select = 0;
//har sub_select = 'y';
float C_area, T_area, R_area, r, b, h, l, w;

system ("cls");

cout << "Press \" 1 \" to calculate circle area." << endl;
cout << "Press \" 2 \" to calculate triangle area." << endl;
cout << "Press \" 3 \" to calculate rectangle area." << endl;
cout << "press \" 4 \" to exit.\n" << endl;

while ( ( main_select = cin.get() ) != '4' ) {

switch ( main_select ) {
case '1':
cout << "Please enter Radius: " << endl;
cin >> r;
C_area = PI * r * r;
cout << "Circle's area is " << C_area << endl;
break;

case '2':
cout << "Please enter Base and Height " << endl;
cin >> b >> h;
T_area = ( b * h ) / 2;
cout << "Triangle's area is " << T_area << endl;
break;

case '3':
cout << "Please enter Length and Width " << endl;
cin >> l >> w;
R_area = l * w;
/* be in khat gir " ; " mide */ cout << "Rectangle's area is " << R_area << endl;
break;

case '4':
cout << " Good Time ! ";
break;

default:
cout << "Incorrect number entered. "
<< " Enter new number. " << endl;
}
}

return 0;
}
[/PHP]

نقل قول:

---

نوشته شده توسط subliz [ برای مشاهده لینک ، با نام کاربری خود وارد شوید یا ثبت نام کنید ]

ممنون از راهنمايي هاتون
با وجود اينكه همش تكراري بود ولي با اين حال تو اين برهه هر چي تكرار كنن اين نكات رو بازم يادم ميره
تو اين مدتي كه اين تاپيك رو زدم هم تغييراتي دادم توش ولي اولي بازم ران نشد
من اگه تو فكر پيچوندن و راه در رو اين حرفا بودم كه از دو طريق حلش نميكردم !

اميدوارم بتونم از تجربيات شما دوستان عزيز استفاده كنم !

فقط يه سوال
برنامه كه ران نشد ولي نظرتون راجع به الگوريتم و طرز فكرم چيه ؟

---

والا من که یک سمیکالن گذاشتم اخر این خط

[PHP]R_area = l * w[/PHP]

درست شد

اگر قبلا برنامه نوسی نکردی و این واقعا اولین برنامه شما هست که خیلی خوبه اما اگر واقعا میخواهی برنامه نویسی با الگوریتم های خوب و ..... داشته باشی باید خیلی تلاش کنی به این اسونی ها نیست

نمیدونم سطح علمی شما چقدره اما در برنامه هایی که قرار فرد یا کاربر یک کاراکتر یا یک عدد تک رقمی رو به عنوان ورودی به سیستم بده بهتره که از دستور cin.get استفاده نکند

چرا ؟!! به این دلیل که مثلا کاربر میاد www رو به عنوان ورودی به برنامه میده بعد برنامه شما میاد w اول رو چک میکنه میبینه که با مقادیر یکی نیست و پیغام خطا چاپ میکنه بعد w دوم رو به عنوان ورودی دریافت میکنه و همان خطا رو دوباره چاپ میکنه و w سوم هم همین روند تکرار میشود

یعنی برنامه شما برای مثال بالا 3 بار پیغام خطا رو با یک بار ورود داده ( اینتر زدن ) چاپ میکنه

با تفاسیر بالا اگر ورودی ww1e باشه چه اتفاقی رخ میدهد ؟

به نظر شما بهتره از چه دستور یا چه روشی استفاده شود که مشکل بالا حل بشود

نقل قول:

---

نوشته شده توسط szh_1367 [ برای مشاهده لینک ، با نام کاربری خود وارد شوید یا ثبت نام کنید ]

نمیدونم سطح علمی شما چقدره اما در برنامه هایی که قرار فرد یا کاربر یک کاراکتر یا یک عدد تک رقمی رو به عنوان ورودی به سیستم بده بهتره که از دستور cin.get استفاده نکند

چرا ؟!! به این دلیل که مثلا کاربر میاد www رو به عنوان ورودی به برنامه میده بعد برنامه شما میاد w اول رو چک میکنه میبینه که با مقادیر یکی نیست و پیغام خطا چاپ میکنه بعد w دوم رو به عنوان ورودی دریافت میکنه و همان خطا رو دوباره چاپ میکنه و w سوم هم همین روند تکرار میشود

یعنی برنامه شما برای مثال بالا 3 بار پیغام خطا رو با یک بار ورود داده ( اینتر زدن ) چاپ میکنه

با تفاسیر بالا اگر ورودی ww1e باشه چه اتفاقی رخ میدهد ؟

به نظر شما بهتره از چه دستور یا چه روشی استفاده شود که مشکل بالا حل بشود

---

استاد من اين برنامه رو قبل از اينكه استاد تدريس نكنه با خوندن صرفا كتاب نوشتم

حالا اين هفته خود استاد تدريس كرد اين فصل رو و من به اشتباهاتم كه اكثرا ناشي از درك غلط فرامين بود پي بردم

اين شد كه از نو دارم مي نويسمش

با اينكه cin.getch استفاده نكردم بازم مشكل سر جاشه

وقتي 2 كاراكتر وارد ميكنم ٢ تا cin رو پر كردم در واقع

چون هنوز اول كاريم و تمرينات صرفا استفاده از فرامينه اين مورد زياد مطرح نيست و نه استاد اشاره اي كرد نه كتاب

اما شديدا رو ذهنم درگير كرده
راه حلش چيه ؟

تو c و scanf طول ميدان بسيار مطرح ميشه اما جايي نديدم اين مسئله رو براي cin مطرح كنن تو اين سطح
راهي هست بگيم فقط يه كاراكتر وارد كنه ؟