Skip to content

Fix BubbleClass - #2

Merged
kou-ukai merged 2 commits into
kou-ukai:takeyamafrom
AotoTakeyama:master
May 17, 2019
Merged

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

Conversation

@AotoTakeyama

Copy link
Copy Markdown
Collaborator

Bubbleクラスのメソッドを修正。
i → j

BubbleSortクラスのsort(int[] ary)メソッドを作成
コードが間違っていたので修正しました。
@kou-ukai

Copy link
Copy Markdown
Owner
Bubbleクラスのメソッドを修正。
i → j

きちんと内容のあるコメントがあり、すばらしいです。
「できました」とかで終わっている方もいるので。

@kou-ukai

Copy link
Copy Markdown
Owner

確認させていただきました。
小規模なコードなのでそこまで必要ありませんが、
コードにコメントを記載するクセをつけてください。
2ヵ月後のあなたと、他者が読みやすくなります。

@kou-ukai
kou-ukai changed the base branch from master to takeyama May 17, 2019 00:36
@kou-ukai
kou-ukai merged commit 643733a into kou-ukai:takeyama May 17, 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