Как правильно сохранить переменную после нажатия кнопки?

Question

[malchikmolodoy]

Всем привет!
Пишу небольшой интерфейс с логами под выдачу телефонов.
Выбираю модель телефона, далее ввожу imei, ФИО, жму "Записать" и в логи при этом пишется всё, кроме модели телефона. Во время нажатия клавиши переменная phone имеет значение, которое выбирается в Combobox, но после переменная сбрасывается. Как мне правильно написать так, чтобы переменная всегда сохранялась?
Вторая проблема: как делать сохранения файла по типу "Дата + ФИО + .log". Когда создаю переменную пути для логов, указываю так:

$txtfile = "C:\scripts\ps\" + $datetext.Text + ".log"

и вместо datetext.text ничего не сохраняет, пустое поле. Только .log

function txtbox ($datetext, $imei, $phone)
    {
        $date = Get-Date -Format "dd.MM.yyyy HH:mm:ss"
        $msg = $date + "`t Выдан телефон: " + $phone + $datetext.text + "`t IMEI: " + $imei.Text + "123123"

        #Записываем всё в тхт файл
        Out-File -FilePath $txtfile -InputObject $msg -Append -Encoding unicode
       
#combobox
    $itembox = New-Object System.Windows.Forms.ComboBox
    $itembox.AutoSize = 1
    $itembox.Location = New-Object System.Drawing.Point(40,40)
    #перечисление устройств в массиве
    $itembox.DataSource = @("Samsung A12", "Redmi Note A6", "Другое:", "4")


#button
    $but_imei = New-Object System.Windows.Forms.Button
    $but_imei.text = "Записать"
    $but_imei.Location = "400,100"
    $but_imei.AutoSize = 1
    $but_imei.Add_Click(
    {
    IMEI
    dateusers 
    $phone = $itembox.SelectedItem
    write-host = $phone
    }
    )  

    }

Comments

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

[System.Windows.Forms.MessageBox]::Show($itembox.Items[$itembox.SelectedIndex].ToString())

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

[malchikmolodoy]

сергей кузьмин, мне эту функцию добавить в add_click?

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

malchikmolodoy, диалог я показал вам чисто для отладки значение куда вам надо туда и девайте

$itembox.Items[$itembox.SelectedIndex].ToString()

[malchikmolodoy]

сергей кузьмин, да, действительно, добавил это в свою функцию и все заработало. Если хочешь, то можешь добавить это в ответы, отмечу решением.
Получается вся проблема в том, что нужно было форматировать эту переменную в строку? или почему он до этого не выводил?

Answer 1

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

$itembox.Items[$itembox.SelectedIndex].ToString()

Answer 2

[MaxKozlov]

Вот этот вот кусок вообще не имеет смысла

$but_imei.Add_Click(
    {
    IMEI
    dateusers 
    $phone = $itembox.SelectedItem
    write-host = $phone
    }
    )

Если IMEI и dateusers это не функции без параметров
а "write-host = $phone" конечно не синтаксическая ошибка благодаря функционалу write-host, но очень на неё похожа

В целом же, если разберётесь с ошибками, код рабочий
Вот в таком виде выдаёт на консоль содержимое комбобокса

add-type -AssemblyName System.Windows.Forms
$Form = New-Object System.Windows.Forms.Form
$Form.Size = New-Object System.Drawing.Size(500,400)
$Form.StartPosition = "CenterParent"
$itembox = New-Object System.Windows.Forms.ComboBox
    $itembox.AutoSize = 1
    $itembox.Location = New-Object System.Drawing.Point(40,40)
$itembox.DataSource = @("Samsung A12", "Redmi Note A6", "Другое:", "4")
$but_imei = New-Object System.Windows.Forms.Button
    $but_imei.text = "Записать"
    $but_imei.Location = "400,100"
    $but_imei.AutoSize = 1
    $but_imei.Add_Click(
    {
      write-host $itembox.SelectedItem -fore green
    }
    )  
$Form.Controls.Add($itembox)
$Form.Controls.Add($but_imei)
$Form.ShowDialog()

Comments

[malchikmolodoy]

Вот полный код
Write-Host для проверки функций пробую.

#функция отправки данных в тхт
    function txtbox ($datetext, $imei, $phone)
    {
        $date = Get-Date -Format "dd.MM.yyyy HH:mm:ss"
        $msg = $date + "`t Выдан телефон: $phone" + $datetext.Text + "`t IMEI: " + $imei.Text

        #Записываем всё в тхт файл
        Out-File -FilePath $txtfile -InputObject $msg -Append -Encoding unicode
       

    }

                                        
#функция вытягивания имеи 
    function IMEI
    {
        $phoneim = $imei.Text
        Write-Host = $phoneim
        
    }

    function test
    {
        Write-Host = "ПРИВЕЕЕЕЕЕЕЕЕЕЕЕЕТ"
    }

    #ФУНКЦИЯ ВВОДА ДАННЫХ
    function dateusers
    {
        $dateuser = New-Object System.Windows.Forms.Form
        $dateuser.text = 'ФИО'
        $dateuser.Width = 300
        $dateuser.Height = 300
        $dateuser.StartPosition = "CenterScreen"
            
       
        $datetext = New-Object System.Windows.Forms.TextBox
        $datetext.Location = New-Object System.Drawing.Point (10,100)
        $datetext.Width = 250
        $datetext.Text = "Введите ФИО"
        

        $but_date = New-Object System.Windows.Forms.Button
        $but_date.Location = New-Object System.Drawing.Point (100,150)
        $but_date.text = "Next"
        $but_date.AutoSize = 0
        $but_date.Add_Click({txtbox $datetext $imei $dateuser.Close()}) 


$dateuser.Controls.Add($but_date)
$dateuser.Controls.Add($datetext)
$dateuser.ShowDialog()

    }

    #Функция проверки телефона
    function checkteleph ($phone)
    {
        if($phonecheck.CheckState -eq 1)
        {
        $phone = "Samsung Galaxy A12"
        }
        else{$phone = "N/A"}
        $phone = "32132131"
        
       
    }






#textbox
    $imei = New-Object System.Windows.Forms.TextBox
    $imei.Width = 200
    $imei.Location = New-Object System.Drawing.Point(100,100)
    $imei.Text = "Введите IMEI"



#combobox
    
    $itembox = New-Object System.Windows.Forms.ComboBox
    $itembox.AutoSize = 1
    $itembox.Location = New-Object System.Drawing.Point(40,40)
    #перечисление устройств в массиве
    $itembox.DataSource = @("Samsung A12", "Redmi Note A6", "Другое:", "4")
    
    


#button

    $but_imei = New-Object System.Windows.Forms.Button
    $but_imei.text = "Записать"
    $but_imei.Location = "400,100"
    $but_imei.AutoSize = 1
    $but_imei.Add_Click(
    {
    IMEI
    dateusers 
    $phone = $itembox.SelectedItem
    Write-Host = $phone
    }
    )





#controlls
    $headwin.Controls.Add($imei)
    $headwin.Controls.Add($but_imei)
    $headwin.Controls.Add($itembox)

$headwin.ShowDialog()

[MaxKozlov]

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

[MaxKozlov]

вот такая фигня не делается, я уже писал
Write-Host = $phoneim
в функции лучше передавать не объект, а значение поля

$but_date.Add_Click({txtbox $datetext.text $imei.text $dateuser.Close()})

в целом же что-то такое пытается сделать, только вот переменная $txtfile - пустая
Если не пытаться в файл писать, а выводить на экран - требуемое содержимое выводит

#Out-File -FilePath $txtfile -InputObject $msg -Append -Encoding unicode
$msg | out-host

Из исправлений для проверки - сместил пониже окно ввода imei и добавил его сразу на форму

$dateuser.Controls.Add($but_date)
$dateuser.Controls.Add($datetext)
$dateuser.Controls.Add($imei)
$dateuser.Controls.Add($itembox)

[malchikmolodoy]

MaxKozlov, "Из исправлений для проверки - сместил пониже окно ввода imei и добавил его сразу на форму"
Не совсем понял для чего это делается.

[malchikmolodoy]

MaxKozlov, и почему именно через tabcontrol?

[MaxKozlov]

malchikmolodoy, потому что легко переключением страниц скрывать и показывать сразу много контролов

[MaxKozlov]

malchikmolodoy, потому что ваши поля для ввода на одном и том же месте и перекрываются. а вложенность создания форм "dateusers" и "headwin" мало того что не работает, так ещё и не оправдана

Answer 3

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

вместо datetext.text ничего не сохраняет

- эта переменная как определяется и какова ее область видимости - весь скрипт или функция txtbox? Тот же вопрос про переменную $phone.
Я так понимаю, что это не весь код, поэтому выглядит он весьма сумбурно
Ещё момент: в предыдущем вопросе вам уже говорили, что использовать двоеточия в именах файлов недопустимо.

P.S. у вас у чистом виде табличные данные, поэтому вместо текстовых файлов я бы сохранял их в Excel или SQL Express

Comments

[malchikmolodoy]

Вот весь код. Я, вроде, в названии и не указываю двоеточия.
Пытался работать с excel, там запары какие - то. Может быть в дальнейшем переделаю на excel, но щас в приоритете проработать все функции)

#функция отправки данных в тхт
    function txtbox ($datetext, $imei, $phone)
    {
        $date = Get-Date -Format "dd.MM.yyyy HH:mm:ss"
        $msg = $date + "`t Выдан телефон: $phone" + $datetext.Text + "`t IMEI: " + $imei.Text

        #Записываем всё в тхт файл
        Out-File -FilePath $txtfile -InputObject $msg -Append -Encoding unicode
       

    }

                                        
#функция вытягивания имеи 
    function IMEI
    {
        $phoneim = $imei.Text
        Write-Host = $phoneim
        
    }

    function test
    {
        Write-Host = "ПРИВЕЕЕЕЕЕЕЕЕЕЕЕЕТ"
    }

    #ФУНКЦИЯ ВВОДА ДАННЫХ
    function dateusers
    {
        $dateuser = New-Object System.Windows.Forms.Form
        $dateuser.text = 'ФИО'
        $dateuser.Width = 300
        $dateuser.Height = 300
        $dateuser.StartPosition = "CenterScreen"
            
       
        $datetext = New-Object System.Windows.Forms.TextBox
        $datetext.Location = New-Object System.Drawing.Point (10,100)
        $datetext.Width = 250
        $datetext.Text = "Введите ФИО"
        

        $but_date = New-Object System.Windows.Forms.Button
        $but_date.Location = New-Object System.Drawing.Point (100,150)
        $but_date.text = "Next"
        $but_date.AutoSize = 0
        $but_date.Add_Click({txtbox $datetext $imei $dateuser.Close()}) 


$dateuser.Controls.Add($but_date)
$dateuser.Controls.Add($datetext)
$dateuser.ShowDialog()

    }

    #Функция проверки телефона
    function checkteleph ($phone)
    {
        if($phonecheck.CheckState -eq 1)
        {
        $phone = "Samsung Galaxy A12"
        }
        else{$phone = "N/A"}
        $phone = "32132131"
        
       
    }






#textbox
    $imei = New-Object System.Windows.Forms.TextBox
    $imei.Width = 200
    $imei.Location = New-Object System.Drawing.Point(100,100)
    $imei.Text = "Введите IMEI"



#combobox
    
    $itembox = New-Object System.Windows.Forms.ComboBox
    $itembox.AutoSize = 1
    $itembox.Location = New-Object System.Drawing.Point(40,40)
    #перечисление устройств в массиве
    $itembox.DataSource = @("Samsung A12", "Redmi Note A6", "Другое:", "4")
    
    


#button

    $but_imei = New-Object System.Windows.Forms.Button
    $but_imei.text = "Записать"
    $but_imei.Location = "400,100"
    $but_imei.AutoSize = 1
    $but_imei.Add_Click(
    {
    IMEI
    dateusers 
    $phone = $itembox.SelectedItem
    Write-Host = $phone
    }
    )





#controlls
    $headwin.Controls.Add($imei)
    $headwin.Controls.Add($but_imei)
    $headwin.Controls.Add($itembox)

$headwin.ShowDialog()