C言語 自作関数の作成について

Question

ボールドテキスト### 自作関数について
プログラム１をプログラム2のように自作関数を使った形に変更したいです。
プログラム１を「素数を判定する関数」と「判定された素数を表示する関数」の2つに分割するといった具合です。

実際にプログラム３のように自分でやってみたのですがなかなかうまく実行できません。ご指摘お願いします。

プログラム１

#include <stdio.h>
 
#define N   97
 
int main(){   
    int i,m,n;
    int data[N + 1], result[N + 1];
    for (i = 0; i < N+1; i++){
        data[i] = 1;
    }
    data[0] = 0;
    data[1] = 0;
    m = 2;
    n = 0;
    do{
        for (i = 2 * m; i < N + 1; i+=m){
            data[i] = 0;
        }
        result[n] = m;
        n++;
        do{
            m++;
        }while (data[m] == 0 && m < N + 1);
    } while (m < N + 1);
    for (i = 0; i < n; i++){
        printf("%3d ", result[i]);
      }
}

###プログラム２

#include <stdio.h> 
#define N 97

int findpn(int a);/*素数を求める*/
void printpn(int result[]);/*素数を表示する*/

int main(){
  int result[N+1];
	findpm(N);
	printpm(result[N+1]);
	return 0;
}

int findpn()
...

void printpn()
...

###プログラム３

#include <stdio.h> 
#define N 97

int findpn(int a);/*素数を求める*/
void printpn(int result[]);/*素数を表示する*/

/*メイン関数*/
int main(){
  int result[N+1];
	findpm(N);
	printpm(result[N+1]);
	return 0;
}



int findpm()
{
	int i,m,n;
	int data[N + 1], result[N + 1];
	 for (i = 0; i < N+1; i++){
        data[i] = 1;
   }
  data[0] = 0;
  data[1] = 0;
  m = 2;
  n = 0;			
  
  do{
        for (i = 2 * m; i < N + 1; i+=m){
            data[i] = 0;
        }
        result[n] = m;
        n++;
        do{
            m++;
        }while (data[m] == 0 && m < N + 1);
  } while (m < N + 1);
	return result[N+1];
}

void printpn(int result[])
{
	int i,n;
	for (i = 0; i < n; i++){
        printf("%3d ", result[i]);
	}
	return 0;			
}

Comments

[y_waiwai]

その提示のコードではどうなるんでしょうか。
まずはそこらへんを説明してください

[int32_t]

「うまく実行できません。」では何が問題なのかわかりません。問題を具体的に書いてください。

[BeatStar]

「うまくいかない」と言うのは『主観的なもの』です。

コンパイルエラーになる、実行結果がわない、急にクラッシュする、例外が飛んでくる…のようにいろんな可能性がある。
さらにそれぞれ対処法も違う。

Answer 1

void printpn(int result[])
{
    int i,n;
    for (i = 0; i < n; i++){
        printf("%3d ", result[i]);
    }
    return 0;            
}

printpn関数のnは、初期値のまま。ゼロ。 初期化されていない。内容は不定。

int findpm()
{
    int i,m,n;

findpm関数のnの値を期待しているのであれば、引数なりなんなりで渡してやらないといけない。

Comments

[WoodenHamlet]

n は初期化されていませんので、「何が入っているか分からない」ですね

[y_yy]

なるほど
質問もままなっていないのに解答ありがとうございます。
助かりました。