while文で処理したことにより、値が変わってしまったものを元に戻したい。

Question

メインメソッドからe=17993とn=90581受け取り、
以下のような処理を行いました。
これは連分数を計算するもので、計算して出た値は
すべてArrayListに格納しております。

    while( e % n > 0) {
      int t = n;
      n = e % n;
      e = t;
      renbunsu.add(e/n);
      int r = renbunsu.get(a);
      System.out.print("," + r);
      a = a + 1;
    }

続いて、次のメソッドで表示するものを書いたのですが、
eとnの値が変わり、n=3、e=1となってしまっています。

  void Show(){
    System.out.print(e + "/" + n + "=[" + e/n);
    for(int i = 1; i < b; i++){
      System.out.print("," + renbunsu.get(i));
    }
    System.out.println("]");
  }
}

while文の処理が終わった時点で、nとeの値をメインメソッドから
もらった値に戻したいです。

どうすればいいでしょうか？

Answer 1

状況がよくわからないですが、作業用の変数を用意すればいいのでは。

int we = e;
int wn = n;

while( we % wn > 0) {
  int t = wn;
  wn = we % wn;
  we = t;
  renbunsu.add(we/wn);
  int r = renbunsu.get(a);
  System.out.print("," + r);
  a = a + 1;
}

リファクタリング

メソッドを使えば値の変化に悩むことはなくなりますよ。メソッド外部へのアクセスは、

private ArrayList<Integer> numSeries = new ArrayList<Integer>();

private void computeSeries(int workNumerator, int workDenominator) {
    int surplus;

    numSeries.add(workNumerator/workDenominator);       // ここと、
    while((surplus = workNumerator % workDenominator) > 0) {
        workNumerator = workDenominator;
        workDenominator = surplus;
        
        numSeries.add(workNumerator/workDenominator);   // ここだけ
    }
}

変に横文字を使おうとして可読性が落ちているのはご愛嬌。

---

連分数の出力は他のメソッドで行った方がスマートです。
見た目にこだわるなら、こんな感じですかね。

public void printNumSeries() {
    if(numSeries.isEmpty()) return;

    System.out.print("[" + numSeries.get(0) + "; ");
    for(int num: numSeries.subList(1, numSeries.size()-1)) {
        System.out.print(num + ", ");
    }
    System.out.println(numSeries.get(numSeries.size()-1) + "]");
}

Comments

[tttmm81121]

こんな単純なことなのに気づきませんでした…
ほんとうにありがとうございます！！

[LouiS0616]

あ、合ってたんですね...
なにか私が勘違いしているのかと思っていました。

[LouiS0616]

あとひとつ、メソッド名はキャメルケースの方がいいですよ。
クラス名：PascalCase
メソッド名／フィールド名：camelCase

Answer 2

whileの中が正しければ、

• while文のスコープ内で新しくローカル変数を定義する
• nとeを別の変数に代入しておく

このどちらかで解決しませんか？

while文の中に違和感があったので、リファクタリングしようかとおもったのですが、うまい解決策を提示できませんでした。すみません。

Comments

[LouiS0616]

while文の中でローカル変数を定義して、何に使うんです？