Skip to content

Js 20 second largest#52

Closed
Klikasoft wants to merge 5 commits intoPyLadiesCZ-Brno:mainfrom
Klikasoft:js-20-second-largest
Closed

Js 20 second largest#52
Klikasoft wants to merge 5 commits intoPyLadiesCZ-Brno:mainfrom
Klikasoft:js-20-second-largest

Conversation

@Klikasoft
Copy link
Contributor

ten GITIGNORE jsem si nastavoval jen pro sebe, složku 05 main nemá, ale já ji chtěl pro své drobnosti, dá se to udělat i soukromě aby to každý neviděl?

Copy link
Contributor

@ZelenyMartin ZelenyMartin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kdyby se ti podařilo samostudiem nastudovat jako udělat interaktivní rebase a pomocí něho smazat první a třetí commit v této branchi, tak by ty bylo úplně nejskvělejší. Ale kdyžtak vyřešíme na srazu.

Comment on lines +8 to +11
for finder in lst:
if finder < lst[0]:
numero = finder
break
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Když už ten seznam řadíš, tak už nepotřebuješ žádný for ani if.

Copy link
Contributor Author

@Klikasoft Klikasoft Apr 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ten for a if jsem tam dal pro pripad, ze by seznam obsahoval duplicity, mel jsem najít druhe nejvetsi cislo, sorted by mi je jen seradil, ale v pripade [10,12,12,12,11,11] bych druhe nejvetsi nenasel, pouhým lst[1] ale uznávám, že se to jistě dalo napsat i líp :)

@Klikasoft Klikasoft closed this Apr 3, 2025
@Klikasoft Klikasoft deleted the js-20-second-largest branch April 3, 2025 13:07
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