The tpexceptionsthomasdonnay from dons9011

Correction

Acquis. Travail complet mais attention à la duplication de code et à ne pas mettre dans un bloc try des instructions ne pouvant lever d'exceptions !
Mets tes regex en constantes

TPexceptionsThomasDonnay/ipi-java-230-ex-batchexo/src/main/java/com/ipiecoles/java/java230/MyRunner.java

Lines 97 to 111 in 3176544

 if (!ligne.matches("^[MTC]{1}.*")) { 

 throw new BatchException("Type d'employé inconnu : " + ligne.charAt(0)); 

 } 

 if (ligne.matches("^[M]{1}.*")) { 

 processManager(ligne); 

 } 

 if (ligne.matches("^[C]{1}.*")) { 

 processCommercial(ligne); 

 } 

 if (ligne.matches("^[T]{1}.*")) { 

 processTechnicien(ligne); 

 }

Tu peux mettre en commun tous les traitements qui sont les mêmes pour tous les types d'employés (matricule, nom, prénom, salaire, date d'embauche),pour éviter le copier coller de code.

Aucune exception ne peut être levée ici...

TPexceptionsThomasDonnay/ipi-java-230-ex-batchexo/src/main/java/com/ipiecoles/java/java230/MyRunner.java

Lines 205 to 211 in 3176544

 try { 

 String strMatriculeman = ligneman[0]; 

 man.setMatricule(strMatriculeman); 

 } catch (Exception e) { 

 throw new BatchException("Probleme avec le matricule : " + ligneman[0]); 

 } 

 ;

Ni là....

TPexceptionsThomasDonnay/ipi-java-230-ex-batchexo/src/main/java/com/ipiecoles/java/java230/MyRunner.java

Lines 213 to 222 in 3176544

 // nom et prenom 

 try { 

 String strPrenomman = ligneman[1]; 

 String strNomman = ligneman[2]; 

 man.setPrenom(strPrenomman); 

 man.setNom(strNomman); 

 } catch (Exception e) { 

 throw new BatchException("Probleme avec les nom et prenom : " + ligneman[1] + "et " + ligneman[2]); 

 } 

 ;

Pourquoi avoir distinguer les deux cas ? Je ne pense pas qu'une autre exception que NumberFormatException puisse être lancée.

TPexceptionsThomasDonnay/ipi-java-230-ex-batchexo/src/main/java/com/ipiecoles/java/java230/MyRunner.java

Lines 237 to 241 in 3176544

 } catch (NumberFormatException p) { 

 throw new BatchException("Probleme avec le parsing du salaire : " + ligneman[4]); 

 } catch (Exception e) { 

 throw new BatchException("Probleme avec le salaire : " + ligneman[4]); 

 }

Là non plus aucune exception possible.

TPexceptionsThomasDonnay/ipi-java-230-ex-batchexo/src/main/java/com/ipiecoles/java/java230/MyRunner.java

Lines 245 to 251 in 3176544

 try { 

 HashSet<Technicien> strEquipeman = new HashSet<Technicien>(); 

 man.setEquipe(strEquipeman); 

 } catch (Exception e) { 

 throw new BatchException("Probleme avec le HashSet de l'equipe "); 

 } 

 ;

A quoi sert cette ligne ? Aussi, tu ne recherches que dans la base, pas dans le fichier.

TPexceptionsThomasDonnay/ipi-java-230-ex-batchexo/src/main/java/com/ipiecoles/java/java230/MyRunner.java

Line 333 in 3176544

Manager managerOftechBidon = managerRepository.findByMatricule(strManagertech);

dons9011 / tpexceptionsthomasdonnay Goto Github PK

tpexceptionsthomasdonnay's People

Contributors

Watchers

tpexceptionsthomasdonnay's Issues

Correction

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

Jobs

	if (!ligne.matches("^[MTC]{1}.*")) {
	throw new BatchException("Type d'employé inconnu : " + ligne.charAt(0));
	}

	if (ligne.matches("^[M]{1}.*")) {
	processManager(ligne);
	}

	if (ligne.matches("^[C]{1}.*")) {
	processCommercial(ligne);
	}

	if (ligne.matches("^[T]{1}.*")) {
	processTechnicien(ligne);
	}

	try {
	String strMatriculeman = ligneman[0];
	man.setMatricule(strMatriculeman);
	} catch (Exception e) {
	throw new BatchException("Probleme avec le matricule : " + ligneman[0]);
	}
	;

	// nom et prenom
	try {
	String strPrenomman = ligneman[1];
	String strNomman = ligneman[2];
	man.setPrenom(strPrenomman);
	man.setNom(strNomman);
	} catch (Exception e) {
	throw new BatchException("Probleme avec les nom et prenom : " + ligneman[1] + "et " + ligneman[2]);
	}
	;

	} catch (NumberFormatException p) {
	throw new BatchException("Probleme avec le parsing du salaire : " + ligneman[4]);
	} catch (Exception e) {
	throw new BatchException("Probleme avec le salaire : " + ligneman[4]);
	}

	try {
	HashSet<Technicien> strEquipeman = new HashSet<Technicien>();
	man.setEquipe(strEquipeman);
	} catch (Exception e) {
	throw new BatchException("Probleme avec le HashSet de l'equipe ");
	}
	;