Skip to content

Update SelectionSort.java - #4

Merged
kou-ukai merged 5 commits into
kou-ukai:takeyamafrom
AotoTakeyama:master
May 24, 2019
Merged

kou-ukai merged 5 commits into
kou-ukai:takeyamafrom
AotoTakeyama:master

Conversation

@AotoTakeyama

Copy link
Copy Markdown
Collaborator

SelectionSortクラスのsortメソッドを実装しました。

BubbleSortクラスは間違えて変更してしまいましたので無視してください。

@kou-ukai kou-ukai left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

昇順(小さい順)でないことを除けば指摘事項はありません

@kou-ukai

Copy link
Copy Markdown
Owner

コメントもよくつけれています。
Indexなど、適切な変数名が使われいる点もよいです。
適切な変数名を考えることは経験が必要なので難しいですが、
他の人のコードを見て、よさそうな名前をつけられるようになりましょう。

@kou-ukai
kou-ukai changed the base branch from master to takeyama May 24, 2019 10:41
@kou-ukai
kou-ukai merged commit d26cbb8 into kou-ukai:takeyama May 24, 2019
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