Início / ubuntu / Perguntas / 1520516
Accepted
Dylan Coulbeck
Asked: 2024-07-14 14:54:21 +0800 CST

Como posso condensar meu script bash para procurar 42 extensões em um diretório

  • 772

Como a pergunta indica, estou lutando para condensar meu script no mínimo de linhas de código possível, em vez de declarar várias ifinstruções.

O objetivo disso é criar um script de automação no audacity para normalizar o áudio da minha biblioteca de música. A detecção de janelas e outras coisas funcionam, só estou tentando descobrir como posso condensá-la para rodar com menos código.

Agradeço antecipadamente a ajuda.

#!/bin/bash

media=0
loop=0
cd "/home/satan/Music/.processing/"
while [[ $media -eq 0 ]]
  do
    if [[ "$media" -gt '0' ]]
    then
      break
    else
      cd *
    fi

    aa=`ls -1 *.aa 2>/dev/null | wc -l`
    if [ $aa != 0 ]
    then
      media=$((aa+media))
      echo $aa "aa files found in"
      pwd
      echo
      ls -1
      echo
    fi

    aac=`ls -1 *.aac 2>/dev/null | wc -l`
    if [ $aac != 0 ]
    then
      media=$((aac+media))
      echo $aac "aac files found in"
      pwd
      echo
      ls -1
      echo
    fi

    aax=`ls -1 *.aax 2>/dev/null | wc -l`
    if [ $aax != 0 ]
    then
      media=$((aax+media))
      echo $aax "aax files found in"
      pwd
      echo
      ls -1
      echo
    fi

    act=`ls -1 *.act 2>/dev/null | wc -l`
    if [ $act != 0 ]
    then
      media=$((act+media))
      echo $act "act files found in"
      pwd
      echo
      ls -1
      echo
    fi

    aiff=`ls -1 *.aiff 2>/dev/null | wc -l`
    if [ $aiff != 0 ]
    then
      media=$((aiff+media))
      echo $aiff "aiff files found in"
      pwd
      echo
      ls -1
      echo
    fi

    alac=`ls -1 *.alac 2>/dev/null | wc -l`
    if [ $alac != 0 ]
    then
      media=$((alac+media))
      echo $alac "alac files found in"
      pwd
      echo
      ls -1
      echo
    fi

    ape=`ls -1 *.ape 2>/dev/null | wc -l`
    if [ $ape != 0 ]
    then
      media=$((ape+media))
      echo $ape "ape files found in"
      pwd
      echo
      ls -1
      echo
    fi

    au=`ls -1 *.au 2>/dev/null | wc -l`
    if [ $au != 0 ]
    then
      media=$((au+media))
      echo $au "au files found in"
      pwd
      echo
      ls -1
      echo
    fi

    dss=`ls -1 *.dss 2>/dev/null | wc -l`
    if [ $dss != 0 ]
    then
      media=$((dss+media))
      echo $dss "dss files found in"
      pwd
      echo
      ls -1
      echo
    fi

    dvf=`ls -1 *.dvf 2>/dev/null | wc -l`
    if [ $dvf != 0 ]
    then
      media=$((dvf+media))
      echo $dvf "dvf files found in"
      pwd
      echo
      ls -1
      echo
    fi

    flac=`ls -1 *.flac 2>/dev/null | wc -l`
    if [ $flac != 0 ]
    then
      media=$((flac+media))
      echo $flac "flac files found in"
      pwd
      echo
      ls -1
      echo
    fi

    gsm=`ls -1 *.gsm 2>/dev/null | wc -l`
    if [ $gsm != 0 ]
    then
      media=$((gsm+media))
      echo $gsm "gsm files found in"
      pwd
      echo
      ls -1
      echo
    fi

    iklax=`ls -1 *.iklax 2>/dev/null | wc -l`
    if [ $iklax != 0 ]
    then
      media=$((iklax+media))
      echo $iklax "iklax files found in"
      pwd
      echo
      ls -1
      echo
    fi

    ivs=`ls -1 *.ivs 2>/dev/null | wc -l`
    if [ $ivs != 0 ]
    then
      media=$((ivs+media))
      echo $ivs "ivs files found in"
      pwd
      echo
      ls -1
      echo
    fi

    m4a=`ls -1 *.m4a 2>/dev/null | wc -l`
    if [ $m4a != 0 ]
    then
      media=$((m4a+media))
      echo $m4a "m4a files found in"
      pwd
      echo
      ls -1
      echo
    fi

    m4b=`ls -1 *.m4b 2>/dev/null | wc -l`
    if [ $m4b != 0 ]
    then
      media=$((m4b+media))
      echo $m4b "m4b files found in"
      pwd
      echo
      ls -1
      echo
    fi

    m4p=`ls -1 *.m4p 2>/dev/null | wc -l`
    if [ $m4p != 0 ]
    then
      media=$((m4p+media))
      echo $m4p "m4p files found in"
      pwd
      echo
      ls -1
      echo
    fi

    mmf=`ls -1 *.mmf 2>/dev/null | wc -l`
    if [ $mmf != 0 ]
    then
      media=$((mmf+media))
      echo $mmf "mmf files found in"
      pwd
      echo
      ls -1
      echo
    fi

    movpkg=`ls -1 *.movpkg 2>/dev/null | wc -l`
    if [ $movpkg != 0 ]
    then
      media=$((movpkg+media))
      echo $movpkg "movpkg files found in"
      pwd
      echo
      ls -1
      echo
    fi

    mpc=`ls -1 *.mpc 2>/dev/null | wc -l`
    if [ $mpc != 0 ]
    then
      media=$((mpc+media))
      echo $mpc "mpc files found in"
      pwd
      echo
      ls -1
      echo
    fi

    mp3=`ls -1 *.mp3 2>/dev/null | wc -l`
    if [ $mp3 != 0 ]
    then
      media=$((mp3+media))
      echo $mp3 "mp3 files found in"
      pwd
      echo
      ls -1
      echo
    fi

    msv=`ls -1 *.msv 2>/dev/null | wc -l`
    if [ $msv != 0 ]
    then
      media=$((msv+media))
      echo $msv "act files found in"
      pwd
      echo
      ls -1
      echo
    fi

    nmf=`ls -1 *.nmf 2>/dev/null | wc -l`
    if [ $nmf != 0 ]
    then
      media=$((nmf+media))
      echo $nmf "nmf files found in"
      pwd
      echo
      ls -1
      echo
    fi

    ogg=`ls -1 *.ogg 2>/dev/null | wc -l`
    if [ $ogg != 0 ]
    then
      media=$((ogg+media))
      echo $ogg "ogg files found in"
      pwd
      echo
      ls -1
      echo
    fi

    oga=`ls -1 *.oga 2>/dev/null | wc -l`
    if [ $oga != 0 ]
    then
      media=$((oga+media))
      echo $oga "oga files found in"
      pwd
      echo
      ls -1
      echo
    fi

    mogg=`ls -1 *.mogg 2>/dev/null | wc -l`
    if [ $mogg != 0 ]
    then
      media=$((mogg+media))
      echo $mogg "mogg files found in"
      pwd
      echo
      ls -1
      echo
    fi

    opus=`ls -1 *.opus 2>/dev/null | wc -l`
    if [ $opus != 0 ]
    then
      media=$((opus+media))
      echo $opus "opus files found in"
      pwd
      echo
      ls -1
      echo
    fi

    ra=`ls -1 *.ra 2>/dev/null | wc -l`
    if [ $ra != 0 ]
    then
      media=$((ra+media))
      echo $ra "ra files found in"
      pwd
      echo
      ls -1
      echo
    fi

    rm=`ls -1 *.rm 2>/dev/null | wc -l`
    if [ $rm != 0 ]
    then
      media=$((rm+media))
      echo $rm "rm files found in"
      pwd
      echo
      ls -1
      echo
    fi

    raw=`ls -1 *.raw 2>/dev/null | wc -l`
    if [ $raw != 0 ]
    then
      media=$((raw+media))
      echo $raw "raw files found in"
      pwd
      echo
      ls -1
      echo
    fi

    rt64=`ls -1 *.rt64 2>/dev/null | wc -l`
    if [ $rt64 != 0 ]
    then
      media=$((rt64+media))
      echo $rt64 "rt64 files found in"
      pwd
      echo
      ls -1
      echo
    fi

    sln=`ls -1 *.sln 2>/dev/null | wc -l`
    if [ $sln != 0 ]
    then
      media=$((sln+media))
      echo $sln "sln files found in"
      pwd
      echo
      ls -1
      echo
    fi

    tta=`ls -1 *.tta 2>/dev/null | wc -l`
    if [ $tta != 0 ]
    then
      media=$((tta+media))
      echo $tta "tta files found in"
      pwd
      echo
      ls -1
      echo
    fi

    voc=`ls -1 *.voc 2>/dev/null | wc -l`
    if [ $voc != 0 ]
    then
      media=$((voc+media))
      echo $voc "voc files found in"
      pwd
      echo
      ls -1
      echo
    fi

    vox=`ls -1 *.vox 2>/dev/null | wc -l`
    if [ $vox != 0 ]
    then
      media=$((vox+media))
      echo $vox "vox files found in"
      pwd
      echo
      ls -1
      echo
    fi

    wav=`ls -1 *.wav 2>/dev/null | wc -l`
    if [ $wav != 0 ]
    then
      media=$((wav+media))
      echo $wav "wav files found in"
      pwd
      echo
      ls -1
      echo
    fi

    wma=`ls -1 *.wma 2>/dev/null | wc -l`
    if [ $wma != 0 ]
    then
      media=$((wma+media))
      echo $wma "wma files found in"
      pwd
      echo
      ls -1
      echo
    fi

    webm=`ls -1 *.webm 2>/dev/null | wc -l`
    if [ $webm != 0 ]
    then
      media=$((webm+media))
      echo $webm "webm files found in"
      pwd
      echo
      ls -1
      echo
    fi

    cda=`ls -1 *.cda 2>/dev/null | wc -l`
    if [ $cda != 0 ]
    then
      media=$((cda+media))
      echo $cda "cda files found in"
      pwd
      echo
      ls -1
      echo
    fi

    threegp=`ls -1 *.3gp 2>/dev/null | wc -l`
    if [ $threegp != 0 ]
    then
      media=$((threegp+media))
      echo $threegp "3gp files found in"
      pwd
      echo
      ls -1
      echo
    fi

    eightsvx=`ls -1 *.8svx 2>/dev/null | wc -l`
    if [ $eightsvx != 0 ]
    then
      media=$((eightsvx+media))
      echo $eightsvx "8svx files found in"
      pwd
      echo
      ls -1
      echo
    fi
done
nohup pcmanfm >/dev/null 2>&1 &
bash

4 respostas

  1. Não consegui descobrir o que você está tentando fazer com suas variáveis loop​​and mediaou the cd *mas, caso contrário, parece ser isso que você está tentando fazer:

    #!/usr/bin/env bash

shopt -s nullglob

media=0
loop=0
cd '/home/whatever/' || exit 1

sfxs=( aa aac aax ... 8svx )
for sfx in "${sfxs[@]}"; do
    files=( *."$sfx" )
    numFiles="${#files[@]}"
    if (( numFiles != 0 )); then
        media=$(( numFiles + media ))
        printf '%d %s files found in %s\n' "$numFiles" "$sfx" "$PWD"
        printf '\n'
        printf '%s\n' "${files[@]}"
        printf '\n'
    fi
done

nohup pcmanfm >/dev/null 2>&1 &

    Obviamente, substitua ...pelo restante dos sufixos de seu interesse, MAS talvez você prefira um script que use apenas os sufixos existentes em seu diretório, em vez de codificar uma lista em seu script, por exemplo, usando GNU sed e GNU sort para -z:

    #!/usr/bin/env bash

shopt -s nullglob

media=0
loop=0
cd '/home/whatever/' || exit 1

sfxs=( $(printf '%s\0' *.* | sed -z 's/.*\.//' | sort -zu | tr '\0' '\n') )

for sfx in "${sfxs[@]}"; do
    files=( *."$sfx" )
    numFiles="${#files[@]}"
    media=$(( numFiles + media ))
    printf '%d %s files found in %s\n' "$numFiles" "$sfx" "$PWD"
    printf '\n'
    printf '%s\n' "${files[@]}"
    printf '\n'
done

nohup pcmanfm >/dev/null 2>&1 &

    Ambos os itens acima não foram testados.

    • 7
  2. Best Answer

    O bash for loop pode lidar com isso.

    suffixes="aa aac aax ..."
for suffix in $suffixes
do
    this=`ls -1 *.$suffix 2>/dev/null | wc -l`
    if [ $this != 0 ]
    then
      media=$((this+media))
      echo $this " files found in"
      pwd
      echo
      ls -1
      echo
    fi
done
    • 1

  3. Se você não se importa em mudar para o shell Z ( zsh), você pode fazer isso de forma muito mais compacta.

    Por exemplo, dada uma matriz files=( *.*(N) )de todos os arquivos não ocultos com "extensões de ponto" no diretório atual, você pode obter uma matriz de suas extensões exclusivas como ${(u)files:e}e pode obter uma matriz de todos os nomes de arquivos que correspondem a uma determinada extensão .extusando ${(M)files:#*.ext}. Juntando tudo e adicionando a #flag para obter o número de elementos em cada resultado:

    #!/usr/bin/zsh

files=( *.*(N) )

for ext ( ${(u)files:e} ) 
  printf '%d %s files found in %s\n' ${#${(M)files:#*.$ext}} $ext $PWD

    Se preferir, você pode colocar o loop em uma função anônima e chamá-lo diretamente na expansão glob:

    (){ for ext ( ${(u)@:e} ) 
      printf '%d %s files found in %s\n' ${#${(M)@:#*.$ext}} $ext $PWD
  } *.*(N)
    • 1

  4. DR: Use a refatoração da função de extração para reduzir código duplicado, como suas 41 instruções if. Ou use a expansão de array do Bash para contar todos os tipos de arquivo como uma linha.

    Cheiro de código e refatoração

    Você identificou 41 vezes um bloco de código muito semelhante. Chamamos isso de cheiro de código: código duplicado .

    Este é o bloco de código que está duplicado (a única diferença específica é a extensão do arquivo):

        aa=`ls -1 *.aa 2>/dev/null | wc -l`
    if [ $aa != 0 ]
    then
      media=$((aa+media))
      echo $aa "aa files found in"
      pwd
      echo
      ls -1
      echo
    fi

    Duplicação de código e função de extração

    A refatoração recomendada para duplicação de código é extract-function .

    Vamos escrever uma função que receba a extensão do arquivo de mídia como aa(para .aaarquivos) como parâmetro e retorne o resultado (ou seja, a contagem de arquivos para esta extensão):

    count_files_with_extension() {
    # returns the file-count as integer
    # use with file-extension as argument like:
    # num = count_files_with_extension("aa")
    extension = $1
    file_count = `ls -1 *.$extension 2>/dev/null | wc -l`
    if [ $file_count != 0 ]
    then
      echo "$file_count .$extension files found in $(pwd)"
    fi
    return file_count
}

    Observação: para condensar a saída, removi a saída da listagem de arquivos echo;ls -1;echoe incorporei o arquivo pwd.

    A acumulação de contagem anterior media=$((aa+media))pode ser obtida adicionando o valor de retorno:

    media_count = $((media_count + count_files_with_extension("aa")))

    Laço

    A seguir, reutilize esta função em um loop conforme já respondido por rfm :

    suffixes="aa aac aax ..."  # all 41 extensions specified without dot, separated by space
for suffix in $suffixes
do
  media_count = $((media_count + count_files_with_extension("$suffix")))
done

    Veja também: Fazer um loop por uma matriz de strings no Bash? - Estouro de pilha .

    Script Completo e Simplificado

    #!/bin/bash
count_files_with_extension() {
    # returns the file-count as integer
    # use with file-extension as argument like:
    # num = count_files_with_extension("aa")
    extension = $1
    file_count = `ls -1 *.$extension 2>/dev/null | wc -l`
    if [ $file_count != 0 ]
    then
      echo "$file_count '.$extension' files found in $(pwd)"
    fi
    return file_count
}

cd "/home/satan/Music/.processing/"
suffixes = "aa aac aax ..."  # all 41 extensions specified without dot, separated by space

media_count = 0
for suffix in $suffixes
do
  media_count = $((media_count + count_files_with_extension("$suffix")))
done

echo "Total media files found: $media_count"

    Alternativa: expansão do array do Bash para lscontar arquivos

    Usually you can use brace-expansion to specify ranges that are expanded by Bash. For example ls *.{mp3,aac} gets expanded to a sequence of 2 commands executed in order: ls *.mp3; ls *.aac.

    You also can use parentheses to assign arrays like suffixes=(mp3 aac) and without braces expand these array elements in a command like ls *.$suffixes.

    Together we can simplify the count of files by all given file-extensions to:

    suffixes=(aa aac aax ...)  # all 41 extensions specified as array in parentheses, separated by space

media_count = `ls *.$suffixes 2>/dev/null | wc -l`
echo "Total media files found: $media_count"

    This prints the total count of all files found by given suffixes.

    References

    1. Refactoring Guru, Code Smells: Duplicate Code
    2. Fowler, Refactoring: Extract Function
    • 0

relate perguntas