Одно событие для многих объектов. Как отловить?

@zhigalkin_p · Регистрация: 26.10.2017

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

День добрый!
На форме 30 panel, в каждой есть TableLayoutPanel в которой уже есть 40 Label. В общей сложности получается рабочих лейблов 1200. Имена у них идут не по порядку (так как на форме помимо этих есть еще и другие).
Вот возник вопрос: как отловить клик мышкой именно на тех лейблах, которые находятся в TableLayoutPanel?
Можно сделать так:

VB.NET
1
2
3
    Private Sub Label1_MouseClick(ByVal sender As Object, ByVal e As System.Windows.Forms.MouseEventArgs) Handles Label1.MouseClick, Label2.MouseClick
        MsgBox("OK!")
End Sub

Но перечислять все 1200 имен - думаю это маразм.

Еще пробовал делать так (в Form1_Load):

VB.NET
1
2
3
4
For Each Lbl As Control In TableLayoutPanel1.Controls
        AddHandler Lbl.Click, AddressOf Lbl_Msg
        TableLayoutPanel1.Controls.Add(Lbl)
        Next

VB.NET
1
2
3
Private Sub Lbl_Msg()
        MsgBox("OK!")
    End Sub

Но что-то не срабатывает. Подскажите как отловить клик?

@ViterAlex · 03.09.2018, 13:29

Сигнатура Lbl_Msg не соответствует сигнатуре события Click
Зачем опять добавляешь Lbl?
Тут целесообразно использовать UserControl, в который вынести TableLayoutPanel со всем содержимым.

@ovva · 03.09.2018, 13:43

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

На форме 30 panel, в каждой есть TableLayoutPanel в которой уже есть 40 Label. В общей сложности получается рабочих лейблов 1200.

Может, что-то в консерватории подправить?

@zhigalkin_p · 03.09.2018, 13:47 **[ТС]**

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

Может, что-то в консерватории подправить?

Никак нельзя. Их именно столько и будет.

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

Тут целесообразно использовать UserControl, в который вынести TableLayoutPanel со всем содержимым.

То есть из TableLayoutPanel создать UserControl? Как с ним потом работать? Я, можно сказать недавно перешел на vb.net.

@ovva · 03.09.2018, 14:10

Тогда м.б. так

VB.NET
1
2
3
4
5
6
7
8
9
10
11
12
Private Sub Form7_Load(sender As System.Object, e As System.EventArgs) Handles MyBase.Load
    For Each tlp As Control In Me.Controls
        If TypeOf tlp Is TableLayoutPanel Then
            For Each Lbl As Control In tlp.Controls
                If TypeOf Lbl Is Label Then AddHandler Lbl.Click, AddressOf Lbl_Msg
            Next
        End If
    Next
End Sub
Private Sub Lbl_Msg()
    MsgBox("OK!")
End Sub

@Элд Хасп · 03.09.2018, 15:12

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

Тогда м.б. так...

Обратите внимание на

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

Сигнатура Lbl_Msg не соответствует сигнатуре события Click

По моему, в таком виде работать не будет

@zhigalkin_p · 03.09.2018, 15:14 **[ТС]**

Неа, не работает

@Элд Хасп · 03.09.2018, 15:20

Надо так:

VB.NET
1
2
3
    Private Sub Lbl_Msg(sender As Label, e As EventArgs)
        MsgBox("Метка " + sender.Name + vbCrLf + "OK!")
    End Sub

Добавлено через 1 минуту

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

Неа, не работает

Что не работает? Какую ошибку выдаёт?

@zhigalkin_p · 03.09.2018, 15:23 **[ТС]**

Сообщение от Элд Хасп

Что не работает? Какую ошибку выдаёт?

Никакую. Просто ничего, как-будто кода нет.

@Элд Хасп · 03.09.2018, 15:29

Вроде, должна быть ещё возможность обрабатывать Click не на уровне Label, а на уровне TableLayoutPanel.

Добавлено через 51 секунду

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

Никакую. Просто ничего, как-будто кода нет.

Сигнатуру обработчика Lbl_Msg поменяли?

Добавлено через 1 минуту
Проверьте в отладчике происходит подсоединение обработчика Click или нет.

Добавлено через 2 минуты
Если обработчик не подсоединяется, проверьте в отладчике циклы в Form7_Load - доходит работа кода до AddHandler... или нет.

@ovva · 03.09.2018, 15:31

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

Неа, не работает

У меня так все работает, в соответствии с заказанным. При клике на Label в TableLayoutPanel имеем Ok.

@Rius · 03.09.2018, 15:31

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

Никак нельзя. Их именно столько и будет.

Откуда появилась такая чудесная форма? Наследие?

@Элд Хасп · 03.09.2018, 15:36

Как у Вас создаются 30 TableLayoutPanel? Если программно, то возможно Вы их создаёте после события Load Формы и, поэтому, обработчик загрузки Form7_Load их не находит.

Добавлено через 2 минуты

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

У меня так все работает...

Получается, сигнатуру обработчика соблюдать не обязательно? Не знал этого.

@zhigalkin_p · 03.09.2018, 15:43 **[ТС]**

Сообщение от Элд Хасп

Как у Вас создаются 30 TableLayoutPanel? Если программно, то возможно Вы их создаёте после события Load Формы и, поэтому, обработчик загрузки Form7_Load их не находит.

Нет, они уже там, вставлены вручную в дизайнере.

@zhigalkin_p · 03.09.2018, 15:56 **[ТС]**

Вот, просто ничего не происходит

@Элд Хасп · 03.09.2018, 16:02

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

Вот, просто ничего не происходит

Скорее всего обработчик не подсоединяется. Надо смотреть последовательность организации Формы.
Скиньте весь проект.

@zhigalkin_p · 03.09.2018, 16:13 **[ТС]**

Вот проект на котором тренируюсь. Сильно не ругайтесь, там много мусора.

@zhigalkin_p · 03.09.2018, 16:34 **[ТС]**

Забыл сказать, у меня VS 2017

@ViterAlex · 03.09.2018, 16:42

zhigalkin_p, у тебя TableLayoutPanel находятся не на форме, а внутри GroupBox. Это нужно учитывать. Но лучше вынести всё в UserControl, как ты уже сделал с TableLabel

@Элд Хасп · 03.09.2018, 16:42

У Вас TableLayoutPanel лежат не в Форме (Form1), а в дочерней панели (Panel1), в GroupBox1. Так же часть Lablel лежит не в TableLayoutPanel, а в TableLable1.
При таких различиях в способах организации элементов у Вас не получится единообразно их обрабатывать. Как то приведите всё к единообразию.
Цикл из решения ovva обрабатывает Lablel лежащие только в TableLayoutPanel, которые в свою очередь лежат непосредственно в родительской форме. У Вас таких нет.

Новые блоги и статьи Все статьи Все блоги /
Как дизайн сайта влияет на конверсию: 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-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .

@ViterAlex 8951 / 4863 / 1886 Регистрация: 11.02.2013 Сообщений: 10,246
	03.09.2018, 13:29
	Сигнатура `Lbl_Msg` не соответствует сигнатуре события `Click` Зачем опять добавляешь `Lbl`? Тут целесообразно использовать `UserControl`, в который вынести `TableLayoutPanel` со всем содержимым. 1

@zhigalkin_p 1 / 1 / 0 Регистрация: 26.10.2017 Сообщений: 108
	03.09.2018, 15:14 [ТС]
	Неа, не работает 0

@zhigalkin_p 1 / 1 / 0 Регистрация: 26.10.2017 Сообщений: 108
	03.09.2018, 15:56 [ТС]
	Вот, просто ничего не происходит Миниатюры 0

@zhigalkin_p 1 / 1 / 0 Регистрация: 26.10.2017 Сообщений: 108
	03.09.2018, 16:34 [ТС]
	Забыл сказать, у меня VS 2017 0

@ViterAlex 8951 / 4863 / 1886 Регистрация: 11.02.2013 Сообщений: 10,246
	03.09.2018, 16:42
	zhigalkin_p, у тебя `TableLayoutPanel` находятся не на форме, а внутри `GroupBox`. Это нужно учитывать. Но лучше вынести всё в `UserControl`, как ты уже сделал с `TableLabel` 0

@Элд Хасп Модератор 16139 / 11263 / 2888 Регистрация: 21.04.2018 Сообщений: 33,109 Записей в блоге: 2
	03.09.2018, 16:42
	У Вас `TableLayoutPanel` лежат не в Форме (`Form1`), а в дочерней панели (`Panel1`), в `GroupBox1`. Так же часть `Lablel` лежит не в `TableLayoutPanel`, а в `TableLable1`. При таких различиях в способах организации элементов у Вас не получится единообразно их обрабатывать. Как то приведите всё к единообразию. Цикл из решения ovva обрабатывает `Lablel` лежащие только в `TableLayoutPanel`, которые в свою очередь лежат непосредственно в родительской форме. У Вас таких нет. 0