for文でのデッドコード

Question

前提・実現したいこと

javaを利用して、挿入ソートで配列に入っている数字を昇順に並べるコードを書いています。

発生している問題・エラーメッセージ

for (int j = i; j > 0; j--) の行において、
デッドコードであると、
Eclipseのコードを打っている欄の左にでてきます。

なぜでしょうか？
また、どう改善するべきでしょうか？

該当のソースコード

for (int i = 1; i < data.length; i++) {
for (int j = i; j > 0; j--) {
if (data[j] < data[j - 1]) {
int tmp = data[j];
data[j] = data[j - 1];
data[j - 1] = tmp;
exchangeNum++;
comparisonNum++;
}
else
comparisonNum++;
break;
}
}

Comments

[javahack]

コードはマークダウン記法を使用して記述してください。
https://teratail.com/help#about-markdown

dataには何が入っているのでしょうか。
実行可能なコードを提示した方が回答が付きやすくなります。

[m.ts10806]

>デッドコードであると表示されます。
どこにどのように表示されるのでしょうか。環境情報と共に追記してください。

[退会済みユーザー]

なんとなくだが、 data.length が　1固定になるコードかな

[ryotazumi]

Eclipseのコードを打っている欄の左にでてきます。

[m.ts10806]

質問は編集できます。

Answer 1

eclipseプラグインの警告

j--がデッドコードです。なぜでしょう。

考え方

• 挿入ソートの内側のループが１回だけ実行されるからj--がデッドコードなのか
• 1回だけだとしたらbreak;を疑う
• elseのあとにブロックが必要か

Comments

[ryotazumi]

回答ありがとうございます。
elseの後にブロックが必要、とはどういった意味でしょうか？

[xebme]

> また、どう改善するべきでしょうか？
「ブロック」はヒントのつもり。知らないなら「if else ブロック」で調べてください。
(ひょっとするとあなたの転記ミスかもしれませんね)
「複合文」とは何か考えてください。

[xebme]

elseで実行するのは次のどの文ですか。ひとつですか、ふたつまとめてですか。
comparisonNum++;
break;

[ryotazumi]

ブロックの意味がわかりました。どちらも実行するつもりでした。ありがとうございました。