Arquitetura de Computadores no script de diálogo

Question

Eu gostaria de criar um programa com um diálogo que mostre minha arquitetura de computador. Mas eu tenho algumas saídas falsas. Este é o meu script:

#!/bin/bash 
# ComputerArchitecture_interactive_dialog: an interactive dialog to see the ComputerArchitecture in a simple way.

DIALOG_CANCEL=1
DIALOG_ESC=255
HEIGHT=0
WIDTH=0

display_result() {
  dialog --title "$1" \
    --no-collapse \
    --msgbox "$result" 0 0
}

while true; do
  exec 3>&1
  selection=$(dialog \
    --backtitle "Computer Architecture list" \
    --title "ComputerArchitectuur" \
    --clear \
    --cancel-label "Exit" \
    --menu "Use [ENTER] to select:" $HEIGHT $WIDTH 4 \
    "1" "Information about Processors and Cores" \
    "2" "Information about RAM-memory" \
    "3" "Information about connected drives and USB-devices" \
    "4" "Inforamtion about the current load" \ 
    2>&1 1>&3)
  exit_status=$?
  exec 3>&-
  case $exit_status in
    $DIALOG_CANCEL)
      clear
      echo "Program stopped."
      exit
      ;;
    $DIALOG_ESC)
      clear
      echo "Program closed." >&2
  exit 1
  ;;
  esac
  case $selection in
    0 )
      clear
      echo "Program stopped."
      ;;
    1 )
      result=$(echo "Processors and Cores"; lscpu)
      display_result "Processors and Cores"
      ;;
    2 )
      result=$(echo "RAM"; dmicode --type 17)
      display_result "RAM"
      ;;
    3 )
     result=$(echo "Connected drives and USB-devices";lsblk \lsusb)
     display_result "Connected drives and USB-devices"
     ;;
   4 )
     result=$(echo "Current load"; top)
     display_result "Current load"
     ;;
 esac
done

e esta é a saída falsa:

Error: Expected 2 arguments, found only 1. 
Use --help to list options. 

    

Answer 1

Você precisa citar duas vezes suas subestações de processo. Todos eles. Você também precisa citar duas vezes suas variáveis quando usá-las (novamente, todas elas - $ selection, $ HEIGHT, $ WIDTH, $ DIALOG_CANCEL, $ DIALOG_ESC e qualquer outra variável que você usar).

por exemplo. não faça isso:

result=$(echo "Processors and Cores"; lscpu)

faça isso:

result="$(echo "Processors and Cores"; lscpu)"

E não faça:

case $selection in

faça isso:

case "$selection" in

Melhor ainda, reescreva sua função display_result para que ela não dependa de uma variável global ( $result ).

por exemplo. algo como:

display_result() {
  # This version of display_result takes multiple args.
  # The first is the title.  The rest are displayed in the 
  # message box, with a newline between each arg.
  # To insert a blank line use an empty string '' between any two args.
  title="$1" ; shift

  dialog --title "$title" \
    --no-collapse \
    --msgbox "$(printf "%s\n" "$@")" 0 0
}

Em seguida, nas suas declarações de caso, você o usaria assim:

...
case "$selection" in
  1) display_result 'Processors and Cores' "$(lscpu)" ;;
  2) display_result 'RAM' "$(dmicode --type 17)" ;;
  ...
esac