どこを変えればいいですか？

質問

先に入力されたデータのチェック

［問題］全部で 20 個の整数データを順に入力するものとする．その際，各データを入力する毎に，新たに入力されたデータが，すでに入力されたデータと異なる場合だけ，そのデータを出力するプログラムを書け．なお，データは，１行に１つずつ出力せよ．
#include <iostream>

int main()
{
int i1, i2, n = 0, x[20], data, sw;

for (i1 = 0; i1 < 20; i1++) {
std::cin >> data;
sw = 0;
for (i2 = 0; i2 < n && sw == 0; i2++) {
if (data == x[i2])
sw = 1;
}
if (sw == 0) {
std::cout << data << std::endl;
x[n] = data;
n++;
}
}

return 0;
}

回答No.1

chie65536（@chie65535） · ベストアンサー

#include <iostream>

int main(void)
{
int i1, i2, x[20];
std::cin >> x[0];
for (i1 = 1; i1 < 20; i1++) {
std::cin >> x[i1];
for (i2 = 0; i2 < i1; i2++) {
if (x[i1] == x[i2]) break;
}
if (i1 == i2) std::cout << x[i1] << std::endl;
}
return 0;
}

最初の１個目を入力した時は「すでに入力されたデータなど存在しない」ので、何も表示してはいけない。

つまり、最初の１個目は「無条件に入力するだけ」になる。

判定を行う必要があるのは２個目から２０個目で、配列添え字で言えば1～19になり
for (i1 = 1; i1 < 20; i1++) {
となる。

判定の相手となるデータは、１個目から始め、現在の入力の１つ手前までになるので
for (i2 = 0; i2 < i1; i2++) {
となる。

同じ物があった場合に内側のループをbreakすると、ループを抜けた後でi1とi2が等しくない状態になる。

逆に言うと、同じ物が無い場合はbreakせずに最後までループが回り、最後まで回った場合はi1とi2が等しくなる。

したがって、ループ終了後、i1とi2が等しいかどうかを判定するだけで、既存データに一致データがあったかどうか判る事になり
if (i1 == i2) std::cout << x[i1] << std::endl;
で良い。

必要最低限の変数とロジックのみで出来ている事に注意すべし。