Ｃ言語で標準入力から完全数を判別する

Question

前提・実現したいこと

標準入力から与えられた数が、"完全数"　"ほぼ完全数" "そのほか"のどれか判別するコードを書きたいと
思っています。
①判別する数がいくつあるのかをまず受け取り
②その後に判別する数を打ち込み
③それを判別する。
という流れです。

そのうえで、コード内で問題のある行に特に配列の定義などに問題が生じているようで手詰まりとなりました。

この場合配列（number)をどのように定義すべきでしょうか。

初心者で至らない点も多いとは思いますがよろしくお願いします。

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

C99のエラー
再定義エラー（ローカル変数ではないのか）
おそらく20行目の配列定義がまずいかもしれない

エラーメッセージ
[vagrant@localhost C.lesson]$ gcc kanzensu.c -o kanzensu
kanzensu.c: In function ‘main’:
kanzensu.c:18: error: ‘for’ loop initial declarations are only allowed in C99 mode
kanzensu.c:18: note: use option -std=c99 or -std=gnu99 to compile your code
kanzensu.c:24: error: ‘for’ loop initial declarations are only allowed in C99 mode
kanzensu.c:26: error: invalid operands to binary % (have ‘int *’ and ‘int’)
kanzensu.c:35: error: redefinition of ‘i’
kanzensu.c:18: note: previous definition of ‘i’ was here
kanzensu.c:35: error: ‘for’ loop initial declarations are only allowed in C99 mode
kanzensu.c:39: 警告: ポインタと整数との比較を行なっています
kanzensu.c:43: 警告: ポインタと整数との比較を行なっています
kanzensu.c:43: 警告: ポインタと整数との比較を行なっています

該当のソースコード

C言語

include <stdio.h>
int main(void)
{


    char str[1000];
    int k;//`回数`
    int sum;
    int number[100]; //`要素の数`



    fgets(str, sizeof(str), stdin);
    sscanf(str, "%d" , &k);   //`13行`



  for(int i = 0; i < k; i++)  //`17行`
  {
        fgets(str, sizeof(str), stdin);
        sscanf(str,"%d", number[i]);    //`20行`


    for(int j = 0; j < k; j++)     //`23行`
    {
        if(number % j == 0)
        {
            sum = sum + j;
        }
    }

  }


  for(int i = 0;  i < k;  i++)  //`再定義エラー？`
  {


    if(sum == number)
    {
        printf("perfect"); //`完全数`
    }
    else if(sum == number - 1 || sum == number + 1)
    {
        printf("nearly"); //`ほぼ完全数`
    }
    else
    {
        printf("neither"); //`どちらでもない`


    }

  }

    return 0;
}

試したこと

scanfを使ってみたが結果が同じでした

一つの要素入力ではできましたが、
いくつ要素を比較するかは
入力者に依存するため
配列定義の仕方がわかりませんでした。

補足情報（FW/ツールのバージョンなど）

Atom1.45.0

Windows10

Comments

[jimbe]

> note: use option -std=c99 or -std=gnu99 to compile your code
と言われていますが, お試しになられましたでしょうか.

Answer 1

C99のエラー

kanzensu.c:18: error: ‘for’ loop initial declarations are only allowed in C99 mode
kanzensu.c:18: note: use option -std=c99 or -std=gnu99 to compile your code

メッセージの通り、for()で変数宣言したいのであれば、オプションに-std=c99か-std=gnu99をつけてください。

・問題点1

if(number % j == 0)
if(sum == number)
else if(sum == number - 1 || sum == number + 1)

コンパイラも指摘していますがポインタと整数の比較になっています。
numberはnumber[i]ですね。

・問題点2
sumが0初期化されていません。
また、2つ目以降の数値の約数も加算されています。

・問題点3
sscanf(str, "%d", number[i]);
は
sscanf(str, "%d", &number[i]);
ポインタを渡すべきところに整数を渡しています。

・問題点4

    for(int j = 0; j < k; j++)     //`23行`
    {
        if(number % j == 0)

0除算が行われています。
また、入力した数値の約数を調べたいはずなのに、判定したい数値の数でループを回してしまっています。

・問題点5

for(int i = 0;  i < k;  i++)  //`再定義エラー？`
(略)

sumと比較していますが、これではすべての入力数値の約数の和との比較になります。
あとでループするのではなく、

for(int i = 0; i < k; i++)  //`17行`

に入れるべきでは？
もしくはsumも配列にして入力値ごとに持ちましょう。

Comments

[L4mbda]

詳しい回答を本当にありがとうございます。
おかげで解決しました。
感謝します。

Answer 2

for(int i = 0; i < k; i++) //17行

for文の中で変数定義できるのはC99以降となります。
GCCのオプションで、C99を指定する必要があります

    if(number % j == 0)

numberは配列です。この条件式は意味不明になってます

Comments

[L4mbda]

回答を本当にありがとうございます。
解決しました。
感謝します。