自身が作ったプログラムを関数化したい

Question

前提・実現したいこと

10000以下の完全数を探す簡単なプログラムを作成しました、そのままプログラムを実行するのではなくそれを関数にしてから実行したいです。

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

最初にあるのが完全数を探すプログラムです。
２つ目がそれを関数にしようとして失敗したものです。関数の戻り値がよくわかっていない状態です。

エラーメッセージ

#include <stdio.h>

int main(void)
{
  int i, j, amari;

  /*1～10000まで探す*/
  for (i = 1; i < 10000; i ++){
    int sum = 0;
    for (j = 1; j < i; j ++){
      amari = i % j;
      
      /*約数だけ足す*/
      if (amari == 0){
        sum = sum + j;
      }
    }
    /*完全数かどうか判定*/
    if (sum == i){
      printf("%dは完全数です\n", sum);
    }
  }
}



/*関数化しようとしたもの*/
#include <stdio.h>

/*完全数を探す関数*/
int kanzen(int kanzen)
{
  int i, j, amari;

  /*1～10000まで探す*/
  for (i = 1; i < 10000; i ++){
    int sum = 0;
    for (j = 1; j < i; j ++){
      amari = i % j;
      
      /*約数だけ足す*/
      if (amari == 0){
        sum = sum + j;
      }
    }
    
  }
  return kanzen;
}

int main(void)
{
  int kanzen;
  printf("%dは完全数です\n", kanzen);
}

```c言語です


### 試したこと

いろいろ書き替えたりしましたが以下の出力が返ってくるだけでした。
「0は完全数です」

１つ目のプログラムは10000万以下の完全数がしっかりと出力されます。

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

https://replit.com/languages/c
こちらのサイトでプログラミングをしています

Comments

[maisumakun]

完全数は「1つ」返せばいいのでしょうか？それとも、元のコードと同じ動作にしたいのでしょうか？

[fana]

関数の仕様を明確にしてください．

> int kanzen(int kanzen)

（とりあえずこの関数を呼んですらいない様子ですが，仮に呼ぶのだとして）
引数には何を与えればよいのですか？
戻り値は何なのですか？

[chocopon]

10000以下の完全数をすべて返したいです

[chocopon]

引数はiとjとamariにしたいです、 int kanzen(int i, int j, int amari)こんな感じでしょうか？
戻り値は10000以下の完全数にしたいです

Answer 1

そもそもなのですが、私だったら指定した数値が完全数か？を判定する関数を作ります。
そのほうが便利じゃないでしょうか？

// 指定した数値nが完全数か？を返す。1=完全数
int is_kanzen( int n)
{
	:
}
int main(void)
{
    for( int i = 1; i <= 123; i++){
        if( is_kanzen(i)){
            printf("%dは完全数です\n", i);
        }
    }
}

Comments

[chocopon]

その視点はありませんでした、そっちのほうが便利ですね、試してやってみたいと思います。
ありがとうございます！

Answer 2

まず，どんな関数を作るのかをちゃんと決めましょう．

関数の機能は何か？　どう使うのか？　入力は何で，出力は何なのか？
関数内で複数の完全数が求まる場合，それをどうやって呼び出し側に返すのか？ etc...

例えば，下記コード例の(1)みたいに，
「ある値nが完全数かどうかを判定する」処理を関数にして，1～10000のループについてはmain側でやる，という形もあり得るでしょうし，
(2)や(3)のように，ループまで関数内に含めるような形もあり得るでしょうし．

とにかく決めねばならない事柄が決まってないのに何となくコードを書いても意味ないです．

//引数の値nが完全数かどうかを判定する．
bool IsPerfectNumber( int n )
{
    int sum = 0;
    for( int j=1; j<n; ++j )
    {
        int mod = n % j;
        if( mod == 0 )sum += j;
    }
    return (sum == n);
}

//値域 first～last の中にある完全数を返す
std::vector<int> EnumPerfectNumbers_1( int first, int last )
{
    std::vector<int> result;
    for( int i=first; i<=last; ++i )
    {
        if( IsPerfectNumber( i ) )result.push_back( i );
    }
    return result;
}

//値域 first～last の中にある完全数を探す．
//完全数が見つかる毎に，その値を引数としてコールバック関数 callback が呼び出される．
//戻り値は見つけた完全数の個数．
int EnumPerfectNubers_2( int first, int last, void (*callback)(int) )
{
    int n = 0;
    for( int i=first; i<=last; ++i )
    {
        if( IsPerfectNumber( i ) ){  callback( i );  ++n;  }
    }
    return n;
}

//main
int main(void)
{
    {//(1) 関数 IsPerfectNumber だけを使い，ループはこっちでやる
        for( int i=1; i<=10000; ++i )
        {
            if( IsPerfectNumber( i ) ){ std::cout << i << std::endl;    }
        }
    }

    //(2) 関数 EnumPerfectNumbers_1 を使う
    auto PNs = EnumPerfectNumbers_1( 1, 10000 );
    for( auto i : PNs ){    std::cout << i << std::endl;    }

    //(3) 関数 EnumPerfectNumbers_2 を使う
    EnumPerfectNubers_2( 1, 10000, [](int i)->void{ std::cout << i << std::endl; } );

    return 0;
}

Comments

[chocopon]

何をすべきか目的意識が甘かったと思います、ありがとうございます。
例にあるように、様々な探し方があるのですね、もう一度目的意識をもって考え直してみます。

[episteme]

僕ならこうするかな

template<typename OutIter>
OutIter EnumPerfectNumbers_3( int first, int last, OutIter out) {
for( int i=first; i<=last; ++i ) {
if( IsPerfectNumber( i ) ) *out++ = i;
}
return out;
}

Answer 3

10000以下の完全数をすべて返したいです

C言語で「いくつになるかわからない個数の値」を扱うには、自分でメモリを確保する必要が出てきて、かなり煩雑な仕事になります。関数の切り出し方で悪戦苦闘しているところから、もう1段階以上進まないといけません。

なお、int kanzen(int i, int j, int amari)という関数では、int1つしか返せません。

Comments

[chocopon]

ありがとうございます、ということは今の自分ではできそうにない感じですね。

[maisumakun]

C++なら、比較的容易に扱う方法もあります。

[chocopon]

なるほど、丁寧にありがとうございます。
とりあえず今はCを勉強しているので頑張ってみます！

Answer 4

関数呼び出しがされていません。
関数呼び出しは
kanzen(100);
等のようにカッコで引数を与えて実行します。
実行結果はkanzen(100)の値を参照して取得します。
int kekka = kanzen(100);
とするとkanzen関数の値がkekka変数に代入されます。
定義されているkanzen関数ですが引数をそのまま返すものとなっていますの
実現したい仕様は満たしていないと思います。

「0は完全数です」と出力されるのは

int kanzen;
printf("%dは完全数です\n", kanzen);

printf関数にてkanzen変数の内容を%dの部分に埋め込んで出力されていますが
kenzen変数は初期化されておらず、たまたま0が入っていたので「0は完全数です」と出力されたのだと思います。

Comments

[chocopon]

もっと勉強します、ありがとうございます！