CMD/BAT

alpap · Конфигурация компьютера

Nilf, чем больше вы будете усложнять код и себе работу тем больше у вас будет проблем и непонимания
зачем вот это

Цитата Nilf:

echo Введите имя пользователя
set /p username=
echo Введите пароль
set /p password=
echo Введите имя нового пользователя
set /p newuser=
echo Введите пароль нового пользователя
set /p newpass= »

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

Elven · Отправлено: **11:12, 25-06-2021** | #3

Ядрёны поммидоры, а зачем goto? И зачем перебирать файл с ip для выяснения количества оных, если все равно используются все?
Особо доставило вот это вот:

Код:

for /f "skip=%NumStr% - 1" %%A IN ("IP.txt") do (set "ipinfile=%%~A"&&goto:M3)
:M3
set /a NumStr = NumStr - 1

Т.е. выяснить количество IP, перебирать файл с ними каждый раз ПОЛНОСТЬЮ но брать только одно значение и уменьшать в следующий раз количество пропускаемых строк, а последующие итерации обламываются безусловным переходом.
Это я, извините, на эмоциях написал, но оно - правда. Теперь собственно к делу, из скрипта удаляем к лешему начиная с rem количество переменных (ip) считывается из файла
до pause, оставляем только это:

Код:

for /f %%A IN ("IP.txt") do (
set "ipinfile=%%~A"
wmic /user:"%username%" /password:"%password%"/node:"%ipinfile%" process call create "cmd /c net user %newuser% /add"
wmic /user:"%username%" /password:"%password%"/node:"%ipinfile%" process call create "cmd /c net user %newuser% %newpass%"
wmic /user:"%username%" /password:"%password%"/node:"%ipinfile%" process call create "cmd /c net localgroup Администраторы %newuser% /add"
wmic /user:"%username%" /password:"%password%"/node:"%ipinfile%" process call create "cmd /c net localgroup Administrators %newuser% /add"
)

ну и комментарии по желанию можно оставить.
Протестить негде, но вроде как если всё остальное в порядке - должно работать.

еще один момент, вот это

Код:

set "ipinfile=%%~A"

тоже является лишним, т.к. можно %%~A подставлять прямо в строчки с wmic, но это уже на усмотрение ТСа.

alpap, согласен. Судя по тому что ТС пытается сделать - у него одноранговая сеть с одинаковой учеткой с правами администратора на хостах, но я откровенно устал писать что запуск подобной, эмммм... автоматизации в одноранговой сети штука довольно спорная и геморная, если так уж хочется выстрелить себе в ногу - кто я такой чтобы мешать? В конце концов сами мы тоже когда-то по граблям ходили, однако помочь с мягкими насадками на рукоять - в наших силах.

Ну и насчет насадок на грабли: есть такая хорошая и полезная штука как ansible - с ней возня в одноранговой сети (если уж нет возможности AD) становится чуть менее болезненной.

alpap · Конфигурация компьютера

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

Код:

192.168.31.2,Vasya,12345
192.168.31.5,Petya,67890
...

Nilf · Отправлено: **11:27, 25-06-2021** | #5

Ок. Понял.

Пример файла IP.txt:

192.168.31.1
192.168.31.2
192.168.31.3
192.168.31.4
....

Т.е. происходит банальное перечисление IP адресов на которые нужно отправить команду о добавлении нового пользователя. одна строка - один IP. Идея была такова что считается количество строк из файла, это значение подставляется в цикл, и по итог когда это значение придет к нулю (т.е. закончиться перебор всех строк) исполнение файла заканчивается.

Цитата:

Nilf, чем больше вы будете усложнять код и себе работу тем больше у вас будет проблем и непонимания
зачем вот это
Цитата Nilf:
echo Введите имя пользователя
set /p username=
echo Введите пароль
set /p password=
echo Введите имя нового пользователя
set /p newuser=
echo Введите пароль нового пользователя
set /p newpass= »

Касаемо этого, мне необходимо задавать логин и пароль нового пользователя, плюс логин и пароль пользователя-администратора, под которым будет запускаться скрипт на удаленном ПК.

Nilf · Отправлено: **12:03, 25-06-2021** | #6

Цитата Elven:

ansible »

Спасибо за разъяснение, будем попробовать. А по поводу того чтобы юзать сторонний софт, к сожалению не получиться, есть такое ограничений что только встроенными в ОС средствами.

Цитата:

непонятно видимо?
Если собираетесь считывать информацию из файла, то только там и должна быть вся информация, причем в таком примерно виде
Код:Выделить весь код
192.168.31.2,Vasya,12345
192.168.31.5,Petya,67890
...

У меня там только IP было.

alpap · Конфигурация компьютера

Цитата Nilf:

закончиться перебор всех строк »

не нужно этого, цикл и так прекратится

Цитата Nilf:

перечисление IP адресов на которые нужно отправить команду о добавлении нового пользователя. одна строка - один IP »

ха, а если на одну машину надо несколько пользователей, я не просто так предложил свой вариант txt, например так может ведь быть?

Код:

192.168.31.1,Vasya,1234
192.168.31.1,Petya,5678
192.168.31.1,Olya,4321
192.168.31.2,Oleg,A1B1C1

но это все подойдет только если пользователи создаются одинаковые
иначе надо составлять файл примерно так:

Код:

Vasya,1234,192.168.31.1,net user UserNew1
Petya,5678,192.168.31.1,net user UserNew2 3476
Olya,4321,192.168.31.1,net localgroup Администраторы UserN
Oleg,A1B1C1,192.168.31.2,net localgroup Administrators NewUser2

тогда код будет выглядеть довольно просто:

Код:

@echo off
for /f "usebackq tokens=1-3* delims=," %%a in ("IP.txt") do (
  wmic /user:"%%~a" /password:"%%~b" /node:"%%~c" process call create "cmd /c %%~d /add"
)
pause

Nilf · Отправлено: **13:17, 25-06-2021** | #8

Цитата alpap:

Цитата Nilf:
перечисление IP адресов на которые нужно отправить команду о добавлении нового пользователя. одна строка - один IP »
ха, а если на одну машину надо несколько пользователей, я не просто так предложил свой вариант txt, например так может ведь быть? »

Да, действительно, так куда удобнее получиться в таком случае, спасибо!

Nilf · Отправлено: **15:14, 25-06-2021** | #9

Цитата Elven:

еперь собственно к делу, из скрипта удаляем к лешему начиная с rem количество переменных (ip) считывается из файла
до pause, оставляем только это:
Код:
for /f %%A IN ("IP.txt") do (
set "ipinfile=%%~A"
wmic /user:"%username%" /password:"%password%"/node:"%ipinfile%" process call create "cmd /c net user %newuser% /add"
wmic /user:"%username%" /password:"%password%"/node:"%ipinfile%" process call create "cmd /c net user %newuser% %newpass%"
wmic /user:"%username%" /password:"%password%"/node:"%ipinfile%" process call create "cmd /c net localgroup Администраторы %newuser% /add"
wmic /user:"%username%" /password:"%password%"/node:"%ipinfile%" process call create "cmd /c net localgroup Administrators %newuser% /add"
)
ну и комментарии по желанию можно оставить.
Протестить негде, но вроде как если всё остальное в порядке - должно работать. »

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

Elven · Отправлено: **15:24, 25-06-2021** | #10

у меня нет под рукой cmd чтобы проверить наверняка. больше всего похоже на какие-то грабли с переменной/переменными.
для проверки значения можно вывести все используемые переменные, например так:

Код:

echo ipinfile = %ipinfile%
echo username = %username%

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

Код:

set ipinfile=%%~A