Make options a dictionary and add validation

[nimcoon.git] / src / nimcoon.nim

diff --git a/src/nimcoon.nim b/src/nimcoon.nim
index 8b1ba591f166a0545053cc420ecf12149c5847ab..2143c12ad6c9d990e15e379b65dd701fc5e32098 100644 (file)
--- a/src/nimcoon.nim
+++ b/src/nimcoon.nim
@@ -1,17 +1,18 @@
 import
   parseopt,
   std/[terminal],
-  strutils
+  strformat,
+  strutils,
+  tables
 
 import config
 import lib
 
-proc parseOptions(): CommandLineOptions =
+
+proc parseArguments(): CommandLineOptions =
   var
     searchQuery = ""
-    musicOnly = false
-    feelingLucky = false
-    fullScreen = false
+    options = to_table({"musicOnly": false, "feelingLucky": false, "fullScreen": false, "download": false})
 
   for kind, key, value in getopt():
     case kind
@@ -19,19 +20,32 @@ proc parseOptions(): CommandLineOptions =
       searchQuery = key
     of cmdShortOption, cmdLongOption:
       case key
-      of "m", "music": musicOnly = true
-      of "l", "lucky": feelingLucky = true
-      of "f", "full-screen": fullScreen = true
-    of cmdEnd:
-      discard
+      of "m", "music": options["musicOnly"] = true
+      of "l", "lucky": options["feelingLucky"] = true
+      of "f", "full-screen": options["fullScreen"] = true
+      of "d", "download": options["download"] = true
+    of cmdEnd: discard
+
+  return (searchQuery, options)
 
-  return (searchQuery, musicOnly, feelingLucky, fullScreen)
+
+proc isValidOptions*(options: Options): bool =
+  # Check for invalid combinations of options
+  var invalidCombinations = [("musicOnly", "fullScreen")]
+  for combination in invalidCombinations:
+    if options[combination[0]] and options[combination[1]]:
+     stderr.writeLine fmt"Incompatible options provided: {combination[0]} and {combination[1]}"
+     return false
+  return true
 
 
 proc main() =
   let
     player = selectMediaPlayer()
-    (searchQuery, musicOnly, feelingLucky, fullScreen) = parseOptions()
+    (searchQuery, options) = parseArguments()
+
+  if(not isValidOptions(options)):
+    quit(1)
 
   if searchQuery.startswith("https:") or searchQuery.startswith("magnet:"):
     directPlay(searchQuery, player)
@@ -40,7 +54,7 @@ proc main() =
   let searchResults = extractTitlesAndUrls(getYoutubePage(searchQuery))
 
   proc getUserInput(): string =
-    if feelingLucky: "0"
+    if options["feelingLucky"]: "0"
     else:
       presentVideoOptions(searchResults[..(limit-1)])
       stdout.styledWrite(fgYellow, "Choose video number: ")
@@ -49,8 +63,8 @@ proc main() =
   # This is a pure function with no side effects
   func buildArgs(number: int): seq[string] =
     var args = @[searchResults[number].url]
-    if musicOnly: args.add("--no-video")
-    if fullScreen: args.add("--fullscreen")
+    if options["musicOnly"]: args.add("--no-video")
+    if options["fullScreen"]: args.add("--fullscreen")
     return args
 
   while(true):
@@ -66,7 +80,7 @@ proc main() =
     # Play the video using the preferred/available media player
     let videoNumber = parseInt(userInput)
     play(player, buildArgs(videoNumber), searchResults[videoNumber].title)
-    if feelingLucky:
+    if options["feelingLucky"]:
       break