refactored if statements to switch case
This commit is contained in:
		
							
								
								
									
										33
									
								
								pman.sh
									
									
									
									
									
								
							
							
						
						
									
										33
									
								
								pman.sh
									
									
									
									
									
								
							@@ -45,9 +45,11 @@ if [ "$command" = "" ]; then
 | 
			
		||||
fi
 | 
			
		||||
 | 
			
		||||
# Evaluate the command arguments
 | 
			
		||||
if [ "$command_args" = "search" ]; then
 | 
			
		||||
case "$command_args" in
 | 
			
		||||
  "search")
 | 
			
		||||
    command_args="-Ss"
 | 
			
		||||
elif [ "$command_args" = "provides" ]; then
 | 
			
		||||
    ;;
 | 
			
		||||
  "provides")
 | 
			
		||||
    # Update the cache
 | 
			
		||||
    su_needed=true
 | 
			
		||||
    command_args="-Fy"
 | 
			
		||||
@@ -55,36 +57,45 @@ elif [ "$command_args" = "provides" ]; then
 | 
			
		||||
 | 
			
		||||
    su_needed=false
 | 
			
		||||
    command_args="-F"
 | 
			
		||||
elif [ "$command_args" = "installed" ]; then
 | 
			
		||||
    ;;
 | 
			
		||||
  "installed")
 | 
			
		||||
    command_args="-Q"
 | 
			
		||||
elif [ "$command_args" = "install" ]; then
 | 
			
		||||
    ;;
 | 
			
		||||
  "install")
 | 
			
		||||
    command_args="-S"
 | 
			
		||||
    if [ "$command" = "pacman" ]; then
 | 
			
		||||
      su_needed=true
 | 
			
		||||
    fi
 | 
			
		||||
elif [ "$command_args" = "update" ]; then
 | 
			
		||||
    ;;
 | 
			
		||||
  "update")
 | 
			
		||||
    command_args="-Sy"
 | 
			
		||||
    if [ "$command" = "pacman" ]; then
 | 
			
		||||
      su_needed=true
 | 
			
		||||
    fi
 | 
			
		||||
elif [ "$command_args" = "upgradable" ]; then
 | 
			
		||||
    ;;
 | 
			
		||||
  "upgradable")
 | 
			
		||||
    command_args="-Qu"
 | 
			
		||||
elif [ "$command_args" = "upgrade" ]; then
 | 
			
		||||
    ;;
 | 
			
		||||
  "upgrade")
 | 
			
		||||
    command_args="-Syu"
 | 
			
		||||
    if [ "$command" = "pacman" ]; then
 | 
			
		||||
      su_needed=true
 | 
			
		||||
    fi
 | 
			
		||||
elif [ "$command_args" = "remove" ]; then
 | 
			
		||||
    ;;
 | 
			
		||||
  "remove")
 | 
			
		||||
    command_args="-Rs"
 | 
			
		||||
    if [ "$command" = "pacman" ]; then
 | 
			
		||||
      su_needed=true
 | 
			
		||||
    fi
 | 
			
		||||
elif [ "$command_args" = "purge" ]; then
 | 
			
		||||
    ;;
 | 
			
		||||
  "purge")
 | 
			
		||||
    command_args="-Rns"
 | 
			
		||||
    if [ "$command" = "pacman" ]; then
 | 
			
		||||
      su_needed=true
 | 
			
		||||
    fi
 | 
			
		||||
elif [ "$more_arguments" = "true" ]; then
 | 
			
		||||
    ;;
 | 
			
		||||
  *)
 | 
			
		||||
    if [ "$more_arguments" = "true" ]; then
 | 
			
		||||
      echo "Invalid command: \"$command $command_args\""
 | 
			
		||||
      echo
 | 
			
		||||
      echo "Usage: pman <command> [args...]"
 | 
			
		||||
@@ -104,5 +115,7 @@ elif [ "$more_arguments" = "true" ]; then
 | 
			
		||||
      echo "  config     - /etc/pacman.conf | edits pacman configuration"
 | 
			
		||||
      exit 1
 | 
			
		||||
    fi
 | 
			
		||||
    ;;
 | 
			
		||||
esac
 | 
			
		||||
 | 
			
		||||
execute_command "${@}"
 | 
			
		||||
 
 | 
			
		||||
		Reference in New Issue
	
	Block a user