При использовании текста или кнопок, клик по картинке больше не работает

@Daemon-Hub · Регистрация: 12.12.2022

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

Хочу сделать игру похожую на Zombie Smash как курсовой проект. Зомби спускаются, а ты должен нажимать на них чтобы давить. Я хотел сделать главное меню с кнопками [Играть], [Правила] и [Выйти], но когда нажимаю на кнопку играть зомби появляются, но на них нажать нельзя, потом игра зависает. Пожалуйста помогите.

Pascal
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
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
uses GraphABC, ABCObjects, ABCButtons;
   
const
  Width: int64 = 512;
  Height: int64 = 512;
  BG: string = 'bg.png';
   
type 
  Zombie = class
    public
      zx,zy: integer;
      zspeed: integer;
      zpic: PictureABC;
      alive: boolean;
      constructor Create(x,y: integer; fname: string);
      begin                
        self.zx := x;
        self.zy := y;
        self.zspeed := Random(10, 30);
        self.zpic := PictureABC.Create(x, y, fname);
        self.alive := true;
        self.zpic.ToFront;
        
      end;
      procedure kill;
      begin
        self.zpic.ChangePicture('blood.png');
        self.zpic.ToBack;
        self.zspeed := 0;
        self.alive := false;
      end;
      procedure Clicked(x,y: integer);
      begin
        if ((self.zpic.Position.X <= x) and ((self.zpic.Center.X*2 - self.zpic.Position.X) >= x)) and 
           ((self.zpic.Position.Y <= y) and ((self.zpic.Center.Y*2 - self.zpic.Position.Y) >= y)) then
          kill;
      end;
      procedure Move;
      begin
        
        if self.zpic = nil then exit;
        if self.zy >= 600 then
          self.zpic.Destroy;
        if self.zspeed <> 0 then
        begin   
          self.zpic.MoveTo(self.zx, self.zy);  
          RedrawObjects;
          self.zy += self.zspeed;         
          Sleep(100);
        end;
      end;
      property isAlive: boolean read alive;
  end;
 
var zomdies: array [1..10] of Zombie;
    btn1, btn2, btn3: ButtonABC;
    title: TextABC;
 
procedure SetWindowConfigurations;
begin
  Window.Title := 'Zombie Smash';
  Window.IsFixedSize := true;
  Window.Width := Width;
  Window.Height := Height;
  Window.Fill(BG);
end;
 
procedure MouseDown(x,y,mb:integer);//нажали кнопку мыши
begin 
  Sleep(1);
  for var i := 1 to 10 do 
    if zomdies[i] <> nil then
      zomdies[i].Clicked(x,y);
end;
 
//procedure MouseUp(x,y,mb:integer);//отпустили кнопку
//begin
//  setpencolor(clRed);
//  Circle(x,y, 20);
//end;
 
procedure InitZombie;
begin
  for var i := 1 to 10 do
    zomdies[i] := Zombie.Create(
        Random(30, 420), // X
        Random(30, 30),  // Y
        'zombie.png'     // file
    );
end;
 
procedure DestroyMenu;
begin
  
  title.Destroy;
  btn1.Destroy;
  btn2.Destroy;
  btn3.Destroy;
  LockDrawingObjects;
  RedrawObjects;
end;
 
procedure StartGame;
begin
  DestroyMenu;
  
  InitZombie;
  
  var cnt := 1;
  while true do
  begin
    var z: Zombie = zomdies[cnt];
    if z.isAlive then
      z.Move();
    
    if cnt = 10 then 
      cnt := 1
    else 
      Inc(cnt);
  end;
end;
 
procedure Rules;
begin
  
end;
 
procedure Menu;
begin
 
  UnLockDrawingObjects;
  title := TextABC.Create(Width div 2 - 130, 70, 40, $'  ZOMBIE{Chr(10)}SMASHER', GColor.Red);
  btn1 := ButtonABC.Create(Width div 2 - 50, Height div 2, 100, 30, 'Играть', GColor.Gainsboro);
  btn2 := ButtonABC.Create(Width div 2 - 50, Height div 2 + 60, 100, 30, 'Правила', GColor.Gainsboro);
  btn3 := ButtonABC.Create(Width div 2 - 50, Height div 2 + 120, 100, 30, 'Выйти', GColor.Gainsboro);
  
  btn1.OnClick := StartGame;
  btn2.OnClick := Rules;
  btn3.OnClick := Window.Close;
  
end;
 
begin
  
  SetWindowConfigurations;
  
  OnMouseDown := MouseDown;
  //OnMouseUp := MouseUp;
  
  Menu;
  //StartGame;
end.

Иногда еще выходит ошибка "Ошибка времени выполнения: Недопустимый параметр" здесь: self.zpic.MoveTo(self.zx, self.zy);

@Sun Serega · 10.06.2024, 23:35

В программе есть несколько потоков выполнения

Есть основной поток, который выполняет вам основной блок begin-end. (с точкой)

В данной программе есть ещё поток окна, который выполняет цикл while true do, но большую часть времени сидит в начале этого цикла, ожидая сообщения винды. Сообщения вроде "пользователь тыкнул мышкой".

Когда обработка этого сообщения вызывает btn1.OnClick - вы там запускаете ещё 1 цикл, не давая следующим сообщениям обрабатываться.
Через некоторое время винда понимает что ваша программа перестала подбирать (а значит и обрабатывать) последние сообщения - и говорит "программа не отвечает".

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

Pascal
1
  btn1.OnClick := ()->System.Threading.Thread.Create(StartGame).Start();

(осторожно - компилировал в уме)

Но вообще это чуток кривая архитектура.
Лучше придумать как сделать чтобы вся логика в основном begin-end. обарабывалась.

К примеру сделать enum состояния игры, и в зависимости от состояния - выполнять разный код.
Или сделать var menu_proc: procedure и циклически выполнять эту переменную когда она не nil

@Daemon-Hub · 11.06.2024, 01:13 **[ТС]**

Я немного переписал игру и использовал ваш совет, вот что получилось:

Pascal
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
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
uses GraphABC, ABCObjects, ABCButtons, System.Windows.Forms;
   
const
  Width: integer = 512;
  Height: integer = 512;
  BG: string = 'bg.png';
  ZombieImage: string = 'zombie.png';
  BloodImage: string = 'blood.png';
  
var score, lives: integer;
    scoreLabel, livesLabel: TextABC;
  
type 
  Zombie = class
    public
      zx,zy: integer;
      zspeed: integer;
      zpic: PictureABC;
      alive: boolean;
      constructor Create(x,y: integer; fname: string);
      begin                
        self.zx := x;
        self.zy := y;
        self.zspeed := Random(4, 10);
        self.zpic := PictureABC.Create(x, y, fname);
        self.alive := true;
        self.zpic.ToFront;
      end;
      procedure Restart;
      begin
        self.zpic.MoveTo(Random(80, 390), Random(5, 30));
        self.zpic.RedrawNow;
      end;
      procedure kill;
      begin
        self.zpic.ChangePicture(BloodImage);
        self.zpic.ToBack;
        self.zspeed := 0;
        self.alive := false;
        score += Random(10, 20);
      end;
      procedure Redraw;
      begin
        self.zpic.RedrawNow;
      end;
      procedure Clicked(x,y: integer);
      begin
        if ((self.zpic.Position.X <= x) and ((self.zpic.Center.X*2 - self.zpic.Position.X) >= x)) and 
           ((self.zpic.Position.Y <= y) and ((self.zpic.Center.Y*2 - self.zpic.Position.Y) >= y)) then
          kill;
      end;
      procedure Move;
      begin
        if self.zpic = nil then exit;
        if self.alive then     
          self.zpic.MoveOn(0, self.zspeed);
      end;
      property isAlive: boolean read alive;
  end;
 
var zombies: array [1..10] of Zombie;
    btn1, btn2, btn3: ButtonABC;
    title: TextABC;
 
procedure SetWindowConfigurations;
begin
  Window.Title := 'Zombie Smash';
  Window.IsFixedSize := true;
  Window.Width := Width;
  Window.Height := Height;
  Window.Fill(BG);
end;
 
procedure MouseDown(x,y,mb:integer);//нажали кнопку мыши
begin 
  for var i := 1 to 10 do 
    if zombies[i] <> nil then
      zombies[i].Clicked(x,y);
end;
 
procedure KeyPressed(key: integer);
begin
  case key of
    VK_Escape: 
      Window.Close;
    else;
   end;
end;
 
procedure UpdateScore;
begin
  scoreLabel.Text := $'Score: {score}';
  livesLabel.Text := $'Lives: {lives}';
end;
 
procedure InitZombie;
begin
  for var i := 1 to 10 do
    zombies[i] := Zombie.Create(
        Random(80, 390), // X
        Random(5, 30),   // Y
        ZombieImage      // file
    );
end;
 
procedure DestroyMenu;
begin
  title.Destroy;
  btn1.Destroy;
  btn2.Destroy;
  btn3.Destroy;
end;
 
procedure InitLabels;
begin
  
  score := 0;
  lives := 3;
  
  scoreLabel := TextABC.Create(10, 10, 20, $'Score: {score}', clCoral);
  livesLabel := TextABC.Create(10, 30, 20, $'Lives: {lives}', clCoral);
end;
 
procedure GameOver;
begin
  LockDrawingObjects;
  
  livesLabel.Text := 'GAME OVER';
  livesLabel.FontSize := 40;
  livesLabel.MoveTo(100, 180);
  livesLabel.Color := clRed;
  livesLabel.ToFront;
  livesLabel.FontStyle := FontStyleType.fsBold;
  
  scoreLabel.Text := $'Total Score: {score}';
  scoreLabel.FontSize := 30;
  scoreLabel.MoveTo(110, 240);
  scoreLabel.Color := clGreen;
  scoreLabel.ToFront;
  livesLabel.FontStyle := FontStyleType.fsBold;
  
  RedrawObjects;
end;
 
procedure StartGame;
begin
  LockDrawingObjects;
  DestroyMenu;
  InitZombie;
  InitLabels;
  RedrawObjects;  
  
  while lives > 0 do
  begin
    for var i := 1 to 10 do
    begin
      LockDrawingObjects;   
      if lives <= 0 then break;
      if zombies[i] <> nil then
      begin
        if zombies[i].IsAlive then
          zombies[i].Move;
        if zombies[i].zpic.Position.Y >= Height then
        begin
          zombies[i].Restart;
          Dec(lives);
        end
        else if not zombies[i].isAlive then
          zombies[i] := Zombie.Create(Random(30, 420), Random(30, 30), ZombieImage); 
        zombies[i].Redraw;
      end;
    end;
      
    UpdateScore;
    UnLockDrawingObjects;
    Sleep(50);
  end;
 
  // Игра окончена
  GameOver;
 
end;
 
 procedure Rules;
 begin
   var msg := $'Все тебе предстоит делать это нажимать на бегущих с верху вниз зомби.{chr(10)}И за это зарабатывать очки.{chr(10)}Игра будет продолжаться пока ты не проиграешь.';
   MessageBox.Show(msg,'Правила', MessageBoxButtons.OK);
 end;
 
procedure Menu;
begin
 
  LockDrawingObjects;
  title := TextABC.Create(Width div 2 - 130, 70, 40, $'  ZOMBIE{Chr(10)}SMASHER', GColor.Red);
  btn1 := ButtonABC.Create(Width div 2 - 50, Height div 2, 100, 30, 'Играть', GColor.Gainsboro);
  btn2 := ButtonABC.Create(Width div 2 - 50, Height div 2 + 60, 100, 30, 'Правила', GColor.Gainsboro);
  btn3 := ButtonABC.Create(Width div 2 - 50, Height div 2 + 120, 100, 30, 'Выйти', GColor.Gainsboro);
  RedrawObjects;
  
  btn1.OnClick := ()->System.Threading.Thread.Create(StartGame).Start();
  btn2.OnClick := Rules;
  btn3.OnClick := Window.Close;
  
end;
 
begin
  
  SetWindowConfigurations;
  
  OnMouseDown := MouseDown;
  OnKeyDown := KeyPressed;
  
  Menu;
 
end.

Когда пытаюсь его запустить с оболочкой, она не запускается а когда через Shift+F9, тоже не всегда, но запускается.
И почему то в процессе игры она как зависнет, и все конец.
Картинки тоже прилагаю, пожалуйста посмотрите как все работает.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Рисуем цветные прямоугольники с помощью рисовальщика SDL3 на Си и C++ 8Observer8 17.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-rectangles-sdl3-c. zip finish-rectangles-sdl3-cpp. zip	Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая ссылка» (hard link),. . .	[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .	делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью
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-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .