Erreur sur les Exceptions personnalisées en JAVA Résolu/Fermé

Question

Bonjour
Je ne comprend pa l'erreur que j'obtient avec l'exception SalarieException  :
package fr.xxxx.yyyy;

import java.util.Enumeration;
import java.util.Hashtable;

import fr.xxxx.exceptions.SalarieException;

public class Application {

public static void main(String[] args) throws SalarieException {
        Hashtable dict = new Hashtable();

Salarie s;
        try {
            s = new Salarie(5, 8, 14, "yyyyyyyyyyyyyyyyy", 1500);
            Integer key = new Integer(s.getM_nMatricule());
            dict.put(key, s);

s = new Salarie(3, 1, 20, "Isodore DUMARC", 10000);
            dict.put(key, s);
            
            s = new Commercial(6, 2, 30, "Mathieu LEBLOND", -17000, 52000, 9);
            dict.put(key, s);
        } catch (SalarieException se) {
            System.err.println(se);
        }
        
        // Ici on ajoute chaque élement dict.element() à l'Enumeration
        for (Enumeration e = dict.elements(); e.hasMoreElements();) {
            Salarie sal = (Salarie) e.nextElement();
            System.out.println(sal);
            System.out.println(sal.calculSalaire());
        }
    }
}package fr.xxxx.exceptions;

import fr.xxxx.yyyy.Salarie;

public class SalarieException extends Exception {
    public SalarieException(Salarie salarie) {
        super();
    }

public String toString(){
        return super.toString() 
                + "
Le salaire ne peut être que positif.";
    }
}
L'erreur se trouve à la ligne 24 à catch (SalarieException se)

KX · Answer

Bonjour,

Tu ne peux pas catch une SalarieException si dans le bloc try tu n'appelles aucun code qui throws cette SalarieException.
Tu ne nous as pas copier le code des classes Salarie et Commercial donc on ne peut pas voir comment elles sont faites, mais leurs constructeurs devraient throws SalarieException pour que ton try/catch ait un sens.

public Salarie(int a, int b, int c, String d, int e) throws SalarieException {
    if (...)
        throw new SalarieException(this);
}

Remarque : Hashtable et Enumeration sont de très vieilles classes Java 1.0 (1996) qui ne devraient plus être utilisées depuis Java 1.2 (1998) quand ont été ajoutées HashMap et Iterator.
Tu as donc plus de 20 ans de retard sur la technologie Java et il ne m'apparaît pas judicieux de coder en 2020 avec un langage Java aussi rustique qu'il ne l'était il y a 20 ans.

Anouchka2586 · Answer

Dans la classe Salarie :
public Salarie(int m_nMatricule, int m_nCategorie, int m_nService, String m_strNom, double m_dSalaire) throws SalarieException {
    super();
    System.out.println("Création du salarié : "+m_strNom);
    this.m_nMatricule = m_nMatricule;
    this.m_nCategorie = m_nCategorie;
    this.m_nService = m_nService;
    this.m_strNom = m_strNom;
    if(this.m_dSalaire < 0) {
        SalarieException se = 
                new SalarieException(this);
    } else this.m_dSalaire = m_dSalaire;
    this.m_nCount = m_nCount + 1;
}

Dans la classe Commercial:
public Commercial(int nMat, int nCatg, int nServ, String strNom, double dSal, double dChiffre,
        int dCommission) throws SalarieException {
    super(nMat, nCatg, nServ, strNom, dSal);
    this.m_dChiffreAffaire = dChiffre;
    this.m_pcCommission = dCommission;
}

Anouchka2586 · Answer

Ok mais quand je met un salaire négatif, l'exception SalarieException n'est pas levée. Aucun message d'erreur

Anouchka2586 · Answer

ça ne marche pas, déjà il souligne en rouge quand je met throw  à throw new SalarieException(this);

Pour votre 2ème proposition cad :
if (m_dSalaire < 0) {    throw new SalarieException(this);} else this.m_dSalaire = m_dSalaire;
Il me renvoit juste le salaire est 0 si je le met en négatif mais ne retourne pas la valeur de toString().
ps: la premièreproposition ne fonctionne pas, il affiche quand même le salaire négatif

Anouchka2586 · Answer

Ah oui ça marche c'est moi qui avais ajouté un mauvais code, merci

Erreur sur les Exceptions personnalisées en JAVA

5 réponses

Discussions similaires