Skip to content

Rendu Kayyissa#36

Open
Korayou wants to merge 4 commits intobcalou:mainfrom
Korayou:main
Open

Rendu Kayyissa#36
Korayou wants to merge 4 commits intobcalou:mainfrom
Korayou:main

Conversation

@Korayou
Copy link
Copy Markdown

@Korayou Korayou commented Nov 24, 2023

No description provided.

@Korayou Korayou marked this pull request as ready for review November 24, 2023 22:18
@bcalou
Copy link
Copy Markdown
Owner

bcalou commented Dec 5, 2023

Réponses claires au TP.
Dans la selection, tu peux utiliser une boucle for à la place du while pour éviter d'avoir à maintenir toi-même l'index.
Dans insertion, beaucoup d'opérations pourraient être évitées (au lieu de déplacer les éléments un par un dans le for, déplacer une fois pour toute quand on trouvé le bon index)
On ne peut pas se passer de commentaires à ce point, même si le code est très propre.
Mais rien de grave, c'est bien.

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