サブプログラムの使い方がわかりません

Question

前提・実現したいこと

サブプログラムをメインプロブラムに組み込むと変な結果が出てしまいます。
intやvoid，returnなどをまだしっかり理解できていないのが原因とは思いますが、この短いプログラムの何がおかしいのかも分からない状態です。
まだプログラムを習い始めて１週間ほどのため、初歩的な内容だとは思いますが、問題点を教えていただけないでしょうか

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

エラーメッセージ
実行結果
Before sorting:4195424 0-94448987232765 0 0 0 0 0

### 該当のソースコード

C言語
#include <stdio.h>
int PrintArray(int i)
{
　int a[i];
  for(i=0;i<10;i++)printf("%3d",a[i]);
  printf("\n");
}
int main(void)
{
  int a[10] = {1,5,8,9,3,4,10,6,7,2};
  int i;
  printf("Before sorting:");

  PrintArray(10);
  return(0);
}

試したこと

intやvoidをかえてみたり、サブプロラムにreturnをつけてみたりしたが、うまくいきませんでした。

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

ここにより詳細な情報を記載してください。

Answer 1

こんにちは、
配列a[MAX_NUMBER] = {1,5,8,9,3,4,10,6,7,2};を関数PrintArrayで宣言する方法があります。(要素数10を記号定数MAX_NUMBERにしました。)

また、エラーになる原因の一つとして、ソースコードのint PrintArrayのところに全角の空白がありました。

include <stdio.h>

#define MAX_NUMBER 10

void PrintArray(int);

int main(void)
{
 
  printf("Before sorting:");

  PrintArray(MAX_NUMBER);
  return 0;
}

void PrintArray(int i)
{

  int a[MAX_NUMBER] = {1,5,8,9,3,4,10,6,7,2};
	
  for(i = 0;i < MAX_NUMBER; i++){
  	printf("%3d",a[i]);
  }
  printf("\n");
  
}

Comments

[audereestfacere]

回答ありがとうございます。
最初にdefineで定義する方法があるんですね。
知りませんでした。
これで一つ知識がつきました。
ありがとうございました。
回答していただいたコードも参考にさせていただきます。

Answer 2

関数PrintArray内で宣言した変数aは、main関数内のそれとは別物です。
名前が同じだからと言って、同じメモリを参照してくれるわけではありません。

グローバル変数として宣言するか、引数として引き渡しましょう。
後者だとこんな感じです。

void PrintArray(int a[], int size)
{
    for(int i = 0; i < size; i++) {
        printf("%3d", a[i]);
    }
    printf("\n");
}

int main(void) {
    ...

    PrintArray(a, 10);

    ...
}

第二引数の取り扱い方もおかしかったので、その部分も変更しています。

Comments

[audereestfacere]

回答ありがとうございます。
どんな関数内でも同じものとして扱えると思っていました。
回答していただいたことをもとに直したらうまくいきました。
ありがとうございました。