CLI fonctionnelle #12
Conversation
cli/cmd/add.go
Outdated
There was a problem hiding this comment.
il y a moyen de mutualiser l'affichage de l'usage, par exemple, en créant une fonction "usage" et en l'utilisant partout où on en a besoin
attention au fait que le programme au final sera un binaire et pas utilisé à partir du code source : donc le "go run" ne convient pas. Remarque, il y a moyen de récupérer depuis le programme le nom du binaire exécuté (via le shell de mémoire)
cli/cmd/delete.go
Outdated
There was a problem hiding this comment.
cf remarque sur add.go pour une fonction "usage", pour y ajouter tous les usages (ici, la suppression)
cli/cmd/get.go
Outdated
There was a problem hiding this comment.
est ce que les templates de Go ne pourraient pas remplacer la génération de texte brut ?
cli/cmd/help.go
Outdated
There was a problem hiding this comment.
cela pourrait être la fonction "usage" dont j'ai parlé dans mes précédents commentaires
cli/cmd/modify.go
Outdated
| fmt.Println("priority : le niveau de priorité de votre tâche entre 1 et 3, du moins au plus urgent") | ||
| os.Exit(1) | ||
| } | ||
| id, err := strconv.Atoi(os.Args[2]) |
There was a problem hiding this comment.
Il faudrait check l'erreur ici et mutualiser l'affichage de l'usage.
De plus, s'il y a une erreur de priorité, le programme continue tout de même son traitement ?
cli/mytodolist
Outdated
cli/user/user.go
Outdated
| func Login(configFilePath string) (u users.User, err error) { | ||
| configData, err := LoadConfig(configFilePath) | ||
| if err != nil { | ||
| fmt.Println(ERR_CFG, err) |
There was a problem hiding this comment.
C'est curieux d'afficher le message d'erreur à ce niveau, vu que la fonction est censée retourner une erreur. Mieux vaut laisser le soin de gérer l'erreur à l'appelant...
Même remarque dans le reste du code.
cli/user/user.go
Outdated
| } | ||
| var confirmmdp string | ||
| fmt.Print("Entrez votre email: ") | ||
| fmt.Scan(&configData.Email) |
ihm/app.svelte
Outdated
| @@ -23,6 +23,8 @@ | |||
| alert(`${text}réessayez`); | |||
| } else if (statusCode == 401) { | |||
There was a problem hiding this comment.
mieux vaut utiliser les codes http de Go que la valeur numérique
metier/todos/todos.go
Outdated
| @@ -77,8 +77,12 @@ func Get(u users.User) ([]Todo, error) { | |||
| } | |||
|
|
|||
| func Add(text Task, priority int, user users.User) (td Todo, err error) { | |||
| td.Task = text | |||
| if priority < 1 || priority > 3 { | |||
| err = fmt.Errorf("priorité de tâche invalide") | |||
There was a problem hiding this comment.
peut être écrit : return td, fmt.Errorf("blabla")
…rme, code plus idiomatique sur l'ensemble du projet
…rité + binaire mytodolist et install.sh pour copier le binaire sous usr/local/bin et pouvoir l'executer depuis n'importe quel emplacement
…ur pouvoir executer l'app cli depuis n'importe quel emplacement
Ajouter/supprimer/modifier les todos + connexion/inscription de l'utilisateur et stockage de ses info dans un fichier caché config.json