回答編集履歴
6
修正
test
CHANGED
|
@@ -72,7 +72,7 @@
|
|
|
72
72
|
|
|
73
73
|
|
|
74
74
|
|
|
75
|
-
void swap(int *a, int *b) {
|
|
75
|
+
inline void swap(int *a, int *b) {
|
|
76
76
|
|
|
77
77
|
int t = *a;
|
|
78
78
|
|
|
@@ -86,7 +86,7 @@
|
|
|
86
86
|
|
|
87
87
|
void selection_sort(int buffer[], int length) {
|
|
88
88
|
|
|
89
|
-
for (int i = 0; i < length; i++) {
|
|
89
|
+
for (int i = 0; i < length - 1; i++) {
|
|
90
90
|
|
|
91
91
|
int max_index = i;
|
|
92
92
|
|
5
修正
test
CHANGED
|
@@ -20,7 +20,7 @@
|
|
|
20
20
|
|
|
21
21
|
- カウンタがその時の気分によって i だったり n だったりする
|
|
22
22
|
|
|
23
|
-
- i
|
|
23
|
+
- iMax を -9 で初期化する意味がわからない
|
|
24
24
|
|
|
25
25
|
|
|
26
26
|
|
4
修正
test
CHANGED
|
@@ -20,11 +20,11 @@
|
|
|
20
20
|
|
|
21
21
|
- カウンタがその時の気分によって i だったり n だったりする
|
|
22
22
|
|
|
23
|
-
- iMax
|
|
23
|
+
- i,n,Max,iMax の初期化の意味がわからない(-9 は iMax ではなく i)
|
|
24
24
|
|
|
25
25
|
|
|
26
26
|
|
|
27
|
-
全般的に「読みにくく保守しにくい」と感じました。
|
|
27
|
+
全般的に「読みにくく保守しにくい」と感じました。「拡張性があって優れている」ということは無く、その逆だと思います。
|
|
28
28
|
|
|
29
29
|
|
|
30
30
|
|
3
修正
test
CHANGED
|
@@ -108,9 +108,7 @@
|
|
|
108
108
|
|
|
109
109
|
|
|
110
110
|
|
|
111
|
-
int main(void)
|
|
111
|
+
int main(void) {
|
|
112
|
-
|
|
113
|
-
{
|
|
114
112
|
|
|
115
113
|
int buffer[N];
|
|
116
114
|
|
2
修正
test
CHANGED
|
@@ -120,13 +120,11 @@
|
|
|
120
120
|
|
|
121
121
|
dump(buffer, length);
|
|
122
122
|
|
|
123
|
+
selection_sort(buffer, length);
|
|
124
|
+
|
|
125
|
+
dump(buffer, length);
|
|
123
126
|
|
|
124
127
|
|
|
125
|
-
selection_sort(buffer, length);
|
|
126
|
-
|
|
127
|
-
|
|
128
|
-
|
|
129
|
-
dump(buffer, length);
|
|
130
128
|
|
|
131
129
|
return 0;
|
|
132
130
|
|
1
修正
test
CHANGED
|
@@ -10,7 +10,7 @@
|
|
|
10
10
|
|
|
11
11
|
- コードがすべて main 関数の中にある
|
|
12
12
|
|
|
13
|
-
- 変数の寿命が無駄に長い
|
|
13
|
+
- 変数の寿命が無駄に長く、再利用までされている。
|
|
14
14
|
|
|
15
15
|
- 大文字・小文字の使い分けが不明瞭(N は定数のように見えるが変数)
|
|
16
16
|
|