Метод пузырьковой сортировки,всегда один и тот же вывод

[spectrum988]

Здравствуйте,реализую программу по поиску элемента x методом"пузырьковой сортировки",но почему то мне говорит что элемента нет,а на самом деле он есть.Что я делаю не та ?

{
int i,n,a[100],poz,x;
bool el;
x=StrToInt(Edit2->Text);
for (i=0; i<n; i++)
{
if (a[i]==x)
{
el=true; poz=i; break;
}
}
if (el==true)
{
ShowMessage("Элемент x присутствует в массиве");
Edit3->Text=IntToStr(poz);
}
else
ShowMessage("Элемент x отсутствует в массиве");
}

[BDA]

Я правильно понимаю, что массив "a" объявлен локально в, скорее всего, обработчике нажатия кнопки?

[phomm]

Да тут и el не инициализируется и n, которая кстати может к некорректному циклу привести.
Локальные переменные надо инициализировать перед чтением из них, запомните на всю жизнь, ну или пока компилятор не станет Вас заставлять это делать.

[spectrum988]

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

[BDA]

То есть у Вас есть функция сортировки со своим локальным массивом и функция поиска элемента со своим?

[spectrum988]

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

[BDA]

Кхм, то есть Вы надеетесь, что два локальных массива чудесным образом станут одним и тем же? Выложите-ка лучше код полностью, как он сейчас есть.

[spectrum988]

void __fastcall TForm1::Button1Click(TObject *Sender)
{
n=StrToInt(Edit1->Text);
if(n<100)
{
ShowMessage("Íå ñîîòâåòñòâóåò óñëîâèþ,ìèíèìàëüíûé ìàññèâ 100");
n=100;
Edit1->Text = "100";
}
StringGrid1->ColCount=n;
for(int i=0; i<n;i++)
 StringGrid1->Cells[i][0]=IntToStr(random(20)-10);
}
//---------------------------------------------------------------------------

void __fastcall TForm1::FormCreate(TObject *Sender)
{
 randomize();
 Edit1->Text=IntToStr(n);
 StringGrid1->ColCount=n;
 Label5->Hide();
Edit3->Hide();
 for(int i=0; i<n;i++)
  StringGrid1->Cells[i][0] = IntToStr(random(100));
}
//---------------------------------------------------------------------------

void __fastcall TForm1::Button2Click(TObject *Sender)
{
int i,j,r;
n=StrToInt(Edit1->Text);
StringGrid2->ColCount=n;
for(i=0; i<n;i++)
 a[i]=StrToInt(StringGrid1->Cells[i][0]);
for (i = 0; i < n-1; i++)
 for (j = i+1; j < n; j++)
  if (a[i] < a[j])
   {
   r = a[i];
   a[i] = a[j];
   a[j] = r;
   }
for (int i=0; i<n; i++)StringGrid2->Cells[i][0]=IntToStr(a[i]);
}
//---------------------------------------------------------------------------

void __fastcall TForm1::Button3Click(TObject *Sender)
{
int i,n,poz,x;
bool el;
x=StrToInt(Edit2->Text);
for (i=0; i<n; i++)
{
if (a[i]==x)
{
el=true; poz=i; break;
}
}
if (el==true)
{
ShowMessage("Ýëåìåíò x ïðèñóòñòâóåò â ìàññèâå");
Label5->Show();
Edit3->Show();
Edit3->Text=IntToStr(poz);
}
else
ShowMessage("Ýëåìåíò x îòñóòñòâóåò â ìàññèâå");
}
//---------------------------------------------------------------------------

void __fastcall TForm1::Button4Click(TObject *Sender)
{
Close();
}

[BDA]

void __fastcall TForm1::Button1Click(TObject *Sender) {
	int n = StrToInt(Edit1->Text);
	if (n < 5) {
		ShowMessage("message");
		n = 5;
		Edit1->Text = "5";
	}
	StringGrid1->ColCount = n;
	for (int i = 0; i < n; i++)
		StringGrid1->Cells[i][0] = IntToStr(random(20) - 10);
}
// ---------------------------------------------------------------------------

void __fastcall TForm1::FormCreate(TObject *Sender) {
	randomize();
	Edit1->Text = "0";
	StringGrid1->RowCount = 2;
	StringGrid1->ColCount = 0;
	Label5->Hide();
	Edit3->Hide();
}
// ---------------------------------------------------------------------------

void __fastcall TForm1::Button2Click(TObject *Sender) {
	int i, j, r;
	int *a;
	int n = StrToInt(Edit1->Text);
	a = new int[n];
	for (i = 0; i < n; i++)
		a[i] = StrToInt(StringGrid1->Cells[i][0]);
	for (i = 0; i < n - 1; i++)
		for (j = i + 1; j < n; j++)
			if (a[i] < a[j]) {
				r = a[i];
				a[i] = a[j];
				a[j] = r;
			}
	for (int i = 0; i < n; i++)
		StringGrid1->Cells[i][1] = IntToStr(a[i]);
	delete[]a;
}
// ---------------------------------------------------------------------------

void __fastcall TForm1::Button3Click(TObject *Sender) {
	int i, n;
	String x;
	n = StrToInt(Edit1->Text);
	x = Edit2->Text;
	for (i = 0; i < n && StringGrid1->Cells[i][1] != x; i++);
	if (i < n) {
		ShowMessage("message1");
		Label5->Show();
		Edit3->Show();
		Edit3->Text = IntToStr(i);
	}
	else
		ShowMessage("message2");
}

[spectrum988]

Спасибо,большое,я уже разобрался со своим кодом,с тем который был,вы были правы,тут моя ошибка(в том что я думал что можно один и тот же массив использовать для двух кнопок)
Скажите,а можно ли чтобы массив,который у меня заполняется рандомом,чтобы его элементы,не повторялись,или это не реально ?

А могу ли я использовать еще такой метод?Правилен ли он?У меня выбивает ошибку возле a[m].

int m,i_key=0,j_key=n-1;;
x=StrToInt(Edit2->Text);

while(i_key< j_key)
{
m = (i_key+ j_key)/2;
if à[m] < x  i_key= m+1
else         j_key = m;
}
if (a[i_key]==x)
{Label5->Show();
Edit3->Show();
Edit3->Text=IntToStr(i_key);
}
else
ShowMessage("нету");
}