Attention c'est String/i64 world #11

New issue

Open

opened 2022-01-13 15:15:12 +01:00 by darnuria · 3 comments

darnuria commented

2022-01-13 15:15:12 +01:00

(Migrated from code.up8.edu)

Dans ton code tu as des String partout c'est un peu dangereux. Limite on préférerais avoir des struct [u8] a qui on donne un type specific (newtype)

Pourquoi c'est dangereux? Les string c'est pour le texte des êtres humains. C'est chiant, ça se partage mal, y'a l'UTF8 a gére et c'est couteux a reallouer.

Souvent on va préférer avoir une structure interne facilement convertissable en String.

pareil pour les i64 pour les dates il existe des lib pour ça genre chrono:

Après bon en l2 sur un bot ça passe. mais au moins faire des newtype pour indiquer que c'est une date ;)

Eventuellement utiliser une/des hashtable|Vec qui contiens toute les string uniques genre les pseudo les channels etc et se balader juste des hash|ID-typé c'est peut-être mieux parceque se balader les &str c'est parfois horrible.

Dans ton code tu as des String partout c'est un peu dangereux. Limite on préférerais avoir des `struct` `[u8]` a qui on donne un type specific (newtype) Pourquoi c'est dangereux? Les string c'est pour le texte des êtres humains. C'est chiant, ça se partage mal, y'a l'UTF8 a gére et c'est couteux a reallouer. Souvent on va préférer avoir une structure interne facilement convertissable en String. pareil pour les i64 pour les dates il existe des lib pour ça genre chrono: Après bon en l2 sur un bot ça passe. mais au moins faire des `newtype` pour indiquer que c'est une date ;) Eventuellement utiliser une/des hashtable|Vec qui contiens toute les string uniques genre les pseudo les channels etc et se balader juste des hash|ID-typé c'est peut-être mieux parceque se balader les `&str` c'est parfois horrible.

darnuria commented

2022-01-13 15:15:27 +01:00

(Migrated from code.up8.edu)

changed the description

darnuria commented

2022-01-13 15:16:29 +01:00

(Migrated from code.up8.edu)

changed the description

darnuria commented

2022-01-13 15:17:06 +01:00

(Migrated from code.up8.edu)

changed the description