60

循環的複雑度を改善するために、をチェックしながら、次ifのをswitch-に変更する必要があります。caseString

String value = some methodx;
if ("apple".equals(value)) {
    method1;
}

if ("carrot".equals(value)) {
    method2;
}

if ("mango".equals(value)) {
    method3;
}

if ("orange".equals(value)) {
    method4;
}

しかし、どのような価値が得られるのかわかりません。

4

13 に答える 13

173

Java(バージョン7より前)は、switch/caseのStringをサポートしていません。ただし、列挙型を使用すると、目的の結果を得ることができます。

private enum Fruit {
    apple, carrot, mango, orange;
}

String value; // assume input
Fruit fruit = Fruit.valueOf(value); // surround with try/catch

switch(fruit) {
    case apple:
        method1;
        break;
    case carrot:
        method2;
        break;
    // etc...
}
于 2012-04-20T05:09:43.760 に答える
20

使い方を学ぶelse

一度に2つの等しくない文字列に等しくなることは決してないためvalue、考えられる結果は5つだけです。1つは気になる値ごとに1つ、もう1つは「上記のどれでもない」です。ただし、コードは合格できないテストを排除しないため、16の「可能な」パス(2 ^テストの数)があり、そのほとんどがたどられることはありません。

を使用elseすると、存在するパスは実際に発生する可能性のある5つだけです。

String value = some methodx;
if ("apple".equals(value )) {
    method1;
}
else if ("carrot".equals(value )) {
    method2;
}
else if ("mango".equals(value )) {
    method3;
}
else if ("orance".equals(value )) {
    method4;
}

switchまたは、ステートメントで文字列を使用する機能を含むJDK7の使用を開始します。もちろん、Javaはとにかく/ likeコンストラクトにコンパイルするだけswitchです...ifelse

于 2012-04-20T05:09:34.640 に答える
20

現在、誰もが少なくともJava 7を使用していますよね?元の問題に対する答えは次のとおりです。

String myString = getFruitString();

switch (myString) {

    case "apple":
        method1();
        break;

    case "carrot":
        method2();
        break;

    case "mango":
        method3();
        break;

    case "orange":
        method4();
        break;
}

ノート

  • caseステートメントは、を使用するのと同じString.equalsです。
  • いつものように、文字列照合では大文字と小文字が区別されます。
  • ドキュメントによると、これは一般に、chained if--elseステートメントを使用するよりも高速です(cHaoの回答のように)。
于 2017-02-09T02:13:45.200 に答える
7

循環的複雑度を減らすには、マップを使用します。

Map<String,Callable<Object>> map = new HashMap < > ( ) ;
map . put ( "apple" , new Callable<Object> () { public Object call ( method1 ( ) ; return null ; } ) ;
...
map . get ( x ) . call ( ) ;

またはポリモーフィズム

于 2012-04-20T07:10:01.423 に答える
2

具体的なエモリーの答えを出すために、実行可能コードは次のとおりです。

  Map<String,Callable<USer>> map = new HashMap<String,Callable<User>>();
  map.put( "test" , new Callable<User> () { public User call (){ return fillUser("test" ); }} ) ;
  map.put( "admin" , new Callable<Utente> () { public Utente call (){  return fillUser("admin" ); }} ) ;

ここで、ユーザーはPOJOであり、

  User user = map.get(USERNAME).call();

最後に、呼び出されたメソッドはどこかにあります:

 private User fillUser(String x){        
        User user = new User();
        // set something in User
        return user;
}
于 2013-10-03T10:15:34.947 に答える
0

Javaは、文字列を使用したSwitch-caseをサポートしていません。このリンクはあなたを助けることができると思います。:)

于 2012-04-20T05:57:25.057 に答える
0

これが1.7より前の可能な方法ですが、私はお勧めできません:

public class PoorSwitch
{
    final static public int poorHash (String s) {
        long l = 0L;
        for (char c: s.toCharArray ()) {
            l = 97*l + c;
        }
        return (int) l;
    }

    public static void main (String args[])
    {
        String param = "foo";
        if (args.length == 1)
        {
            param = args[0];
        }
        // uncomment these lines, to evaluate your hash
        // test ("foo");
        // test ("bar");
        switch (poorHash (param)) {
            // this doesn't work, since you need a literal constant
            // so we have to evaluate our hash beforehand:
            // case poorHash ("foo"): {
            case 970596: {
                System.out.println ("Foo!");
                break;
            }
            // case poorHash ("bar"): {
            case 931605: {
                System.out.println ("Bar!");
                break;
            }
            default: {
                System.out.println ("unknown\t" + param);
                break;
            }
        }
    }

    public static void test (String s)
    {
        System.out.println ("Hash:\t " + s + " =\t" + poorHash (s));
    }
}

たぶん、生成されたコードでそのようなトリックを扱うことができます。そうでなければ私はそれをお勧めすることはできません。ハッシュの衝突の可能性が心配になるほどで​​はありませんが、何かが混同されている場合(カットアンドペースト)、エラーを見つけるのは困難です。931605は適切なドキュメントではありません。

好奇心と同じように、概念実証と同じように受け止めてください。

于 2012-04-20T09:06:16.567 に答える
0

Switchは、データ型互換のint:short、Shor、byte、Byte、int、Integer、char、Character、またはenum型にのみ適用できます。

于 2013-06-23T11:04:50.820 に答える
0

Stringswitchステートメントを使用した変数の評価はJavaSE7で実装されているため、Java7でのみ機能します。この新機能がJDK7でどのように実装されているかも確認できます。

于 2013-09-05T11:48:22.893 に答える
0

Java8は文字列switchcaseをサポートしています。

String type = "apple";

switch(type){
    case "apple":
       //statements
    break;
    default:
       //statements
    break; }
于 2018-04-13T21:45:25.667 に答える
0
    String name,lname;
 name= JOptionPane.showInputDialog(null,"Enter your name");
   lname= JOptionPane.showInputDialog(null,"Enter your father name");
    if(name.equals("Ahmad")){
       JOptionPane.showMessageDialog(null,"welcome "+name);
    }
    if(lname.equals("Khan"))
   JOptionPane.showMessageDialog(null,"Name : "+name +"\nLast name :"+lname ); 

    else {
       JOptionPane.showMessageDialog(null,"try again " );
    } 
  }}
于 2018-05-21T03:57:11.340 に答える
-1

あまりきれいではありませんが、ここに別の方法があります:

String runFct = 
        queryType.equals("eq") ? "method1":
        queryType.equals("L_L")? "method2":
        queryType.equals("L_R")? "method3":
        queryType.equals("L_LR")? "method4":
            "method5";
Method m = this.getClass().getMethod(runFct);
m.invoke(this);
于 2017-06-01T06:26:57.040 に答える
-11
String value = someMethod();
switch(0) {
default:
    if ("apple".equals(value)) {
        method1();
        break;
    }
    if ("carrot".equals(value)) {
        method2();
        break;
    }
    if ("mango".equals(value)) {
        method3();
        break;
    }
    if ("orance".equals(value)) {
        method4();
        break;
    }
}
于 2015-05-20T12:09:51.030 に答える