Есть проблемка с отладкой программы

[warder]

Написал программу, исправил ошибки, но программка вылетает, помогите решить проблемку плз:

#include <iostream>
#include <ctime>
#include <cstdlib>
#include <clocale>
#include <conio.h>
using namespace std;
  int main()
  {
setlocale(LC_CTYPE, "rus");
srand(time(0));
double x,y;
int Vubor;
for (int Zac=-1;Zac<1;Zac--)
{
	cout<<"1. Введите координаты точек.\n";
	cout<<"2. Автоматическое введение данных. \n";
	cout<<"0. Выход. \n";
	if(Vubor==0){break;}
	if(Vubor==1)
	{
   cout<<"Введите координату Y >";
  cin>>x;
   cout<<"\nВведите координату X >";
  cin>>y;
	}
	else if(Vubor==2)
	{
		{x=rand()%10-5+1;}
		{y=rand()%15-10+1;}
		cout<< " Авто ввод данных закончен.\n";
		cout<< "X="<<x<<"\t\t\tY="<<y<<"/n";
	}
      // проверка условий
   if ( (y>=5-x) && (x*x+y*y<=2) )
   cout<<"Точка не попадает в область \n";
     else 
   cout<<"Точка не попадает в область \n";
   {   getch();
    return 0;
}
}
  }

[Пепел Феникса]

у вас Vibor чему равно?)

[warder]

Сообщение от Пепел Феникса

у вас Vibor чему равно?)

ну я эту переменную использовал для выбора в меню, помогите кто то с этим меню)

[Пепел Феникса]

у вас эта переменная только обьявлена.
у вас ей значение присвоенно какое то?

[warder]

ну я попытался зделать так: если ей присваивается какое то значение то будет то или иное действие

[Гром]

Оно у вас вообще нигде не присваивается. И никакой выбор как таковой нигде не делается в вашем коде.

[coNsept]

Почему не используешь switch() для выбора той или иной операции ? Я думаю это довольно удобней, чем делать все проверками

switch ( ... )
{
case X:
    {
        ...
    }
    break;
...
}

В твоем случае можно сделать так:

switch ( _getch() )
{

case '1':
    {
        break;
    }

case '2':
    {
        cout<<"Введите координату Y >";
        cin>>x;
        cout<<"\nВведите координату X >";
        cin>>y;
    }

case '3':
    {
	/*{*/ x=rand()%10-5+1; /*}*/ // Скобки здесь не в эту, не в красную армию...
	/*{*/ y=rand()%15-10+1; /*}*/
	cout << " Авто ввод данных закончен.\n";
	cout << "X=" << x << "\t\t\tY=" << y << "/n";
    }

default: { return; }
}

[warder]

Сообщение от coNsept

Почему не используешь switch() для выбора той или иной операции ? Я думаю это довольно удобней, чем делать все проверками

switch ( ... )
{
case X:
    {
        ...
    }
    break;
...
}

а как оно будет на моем примере примерно, я просто с этим оператором не знаком

вообще я с шаблон с проверками взял отсюда:

#include <iostream>
#include <ctime>
#include <cstdlib>
#include <clocale>
using namespace std;
void main()
{
 setlocale(LC_CTYPE, "rus");
 srand(time(0));
 //////////////////////////////////////////////////
 int Vubor,
  Kchi[100];
 for(int h=0;h<100;h++){Kchi[h]=-666;}
 for (int Zah=-1;Zah<1;Zah--)
 {
  cout<<"1. Вести элементы самому.\n";
  cout<<"2. Автозаполнение.\n";
  cout<<"3. Печать.\n";
  cout<<"0. Выход.\n";
  cin>>Vubor;
  if(Vubor==0){break;}
  if(Vubor==1)
  {
   cout<<"-66. Отмена.\n";
   for(int h=0;h<100;h++)
   {
    cout<<"Ведите "<<h+1<<" Элемент: ";
    cin>>Kchi[h];
    if(Kchi[h]==-66){Kchi[h]=-666;break;}
    if(Kchi[h]>50 || Kchi[h]<-50){cout<<"\tError: Минимум -50, Максимум 50.\n";h--;}
   }
  }
  else if(Vubor==2)
  {
   for(int h=0;h<100;h++)
   {Kchi[h]=rand()%100-50+1;}
   cout<<"\tDone...\n";
  }
  else if(Vubor==3)
  {
   for(int h=0;h<100;h++)
   {
    if(Kchi[h]==-666){break;}
    cout<<h+1<<" - "<<Kchi[h]<<"\n";
   }
  }
  else cout<<"\tError!!!\n";
 }
}

[Гром]

 cin>>Vubor;

вам ни о чем не подсказывает?

switch (X)
{
case X1: {/**/} break;
case X2: {/**/} break;
//...
default: {/**/}
}

почти то же самое, что и

if (X == X1) {/**/}
else if (X == X2) {/**/}
//...
else {/**/}

Можете про него, например, здесь подробнее почитать: Инструкции (операторы) сравнения if и switch

[warder]

Гром, во спс тебе с cin>>Vubor; завтыкал)