From 4bbe2996afe33ee0213bbfd72127d3d918a09390 Mon Sep 17 00:00:00 2001
From: Mylloon <kennel.anri@tutanota.com>
Date: Mon, 3 Jan 2022 10:49:11 +0100
Subject: [PATCH] apply clippy reccomendations

---
 src/main.rs       | 29 ++++++++++++++++++-----------
 src/mattermost.rs |  2 +-
 2 files changed, 19 insertions(+), 12 deletions(-)

diff --git a/src/main.rs b/src/main.rs
index 98ce77c..9996cfe 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -5,7 +5,12 @@ mod mattermost;
 
 #[tokio::main]
 async fn main() {
-    dotenv().expect(&message_erreur("Le fichier .env n'a pas été trouvé...")); // vérification si fichier .env trouvé
+    dotenv().unwrap_or_else(|_| {
+        panic!(
+            "{}",
+            message_erreur("Le fichier .env n'a pas été trouvé ou invalide...")
+        )
+    }); // vérification si fichier .env trouvé
 
     // Initialisation des variables token, pseudo et mot de passe
     let mut token = "".to_string();
@@ -14,12 +19,9 @@ async fn main() {
 
     // Récupération du token
     let mut token_recuperer = false;
-    match env::var("TOKEN") {
-        Ok(string) => {
-            token = string;
-            token_recuperer = true;
-        }
-        _ => (),
+    if let Ok(string) = env::var("TOKEN") {
+        token = string;
+        token_recuperer = true;
     }
 
     // Récupération du pseudo
@@ -61,16 +63,21 @@ async fn main() {
             }
         }
     }
-    let mut url = env::var("URL").expect(&message_erreur("URL non trouvé dans le fichier .env")); // récupération de l'url
-    if url.chars().last().unwrap() == '/' {
+    let mut url = env::var("URL")
+        .unwrap_or_else(|_| panic!("{}", message_erreur("URL non trouvé dans le fichier .env"))); // récupération de l'url
+    if url.ends_with('/') {
         // si l'url finit par un '/'
         url = url[0..url.len() - 1].to_string(); // on retire le /
     }
 
     let api = mattermost::connexion(&token, &pseudo, &mot_de_passe, &url).await; // connexion à mattermost
 
-    let equipe =
-        env::var("EQUIPE").expect(&message_erreur("Équipe non trouvé dans le fichier .env")); // récupération de l'équipe
+    let equipe = env::var("EQUIPE").unwrap_or_else(|_| {
+        panic!(
+            "{}",
+            message_erreur("Équipe non trouvé dans le fichier .env")
+        )
+    }); // récupération de l'équipe
     mattermost::test(&api, &equipe).await;
 }
 
diff --git a/src/mattermost.rs b/src/mattermost.rs
index dfad5c6..47aa5ad 100644
--- a/src/mattermost.rs
+++ b/src/mattermost.rs
@@ -8,7 +8,7 @@ pub async fn connexion(
     url: &str,
 ) -> mattermost_api::client::Mattermost {
     let auth;
-    if token.len() > 0 {
+    if !token.is_empty() {
         auth = AuthenticationData::from_access_token(token);
         println!("Login via token");
     } else {