Quite a unwrap_or_else _ + panic autant expect #3

New issue

Open

opened 2022-01-13 14:45:56 +01:00 by darnuria · 2 comments

darnuria commented

2022-01-13 14:45:56 +01:00

(Migrated from code.up8.edu)

Quand tu fait:

unwrap_or_else(|_| {
        // Récupération du salon
        panic!(
            "{}",
            erreur::message_erreur("Salon non trouvé dans le fichier .env")
        )

autant expect t'es dans le main, dans une app tu sais que tu va crash donc bon. tu peux format le msg de expect avec format!.

Quand tu fait: ```rust unwrap_or_else(|_| { // Récupération du salon panic!( "{}", erreur::message_erreur("Salon non trouvé dans le fichier .env") ) ``` autant `expect` t'es dans le main, dans une app tu sais que tu va crash donc bon. tu peux format le msg de expect avec `format!`.

Anri commented

2022-01-13 16:18:23 +01:00

(Migrated from code.up8.edu)

oui, c'est ce que je faisais au début mais clippy était pas content

oui, c'est ce que je faisais [au début](https://code.up8.edu/Anri/matter_bot/-/commit/4bbe2996afe33ee0213bbfd72127d3d918a09390) mais clippy était pas content

darnuria commented

2022-01-28 11:52:02 +01:00

(Migrated from code.up8.edu)

C'est un classique de débutant et doonc normal, on veux contenter le compilo/checker et on fait pire que prévu.
Ici l'idéal ça serais retourner un Result même dans le main et avec un peu de magie on a une fonction qui attrape les crash et decide quoi faire ;)

C'est un classique de débutant et doonc normal, on veux contenter le compilo/checker et on fait pire que prévu. Ici l'idéal ça serais retourner un `Result` même dans le `main` et avec un peu de magie on a une fonction qui attrape les crash et decide quoi faire ;)

This repo is archived. You cannot comment on issues.