引数の受け取り方について

Question

前提・実現したいこと

kadaimainクラスでkadaiクラスにあるunionメソッドを使用します。
unionメソッドで実現したいことはメソッドに入っているr1（x1,y1,x2,y2)と引数として与えたr2（x1,y1,x2,y2）を変数名が一致しているもの同士で比較するというメソッドです。r3とr4を使って同様の処理も行いたいです。

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

unionメソッド上でどうやって引数を受け取り、r1とr2に入っている数字の大小関係を比べるか教えていただきたいです。

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

java
public class kadaiMain {
	public static void main(String[] args) {
		kadai r1 = new kadai(1, 2, 3, 4);
		kadai r2 = new kadai(2, 3, 5, 6);
　　　　　　　　　kadai r3 = new kadai(1, 4, 6, 7);
		kadai r4 = new kadai(3, 3, 1, 6);
		kadai u1 = r1.union(r2);
                kadai u1 = r3.union(r4);
		u1.println();
		}

}

-----------------------------------------------------------------------
public class kadai {

 private int x1;
 private int y1;
 private int x2;
 private int y2;

 public kadai(int x1,int y1,int x2,int y2){

  this.x1 = x1;
  this.y1 = y1;
  this.x2 = x2;
  this.y2 = y2;
 }

  public kadai union(kadai r) {
   kadai uniX1 = Math.min(x1, x1); //一つ目のx1にr1の1、二つ目のx1にr2の2
　　　　　　　　　　　　　　　　　　　   を代入したい。　　
   
   kadai uniY1 = Math.min(y1, y1);

   kadai uniX2 = Math.max(x2, x2);
   
   kadai uniY2 = Math.max(y2, y2);


   

    
  return    //kadai型の四つの値が並んだ形
  }

  public void plintln() {

  System.out.println("[" + uniX1 +"," + uniY1 ":" uniX2 + "," + uniY2 + "]")

試したこと

ここに問題に対して試したことを記載してください。

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

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

Comments

[m.ts10806]

コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

[yamamoto111]

ご指摘ありがとうございます。以後気を付けます。

Answer 1

kadai uniX1 = Math.min(x1, x1);となっていますが、Math.minで得られる結果は数値であってkadaiではありません。

Comments

[maisumakun]

unionはkadaiのメソッドですので、r2.x1のようなprivateへのアクセスが可能です。

[yamamoto111]

回答ありがとうございます。　質問させていただきたいことがあります。
写しミスでメソッド側の引数がr2に限定されていました。
kadaimain 内　に r3(2, 2, 4, 4) r4 (5, 2, 3, 9)を作成しこの二つで同様の処理を行いたい場合メソッド側の引数をr2に限定してしまうとできないと考えているのですがどう表現するのが適切ですか？

[maisumakun]

> 写しミスでメソッド側の引数がr2に限定されていました。

限定も何も、もとからメソッドの引数は1つしかないので、特に条件は変わらないかと思います。

[yamamoto111]

下記のように書き換えればよいという認識でよろしいでしょうか？

　　　public Rect union(Rect r) {

　　　Rect uniX1 = Math.min(x1, r.x1);

　　　Rect uniY1 = Math.min(y1, r.y1);

　　Rect uniX2 = Math.max(x2, r.x2);

　　Rect uniY2 = Math.max(y2, r.y2);

[gentaro]

「こうかな？」と思う方法があればまず自分で試して、それで望み通りの動作をしてるのかどうかぐらい確認してから質問しましょう。
自分で試してわからない点があったのならそれを別途質問すればいいだけです。

[yamamoto111]

maisumakunの指摘を参考にして無事に解決できました。
ありがとうございました。