Logiczne OR w moim skrypcie

Question

Mój skrypt:

#!/bin/bash 

for file in *.ats; 
do 
    if [[ ("${file}" = THx) || ("${file}" = THy)]] 
    then cp $file /home/milenko/procmt 
    fi 
done 

Pliki w katalogu

262_V01_C00_R000_TEx_BL_128H.ats 
262_V01_C01_R000_TEy_BL_128H.ats 
262_V01_C02_R000_THx_BL_128H.ats 
262_V01_C03_R000_THy_BL_128H.ats 

Chciałem jest skopiowanie plików, które zawierają THX lub THY, ale pliki nie są kopiowane. Dlaczego?

Answer 1

Co powiesz na używanie rozszerzonego globowania przy użyciu extglob? W ten sposób można wykorzystać sam for aby uzyskać wymagane rozszerzenia:

shopt -s extglob 
for file in *TH?(x|y)*.ats; do 
    # do things with "$file" ... 
done 

*TH?(x|y)*.ats rozszerza się do tych plików zawierających <something> + TH + either x or y + <something> + .ats

---

Twój skrypt nie działa, ponieważ masz literówkę w nim:

if [[ ("${file}" = THx) || ("${file}" = THy)]] 
#          ^
#        missing space 

To jest w porządku:

$ d="hi" 
$ [[ ($d == hi) || ($d == ha) ]] && echo "yes" 
yes 

Chociaż nawiasy są zbędne:

$ [[ $d == hi || $d == ha ]] && echo "yes" 
yes 

Answer 2

Można użyć klasę postaci w glob IE *TH[xy]* aby sprawdzić, czy $file zawiera THx lub THy:

for file in *.ats; do 
    if [[ $file == *TH[xy]* ]]; then 
     cp "$file" /home/milenko/procmt 
    fi 
done 

Answer 3

Twoje pytanie określa „pliki, które zawierają THX lub Thy” ... ale kod określa, że ​​plik nazwa jest THx lub THy.

Answer 4

myślę, że można uniknąć pętli w całości tutaj:

cp *TH[xy]*.ats /home/milenko/procmt 

nie ma potrzeby pętli wyników, a następnie zrobić oddzielną porównania ; pojedynczy glob rozwinie się do listy żądanych plików.

Było kilka problemów ze swoim oryginalnym podejściem:

• Po pierwsze, próbują testować dokładnych dopasowań, więc warunek nie byłby prawdziwy.
• Należy również uwzględnić spacje: ]] jest słowem kluczowym w komendzie złożonej [[, więc musi być oddzielnym słowem (tj. Otoczonym spacjami).