如何在while循环中从grep输出中分配变量

Question

我想为Usernames.txt文件中的每个grep regex行创建一个变量。文本包含这个

mario,visitor
bobmarley,staff
luigi,visitor
hailey,visitor

等等

我希望逗号前的名称作为用户名，访问者作为用户组

#!/bin/bash
sudo addgroup visitor
sudo addgroup staff
filename='Usernames.txt'
while read line; do
username=$(echo $line | grep -o '^[a-z]*[^,]')
group=$(echo $line | grep -o '[^,][a-z]*$')
sudo useradd $username -G $group -p $username
done < $filename

但是输出说没有找到命令用户名。所以我尝试了不太有效的方法

while read line; do
sudo useradd $(echo $line | grep -o '^[a-z]*[^,]') -G $(echo $line | grep -o '[^,][a-z]*$') -p $(echo $line | grep -o '^[a-z]*[^,]')
done < $filename

我希望每个循环的结果像这样

sudo useradd mario -G visitor -p mario

我如何改善这一点?谢谢!

Answer 1

在每次循环迭代中分叉多个进程(例如grep)通常是一个坏主意。此外，简单的单词拆分可以直接在Bash中完成，而不需要任何外部进程。

选项1:

while IFS= read -r line; do
username="${line%,*}"
group="${line#*,}"
echo sudo useradd "$username" -G "$group" -p "$username"
done < "$filename"

选项2:

while IFS=, read -r username group; do
echo sudo useradd "$username" -G "$group" -p "$username"
done < "$filename"

Answer 2

试试这个方法

while read -r name role; do
sudo useradd $name -G $role -p $name
done < <(sed 's/,/ /' $filename)

实际上sed是足够的:

sudo sed 's/(.*),(.*)/useradd 1 -G 2 -p 1/e' $filename