Skip to content

Add files via upload#40

Open
LifestreamVII wants to merge 1 commit intobcalou:mainfrom
LifestreamVII:main
Open

Add files via upload#40
LifestreamVII wants to merge 1 commit intobcalou:mainfrom
LifestreamVII:main

Conversation

@LifestreamVII
Copy link
Copy Markdown

No description provided.

@bcalou
Copy link
Copy Markdown
Owner

bcalou commented Dec 5, 2023

Les conventions pep8 ne sont pas toujours respectées.
Les explications au sein du readme ne sont pas toujours très claires et il aurait fallu des tableaux (et des temps d'éxecution) plus grands pour mettre les complexités en évidence sur les graphiques.
Le tri par sélection est un peu plus complexe que nécessaire.
Dans insertion, inutile de décaler les éléments un par un (au sein du while), on peut le faire en une fois après le while quand on a trouvé la bonne position.
Par contre le code est soigné et bien commenté.
Donc pas mal dans l'ensemble mais attention aux détails (et si possible faire une PR en avance pour avoir des retours de ma part).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants