- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
- 9
- 10
- 11
procedure TForm1.Image1Click(Sender: TObject);
begin
sPictureName := Label2.Caption;
Form2.Show;
end;
procedure TForm1.Image2Click(Sender: TObject);
begin
sPictureName := Label3.Caption;
Form2.Show;
end;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
0
procedure TForm1.Image1Click(Sender: TObject);
begin
sPictureName := Label2.Caption;
Form2.Show;
end;
procedure TForm1.Image2Click(Sender: TObject);
begin
sPictureName := Label3.Caption;
Form2.Show;
end;
Знаю конечно что уныло, но по моему делать одну процедуру из-за этого слишком геморройно. С тем же массивом из 2-х элементов хотя бы. Говно ли?
а еще физическая венгерка говно
не?
циклы? процедуры? o_O just copy and paste it
Blaise Pascal - mathématicien, physicien, inventeur, philosophe, moraliste et théologien français.
зы: либо логическую (запись, класс), либо физическую (контрол-контейнер)
If (Sender Is TImage) или как там правильно в дельфях
Сегодня это TImage, завтра TAdvImage и т.д. Надежней разбить на два обработчика и общий метод.
PS: Хотя, автору виднее...
http://govnokod.ru/7345#comment99135
http://govnokod.ru/7345#comment99157
А в событиях Image2, на OnClick поставить рефер Image1Click.
Таким образом вы автоматом (без почвенно) определяете зависимости
1. Sender должен быть TImage.
2. TImage должен иметь определенный Name.
2. Уже пофиксил.