web-dev-qa-db-ja.com

Integer.parseInt()をカプセル化する良い方法

Stringをintに変換するためにInteger.parseInt()をよく使用するプロジェクトがあります。問題が発生した場合(たとえば、Stringは数字ではなく、文字aなど)、このメソッドは例外をスローします。ただし、コード内のすべての場所で例外を処理する必要がある場合、これは非常にquicklyく見え始めます。これをメソッドに入れたいのですが、変換が失敗したことを示すためにクリーンな値を返す方法がわかりません。

C++では、intへのポインタを受け入れるメソッドを作成し、メソッド自体がtrueまたはfalseを返すようにすることができました。ただし、私が知る限り、これはJavaでは不可能です。 true/false変数と変換された値を含むオブジェクトを作成することもできますが、これも理想的ではありません。同じことがグローバルな値にも当てはまります。これにより、マルチスレッドで問題が発生する可能性があります。

これを行うためのクリーンな方法はありますか?

83
Bad Horse

Integerの代わりにintを返すことができ、解析の失敗時にnullを返します。

それは残念ですJavaは、内部的にスローされる例外がなくてもこれを行う方法を提供しません-例外を隠すことができます(キャッチしてnullを返すことで)が、それでもパフォーマンスの問題になる可能性があります数十万ビットのユーザー提供データを解析している場合。

編集:そのようなメソッドのコード:

public static Integer tryParse(String text) {
  try {
    return Integer.parseInt(text);
  } catch (NumberFormatException e) {
    return null;
  }
}

textがnullの場合、これが何をするのかわからないことに注意してください。あなたはそれを考慮する必要があります-それがバグを表す場合(つまり、コードが無効な値を渡す可能性が高いが、決してnullを渡してはならない)、例外をスローするのが適切です;バグを表していない場合は、他の無効な値の場合と同様に、おそらくnullを返す必要があります。

もともと、この答えはnew Integer(String)コンストラクターを使用していました。 Integer.parseIntとボクシング操作を使用するようになりました。この方法では、小さな値がキャッシュされたIntegerオブジェクトにボックス化され、そのような状況でより効率的になります。

132
Jon Skeet

数字ではない場合、どのような動作を期待しますか?

たとえば、入力が数値ではないときに使用するデフォルト値がある場合は、次のような方法が便利です。

public static int parseWithDefault(String number, int defaultVal) {
  try {
    return Integer.parseInt(number);
  } catch (NumberFormatException e) {
    return defaultVal;
  }
}

入力を解析できない場合、異なるデフォルトの動作に対して同様のメソッドを作成できます。

34
Joachim Sauer

場合によっては、解析エラーをフェイルファーストの状況として処理する必要がありますが、アプリケーション構成などの他の場合では、 Apache Commons Lang 3 NumberUtils を使用して、欠落している入力をデフォルト値で処理することを好みます。

int port = NumberUtils.toInt(properties.getProperty("port"), 8080);
24
Bryan W. Wagner

例外の処理を回避するには、正規表現を使用して、最初にすべての数字があることを確認してください。

if(value.matches("\\d+") {
    Integer.parseInt(value);
}
15
Daddyboy

Guava にはInts.tryParse()があります。数値以外の文字列では例外をスローしませんが、null文字列では例外をスローします。

10
husayt

質問への回答を読んだ後、parseIntメソッドをカプセル化またはラップする必要はないと思います。

Jonが示唆したように 'null'を返すこともできますが、それは多かれ少なかれtry/catch構造をnullチェックに置き換えます。エラー処理を「忘れる」場合、動作にわずかな違いがあります。例外をキャッチしない場合、割り当てはなく、左側の変数は古い値を保持します。 nullをテストしないと、JVM(NPE)にヒットする可能性があります。

ヤーンの提案は私にはもっとエレガントに見えます。なぜなら、いくつかのエラーや例外的な状態を知らせるためにnullを返すのは嫌だからです。ここで、問題を示す定義済みオブジェクトとの参照の等価性を確認する必要があります。しかし、他の人が主張するように、再びチェックを忘れて文字列が解析できない場合、プログラムは「ERROR」または「NULL」オブジェクト内のラップされたintを継続します。

Nikolayのソリューションはさらにオブジェクト指向であり、他のラッパークラスのparseXXXメソッドでも機能します。しかし、最終的に、彼はNumberFormatExceptionをOperationNotSupported例外に置き換えただけです-解析できない入力を処理するにはtry/catchが必要です。

したがって、単純なparseIntメソッドをカプセル化しないという私の結論です。 (アプリケーションに依存する)エラー処理も追加できる場合にのみカプセル化します。

4
Andreas_D

次のようなものを使用できます。

public class Test {
public interface Option<T> {
    T get();

    T getOrElse(T def);

    boolean hasValue();
}

final static class Some<T> implements Option<T> {

    private final T value;

    public Some(T value) {
        this.value = value;
    }

    @Override
    public T get() {
        return value;
    }

    @Override
    public T getOrElse(T def) {
        return value;
    }

    @Override
    public boolean hasValue() {
        return true;
    }
}

final static class None<T> implements Option<T> {

    @Override
    public T get() {
        throw new UnsupportedOperationException();
    }

    @Override
    public T getOrElse(T def) {
        return def;
    }

    @Override
    public boolean hasValue() {
        return false;
    }

}

public static Option<Integer> parseInt(String s) {
    Option<Integer> result = new None<Integer>();
    try {
        Integer value = Integer.parseInt(s);
        result = new Some<Integer>(value);
    } catch (NumberFormatException e) {
    }
    return result;
}

}
4
Nikolay Ivanov

また、非常に簡単にしたいC++の動作を複製することもできます。

public static boolean parseInt(String str, int[] byRef) {
    if(byRef==null) return false;
    try {
       byRef[0] = Integer.parseInt(prop);
       return true;
    } catch (NumberFormatException ex) {
       return false;
    }
}

次のようなメソッドを使用します。

int[] byRef = new int[1];
boolean result = parseInt("123",byRef);

その後、変数resultがすべて正常に進み、byRef[0]に解析された値が含まれている場合はtrueです。

個人的には、例外をキャッチすることに固執します。

2
Wilian Z.

私のJavaは少し錆びていますが、正しい方向に向けることができるかどうか見てみましょう。

public class Converter {

    public static Integer parseInt(String str) {
        Integer n = null;

        try {
            n = new Integer(Integer.tryParse(str));
        } catch (NumberFormatException ex) {
            // leave n null, the string is invalid
        }

        return n;
    }

}

戻り値がnullの場合、不正な値があります。それ以外の場合、有効なIntegerがあります。

1
Adam Maras

Jon Skeetによる答えは問題ありませんが、null Integerオブジェクトを返すのは好きではありません。私はこれを使用するのがわかりにくいと思います。 Java 8以降、(私の意見では)OptionalIntを使用するより良いオプションがあります:

public static OptionalInt tryParse(String value) {
 try {
     return OptionalInt.of(Integer.parseInt(value));
  } catch (NumberFormatException e) {
     return OptionalInt.empty();
  }
}

これにより、値が利用できない場合を処理する必要があることが明示されます。この種の関数が将来Javaライブラリに追加されることを希望しますが、それが起こるかどうかはわかりません。

1
Marc

Java 8以降を使用している場合は、リリースしたばかりのライブラリ https://github.com/robtimus/try-parse を使用できます。例外のキャッチに依存しないint、long、およびbooleanのサポートがあります。 GuavaのInts.tryParseとは異なり、 https://stackoverflow.com/a/38451745/1180351 のようにOptionalInt/OptionalLong/Optionalを返しますが、より効率的です。

1
Rob Spoor

parseIntの分岐メソッドはどうですか?

簡単です。IntegerまたはOptional<Integer>を返す新しいユーティリティに内容をコピーして貼り付け、スローをリターンに置き換えます。基礎となるコードに例外はないようですが、より良いチェックです。

例外処理をすべてスキップすることで、無効な入力の時間を節約できます。また、このメソッドはJDK 1.0以降に存在するため、最新の状態を維持するために多くのことを行う必要はほとんどありません。

1
Vlasec

私がこの問題を処理する方法は再帰的です。たとえば、コンソールからデータを読み取る場合:

Java.util.Scanner keyboard = new Java.util.Scanner(System.in);

public int GetMyInt(){
    int ret;
    System.out.print("Give me an Int: ");
    try{
        ret = Integer.parseInt(keyboard.NextLine());

    }
    catch(Exception e){
        System.out.println("\nThere was an error try again.\n");
        ret = GetMyInt();
    }
    return ret;
}
0
Boboman

例外を回避するには、JavaのFormat.parseObjectメソッドを使用できます。以下のコードは、基本的にApache Commonの IntegerValidator クラスの簡易バージョンです。

public static boolean tryParse(String s, int[] result)
{
    NumberFormat format = NumberFormat.getIntegerInstance();
    ParsePosition position = new ParsePosition(0);
    Object parsedValue = format.parseObject(s, position);

    if (position.getErrorIndex() > -1)
    {
        return false;
    }

    if (position.getIndex() < s.length())
    {
        return false;
    }

    result[0] = ((Long) parsedValue).intValue();
    return true;
}

好みに応じて、AtomicIntegerまたはint[]配列トリックを使用できます。

これを使用するテストは次のとおりです-

int[] i = new int[1];
Assert.assertTrue(IntUtils.tryParse("123", i));
Assert.assertEquals(123, i[0]);
0
Josh Unger

私も同じ問題を抱えていました。これは、ユーザーに入力を要求し、整数でない限り入力を受け入れないように作成したメソッドです。私は初心者なので、コードが期待どおりに機能しない場合は、私の経験不足を非難してください!

private int numberValue(String value, boolean val) throws IOException {
    //prints the value passed by the code implementer
    System.out.println(value);
    //returns 0 is val is passed as false
    Object num = 0;
    while (val) {
        num = br.readLine();
        try {
            Integer numVal = Integer.parseInt((String) num);
            if (numVal instanceof Integer) {
                val = false;
                num = numVal;
            }
        } catch (Exception e) {
            System.out.println("Error. Please input a valid number :-");
        }
    }
    return ((Integer) num).intValue();
}
0
Abhinav Mathur

次のような方法を検討することをお勧めします

 IntegerUtilities.isValidInteger(String s)

必要に応じて実装します。結果を持ち帰りたい場合-おそらくとにかくInteger.parseInt()を使用するため-配列トリックを使用できます。

 IntegerUtilities.isValidInteger(String s, int[] result)

result [0]をプロセスで見つかった整数値に設定します。

これは、質問8391979、「Javaには不正なデータに対して例外をスローしないint.tryparseがありますか?[重複]」に対する回答であり、この質問にリンクされています。

Edit 2016 08 17:ltrimZeroesメソッドを追加し、tryParse()で呼び出しました。 numberStringに先行ゼロがないと、誤った結果になる場合があります(コードのコメントを参照)。また、正および負の「数値」に対して機能するpublic static String ltrimZeroes(String numberString)メソッドもあります(END Edit)

以下に、文字列自体を解析し、JavaのInteger.parseInt(String s)よりも少し高速な、非常に高速に最適化されたtryParse()メソッド(C#と同様)を備えたintの初歩的なWrapper(ボクシング)クラスを見つけます。

public class IntBoxSimple {
    // IntBoxSimple - Rudimentary class to implement a C#-like tryParse() method for int
    // A full blown IntBox class implementation can be found in my Github project
    // Copyright (c) 2016, Peter Sulzer, Fürth
    // Program is published under the GNU General Public License (GPL) Version 1 or newer

    protected int _n; // this "boxes" the int value

    // BEGIN The following statements are only executed at the
    // first instantiation of an IntBox (i. e. only once) or
    // already compiled into the code at compile time:
    public static final int MAX_INT_LEN =
            String.valueOf(Integer.MAX_VALUE).length();
    public static final int MIN_INT_LEN =
            String.valueOf(Integer.MIN_VALUE).length();
    public static final int MAX_INT_LASTDEC =
            Integer.parseInt(String.valueOf(Integer.MAX_VALUE).substring(1));
    public static final int MAX_INT_FIRSTDIGIT =
            Integer.parseInt(String.valueOf(Integer.MAX_VALUE).substring(0, 1));
    public static final int MIN_INT_LASTDEC =
            -Integer.parseInt(String.valueOf(Integer.MIN_VALUE).substring(2));
    public static final int MIN_INT_FIRSTDIGIT =
            Integer.parseInt(String.valueOf(Integer.MIN_VALUE).substring(1,2));
    // END The following statements...

    // ltrimZeroes() methods added 2016 08 16 (are required by tryParse() methods)
    public static String ltrimZeroes(String s) {
        if (s.charAt(0) == '-')
            return ltrimZeroesNegative(s);
        else
            return ltrimZeroesPositive(s);
    }
    protected static String ltrimZeroesNegative(String s) {
        int i=1;
        for ( ; s.charAt(i) == '0'; i++);
        return ("-"+s.substring(i));
    }
    protected static String ltrimZeroesPositive(String s) {
        int i=0;
        for ( ; s.charAt(i) == '0'; i++);
        return (s.substring(i));
    }

    public static boolean tryParse(String s,IntBoxSimple intBox) {
        if (intBox == null)
            // intBoxSimple=new IntBoxSimple(); // This doesn't work, as
            // intBoxSimple itself is passed by value and cannot changed
            // for the caller. I. e. "out"-arguments of C# cannot be simulated in Java.
            return false; // so we simply return false
        s=s.trim(); // leading and trailing whitespace is allowed for String s
        int len=s.length();
        int rslt=0, d, dfirst=0, i, j;
        char c=s.charAt(0);
        if (c == '-') {
            if (len > MIN_INT_LEN) { // corrected (added) 2016 08 17
                s = ltrimZeroesNegative(s);
                len = s.length();
            }
            if (len >= MIN_INT_LEN) {
                c = s.charAt(1);
                if (!Character.isDigit(c))
                    return false;
                dfirst = c-'0';
                if (len > MIN_INT_LEN || dfirst > MIN_INT_FIRSTDIGIT)
                    return false;
            }
            for (i = len - 1, j = 1; i >= 2; --i, j *= 10) {
                c = s.charAt(i);
                if (!Character.isDigit(c))
                    return false;
                rslt -= (c-'0')*j;
            }
            if (len < MIN_INT_LEN) {
                c = s.charAt(i);
                if (!Character.isDigit(c))
                    return false;
                rslt -= (c-'0')*j;
            } else {
                if (dfirst >= MIN_INT_FIRSTDIGIT && rslt < MIN_INT_LASTDEC)
                    return false;
                rslt -= dfirst * j;
            }
        } else {
            if (len > MAX_INT_LEN) { // corrected (added) 2016 08 16
                s = ltrimZeroesPositive(s);
                len=s.length();
            }
            if (len >= MAX_INT_LEN) {
                c = s.charAt(0);
                if (!Character.isDigit(c))
                    return false;
                dfirst = c-'0';
                if (len > MAX_INT_LEN || dfirst > MAX_INT_FIRSTDIGIT)
                    return false;
            }
            for (i = len - 1, j = 1; i >= 1; --i, j *= 10) {
                c = s.charAt(i);
                if (!Character.isDigit(c))
                    return false;
                rslt += (c-'0')*j;
            }
            if (len < MAX_INT_LEN) {
                c = s.charAt(i);
                if (!Character.isDigit(c))
                    return false;
                rslt += (c-'0')*j;
            }
            if (dfirst >= MAX_INT_FIRSTDIGIT && rslt > MAX_INT_LASTDEC)
                return false;
            rslt += dfirst*j;
        }
        intBox._n=rslt;
        return true;
    }

    // Get the value stored in an IntBoxSimple:
    public int get_n() {
        return _n;
    }
    public int v() { // alternative shorter version, v for "value"
        return _n;
    }
    // Make objects of IntBoxSimple (needed as constructors are not public):
    public static IntBoxSimple makeIntBoxSimple() {
        return new IntBoxSimple();
    }
    public static IntBoxSimple makeIntBoxSimple(int integerNumber) {
        return new IntBoxSimple(integerNumber);
    }

    // constructors are not public(!=:
    protected IntBoxSimple() {} {
        _n=0; // default value an IntBoxSimple holds
    }
    protected IntBoxSimple(int integerNumber) {
        _n=integerNumber;
    }
}

クラスIntBoxSimpleのテスト/サンプルプログラム:

import Java.io.BufferedReader;
import Java.io.IOException;
import Java.io.InputStreamReader;
public class IntBoxSimpleTest {
    public static void main (String args[]) {
        IntBoxSimple ibs = IntBoxSimple.makeIntBoxSimple();
        String in = null;
        BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
        do {
            System.out.printf(
                    "Enter an integer number in the range %d to %d:%n",
                        Integer.MIN_VALUE, Integer.MAX_VALUE);
            try { in = br.readLine(); } catch (IOException ex) {}
        } while(! IntBoxSimple.tryParse(in, ibs));
        System.out.printf("The number you have entered was: %d%n", ibs.v());
    }
}
0
Peter Sulzer

正規表現とデフォルトのパラメーター引数を試してください

public static int parseIntWithDefault(String str, int defaultInt) {
    return str.matches("-?\\d+") ? Integer.parseInt(str) : defaultInt;
}


int testId = parseIntWithDefault("1001", 0);
System.out.print(testId); // 1001

int testId = parseIntWithDefault("test1001", 0);
System.out.print(testId); // 1001

int testId = parseIntWithDefault("-1001", 0);
System.out.print(testId); // -1001

int testId = parseIntWithDefault("test", 0);
System.out.print(testId); // 0

apache.commons.lang3を使用している場合は、 NumberUtils を使用します。

int testId = NumberUtils.toInt("test", 0);
System.out.print(testId); // 0
0
Krunal

あなたはcouldできますが、commons langのStringUtils.isNumeric()method を使用するのと同じくらい簡単です。 Character.isDigit() を使用して、ストリング内の各文字を反復処理します。

0
James Bassett

私は特に整数を要求する場合に機能する別の提案を投げ込みたいと思います:単にlongを使用し、エラーの場合はLong.MIN_VALUEを使用します。これは、Reader.read()がcharの範囲の整数を返すReaderの文字に使用されるアプローチに似ています。リーダーが空の場合は-1を返します。

FloatおよびDoubleの場合、NaNは同様の方法で使用できます。

public static long parseInteger(String s) {
    try {
        return Integer.parseInt(s);
    } catch (NumberFormatException e) {
        return Long.MIN_VALUE;
    }
}


// ...
long l = parseInteger("ABC");
if (l == Long.MIN_VALUE) {
    // ... error
} else {
    int i = (int) l;
}
0
Searles

値の検証に例外を使用しないでください

単一の文字の場合、簡単な解決策があります。

Character.isDigit()

値が長い場合は、いくつかのユーティリティを使用することをお勧めします。 Apacheが提供するNumberUtilsはここで完全に機能します。

NumberUtils.isNumber()

確認してください https://commons.Apache.org/proper/commons-lang/javadocs/api-2.6/org/Apache/commons/lang/math/NumberUtils.html

0

これは、ニコライのソリューションにいくらか似ています。

 private static class Box<T> {
  T me;
  public Box() {}
  public T get() { return me; }
  public void set(T fromParse) { me = fromParse; }
 }

 private interface Parser<T> {
  public void setExclusion(String regex);
  public boolean isExcluded(String s);
  public T parse(String s);
 }

 public static <T> boolean parser(Box<T> ref, Parser<T> p, String toParse) {
  if (!p.isExcluded(toParse)) {
   ref.set(p.parse(toParse));
   return true;
  } else return false;
 }

 public static void main(String args[]) {
  Box<Integer> a = new Box<Integer>();
  Parser<Integer> intParser = new Parser<Integer>() {
   String myExclusion;
   public void setExclusion(String regex) {
    myExclusion = regex;
   }
   public boolean isExcluded(String s) {
    return s.matches(myExclusion);
   }
   public Integer parse(String s) {
    return new Integer(s);
   }
  };
  intParser.setExclusion("\\D+");
  if (parser(a,intParser,"123")) System.out.println(a.get());
  if (!parser(a,intParser,"abc")) System.out.println("didn't parse "+a.get());
 }

Mainメソッドはコードをデモします。 Parserインターフェースを実装する別の方法は、明らかに、構築から「\ D +」を設定し、メソッドに何もさせないことです。

0
Carl