ComboBox.SelectedItem является нулем - PullRequest
Новая
       10

ComboBox.SelectedItem является нулем

0 голосов
/ 24 мая 2019

Я заполнил комбинированный список элементами AD SamAccountName. При выборе одного из элементов можно нажать кнопку, чтобы получить информацию из этой учетной записи. Тем не менее, при нажатии на кнопку я получаю следующее ошибка : 

Cannot validate argument on parameter 'Identity'. The argument is null. Provide a valid value for the argument, and then try running the command again.

Команда, к которой относится ошибка: 

$Label_User_ItemContent.Text = (Get-ADUser -Identity $ComboBox.SelectedItem).SamAccountName

критическая часть кода : 

        $ComboBox = New-Object -TypeName System.Windows.Forms.ComboBox
        $ComboBox.Width = 300
        $ComboBox.Location = New-Object -TypeName System.Drawing.Point(250, 25)

        $Users = Get-ADUser -Filter * | Where-Object {$_.SamAccountName -match '^adminA'}
        ForEach ($User in $Users){
            $ComboBox.Items.Add($User.SamAccountName)
        }
        $ComboBox.SelectedIndex = 1

            $MainWindow.Controls.Add($ComboBox)

        $Button_Check_TEST = New-Object -TypeName System.Windows.Forms.Button
        $Button_Check_TEST.Location = New-Object -TypeName System.Drawing.Size(350, 150)
        $Button_Check_TEST.Size = New-Object -TypeName System.Drawing.Size(150, 50)
        $Button_Check_TEST.Text = 'Check'

        $MainWindow.Controls.Add($Button_Check_TEST)

        $Button_Check_TEST.Add_Click({
            Try{
                $Label_User_ItemContent.Text = (Get-ADUser -Identity $ComboBox.SelectedItem).SamAccountName
            }
            Catch{
                Write-Verbose -Verbose $_.Exception.Message
            }
        })

Проблема в том, что мне нужно два слоя. По сути, должно быть меню с четырьмя различными опциями, одна из которых - «Пользователь». При нажатии на «Пользователь» должны появиться ComboBox и кнопка «Клик». Использование приведенного выше кода без кнопки «Пользователь» работает нормально.

Вопрос: Почему ComboBox.SelectedItem не работает, когда я использую кнопку для «создания» ComboBox?

Полный код ниже: 

    $font = New-Object -TypeName System.Drawing.Font("Times New Roman", 18, [System.Drawing.FontStyle]::Bold)

    $MainWindow = New-Object -TypeName System.Windows.Forms.Form
    $MainWindow.Text = 'PIM v10 Administrator Window'
    $MainWindow.Width = 600
    $MainWindow.Height = 555
    $MainWindow.AutoSize = $true

    $Button_User = New-Object -TypeName System.Windows.Forms.Button
    $Button_User.Location = New-Object -TypeName System.Drawing.Size(25, 25)
    $Button_User.Size = New-Object -TypeName System.Drawing.Size(200, 75)
    $Button_User.Text = 'User'
    $Button_User.Font = $font

        $MainWindow.Controls.Add($Button_User)

    $Button_User.Add_Click({
    $Label_User = New-Object -TypeName System.Windows.Forms.Label
    $Label_User.Text = 'Given Name:'
    $Label_User.Location = New-Object -TypeName System.Drawing.Point(250, 50)
    $Label_User.AutoSize = $true

        $MainWindow.Controls.Add($Label_User)

    $Label_User_ItemContent = New-Object -TypeName System.Windows.Forms.Label
    $Label_User_ItemContent.Text = ''
    $Label_User_ItemContent.Location = New-Object -TypeName System.Drawing.Point(250, 100)

        $MainWindow.Controls.Add($Label_User_ItemContent)

    $ComboBox = New-Object -TypeName System.Windows.Forms.ComboBox
    $ComboBox.Width = 300
    $ComboBox.Location = New-Object -TypeName System.Drawing.Point(250, 25)

    $Users = Get-ADUser -Filter * | Where-Object {$_.SamAccountName -match '^adminA'}
     ForEach ($User in $Users){
         $ComboBox.Items.Add($User.SamAccountName)
        }
     $ComboBox.SelectedIndex = 1

        $MainWindow.Controls.Add($ComboBox)

     $Button_Check_TEST = New-Object -TypeName System.Windows.Forms.Button
     $Button_Check_TEST.Location = New-Object -TypeName System.Drawing.Size(350, 150)
     $Button_Check_TEST.Size = New-Object -TypeName System.Drawing.Size(150, 50)
     $Button_Check_TEST.Text = 'Check'

     $MainWindow.Controls.Add($Button_Check_TEST)

     $Button_Check_TEST.Add_Click({
            Try{
                $Label_User_ItemContent.Text = (Get-ADUser -Identity $ComboBox.SelectedItem).SamAccountName
            }
            Catch{
                Write-Verbose -Verbose $_.Exception.Message
            }
        })

     if (-not ($ComboBox.SelectedItem -eq $null)){
         $Label_User_ItemContent.Text = (Get-ADUser -Identity $ComboBox.SelectedItem).SamAccountName
     }
     else {
        Write-Host -Object "Object is null"
     }
    })
    $MainWindow.ShowDialog()

1 Ответ

1 голос
/ 24 мая 2019

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

$Button_User.Add_Click({
    $ComboBox = New-Object -TypeName System.Windows.Forms.ComboBox
    $Label_User_ItemContent = New-Object -TypeName System.Windows.Forms.Label
    $MainWindow.Controls.Add($ComboBox)
    $Label_User_ItemContent = New-Object -TypeName System.Windows.Forms.Label
    $Button_Check_TEST.Add_Click({
        $Label_User_ItemContent.Text = (Get-ADUser -Identity 
        $ComboBox.SelectedItem).SamAccountName
    )}
})

Поскольку вы создаете Combo и Lable внутри действия Add_click.Эти значения существуют только тогда, когда действие выполнено и в $MainWindows.Controls.Затем элементы удаляются из памяти

Когда вы запускаете следующее действие $Button_Check_TEST.Add_Click(), так как переменные очищаются, тогда $ComboBox и $Label_User_ItemContent ничего не равны.

Исправление будет помещеноони вне $Button_User.Add_Click() события 

$ComboBox = New-Object -TypeName System.Windows.Forms.ComboBox
$Label_User_ItemContent = New-Object -TypeName System.Windows.Forms.Label
$Button_User.Add_Click({
    $MainWindow.Controls.Add($ComboBox)
    $Label_User_ItemContent = New-Object -TypeName System.Windows.Forms.Label
    $Button_Check_TEST.Add_Click({
        $Label_User_ItemContent.Text = (Get-ADUser -Identity 
        $ComboBox.SelectedItem).SamAccountName
    )}
})

Вот весь сценарий в рабочем состоянии сейчас 

$font = New-Object -TypeName System.Drawing.Font("Times New Roman", 18, [System.Drawing.FontStyle]::Bold)

$MainWindow = New-Object -TypeName System.Windows.Forms.Form
$MainWindow.Text = 'PIM v10 Administrator Window'
$MainWindow.Width = 600
$MainWindow.Height = 555
$MainWindow.AutoSize = $true

$Button_User = New-Object -TypeName System.Windows.Forms.Button
$Button_User.Location = New-Object -TypeName System.Drawing.Size(25, 25)
$Button_User.Size = New-Object -TypeName System.Drawing.Size(200, 75)
$Button_User.Text = 'User'
$Button_User.Font = $font

$MainWindow.Controls.Add($Button_User)

$ComboBox = New-Object -TypeName System.Windows.Forms.ComboBox
$Label_User_ItemContent = New-Object -TypeName System.Windows.Forms.Label

$Button_User.Add_Click({
    $Label_User = New-Object -TypeName System.Windows.Forms.Label
    $Label_User.Text = 'Given Name:'
    $Label_User.Location = New-Object -TypeName System.Drawing.Point(250, 50)
    $Label_User.AutoSize = $true

    $MainWindow.Controls.Add($Label_User)


    $Label_User_ItemContent.Text = ''
    $Label_User_ItemContent.Location = New-Object -TypeName System.Drawing.Point(250, 100)

    $MainWindow.Controls.Add($Label_User_ItemContent)


    $ComboBox.Width = 300
    $ComboBox.Location = New-Object -TypeName System.Drawing.Point(250, 25)

    $Users = Get-ADUser -Filter * | Where-Object {$_.SamAccountName -match '^adminA'}
    $MainWindow.Controls.Add($ComboBox)
    ForEach ($User in $Users){
        $ComboBox.Items.Add($User.SamAccountName)
    }
    $MainWindow.Controls.Add($ComboBox)
    $ComboBox.SelectedIndex = 0   
    $Button_Check_TEST = New-Object -TypeName System.Windows.Forms.Button
    $Button_Check_TEST.Location = New-Object -TypeName System.Drawing.Size(350, 150)
    $Button_Check_TEST.Size = New-Object -TypeName System.Drawing.Size(150, 50)
    $Button_Check_TEST.Text = 'Check'

    $MainWindow.Controls.Add($Button_Check_TEST)

    $Button_Check_TEST.Add_Click({
        Try{
            $Label_User_ItemContent.Text = (Get-ADUser -Identity $ComboBox.SelectedItem).SamAccountName
        }
        Catch{
            Write-Verbose -Verbose $_.Exception.Message
        }
    })

    if (-not ($ComboBox.SelectedItem -eq $null)){
        $Label_User_ItemContent.Text = (Get-ADUser -Identity $ComboBox.SelectedItem).SamAccountName
    }
    else {
        Write-Host -Object "Object is null"
    }
})
$MainWindow.ShowDialog()
...