javaで数値や文字列の比較がしたい。

Question

お世話になっております。

javaの学習サイト等を回っていると数値や文字列の比較が必要となる問題によく当たるのですがこのような比較対象が3つ以上あるケースはif文以外で何かもっとスマートな書き方はあるでしょうか。

比較が二つしかない場合は
数値比較ですと　number1 == number2 ;
文字列比較ですと　str1.equals(str2);
と書くのが一番シンプルだという認識です。

もし下記のようなif文よりシンプルな記法がありましたらご教示いただきたいです。

    public static void main(String[] args) {
        Scanner sc = new Scanner(System.in);
        int N1 = 10; 
        int N2 = 10;
        int N3 = 10;
        
        System.out.println(check (N1,N2,N3));
        }
//自己定着の為メソッド化しただけです。
        public static String check(int X,int Y,int Z){
        if (X==Y && Y == Z){
            return "Same";
        }else{
            return "Different";
        }
    }

みなさま、多数のアドバイスをありがとうございます。
直接リプライを送るべきなのですが繰り広げられている議論に知識的についていけず的外れな回答になりそうなのでこちらでお礼申し上げます。

どれも興味が湧く内容でしたので別途試して理解できるよう学習をしていこうと思います。

Answer 1

質問の方法が一番シンプルなように思います。
ただ似たような判定を多用するのであれば、メソッドに切り出しても良いでしょう。

static boolean allSame(int... args) {
    if(args.length <= 1) {
        return true;
    }
        
    for(int i = 1; i < args.length; ++i) {
        if(args[i-1] != args[i]) {
            return false;
        }
    }
        
    return true;
}

こんなふうに使います。

jshell> allSame()
$ ==> true

jshell> allSame(1)
$ ==> true

jshell> allSame(1, 1)
$ ==> true

jshell> allSame(1, 1, 1)
$ ==> true

jshell> allSame(1, 1, 1, 2)
$ ==> false

配列をそのまま渡しても構いません。

jshell> int[] arr = {1, 1, 1, 1};
arr ==> int[4] { 1, 1, 1, 1 }

jshell> allSame(arr)
$ ==> true

---

StackOverflowにも同様の議論がありました。
様々な方法が提案されていますが、どれも直接比較よりは分かりづらい印象です。
Java all determine elements are same in a list - Stack Overflow

Comments

[hermit19901127]

連続されて入力されてくるだろう文字を単なる「個別に宣言した変数」として扱うか、「いったんArrayなりに加工した上」で扱うかにもよりますね…
そのうえでLouiS0616さんのおっしゃる通り、関数で処理などするなど、対応方法はいくらでもあるかと思う次第です…

[LouiS0616]

常用する分には可変長が一番使いやすいのかな、という印象です。
int... と Object... でオーバーロードするとすぐにエラーを吐くのが難点ですが。

[hermit19901127]

実はこちらにコメントして、後で気づいたのですが、「可変長」が引数として使えるのを完全に失念してた次第です…
最初に入った会社でJava使ってたのですが、Javaプログラマー失格です…お恥ずかしい…

N1、N2、N3…と際限なく単体のint型が宣言された上で、まとめて与えられると考えると
「不定の数の単体のint型が与えられる」状態に対応した、たしかにLouiS0616さんの提示する通りの関数を作るが最適解のように思えます…

[Zuishin]

おかしい。
> javaで数値や文字列の比較がしたい。
という質問なのになぜこちらが +2 で私の回答が 0 なんだろう。
Stream を使うのはなるべく避けるべきという理由や文化がありますか？

[LouiS0616]

> Stream を使うのはなるべく避けるべきという理由や文化がありますか？

たまにそのような主張をする人はいますが、納得できる理由を聞いたことはありません。
私が回答にStreamを用いていないのは、回答の主眼を『メソッド化』に置いていて、実装はあくまで例として提示しているだけだからです。

[Zuishin]

ありがとうございます。特に問題はないということですね。

[LouiS0616]

高評価数については当人に聞いてみないと何とも。
私自身は私の回答の方が特別優れているとは思っていません。

[Zuishin]

私は回答に問題なければそれで大丈夫です。何か間違っていてもわからないので聞いてみました。

[hermit19901127]

LouiS0616さんに高評価をいたしました…
その理由ですが、質問主さんの以降として「N1、N2、N3...」と現時点で「ああ…質問主、これを増やすのであろうな…」という様子に対し
LouiS0616さんの「可変長配列を使った関数」というソリューションが最も理解しやすい解答だなと思った次第です…。

もしかするとLouiS016さんの解答に問題があるかもしれないし
Zuishinの解答にメモリ効率や処理速度的な利点がある可能性もあるかもしれませんが、
質問主が求めるソリューションとして「わかりやすい概念ではないのか」と思った次第です…。

ここは、デキるプログラマとしては歯がゆい次第だと思いますが、
初心者にとって求めるのはメモリ効率や処理速度より「概念的に分かりやすいか」が重要だと思う次第です…

[LouiS0616]

一つだけ気になる点があるとしたらオートボクシングのオーバーヘッドくらいですが、実装を簡潔にするためには充分無視できる範囲かと思います。これを避けるとなると各プリミティブ型に対応したメソッドをいちいち用意しなければならないので...

[hermit19901127]

質問主のデータを入力時点で適切に処理すれば、Zuisinさんの解決策に乗せたり等、効率のいいソリューションに持っていけると思えますが、
現状、「N1、N2、N3…」と無駄にint型が増えていくんだろうな…という現実を垣間見た結果、これに対応できるのがLouiS0616さんの手法が「現実的」だろうなと思った次第です。

[Zuishin]

ジェネリクスを使ってもボックス化するというのが今ひとつ馴染めません。勉強しました。

[Zuishin]

hermit19901127 さん、私の回答読んでないでしょう。

[hermit19901127]

読んでないというより、質問主が求めてる解答に対して、貴殿の説明が、技巧がある上に処理速度やパフォーマンスの面で有利なのかもしれない印象はありますが、それ以前に
「これ、初心者が見て学びたい、または真似したいか？」という点で疑問に思った次第です…。
特に処理速度やパフォーマンス面の補足等が無い為、「なんか変態的なことをしているのだろう…」程度の印象しか抱けなかったです…。

そちらのプログラムに対する美学はあるかもしれませんが、個人的には
「ぱっと見が分かりやすいか」が大事だと思うのと、
それは、特にプログラムのプの字もわからない初心者さんであれば、なおさらだと思う次第です…

[Zuishin]

なるほど Stream は変態的で、それが書いてあるだけで初心者はアレルギーを起こすため、可変長引数が見えなくなるということがわかりました。

私の回答はこれに答えたものです。
> もし下記のようなif文よりシンプルな記法がありましたらご教示いただきたいです。

[hermit19901127]

分かればZuishinさんのコードがコード量が少なく、もっとも理解できますが、
そもそも、このような質問をする初心者に対して
「よー分からんけどコピペしよｗ」みたいな学を重んじない適当な人であればまぁいいのですが、
「可変長変数」どころか「ジェネリック」、さらに「Stream」まで理解を求めるような概念は、さすがにハードルが重い印象です…。
アレルギーというかアレですが、嫌いな食べ物がある人に対して、ある程度の段階は必要と言う印象ですね…。今回の場合、「可変長引数」のみの理解が、
無駄に増える個別のint定数（個人的にはまずこれを管理しろという印象）に対して、関数の宣言方法程度で対応できるLouiS0616さんのシンプルなやり方が「最適」ではないですが「相応」なのだと思った次第です…

Answer 2

こんなのはどうでしょうか。

Wandbox

import java.util.*;

class Wandbox
{
    public static void main(String[] args)
    {
        System.out.println(check(1, 1, 1)); // true
        System.out.println(check(1, 2, 3, 4)); // false
    }
    
    @SafeVarargs
    static <T> boolean check(T ... args)
    {
        return Arrays.stream(args).distinct().count() == 1;
    }
}

追記

ジェネリクスを使わないバージョン。

Wandbox(リンク先に実行結果があります)

import java.util.*;

class Wandbox
{
    public static void main(String[] args)
    {
        System.out.println(check(1, 1, 1)); // true
        System.out.println(check(1, 2, 2, 1, 1, 1)); // false
        System.out.println(check("abc", "abc", "abc")); // true
    }
    
    static boolean check(Object ... args) // 引数がいくつあっても配列として取得できる
    {
        return Arrays
            .stream(args)  // 可変長引数 args を Stream に変換
            .distinct()    // 重複した要素を削除([1, 2, 2, 1, 1, 1] → [1, 2])
            .count() == 1; // 残った要素数が一つであればすべて同じだったと判定
    }
}

Stream API について

Stream API は関数型プログラミングの手法を取り入れた、モダンで読みやすく堅牢なプログラミングを可能にする技術です。これからのプログラマーは必ずマスターしましょう。

むしろ、これが読めないと、今後は老害と呼ばれたり、人よりプログラミング効率が落ちたり仕事が減ったりと、困ったことになります。決して一部の人が誤解しているような、特殊な人が特殊な趣味で使うものではありませんし、新しすぎる尖った手法でもありません。Java 8 から存在し、ごくごく普通に使われているものです。

参考: Java 11 時代の Java プログラミングスタイルガイド - Qiita

極力 for 文を使用せず、Stream API によって処理する。

Java 8 での Stream API の出現によって for 文が活躍する機会はほとんどなくなった。Stream API の宣言的な記述はうまく使用すれば可読性が高く、パフォーマンスの観点からも優れるため、例外的な場面を除いてコレクション操作は Stream API によって行うのがよいだろう。

Comments

[LouiS0616]

ジェネリクスを使わずとも、Object... で受け取って充分では？

[Zuishin]

ジェネリクスを使っても Object になるんですね。C# であれば check(1, "abc") のように違う型の引数を渡すとコンパイルエラーにしてくれるので。
それなら Object でもいいと思います。

[Zuishin]

ジェネリクスを使わないコードにコメントを入れ、また Stream API について追記しました。
引用しているのは Qiita の記事で、公式のものではありません。読まれた方が自分で判断してください。

Answer 3

入出力部分で、どうデータを整形するにもよりますが…

        int N1 = 10; 
        int N2 = 10;
        int N3 = 10;
        
        int[] ia = new int[] { N1 , N2 , N3 } ;
        
        boolean allEqual = true ;
        
        for ( int i = 0 ; i < ia.length ; i++ )
        {
            if( ia[i] != N1 ) { allEqual = false ; break ; }
        }
        
        System.out.println( allEqual );

こう記述すれば、N4,N5と増えようが、iaに追加し、
N1を基準として、１つでも違うデータがあれば、allEqualがfalseを返すと思います…

Answer 4

Streamを使うにしても、「すべて等しい」ならこっちの書き方のほうが自然だと思います。

public static boolean allSame(int... values) {
    if(values.length == 0) {
        return true;  // ここは要件次第
    }
    return IntStream.of(values).allMatch(v -> v == values[0]);
}

なお、一般の型に対してはnullセーフも考えて

public static boolean allSame(Object... values) {
    if(values.length == 0) {
        return true;  // ここは要件次第
    }
    return Stream.of(values).allMatch(v -> Objects.equals(v, values[0]));
}

Comments

[Zuishin]

> もし下記のようなif文よりシンプルな記法がありましたらご教示いただきたいです。

という質問のはずだったので。

Answer 5

問題が有ったので削除しました。申し訳ないm(")M