fix: argument parsing with clap; notifies user if file is not .fit

Cargo.toml

@@ -13,6 +13,7 @@ license = "MIT"
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 
 [dependencies]
+clap = { version = "4.5.8", features = ["derive"] }
 fit-rust = "0.1.8"
 geo-types = "0.7.13"
 gpx = "0.10.0"

src/main.rs

@@ -1,3 +1,4 @@
+use clap::Parser;
 use fit_rust::{
     protocol::{message_type::MessageType, value::Value, DataMessage, FitMessage},
     Fit,
@@ -7,6 +8,12 @@ use gpx::{Gpx, GpxVersion, Track, TrackSegment, Waypoint};
 use std::{fs, fs::File, io::BufWriter};
 use time::OffsetDateTime;
 
+#[derive(Parser)]
+#[command(version, about, long_about = None)]
+struct Args {
+    pub files: Vec<String>,
+}
+
 #[derive(Clone, Copy, Default, Debug, PartialEq)]
 struct RecordData {
     /// latitude
@@ -104,10 +111,14 @@ impl From<RecordData> for Waypoint {
 
 fn main() {
     // collecting cli args
-    let args = std::env::args().collect::<Vec<_>>();
+    let args = Args::parse();
 
     let mut handles = vec![];
-    for file in args.iter().skip(1).filter(|f| f.ends_with(".fit")) {
+    for file in args.files.iter() {
+        if !file.ends_with(".fit") {
+            eprintln!("invalid file: {file:?}");
+            continue;
+        }
         let file = file.clone();
         let jh = std::thread::spawn(move || {
             fit2gpx(&file);