Зависание программы при нажатии кнопки

@more · Регистрация: 30.09.2011

Студворк — интернет-сервис помощи студентам

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
//---------------------------------------------------------------------------
 
#include <vcl.h>
#pragma hdrstop
 
#include "Unit1.h"
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma link "CSPIN"
#pragma resource "*.dfm"
TForm1 *Form1;
int i,j,x,y,a,b,n;
float *mas = new float[n],*mas1 = new float[n-1],**matr1, max,**matr;
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::Button1Click(TObject *Sender)
{ // ñîçäàíèå ìàññèâà äâóìåðíîãî
 n=CSpinEdit1->Value;
 matr = new float*[n];
 for(i=0;i<n; i++)
 matr[i] = new float[n];
//îñâîáîæäåíèå ïàìÿòè
  delete[] mas;
for(i=0; i<n; i++)
 delete[] matr[i];
delete[] matr;
// Çàïîëíÿåì ìàññèâ
 StringGrid1->ColCount=n;
 StringGrid1->RowCount=n;
 // âòîðàÿ ìàòðèöà
  matr1 = new float*[n-1];
 for(i=0;i<n-1; i++)
 matr1[i] = new float[n-1];
//îñâîáîæäåíèå ïàìÿòè
  delete[] mas1;
for(i=0; i<n-1; i++)
 delete[] matr1[i];
delete[] matr1;
// Çàïîëíÿåì ìàññèâ
 StringGrid2->ColCount=n-1;
 StringGrid2->RowCount=n-1;
 }
 
void __fastcall TForm1::Button2Click(TObject *Sender)
{
 matr = new float*[n];
 for(i=0;i<n; i++)
 matr[i] = new float[n];
{
Randomize;
 for (i=0;i<n;i++)
  for (j=0;j<n;j++)
 {
    mas[i,j]=random(100);
    StringGrid1->Cells[i][j] = FloatToStr(mas[i,j]);
  }
}
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::Button3Click(TObject *Sender)
{
int i,j;
{
for (i=0;i<n;i++)
    for (j=0;j<n;j++)
      mas[i,j]=StrToFloat(StringGrid1->Cells[i][j]);
}
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::Button4Click(TObject *Sender)
{
{
max=mas[0,0];
a=0;
b=0;
for (i=0;i<n;i++)
for (j=0;j<n;i++)
if (mas[i,j]>max)
{
max=mas[i,j];
a=i;b=j;
}
x=0;y=0;
for (i=0;i<n;i++)
if (i!=a)
{
for (j=0;j<n;i++)
if (j!=b)
{
mas1[x,y]=mas[i,j];
StringGrid2->Cells[x][y]=mas[i,j];
y=y+1;
}
x=x+1;
}
}
}
//---------------------------------------------------------------------------

когда нажимаю на button4 ничего не работает, просто повисает программа, с чем может быть связано, заранее спасибо!!

@George22 · 13.04.2012, 18:54

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

float *mas = new float[n],*mas1 = new float[n-1],**matr1, max,**matr;

Это что за головоломка? n не определено - такой массив может всю память сожрать.

@BRcr · 14.04.2012, 10:52

Так делать нельзя, правильно George22 говорит:

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

C++
1
2
int i,j,x,y,a,b,n;
float *mas = new float[n],*mas1 = new float[n-1],**matr1, max,**matr;

Можно так:

C++
1
2
int i, j, x, y, a, b, n=20;
float *mas = new float[n], *mas1 = new float[n - 1], **matr1, max, **matr;

, хоть и неверно в этом месте память под массивы выделять, надо ее там выделять, где размерность его известна.

Button4Click у вас виснет, потому что циклы невнимательно прописаны:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
void __fastcall TForm1::Button4Click( TObject *Sender ) {
    max = mas[0, 0];
    a = 0;
    b = 0;
    for ( i = 0; i < n; i++ )
        for ( j = 0; j < n; j++ )   // <--- у вас здесь i++
            if ( mas[i, j] > max ) {
                max = mas[i, j];
                a = i;
                b = j;
            }
    x = 0;
    y = 0;
    for ( i = 0; i < n; i++ )
        if ( i != a ) {
            for ( j = 0; j < n; j++ ) // <--- и здесь тоже i++ было
                if ( j != b ) {
                    mas1[x, y] = mas[i, j];
                    StringGrid2->Cells[x][y] = mas[i, j];
                    y = y + 1;
                }
            x = x + 1;
        }
}

@more · 16.04.2012, 15:01 **[ТС]**

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
//---------------------------------------------------------------------------
 
#include <vcl.h>
#pragma hdrstop
 
#include "Unit1.h"
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma link "CSPIN"
#pragma resource "*.dfm"
TForm1 *Form1;
int i,j,x,y,a,b,n;
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::Button1Click(TObject *Sender)
{ // ñîçäàíèå ìàññèâà äâóìåðíîãî
 n=CSpinEdit1->Value;
 float *mas = new float[n],*mas1 = new float[n-1],**matr1, max,**matr;
 matr = new float*[n];
 for(i=0;i<n; i++)
 matr[i] = new float[n];
// Çàïîëíÿåì ìàññèâ
 StringGrid1->ColCount=n;
 StringGrid1->RowCount=n;
 // âòîðàÿ ìàòðèöà
  matr1 = new float*[n-1];
 for(i=0;i<n-1; i++)
 matr1[i] = new float[n-1];
// Çàïîëíÿåì ìàññèâ
 StringGrid2->ColCount=n-1;
 StringGrid2->RowCount=n-1;
 }
void __fastcall TForm1::Button2Click(TObject *Sender)
{ float *mas = new float[n],*mas1 = new float[n-1],**matr1, max,**matr;
 matr = new float*[n];
 for(i=0;i<n; i++)
 matr[i] = new float[n];
{
Randomize;
 for (i=0;i<n;i++)
  for (j=0;j<n;j++)
 {
    mas[i,j]=random(100);
    StringGrid1->Cells[i][j] = FloatToStr(mas[i,j]);
  }
  max=mas[0,0];
}
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::Button3Click(TObject *Sender)
{
int i,j;
float *mas = new float[n],*mas1 = new float[n-1],**matr1, max,**matr;
{
for (i=0;i<n;i++)
    for (j=0;j<n;j++)
      mas[i,j]=StrToFloat(StringGrid1->Cells[i][j]);
}
}
 
//---------------------------------------------------------------------------
 
void __fastcall TForm1::Button4Click(TObject *Sender)
{
float *mas = new float[n],*mas1 = new float[n-1],**matr1, max,**matr;
{
{
max=StrToFloat(StringGrid1->Cells[0][0]);
a=0;
b=0;
for (i=0;i<n;i++)
for (j=0;j<n;i++)
if (StrToFloat(StringGrid1->Cells[i][j])>max)
{
max=StrToFloat(StringGrid1->Cells[i][j]);
a=i;b=j;
}
x=0;y=0;
for (i=0;i<n;(i++)&&(x++))
if (i!=a)
{
for (j=0;j<n;(j++)&&(y++))
if (j!=b)
{
mas1[x,y]=StrToFloat(StringGrid1->Cells[i][j]);
StringGrid2->Cells[x][y]=mas1[x,y];
}
}
}
}
}
//---------------------------------------------------------------------------

сделал так, и все равно выскакивает ошибка, о пустом символе на button4.

@BRcr · 16.04.2012, 16:09

Я ж вам уже показывал, где ерунда написана. Еще тогда я ваш код запускал и у меня он отрабатывал как положено.
Всмотритесь вот в эту строчку и постарайтесь понять логику ее действий:

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

C++
1
for (j=0;j<n;i++) // цикл по j, а инкрементируется i

Вот это тоже совершенно лишнее, булевая алгебра тут ни к чему, только путаницу вносит.

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

C++
1
for (i=0;i<n;(i++)&&(x++))

Просто вот так пишите:

C++
1
for ( i = 0; i < n; i++, x++ )

Добавлено через 4 минуты
И все, что вы создаете оператором new, должно быть в конце области видимости указателя удалено, в данном случае в конце тела функции Button4Click

C++
1
2
3
4
float *mas = new float[n],*mas1 = new float[n-1],**matr1, max,**matr;
// ...
delete mas;
delete mas1;

@more · 17.04.2012, 10:14 **[ТС]**

и все равно ругается)

@nick42 · 17.04.2012, 10:19

а здесь не опечатка у Вас (во втором цикле i а не j)__

C++
1
for (i=0;i<n;i++) for (j=0;j<n;i++)

@BRcr · 17.04.2012, 10:29

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

и все равно ругается)

more, здесь экстрасенсов нету, выражайтесь точнее. Приводите текст ошибки и указывайте строчку, в которой она появляется.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

@more 1 / 1 / 0 Регистрация: 30.09.2011 Сообщений: 44
	17.04.2012, 10:14 [ТС]
	и все равно ругается) 0