Início / unix / Perguntas / 450796
Accepted
fauve
Asked: 2018-06-20 23:18:37 +0800 CST

encontrar dentro da função shell

  • 772

Eu tento colocar finddentro da função e pegar um argumento passado para esta função com o seguinte exemplo de trabalho mínimo:

função DO
{
    ls $(find. -type f -name "$@" -exec grep -IHl "TODO" {} \;)
}

Mas, quando executo DO *.tex, recebo “find: os caminhos devem preceder a expressão:”. Mas quando eu faço diretamente:

ls $(find . -type f -name "*.tex" -exec grep -IHl "TODO" {} \;)

então recebo todos os arquivos TeX que contêm "TODO".

Eu tento muitas coisas na DOfunção, como \"$@\", '$@', mudo as aspas, mas o comportamento continua o mesmo.

Então, o que fazer para forçar a encontrar o trabalho dentro da função?

bash alias

2 respostas

  1. Best Answer

    Existem alguns problemas no seu código:

    1. O *.texpadrão será expandido ao chamar a função DO, se corresponder a qualquer nome de arquivo no diretório atual. Você terá que citar o padrão como '*.tex', "*.tex"ou \*.texao chamar a função.

    2. O lsnão é necessário. Você já tem os dois finde greppode relatar os nomes de caminho dos arquivos encontrados.

    3. -name "$@"só funciona corretamente se "$@"contiver um único item. Seria melhor usar -name "$1". Para uma solução que permite vários padrões, veja abaixo.

    A função pode ser escrita

    DO () {
   # Allow for multiple patterns to be passed,
   # construct the appropriate find expression from all passed patterns

   for pattern do
       set -- "$@" '-o' '-name' "$pattern"
       shift
   done

   # There's now a -o too many at the start of "$@", remove it
   shift

   find . -type f '(' "$@" ')' -exec grep -qF 'TODO' {} ';' -print
}

    Chamando esta função como

    DO '*.tex' '*.txt' '*.c'

    vai fazê-lo executar

    find . -type f '(' -name '*.tex' -o -name '*.txt' -o -name '*.c' ')' -exec grep -qF TODO {} ';' -print

    Isso geraria uma lista de nomes de caminho de arquivos com esses sufixos de nome de arquivo, se os arquivos contivessem a string TODO.

    Para usar grepem vez de findimprimir os nomes de caminho encontrados, altere o -exec ... -printbit para -exec grep -lF 'TODO' {} +. Isso será mais eficiente, especialmente se você tiver um grande número de nomes de arquivos correspondentes à(s) expressão(ões) fornecida(s). Em ambos os casos, você definitivamente não precisa usar ls.

    Para permitir que o usuário use

    DO tex txt c

    sua função pode ser alterada para

    DO () {
   # Allow for multiple patterns to be passed,
   # construct the appropriate find expression from all passed patterns

   for suffix do
       set -- "$@" '-o' '-name' "*.$suffix"   # only this line (and the previous) changed
       shift
   done

   # There's now a -o too many at the start of "$@", remove it
   shift

   find . -type f '(' "$@" ')' -exec grep -qF 'TODO' {} ';' -print
}
    • 5

  2. Você precisa citar o parâmetro para a função, caso contrário, o shell o expande e a função obtém uma lista de arquivos, não uma máscara curinga:

    DO "*.tex"

    Você pode usar apenas "$1"em vez de "$@", pois haverá apenas um parâmetro.

    • 0

relate perguntas