Início / unix / Perguntas / 750146
Accepted
Nickotine
Asked: 2023-06-29 14:32:50 +0800 CST

Minhas funções de uso estão implementadas corretamente?

  • 772

Tenho várias funções no meu ~/.bash_profileque uso como comandos e também criei funções de uso para elas.

Gostaria de saber se implementei as funções de uso corretamente, já que as desenvolvi por conta própria.

Exemplo

IFS=$'\n'

usageSliceArr() {
  if [ $# != 3 ]; then
      echo 'name, index 1, index 2'
      return 1
  fi
 }

slicearr() {
  if (usageSliceArr $1 $2 $3); then 
      declare -n name=$1
      declare -i fst=$2 lst=$3
      echo ${name[@]:fst:lst}
  fi
 }
bash

2 respostas

    1. Coloque aspas duplas em suas variáveis ​​ao usá-las. Caso contrário, eles serão interrompidos se você passar valores que contenham espaços em branco. Além disso, não há necessidade desse subshell. Então, em vez de if (usageSliceArr $1 $2 $3); thenvocê deve escreverif usageSliceArr "$1" "$2" "$3"; then
    2. Use -nepara comparar números, !=para strings
    3. Não se esqueça que há https://shellcheck.net/
    4. Coloque aspas duplas em suas variáveis ​​ao usá-las
    5. Coloque a mensagem de uso dentro da função que faz o trabalho, para que o código se torne autodocumentado
    6. Escreva mensagens de erro para stderr em vez de stdout
    7. Considere escrever comandos como scripts de shell que vivem em seu $PATH, em vez de funções disponíveis apenas se o seu ~/.bash_profilefor executado
    8. Eu mencionei que você deve colocar aspas duplas em suas variáveis ​​ao usá-las?

    Aqui está uma abordagem:

    ########################################################################
# slicearr name index count
#
# Return <count> space-separated and tokenised elements from the <name>
# array starting at <index>
#
slicearr() {
    if [ $# -ne 3 ]
    then
        echo 'Usage: name index count' >&2
        exit 1
    fi

    declare -n name=$1
    printf "%s " "${name[@]:$2:$3}"; echo
}

    A relevância da cláusula separada por espaço e tokenizada também se aplica ao seu código original. Considere uma matriz

    a=( one two 'twenty three' )

    O "vinte e três" será extraído como um único elemento, mas retornado como dois valores separados por espaços "vinte" e "três".

    • -1
  2. Best Answer

    Usando algumas boas sugestões de @StéphaneChezalas, vou fazer isso agora:

    IFS=$'\n'

usageSliceArr() {
  if (( $# != 3 )); then
      echo 'name, index 1, index 2'
      return 1
  fi
 }

slicearr() {
  usageSliceArr $@ &&
    declare -n name=$1
    declare -i fst=$2 lst=$3
    echo ${name[@]:fst:lst}
 }

    Lembre-se de que tenho muitas funções, incluindo esta no meu, ~/.bash_profilenão é apenas um script único.

    • -1

relate perguntas