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

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

ただいまの
回答率

90.51%

  • Java

    13829questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • for

    231questions

    for文は、様々なプログラミング言語で使われている制御構造です。for文に定義している条件から外れるまで、for文内の命令文を繰り返し実行します。

  • if

    205questions

    if文とは様々なプログラミング言語で使用される制御構文の一種であり、条件によって処理の流れを制御します。

Java言語 帯分数と整数を求めたいです。

解決済

回答 3

投稿

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

ssssa

score 27

2つの分数同士の足し算をし、その結果を分数で表示するプログラムを作りなさい。
このとき、分子、分母共に最大値が10で、最小値は分母が2、分子が1とする。それらの数値をランダムに発生させ、以下のように結果を表示させなさい。
ただし、計算結果は、分子と分母がきちんと約分されていることとする。
また、分子が分母の数で割り切れる場合は、整数として表示するものとする。


1/5 + 2/3 = 13/15  ← 通常のケース
2/3 + 3/8= 1.1/24  ← 帯分数になるケース
1/6 + 1/3= 1/2     ← 約分されるケース
1/2 + 1/2 = 1      ← 整数になるケース

public class Prob8_6 {

/**

  • @param args
    */
    public static void main(String[] args) {
    int a = (int) (Math.random() * 10) + 1;
    int b = (int) (Math.random() * 10) + 2;
    int c = (int) (Math.random() * 10) + 1;
    int d = (int) (Math.random() * 10) + 2;

System.out.println("分子" + a);
System.out.println("分母" + b);
System.out.println("分子" + c);
System.out.println("分母" + d);

int e = (a * b) + (c * d);
int o = e;
int f = (b * d);
int p = f;
System.out.println("");
System.out.println("-----------通分-----------");
System.out.println("分子" + e);
System.out.println("分母" + f);
System.out.println(a + "/" + b + "+" + c + "/" + d + "=" + e + "/" + f);
System.out.println("");
System.out.println("----------約分-------------");
if (e < f) {
int tmp = e;
e = f;
f = tmp;
}
int r = e % f;
while (r != 0) {
e = f;
f = r;
r = e % f;
}
System.out.println("最大公約数" + f);
System.out.println("分子" + o / f);
System.out.println("分母" + p / f);
System.out.println(" ");
System.out.println("--------帯分数----------");
if (e % f != 0) {
System.out.println(e + "/" + f);

}
System.out.println(" ");
System.out.println("--------整数-------------");
if (e == f) {
System.out.println(e);
}

}

}
上記のように書いてみましたが帯分数と整数表示が何も表示されずどこをなおせばいいのかわからないです。
どなたかご教示いただけないでしょうか?

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • mts10806

    2018/07/12 09:16 編集

    https://teratail.com/help/avoid-asking >何かを作りたいのでコードを書いてほしい、学校の課題を解いてほしい等の質問は、具体的にプログラミングで困っている質問ではないと考え、推奨していません。

    キャンセル

回答 3

+2

if (e % f != 0)
if (e == f)


このコードに到達する時点でeとfは互除法をやった残りかすだということに注意してください。
前者は必ずeがfで割り切れる状態なので、このifには入れません。
後者は最初の段階でe==f、つまり2つの分数の和が1になるときしか成立しません。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/07/12 22:32

    ご指摘また的確な修正内容ありがとうございます。
    変数名が何を意味するのかしっかり考え問題を解きたいと思います。
    見ていただき本当にありがとうございました。

    キャンセル

checkベストアンサー

+1

こんにちは、

 まず、コードを見て気づいたところですが、

1

 System.out.println("分子" + a);
 System.out.println("分母" + b);
 System.out.println("分子" + c);
 System.out.println("分母" + d); 

int e = (a * b) + (c * d);


とありますが、通分の際の分子を求める式が間違っています。
a / b + c / d の計算をするならば、int e = a * d + b * cです。

2.変数名をもっとわかりやすいものにしてください。a,b,c,d,e,fだと後でコードを見たときにその変数が何を指しているのかわからなくなる恐れがあります。例えば、帯分数の整数部分をintegerPartなどにするといいでしょう。

 では、私がコードを書く際に工夫した点を述べます。

1.a,b,c,dを配列にする。
配列にすることで、

for(int i = 0; i < 4; i++){
   data[i] = rand.nextInt(10) + 1 + i % 2;
}


と簡潔に書くことができます。

また、その後の通分の式も

for(int i = 0; i < 2; i++){
   num += data[i] * data[3 - i];
   den *= data[2 * i + 1];
}


のように一つの式が短くなるメリットがあります。

2.約分

最大公約数を求める際、その個所をメソッドに切り出しました。

3.後は、答えが真分数になるのか、整数になるのかをif文を使って分岐させればよいでしょう。

約分は済ませたので答えは
(i)整数
(ii)真分数
(iii)帯分数
の3つに場合分けできます。

それぞれの条件は以下の通りなのでこれをもとにif文で分岐させましょう。

整数部分 分子
整数 0
真分数 0
帯分数

細かいところですが、質問欄にある 帯分数を表示するとき 1.1/24  と書いてありますが、
「 1 と 1 / 24 」または、「 1 and 1 / 24 」と書くことをおすすめします。
初見、"1.1"という小数を24で割ったものと勘違いしました。(小数点は小数を表すときに使うので)

以下、私が書いたソースコードを載せておきます。長々と書いたのでどこか間違ったことを
書いているかもしれません。その時はご指摘ください。

import java.util.Random;

public class Main {
    public static void main(String[] args){
        // Your code here!
        Random rand = new Random();
        int []data = new int[4];
        String []text = {"分子","分母"};
        for(int i = 0; i < 4; i++){
            data[i] = rand.nextInt(10) + i % 2 + 1;
            //System.out.println(text[i % 2] + (i / 2 + 1) + ":" + data[i]);
        }
        int num = 0; //分子
        int den = 1; //分母

        for(int i = 0; i < 2; i++){
            num += data[i] * data[3 - i];
            den *= data[2 * i + 1];
        }
        int integerPart = num / den; //整数部分
        num = num % den;

        //System.out.println(num + " " + den);
        int tmp = my_gcd(num,den);
        num /= tmp;
        den /= tmp;

        System.out.print(data[0] + " / " + data[1] + " + " + data[2] + " / " + data[3] + " = ");

        if(integerPart > 0){
            System.out.print(integerPart);
            if(num > 0){
                System.out.print(" and ");
            }
        }
        if(num > 0){
            System.out.println(num + " / " + den);
        }


    }
    public static int my_gcd(int a, int b){

        int r = a % b;
        while (r != 0) {
            a = b;
            b = r;
            r = a % b;
        }
        return b;
    }
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/07/12 18:03

    最大公約数は互除法で求めてるからそこは変更の必要がないと思うのですが。

    キャンセル

  • 2018/07/12 18:20

    私がやった方法だと新しく用意する変数は1つで済むのでこちらで書きました。
    もちろん互除法のほうがループ回数は少なくて処理は速いですが、新しく3つの変数を準備しなければいけないのでどっちで書こうか迷いました。

    キャンセル

  • 2018/07/12 18:20

    メソッドに切り出したい

    キャンセル

  • 2018/07/12 18:22

    私も たった今 メソッドにしようと思いました。直しておきます。

    キャンセル

  • 2018/07/12 18:28

    訂正しました。ご指摘ありがとうございます。

    キャンセル

  • 2018/07/12 22:30

    指摘やわかりやすいコードなどありがとうございます。
    完成度が高く感動いたしました。
    本当にありがとうございます。

    キャンセル

+1

random で数値を設定していると、テストやデバッグがしにくいです。
数値を設定する部分と、計算・表示をする部分を別けてみました。

public class Prob8_6 {
    public static void main(String[] args) {
        int n1 = (int) (Math.random() * 10) + 1;
        int d1 = (int) (Math.random() * 10) + 1;
        int n2 = (int) (Math.random() * 10) + 1;
        int d2 = (int) (Math.random() * 10) + 1;

        my_task(n1, d1, n2, d2);
        // テスト
        my_task(1, 3, 1, 2); // 1/3 * 1/2 = 5/6
        my_task(1, 3, 2, 3); // 1/3 + 2/3 = 1
        my_task(1, 3, 3, 4); // 1/3 + 3/4 = 1 と 1/12
        my_task(1, 12, 3, 18); // 1/12 + 3/18 = 1/4
    }

    public static void my_task(int n1, int d1, int n2, int d2) {
        System.out.println("\n-----------通分での足し算-----------");
        int n3 = (n1 * d2) + (n2 * d1);
        int d3 = (d1 * d2);
        System.out.println("" + n1 + "/" + d1 + " + " + n2 + "/" + d2 + " = " + n3 + "/" + d3);

        System.out.println("----------約分して 整数、帯分数, 分数で表示する-------------");
        int cd = gcd(n3, d3);
        System.out.println("最大公約数 " + cd);

        int n4 = n3 / cd;
        int d4 = d3 / cd;
        if (n4 % d4 == 0) {
            System.out.println("  --------整数-------------");
            System.out.println(n4 / d4);
        } else if (n4 > d4) {
            System.out.println("  --------帯分数----------");
            int n = n4 / d4;
            System.out.println("" + n + " と " + (n4 - n * d4) + "/" + d4);
        } else {
            System.out.println("  --------分数----------");
            System.out.println("" + n4 + "/" + d4);
        }
    }

    // 最大公約数を求める (ユークリッドの互除法)
    public static int gcd(int a, int b) {
        if (a < b) {
            // 引数の大小入替 (a >= b にする)
            int temp = a;
            a = b;
            b = temp;
        }
        if (b < 1) {
            return -1;
        }
        if (a % b == 0) {
            return b;
        }
        return gcd(b, a % b);
    }
}


実行例

-----------通分での足し算-----------
7/9 + 6/8 = 110/72
----------約分して 整数、帯分数, 分数で表示する-------------
最大公約数 2
  --------帯分数----------
119/36

-----------通分での足し算-----------
1/3 + 1/2 = 5/6
----------約分して 整数、帯分数, 分数で表示する-------------
最大公約数 1
  --------分数----------
5/6

-----------通分での足し算-----------
1/3 + 2/3 = 9/9
----------約分して 整数、帯分数, 分数で表示する-------------
最大公約数 9
  --------整数-------------
1

-----------通分での足し算-----------
1/3 + 3/4 = 13/12
----------約分して 整数、帯分数, 分数で表示する-------------
最大公約数 1
  --------帯分数----------
11/12

-----------通分での足し算-----------
1/12 + 3/18 = 54/216
----------約分して 整数、帯分数, 分数で表示する-------------
最大公約数 54
  --------分数----------
1/4

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/07/21 13:45

    わかりやすくご丁寧な回答ありがとうございます。参考にさせていただきました。
    本当にありがとうございます。

    キャンセル

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

  • ただいまの回答率 90.51%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

  • 解決済

    java 乱数 配列

            }     } なるべく短いプログラムにしたいです。よろしくお願いします。

  • 解決済

    java ループ

    コード java public void play(){     Random rnd = new Random();         int run = rnd.nextIn

  • 解決済

    メソッドの切り離しについて (Java学習中)

    こんばんは。 今日は、メソッドについての質問です。 とても簡単なプログラミングで、数値を入力してもらってそれを左から順番に小さい方から並べるというものを作ろうとしました。 こ

  • 受付中

    ループ化の方法

    public class Gohkaku {     public static void main(String[] args){         int math = ne

  • 受付中

    java For文が正常に動きません。

    初心者なのですが、forが処理できません。なぜでしょうか? 教えてくださいよろしくお願いします。 package game; import java.util.Rando

  • 解決済

    乱数表示から最大値を表示する Java

    いつもお世話になっております。 現在変数を表示させてから、最大値を求めるプログラムを組んでいます。 乱数は表示できる状態です。 開発環境Eclipse、言語はJavaです

  • 受付中

    ENTER実行処理

    Enterキーを押して、次の処理に移るCUIを作りたいのですが、思うようにいきません。勇者の攻撃のあとと、HP、MP表示のあとEnter押すまで次の処理しないようにしたのですが、教

  • 解決済

    00

    oooooooooooooooooooooooooooooooooooooooo

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

  • Java

    13829questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • for

    231questions

    for文は、様々なプログラミング言語で使われている制御構造です。for文に定義している条件から外れるまで、for文内の命令文を繰り返し実行します。

  • if

    205questions

    if文とは様々なプログラミング言語で使用される制御構文の一種であり、条件によって処理の流れを制御します。