Merge pull request #4 from CerebruxCode/develop

Shellcheck review

.github/workflows/main.yml

@@ -1,7 +1,8 @@
 name: CI
 
 # Controls when the action will run. Triggers the workflow on push or pull request
-# events but only for the master branch
+# events but only for the master branch 
+# https://github.com/azohra/shell-linter and https://github.com/marketplace/actions/shell-linter
 on:
   push:
   pull_request:
@@ -16,4 +17,5 @@ jobs:
       - name: Run Shellcheck
         uses: azohra/shell-linter@latest
         with:
-          args: --external-sources
+          args: --external-sources
+          path: "shelldio" 

.github/workflows/reviewdog.yml

@@ -19,7 +19,8 @@ jobs:
         with:
           github_token: ${{ secrets.github_token }}
           reporter: github-check # Change reporter.
-          pattern: "*.sh" # Optional.
+          #path: "shelldio"
+          #pattern: "*.sh" # Optional.
           exclude: "./.git/*" # Optional.
           shellcheck_flags: --external-sources
           #level: info
@@ -36,6 +37,7 @@ jobs:
         with:
           github_token: ${{ secrets.github_token }}
           reporter: github-pr-review 
-          pattern: "*.sh" # Optional.
+          #path: "shelldio"
+          #pattern: "*.sh" # Optional.
           exclude: "./.git/*" # Optional.
-          shellcheck_flags: --external-sources
+          shellcheck_flags: --external-sources

shelldio → shelldio.sh

@@ -16,7 +16,7 @@ do
 terms=0
 trap ' [ $terms = 1 ] || { terms=1; kill -TERM -$$; };  exit' EXIT INT HUP TERM QUIT 
 
-if [ "$#" -eq "0" ]		    #στην περίπτωση που δε δωθεί όρισμα παίρνει το προκαθορισμένο αρχείο
+if [ "$#" -eq "0" ]		    #στην περίπτωση που δε δοθεί όρισμα παίρνει το προκαθορισμένο αρχείο
 	then
 	stations="gr_stations.txt"
 	else
@@ -33,7 +33,7 @@ fi
 
 info() {
 tput civis      -- invisible  # Εξαφάνιση cursor
-echo -ne "| Η ώρα είναι "$(date +"%T")"\n| Ακούτε "$stathmos_name"\n| Πατήστε Q/q για έξοδο ή R/r για επιστροφή στη λίστα σταθμών"
+echo -ne "| Η ώρα είναι '$(date +"%T")'\n | Ακούτε '$stathmos_name'\n| Πατήστε Q/q για έξοδο ή R/r για επιστροφή στη λίστα σταθμών"
 }
 echo "---------------------------------------------------------"
 echo "Shelldio - ακούστε online ραδιόφωνο από το τερματικό"
@@ -46,9 +46,9 @@ echo "---------------------------------------------------------"
 num=0 
 
 while IFS='' read -r line || [[ -n "$line" ]]; do
-    num=$(( $num + 1 ))
-    echo "["$num"]" $line | cut -d "," -f1
-done < $stations
+    num=$(( num + 1 ))
+    echo ["$num"] "$line" | cut -d "," -f1
+done < "$stations"
 echo "---------------------------------------------------------"
 read -rp "Διαλέξτε Σταθμό (Q/q για έξοδο): " input
 
@@ -59,10 +59,10 @@ if [[ $input = "q" ]] || [[ $input = "Q" ]]
 	exit 0
 fi
 
-if [ $input -gt 0 -a $input -le $num ]; #έλεγχος αν το input είναι μέσα στο εύρος της λίστας των σταθμών
+if [ "$input" -gt 0 ] && [ "$input" -le $num ]; #έλεγχος αν το input είναι μέσα στο εύρος της λίστας των σταθμών
 	then
-	stathmos_name=$(cat $stations | head -n$(( $input )) | tail -n1 | cut -d "," -f1)
-	stathmos_url=$(cat $stations | head -n$(( $input )) | tail -n1 | cut -d "," -f2)
+	stathmos_name=$(< "$stations" head -n$(( "$input" )) | tail -n1 | cut -d "," -f1)
+	stathmos_url=$(< "$stations" head -n$(( "$input" )) | tail -n1 | cut -d "," -f2)
 	break
 	else
 	echo "Αριθμός εκτός λίστας"
@@ -71,14 +71,14 @@ if [ $input -gt 0 -a $input -le $num ]; #έλεγχος αν το input είνα
 fi
 done
 
-$player $stathmos_url &> /dev/null &
+$player "$stathmos_url" &> /dev/null &
 while true
 
 do 
 	   clear 
 	   info
 	   sleep 0
-	   read -n1 -t1 input          # Για μικρότερη αναμονή της read
+	   read -r -n1 -t1 input          # Για μικρότερη αναμονή της read
 	   if [[ $input = "q" ]] || [[ $input = "Q" ]] 
    		then
 		clear
@@ -89,7 +89,7 @@ do
 
 	   if [[ $input = "r" ]] || [[ $input = "R" ]]
 		then
-		killall -9 $player &> /dev/null
+		killall -9 "$player" &> /dev/null
 		clear
 		echo "Επιστροφή στη λίστα σταθμών"
 		tput cnorm   -- normal  # Εμφάνιση cursor