Почему не появляется скрытый comboBox?

Question

[atm87]

Есть форма. По умолчанию $comboBox скрыт от пользователя. Нужно, чтобы при нажатии на кнопку $radioButton1 $comboBox становился видимым. В коде я прописал:

if ($radioButton1.Checked -eq $true) {
    $comboBox.Visible = $true
}

но $comboBox не появляется. В чём причина?

Add-Type -assembly System.Windows.Forms

$listComputersForm = New-Object System.Windows.Forms.Form
$listComputersForm.Text = "Мои компьютеры"
$listComputersForm.Width = 250
$listComputersForm.Height = 380
$listComputersForm.AutoSize = $true

$groupBox1 = New-Object System.Windows.Forms.GroupBox
$groupBox1.Text = "Выбор сети"
$groupBox1.AutoSize = $true
$groupBox1.Width = 240
$groupBox1.Height = 80
$groupBox1.Location = New-Object System.Drawing.Point(5,5)
$radioButton1 = New-Object System.Windows.Forms.RadioButton
$radioButton1.Text = "Из Active Directory"
$radioButton1.Location = New-Object System.Drawing.Point(10,20)
$radioButton1.AutoSize = $true
$radioButton2 = New-Object System.Windows.Forms.RadioButton
$radioButton2.Text = "Одноранговая сеть"
$radioButton2.Location = New-Object System.Drawing.Point(10,50)
$radioButton2.AutoSize = $true
$groupBox1.Controls.Add($radioButton1)
$groupBox1.Controls.Add($radioButton2)
if ($radioButton1.Checked -eq $true) {
    $comboBox.Visible = $true
}

$groupBox2 = New-Object System.Windows.Forms.GroupBox
$groupBox2.Text = "Список компьютеров"
$groupBox2.AutoSize = $true
$groupBox2.Width = 240
$groupBox2.Height = 80
$groupBox2.Location = New-Object System.Drawing.Point(5,100)
$radioButton3 = New-Object System.Windows.Forms.RadioButton
$radioButton3.Text = "Диапазон IP-адресов"
$radioButton3.Location = New-Object System.Drawing.Point(10,20)
$radioButton3.AutoSize = $true
$radioButton4 = New-Object System.Windows.Forms.RadioButton
$radioButton4.Text = "Из текстового файла"
$radioButton4.Location = New-Object System.Drawing.Point(10,50)
$radioButton4.AutoSize = $true
$groupBox2.Controls.Add($radioButton3)
$groupBox2.Controls.Add($radioButton4)
$groupBox2.Enabled = $false

$richTextBox = New-Object System.Windows.Forms.RichTextBox
$richTextBox.Text = "Для выбора списка компьютеров укажите группу безопасности Active Directory. Загрузка списка из тысяч компьютеров может занять продолжительное время."
$richTextBox.Width = 240
$richTextBox.Height = 65
$richTextBox.BorderStyle = 'None'
$richTextBox.Location  = New-Object System.Drawing.Point(5,190)
$richTextBox.Enabled = $false

$comboBox = New-Object System.Windows.Forms.ComboBox
$comboBox.Text = "Группы безопасности Active Directory"
$comboBox.Width = 240
$comboBox.Location = New-Object System.Drawing.Point(5,260)
$comboBox.Visible = $false

$buttonOK = New-Object System.Windows.Forms.Button
$buttonOK.Text = "OK"
$buttonOK.Location = New-Object System.Drawing.Point(50,290)
$buttonCancel = New-Object System.Windows.Forms.Button
$buttonCancel.Text = "Отмена"
$buttonCancel.Location = New-Object System.Drawing.Point(130,290)

$listComputersForm.Controls.Add($groupBox1)
$listComputersForm.Controls.Add($groupBox2)
$listComputersForm.Controls.Add($richTextBox)
$listComputersForm.Controls.Add($comboBox)
$listComputersForm.Controls.Add($buttonOK)
$listComputersForm.Controls.Add($buttonCancel)
$listComputersForm.ShowDialog()

Comments

[Роман Безруков]

как вариант - оперировать свойством Enabled вместо Visible: отрисовать combobox на форме, свойствo Enabled выставить в false, при выборе радиокнопки переключать comboBox.Enabled в true

[atm87]

Свойство Enabled пробовал. Тоже самое...

[Роман Безруков]

atm87, вот так должно работать:

$radioButton1.Add_Click ({
  if ($radioButton1.Checked -eq $true) {
    $comboBox.Visible = $true
})

[сергей кузьмин]

Роман Безруков, можно посоветовать правильное решение ?

( atm87 квам то же относится

Answer 1

[MaxKozlov]

Вы не повесили свой код на событие, вы его просто написали в произвольном месте скрипта

Пример, как вешать на событие

# Это код, который будет выполняться по событию и присвоение его переменной
$radioButton1_OnClick = {
  $comboBox.Visible = $radioButton1.Checked
}
# Это подписка на само событие. Событие OnClick, выполняемый код из $radioButton1_OnClick
$radioButton1.add_Click($radioButton1_OnClick)
$radioButton2.add_Click($radioButton1_OnClick)

Comments

[сергей кузьмин]

MaxKozlov Куратор тега PowerShell
можно вма посоветовать правильное решение ?

спойлер вы почти попали

[MaxKozlov]

сергей кузьмин, Вы уверены, что отвечаете тому человеку ? У меня подобных задач не стоит :)

Но свой вариант вы всегда можете опубликовать как в комментарии, так и в ответе. там и обсудим правильность

[atm87]

А всё же, как лучше?)

[сергей кузьмин]

использовать правильное событие:

$r1.Add_CheckedChanged({
param(
  [Object]$sender,
  [System.EventArgs]$e
)
  if ($r1.Checked -eq $true) {
    $c.Visible = $true
} else { 
    $c.Visible = $false
}

})

если помогло - я положу в ответы вы отметите решением

[atm87]

сергей кузьмин, Работает! Спасибо.

[MaxKozlov]

сергей кузьмин, Ну, ваш вариант лучше только более правильным событием,
с точки зрения atm87 будет функционально то же самое.

Но вот это вот - фу :) Ничему не помогает, только загромождает код

if ($r1.Checked -eq $true) {
    $c.Visible = $true
} else { 
    $c.Visible = $false
}

[сергей кузьмин]

MaxKozlov, вы абсолютно правы улучшение только в том что для радио трудно клик отрабатывать

"код" опитимизировать я не пытался времени не было и на мой взгляд код это что то большое а то что улучшали это скрипт диагог написовать

[сергей кузьмин]

atm87, не забудьте отметить решением

Answer 2

[сергей кузьмин]

использовать правильное событие - для радио это

AppearanceChanged,CheckedChanged,DoubleClick,MouseDoubleClick
https://learn.microsoft.com/en-us/dotnet/api/syste...

:

$r1.Add_CheckedChanged({
param(
  [Object]$sender,
  [System.EventArgs]$e
)
  if ($r1.Checked -eq $true) {
    $c.Visible = $true
} else { 
    $c.Visible = $false
}

})