Как правильно составить скрипт PowerShell?

Question

[Knorke_02]

Суть такова, нужно написать скрипт, чтобы при помощи него создавались пользователи в Active Directory:
1. Количество пользователей
2. Имена пользователей
3. Пароль для пользователей
Я написал такой скрипт, вижу в нем ошибку, но не получается сделать, прошу помощи
Import-Module ActiveDirectory

# Переменные
$number= Read-Host "Enter the number of users"
$input = Read-Host 'Enter a name'
$pass = Read-Host 'Enter the password'
$count=1..$number

# Cоздаем OU
New-ADOrganizationalUnit -Name "DemoOffice" -Path “DC=demo,DC=lab”
New-ADOrganizationalUnit -Name "Users" -Path “OU=DemoOffice,DC=demo,DC=lab”
New-ADOrganizationalUnit -Name "Computers" -Path “OU=DemoOffice,DC=demo,DC=lab”

# Цикл с пользователями
foreach ($i in $count) {

$Users = @{

Name = "$input"
GivenName = "$input"
UserPrincipalName = "$[e-mail]"
Path = "OU=Users,OU=DemoOffice,DC=demo,DC=lab"
ChangePasswordAtLogon = $true
AccountPassword = "$pass" | ConvertTo-SecureString -AsPlainText -Force
Enabled = $true

}

New-ADUser @Users

}

Comments

[Alexey Dmitriev]

Открываете скрипт в PowerShell ISE, выделяете любой блок, запускаете блок по F8 и проверяете - получается ли ожидаемый результат в конкретной части скрипта. Если да-переходите дальше, если нет - исправляете

"Вижу в нем ошибку" - какую именно-не считаете ли логичным, что нужно было ее привести здесь?

[Евгений]

$input = Read-Host 'Enter a name' Вы сюда заносите имена пользователей, тут только одно имя или несколько? Если несколько, то есть ли у них какой либо разделитель? Мне кажется тут получится тип данных String и все имена будут восприниматься как одна строка. Самое оптимальное, как мне кажется, это создать CSV/текстовый файл, со списком имен пользователей, где каждое имя будет на новой строке. После этого подгружать эти имена в скрипт как массив и делать Foreach не по количеству указанному вами вручную, а по всем элементам загруженного массива пользователей где будет по отдельности браться каждое имя.

[Knorke_02]

@BeatHazard, Alexey Dmitriev, Вот проблема в том, чтобы вводить пользователей прям в консоли powershell обычно их не больше 6 бывает.

[Knorke_02]

Он успевает создать 1 пользователя, но потом в цикл начинает перебирать одно и тоже имя

[Евгений]

Так, смотрите, вы указали создать 2 записи с одним именем. В AD не может быть две записи с одним и тем же именем, о чем вам консоль и говорит в самой первой строке ошибки.

[Knorke_02]

Евгений, Да я понимаю это, но как это решить?

[Евгений]

У вас есть Телеграмм? Найдите меня по имени @Top_EV я вас проконсультирую голосом, переписываться не продуктивно =)

Answer 1

[Knorke_02]

Вопрос решен, Спасибо Евгений
Import-Module ActiveDirectory

# Cоздаем OU
New-ADOrganizationalUnit -Name "DemoOffice" -Path “DC=demo,DC=lab”
New-ADOrganizationalUnit -Name "Users" -Path “OU=DemoOffice,DC=demo,DC=lab”
New-ADOrganizationalUnit -Name "Computers" -Path “OU=DemoOffice,DC=demo,DC=lab”

$number = Read-Host "Введите количество пользователей"
$count=1..$number
$users = foreach ($i in $count)
{
Read-Host "Введите имя пользователя номер $i"
}
$pass = Read-Host 'Enter the password'

# Цикл с пользователями
foreach ($user in $users) {
$Username = @{

Name = "$user"
GivenName = "$user"
UserPrincipalName = "$[e-mail]"
Path = "OU=Users,OU=DemoOffice,DC=demo,DC=lab"
ChangePasswordAtLogon = $true
AccountPassword = "$pass" | ConvertTo-SecureString -AsPlainText -Force
Enabled = $true

}

New-ADUser @Username

}

Answer 2

[MaxKozlov]

Заполнение $input из read-host перенесите внутрь цикла.
И, для лучшего понимания, стоит $users переименовать в $user потому что по смыслу это про одного пользователя