Quelle est la bonne façon de traiter une NumberFormatException lorsqu'il est prévu?

Je suis en cours d'exécution dans cette situation où j'ai besoin d'analyser un String dans un int et je ne sais pas quoi faire avec le NumberFormatException. Le compilateur ne se plaint pas lorsque je n'ai pas l'attraper, mais je veux juste m'assurer que je suis de la manipulation de cette situation correctement.

private int getCurrentPieceAsInt() {
    int i = 0;
    try {
        i = Integer.parseInt(this.getCurrentPiece());
    } catch (NumberFormatException e) {
        i = 0;
    }
    return i;
}

Je veux juste simplifier mon code comme ceci. Le compilateur n'a pas un problème avec elle, mais le thread meurt sur la NumberFormatException.

private int getCurrentPieceAsInt() {
    int i = 0;
    i = Integer.parseInt(this.getCurrentPiece());
    return i;
}

Google CodePro veut me connecter à l'exception d'une certaine façon, et je suis d'accord que c'est la meilleure pratique.

private int getCurrentPieceAsInt() {
    int i = 0;
    try {
        i = Integer.parseInt(this.getCurrentPiece());
    } catch (NumberFormatException e) {
        i = 0;
        e.printStackTrace();
    }
    return i;
}

Je veux cette méthode pour retourner 0 lorsque la pièce n'est pas un nombre ou ne peut pas être analysé. Quand je n'attrape pas la NumberFormatException explicitement, n'est-il pas attribuer à la variable i? Ou est-il une valeur par défaut qui Integer.parseInt() retourne?

Style général dit que si j'attrape une exception, je devrais l'enregistrer quelque part. Je ne veux pas de journal. C'est un fonctionnement normal de cette exception, parfois, qui également ne pas s'asseoir bien avec moi. Je ne trouve pas une fonction, cependant, qui me dira si Integer.parseInt() lèvera une exception. Donc, mon seul recours semble être juste de l'appeler et d'intercepter l'exception.

La javadoc pour parseInt ne l'aide pas beaucoup.

Voici les questions que je voudrais savoir:

  • Est-il une méthode que je peux appeler qui va me dire si Integer.parseInt() va jeter un NumberFormatException avant de l'appeler? Ensuite, je n'aurais pas de problème de connexion, puisque cela ne devrait jamais se produire.
  • Si je n'ai tout simplement pas attraper l'exception, l'valiable pas assigné? Alors je vais simplement l'initialiser à la valeur que je veux quand ce n'est pas un nombre et ne pas intercepter l'exception.
  • Est-il un moyen de marquer l'exception d'une certaine manière explicite que je ne m'inquiète pas à ce sujet? Je suis pensant que ce serait quelque chose de similaire AWTEvent.consume(). Si oui, alors je vais faire ce que Google CodePro ne voyons pas cela comme "anonymes".
"Si je n'ai tout simplement pas attraper l'exception, l'valiable pas assigné? Alors je vais tout simplement pas attraper l'exception." - si vous n'êtes pas sûr de savoir si c'est une option viable, je vous suggère de l'essayer (et de procédure pas à pas avec un débogueur) pour être 100% sûr que vous comprenez ce qui se passe dans ce cas. Je ne veux pas sonner comme je parle bas, mais j'ai l'impression que d'avoir une solide compréhension des exceptions est important.
Je ne suis pas un programmeur Java, mais en C#, Entier a une TryParse() méthode qui tente de l'analyser de l'int et renvoie un booléen si elle a réussi. Sûrement, c'est mieux que d'attendre une exception.
J'aimerais si il y avait un tryParse() méthode. Je crois que je comprends pourquoi il n'y a pas, puisqu'il s'agit essentiellement de faire le travail deux fois. Aussi, quand je l'ai essayer, la portion non utilisée du NumberFormatException tue le fil sur le spot. J'ai mis à jour la question pour en tenir compte.

OriginalL'auteur Erick Robertson | 2010-12-10

  1. 13
    • Est-il une méthode que je peux appeler qui va me dire si le nombre Entier.parseInt() va lancer une NumberFormatException avant de l'appeler? Ensuite, je n'aurais pas de problème de connexion, puisque cela ne devrait jamais se produire.

    Malheureusement, non. Au moins pas dans le noyau de l'API Java. Il est facile d'en écrire un, cependant - il suffit de modifier le code ci-dessous.

    • Si je n'ai tout simplement pas attraper l'exception, l'valiable pas assigné? Alors je vais simplement l'initialiser à la valeur que je veux quand ce n'est pas un nombre et ne pas intercepter l'exception.

    Si vous ne le faites pas l'exception, puis la pile détendre jusqu'à ce qu'il frappe un bloc catch qui va la traiter, ou il va se détendre complètement et arrêter le fil. La variable sera, en fait, ne pas être affecté, mais ce n'est pas exactement ce que vous voulez.

    • Est-il un moyen de marquer l'exception d'une certaine manière explicite que je ne m'inquiète pas à ce sujet? Je suis pensant que ce serait quelque chose de similaire AWTEvent.consommer(). Si oui, alors je vais faire ce que Google CodePro ne voyons pas cela comme "anonymes".

    Il y a peut être un moyen de dire CodePro d'ignorer cet avertissement. Certainement avec des outils comme FindBugs et Checkstyle vous pouvez désactiver les avertissements dans des endroits spécifiques. (EDIT: @Andy a souligné comment le faire.)

    Je soupçonne ce que vous voulez, c'est quelque chose comme l'Commons lang paquet mentionné par @daveb. Il est assez facile d'écrire une telle fonction:

    int parseWithDefault(String s, int def) {
    try {
        return Integer.parseInt(s);
    }
    catch (NumberFormatException e) {
        //It's OK to ignore "e" here because returning a default value is the documented behaviour on invalid input.
        return def;
    }
}
    CodePro est incorrect en nécessitant un enregistrement pour chaque prise. Dans ce cas, il est parfaitement normal de manger de l'exception, car c'est souhaitée et documenté comportement de la méthode.

    OriginalL'auteur Cameron Skinner

  2. 9

    Il est NumberUtils.toInt(String, int) dans commons lang qui va faire exactement ce que vous voulez.

    NumberUtils.toInt("123", 42) ==> 123
NumberUtils.toInt("abc", 42) ==> 42
    Je ne peux pas le faire d'une certaine façon, sans l'importation de Apache Commons?
    Yep, mais le nombre de projets commons lang ne soyez donc pas besoin de rouler leur propre impl.
    J'ai évité jusqu'à présent de faire, soit.
    NumberUtils juste l'attrape. Ne pas obtenir autour de l'attraper. 🙂

    OriginalL'auteur daveb

  3. 3
    * Is there a way to mark the exception somehow explicitly that I don't care about it? I'm thinking this would be something similar to AWTEvent.consume(). If so, then I will do this so that Google CodePro doesn't see this as "unlogged".

    Oui, vous pouvez désactiver localement un CodePro règle d'audit pour une ligne de code:

    http://code.google.com/javadevtools/codepro/doc/features/audit/locally_disabling_audit_rules.html

    Cela dit, il n'est pas nécessaire d'inclure la journalisation des diagnostics dans chaque exception bloc catch. Parfois, le mieux est de prendre un défaut cours. Parfois, il est d'interagir avec l'utilisateur. Il dépend.

    OriginalL'auteur Andy Thomas

  4. 1

    Créer votre propre méthode de convenance pour maintenant et pour le futur:

    public static int parseInt(final /*@Nullable*/ String s, final int valueIfInvalid) {
    try {
        if (s == null) {
            return valueIfInvalid;
        } else {
            return Integer.parseInt(s);
        }
    } catch (final NumberFormatException ex) {
        return valueIfInvalid;
    }
}

    Est-il une méthode que je peux appeler qui va me dire si le nombre Entier.parseInt() va lancer une NumberFormatException avant de l'appeler? Ensuite, je n'aurais pas de problème de connexion, puisque cela ne devrait jamais se produire.

    Pas que je suis au courant. Gardez à l'esprit que si il y avait susceptible de mettre fin à l'analyse de la valeur deux fois (une fois pour valider et une fois pour l'analyser). Je comprends que vous voulez éviter d'exception, mais dans ce cas, c'est la capture de l'exception est le standard dans le langage Java, et il ne fournit pas un autre (au moins que je sache).

    Si je n'ai tout simplement pas attraper l'exception, l'valiable pas assigné? Alors je vais simplement l'initialiser à la valeur que je veux quand ce n'est pas un nombre et ne pas intercepter l'exception.

    Vous devez attraper l'exception (même si elle ne fait rien), ou il va s'échapper de la bloquer et de les jeter à travers la pile.

    Est-il un moyen de marquer l'exception d'une certaine manière explicite que je ne m'inquiète pas à ce sujet? Je suis pensant que ce serait quelque chose de similaire AWTEvent.consommer(). Si oui, alors je vais faire ce que Google CodePro ne voyons pas cela comme "anonymes".

    Je ne sais pas du tout. Je voudrais utiliser au-dessus de la méthode de convenance (j'ai quelque chose de similaire dans une petite collection de services d'utilité générale que j'ai à disposition pour une utilisation sur tous mes projets).

    Je ne voudrais pas vous connecter si ses vraiment une condition normale que vous manipulez. Je ne suis pas familiiar avec Google CodePro, mais j'espère que il ya un moyen de supprimer l'avertissement, par exemple, une sorte de @SuppressWarnings("xxx") annotation/mot-clé.

    Edit: j'ai voulu montrer ces commentaires dans les commentaires ci-dessous

    Cette approche ne fonctionne toujours pas gérer l'exception. Il est mal vu pour intercepter une exception et ne rien faire avec elle. C'est pourquoi je suis à la recherche d'une meilleure solution

    .

    ... L'exception (la situation) est en cours de traitement par le retour de la a indiqué valueIfInvalid. Le "mauvaise forme" vous faites référence à l' mauvaises pratiques de l'aveugle et de manière non intentionnelle de l'écriture de vide blocs catch et de ne jamais retourner à véritablement d'examiner et de traiter le cas. Si le situation d'exception est considéré et fait la bonne chose de la situation (même si la bonne chose est de ne rien faire), puis vous ai "géré" l'exception.

    Cette approche ne fonctionne toujours pas gérer l'exception. Il est mal vu pour intercepter une exception et ne rien faire avec elle. C'est pourquoi je suis à la recherche d'une meilleure solution.
    C'est pas mal si vous savez que vous ne voulez vraiment pas faire quelque chose avec elle. C'est seulement une mauvaise forme à ignorer les exceptions que vous devriez faire quelque chose d'utile avec.
    Je suis d'accord avec @Cameron. L'exception (la situation) est gérée par le retour de la a indiqué valueIfInvalid. La notion générale de "exception non gérée" fait référence à la mauvaise pratique de l'aveugle et de unthinkly écriture vide blocs catch et de ne jamais retourner à véritablement d'examiner et de traiter le cas. Si la situation d'exception est considéré et fait la bonne chose pour la situation (même si la chose est de ne rien faire), vous avez "manipulé" l'exception à la règle.
    Le "final" est juste le désordre.
    Steve Kuo - finale peut jamais être le désordre

    OriginalL'auteur Bert F

  5. 0

    Vous devez attraper l'Exception que vous êtes en train de faire. C'est ennuyeux, mais la meilleure approche.

    Il n'y a pas de Java API méthode qui retourne 0 lorsque la chaîne n'est pas valide int.

    Lorsque la chaîne n'est pas un int, une exception sera levée si votre int variable ne seront pas fixés, sauf si vous intercepter l'exception que vous êtes en train de faire.

    Alors que dois-je faire avec elle après que je l'attraper? Je ne voulez pas vous connecter, parce que c'est normal. Ce n'est pas un cas d'exception. Laissant anonymes fonctionne, mais l'interception d'une exception et ne rien faire-c'est mauvais. Je suis à la recherche d'une meilleure réponse.

    OriginalL'auteur Marcus Leon

  6. 0

    Si il n'est pas clair comment vous devez gérer à partir de la lecture, vous ne devriez pas l'attraper et l'appelant de traiter avec elle à la place. Si vous savez comment elle doit être traitée vous devriez faire cela. La journalisation, il peut ne pas être nécessaire ou très utile dans ce cas.

    L'enregistrement d'une exception est plus utile si vous ne savez pas comment gérer l'exception et de vous laisser à la personne la lecture des journaux.

    Précisément. C'est pourquoi je suis à la recherche d'un moyen de marquer l'exception, comme manipulé sans enregistrement.

    OriginalL'auteur Peter Lawrey

  7. 0

    Votre premier bloc de code est correct. i de ne pas être implicitement converti à 0 lorsqu'une exception se produit et que vous avez à les attraper cette exception. Réglage i à 0 à l'intérieur de catch est correct, bien que vous pouvez simplement remplacer i = 0; avec return 0;. Vous ne pouvez pas éviter la manipulation d'exception dans ce cas.

    Pour clarifier, vous pouvez utiliser ceci:

    private int getCurrentPieceAsInt() {
    int i = 0;
    try {
        i = Integer.parseInt(this.getCurrentPiece());
    } catch (NumberFormatException e) {
        //log that an exception occured if it's needed
        return 0;
    }
    return i;
}
    Que puis-je faire, à l'exception de marquer explicitement comme manipulés? C'est mauvais pour intercepter une exception et ne rien faire avec elle, donc je suis à la recherche d'une meilleure solution.
    non, il n'est pas mauvais pour intercepter une exception et ne rien faire avec elle. Le point de checked exceptions est de sorte que vous pouvez les attraper et à les gérer de la façon dont votre application a besoin d' - parfois, qui implique l'enregistrement ou de la récupération, mais parfois, vous savez qu'il n'a pas d'importance.

    OriginalL'auteur darioo

  8. 0

    Comme d'autres l'ont mentionné, il n'est pas intégré de base de l'API Java de la méthode que vous pouvez appeler pour valider un entier, mais vous pouvez utiliser le Character classe pour valider votre entrée sans à l'aide de la gestion des exceptions. Par exemple:

    package com.example.parseint;

public class ValidateIntExample {
    public static boolean isInteger(String s) {
        if (s == null) {
            return false;
        }

        s = s.trim();

        if (s.length() == 0) {
            return false;
        }

        int start = 0;
        if (s.charAt(0) == '-') { //handle negative numbers
            if (s.length() == 1) {
                return false;
            }
            else {
                start = 1;
            }
        }

        for (int i = start; i < s.length(); i++) {
            if (! Character.isDigit(s.charAt(i))) {
                return false;
            }
        }

        return true;
    }
}

    En fait, parseInt lui-même utilise Character.isDigit en interne, ce que vous pouvez vérifier dans la JRE de code source. (Désolé, j'aurais inclus le parseInt méthode ici, mais je ne sais pas si je suis autorisé sous les termes de la licence.) Si vous êtes en utilisant Eclipse et vous avez le JRE code source connectée à votre projet, vous pouvez droit-cliquez sur la méthode Integer.parseInt dans votre code et cliquez sur Ouvrir Déclaration.

    OriginalL'auteur rob