J'ai fait une fourchette du dépôt pour modifier l'intégration continue afin qu'elle vérifie correctement les défauts relevés par clippy. Jusqu'à maintenant, elle se contentait d'afficher des avertissements :stuck_out_tongue:

J'ai aussi retiré les commandes qui venaient modifier la version de Rust. Comme ça, le résultat est plus consistant entre les exécutions, c'est une version 1.64.0 de Rust et c'est tout. J'ai donc corrigé les derniers défauts qui se sont déclarés entre la version de Rust que j'utilise 1.58.0 et celle de l'intégration continue

The for loop was taking ownership of the grid with an implicit call to "into_iter". Adding an explicit "iter_mut" fixed the ownership

Si ça ne te dérange pas, on partirait sur ce squelette pour la résolution. Il permet d'ajouter facilement des tests et tous les puzzles s'y adaptent bien :smile:

Pour te motiver, je t'ai même donné une petite chips, regarde comment elle est belle ma résolution de partie 2 :wink:

Hello, j'ai ajouté une solution à un puzzle de 2015. Elle est un peu bizarre parce qu'elle n'est théoriquement pas ce qui est demandé, mais elle fonctionne. Je te propose de l'essayer aussi pour voir ? La vraie solution me semble trop compliquée algorithmiquement. Elle part vite dans les choux parcequ'il y a trop de possibilité (ou alors j'ai loupé un truc). Le hic, c'est que du coup, les exemples de base ne fonctionnent pas (d'où les deux tests ignorés).

J'ai pris la liberté de modifier la gestion du fichier d'entrée, je trouvais le code tout moche et pas pratique. Là c'est pratique pour faire des tests à la pelle en rajoutant des fichiers.

J'ai aussi abandonné l'idée du fichier input.txt, vu que c'est censé être ton input de puzzle. Autant le laisser comme un test supplémentaire, non ? Mon input est en test05.txt par exemple.

Idéalement, on ferait une librairie qui donne accès à une classe qui lit les fichiers d'entrée et donne un vecteur de strings. Je pense que c'est la même chose pour tous les problèmes.

Sinon, c'est pas encore prêt à merger, je dois rajouter la documentation des petites étoiles

I tried to write the same program in C++ as much as I could, with the right types and some error checking.

In C++ there are fewer checks, especially on integer arithmetics. You can mix unsigned/signed operands, different bit sizes, overflow, without any compiler complaint.

The vector operator [] could segfault if the index is out of the limits.

Moreover, in the solve inner loop (when the monkey throws items) if the input is not correctly chosen, we can stall in a forever ping-pong party. In Rust, the borrowing mechanism prevents this and forces developers to write their code differently.

Mise à jour de la page README pour que les années 2015 et 2016 soient comme la 2021. J'en ai profité pour harmoniser la documentation en début de fichier.

Est-ce qu'on peut virer les puzzles en Python quand ils sont fait en Rust ? Ils me font souvent chier quand je navigue et puis ça s'appelle advent-of-rust, pas advent-of-rust-and-sometimes-python. Mais si tu y tiens, on les laisse bien entendu :wink:

Clippy pedantic warnings have been fixed.

When checking CI pipeline, we still see warnings because they come from the nightly version of rust toolchain. I'm using the rust toolchain coming along with Ubuntu 22.04 package repository (rust 1.58.1) :smile:

• fixed part1 of day5 challenge
• fixed progress spinner display for day5 challenge, the tick is not supposed to be called when a steady tick is set. However it should not have messed the display
• added a second test for day14 challenge, but commented it as it takes some time to complete
• fixed day19 challenge when input has final newline
• ran every test associated with the modified challenges (and those that use assembunny)