質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.34%

  • C#

    7696questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

ユーグリッドの互除法を求める二つの関数の違い

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 172

ryu72639

score 6

下の二つの関数の違いを教えて下さい。上の方は数字が大きくなると正しい答えが出ません

#include <stdio.h>
int yugurid(int number1, int number2) /*ユーグリッドの互除法による最大公約数*/
{
  int r, a, b;

  if (number1 < number2){
    a = number1;
    b = number2;
  }
  else {
    a = number2;
    b = number1;
  }

  r = b % a;

  while (r != 0)
    {
      a = b;
      b = r;
      r = b % a;
    }
  return a; /* a = 最大公約数 */
}
#include <stdio.h>

int yugurid(int x, int y)
{
  if(x % y == 0)
    return y;
  return yugurid(y, x % y);
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • HogeAnimalLover

    2018/06/30 19:39

    具体的に正しい結果が出ないときの、状況(number1,number2の値)は何ですか?

    キャンセル

回答 3

checkベストアンサー

0

言語タグはC#じゃなくてCですよね?

C#で動かしてみて、関数1ではaとbとrを、
関数2ではxとyとx % yを標準出力で確認してみました

yugurid1(123, 127)
a=123, b=127, r=4
a=127, b=4, r=4
a=4, b=4, r=0
結果=4
yugurid2(123, 127)
x=123, y=127, x % y=123
x=127, y=123, x % y=4
x=123, y=4, x % y=3
x=4, y=3, x % y=1
x=3, y=1, x % y=0
結果=1

ループ内でaとbに代入する際に大小関係を間違えていませんか?
こちらで間違いを直せば関数2と同じ結果になりましたので

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/30 21:15

    最初に
    if (number1 < number2){
    a = number1;
    b = number2;
    }
    else {
    a = number2;
    b = number1;
    }
    として、大%小としているのですが、それだとダメなのですか?

    キャンセル

  • 2018/06/30 22:07

    whileの中の話をしています
    aに大きい方の値を代入して、bに余りを代入していますよね?
    次のループでは小さい方の数値が消えてaに前回の余りを代入し、bに今の余りを代入していますよね
    2巡目で元の数値は完全に消えてしまいました
    いったい何の計算をしようとしているのでしょうか?

    キャンセル

  • 2018/06/30 22:09

    開発環境が分からないのでやり方は何とも言えませんが、おかしいと思ったのなら変数の値の変化を確認するようにしましょう

    キャンセル

  • 2018/06/30 22:20

    ループ前:a=数値(小)、b=数値(大),r=数値(大)÷数値(小)の余り//以降は余り(1)
     1回目:a=数値(大)、b=余り(1),r=余り(1)÷数値(大)の余り//当然 数値(大)>余り(1)なので余り(1)と同じ値
     2回目:a=余り(1)、b=余り(1),r=余り(1)÷余り(1)の余り//同じ数値で割るためあまり0

    キャンセル

  • 2018/06/30 22:23 編集

    なお、理解せずともコピペできる回答ではなく、なぜなのかを考えてわかってもらえるようにと回答しています

    キャンセル

  • 2018/07/01 17:48

    なるほど、確かにそうですね。わかりやすい回答ありがとうございます

    キャンセル

0

bを大きい数、aを小さい数とするなら、値の交換は

b = a;
a = r;
r = b % a;


とすべきなのでは?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

ループ内で、aとbの更新が変です。bにaを、aにはrを代入します。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

同じタグがついた質問を見る

  • C#

    7696questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。