C言語にてユークリッドの互除法を用いず最小公倍数、最大公約数を表示したい。

Question

前提・実現したいこと

プロトタイプ宣言を使用して入力された任意の二つの整数n1,n2の最小公倍数、最大公約数を表示したい。
ただし、最大公約数はユークリッドの互除法を用いずに計算したい。

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

ソースコードのMAXgcd（最大公約数）に一度該当する値が入力されたらそれ以降の値が入力されていないのだと推測しています。
例を挙げるとn1=30、n2=15とした場合、for文がm=1の場合はMAXgcd=1となる。ここでm=7とすればMAXgcd=15となる。
おそらく一度if文が真となるmが入力されればfor文から抜けているのではないか。
コンパイルエラーメッセージは下記の通りです。

Main.c:13:18: warning: variable 'm' is uninitialized when used here [-Wuninitialized]
    MAXgcd = gcd(m,n1,n2);
                 ^
Main.c:9:10: note: initialize the variable 'm' to silence this warning
    int m,n1,n2,MAXgcd;
         ^
          = 0
Main.c:26:1: warning: control may reach end of non-void function [-Wreturn-type]
}

該当のソースコード

//6-9
#include <stdio.h>

//プロトタイプ宣言
int gcd(int,int,int);
int lcm(int,int,int);
  
int main(void){
    int m,n1,n2,MAXgcd;
    scanf("%d %d",&n1,&n2);
    printf("1つ目の値を入力してください%d\n",n1);
    printf("2つ目の値を入力してください%d\n",n2);
    MAXgcd = gcd(m,n1,n2);
    printf("2つの数の最大公約数は、%dです。\n",MAXgcd);
    printf("2つの数の最小公倍数は、%dです。\n",lcm(n1,n2,MAXgcd));
}


//最大公約数を計算する関数
int gcd(int m,int n1,int n2){
    for(m = 8 ; m <= n1 || m <= n2 ; m++){
        if(n1 % m == 0 && n2 % m == 0){
            return m;
        }
    }
}

//最小公倍数を計算する関数
int lcm(int n1,int n2,int MAXgcd){
    return (n1 * n2) / MAXgcd;
}

試したこと

プロトタイプ宣言関係の処理を削除した下記内容では問題なく動きました。
そのため、おそらくプロトタイプ宣言関係の処理が上手くいってないのだと思います。

#include <stdio.h>
  
int main(void){
    int m,n1,n2,MAXgcd,MINlcm;
    scanf("%d %d",&n1,&n2);
    printf("1つ目の値を入力してください%d\n",n1);
    printf("2つ目の値を入力してください%d\n",n2);
    //最大公約数を計算する関数
    for(m = 1 ; m <= n1 || m <= n2 ; m++){
        if(n1 % m == 0 && n2 % m == 0){
            MAXgcd = m;
        }
    }
//最小公倍数を計算する関数
    MINlcm = (n1 * n2) / MAXgcd;
    printf("2つの数の最大公約数は、%dです。\n",MAXgcd);
    printf("2つの数の最小公倍数は、%dです。\n",MINlcm);
}

注文が多いですが、お力添えしていただけると助かります。

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

用いているツールはpaizaです。

上記処理の解決方法

#include <stdio.h>

//プロトタイプ宣言
int gcd(int,int);
int lcm(int,int,int);
  
int main(void){
    int n1,n2,MAXgcd;
    scanf("%d %d",&n1,&n2);
    printf("1つ目の値を入力してください%d\n",n1);
    printf("2つ目の値を入力してください%d\n",n2);
    MAXgcd = gcd(n1,n2);
    printf("2つの数の最大公約数は、%dです。\n",MAXgcd); //mのスコープ範囲をint bcdに移動
    printf("2つの数の最小公倍数は、%dです。\n",lcm(n1,n2,MAXgcd));
}


//最大公約数を計算する関数
int gcd(int n1,int n2){
    int m,MAX;
    for(m = 1 ; m <= n1 || m <= n2 ; m++){
        if(n1 % m == 0 && n2 % m == 0){
            MAX = m; //最大公約数をint型MAXに格納
        }
    }
    int r = MAX;
    return r; //最大公約数MAXをreturnで返す
}

//最小公倍数を計算する関数
int lcm(int n1,int n2,int MAXgcd){
    return (n1 * n2) / MAXgcd;
}

Answer 1

for(m = 8 ; m <= n1 || m <= n2 ; m++){

m は、下は 1 なのに、なぜここでは 8 ?

for(m = 1 ; m <= n1 && m <= n2 ; m++){

でいいんじゃないですか？

指摘済みのところですが、
return m のところは、ほかの保持する変数に格納する必要があります。
最後にその値をreturn します。

Comments

[hayato208]

m = 8は私の誤字でした、すみません。
return mの部分を他の変数に保持し、最後にreturnすればよかったのですね、それを勘違いしておりました。
その勘違いを正してくださったおかげで無事解決しました、ありがとうございました。

Answer 2

MAXgcd = gcd(m,n1,n2);

なぜ gcd は m をパラメータで貰っているのでしょうか. しかもその m は初期化されていません.
さらに, gcd は( if 文に引っかからずに) for を回り切った場合にも int 値を返す必要がありますが, それがありません.

Comments

[hayato208]

mainの部分にmを表示するためにmを貰っておりましたが、ご指摘を踏まえてint gcdのローカル変数で変えてみたところエラーはなくなりました。
また、forの中でintを返しておりましたが、それをfor外にしたところ無事動きました。
ありがとうございました。