Une meilleure boucle sans interruption;

Question

Je me demandais s'il y avait une meilleure alternative à l'utilisation de cette implémentation de rupture. J'essaie d'améliorer mes capacités et ce qui suit fonctionne vraiment. C'est juste que je n'arrête pas d'entendre parler de Break; comme étant la sortie facile et produit le code spaghetti potentiel qui n'a pas eu lieu ici mais encore. Sans la coupure, le message «Rien vendu au cours des deux dernières semaines» ne produirait la quantité stockée dans le groupement.

Answer 1

Puisque vous ne faites rien après la pause, si cela se produit. Vous pourriez aussi bien le remplacer par le retour.

public void getWoodSoldRecently(){ 
Calendar cal = Calendar.getInstance(); 
cal.add(Calendar.WEEK_OF_YEAR, -2); 
for(Tree t : theTrees){ 
    if(t.getSimpleDateSold().getTime().after(cal.getTime()) && t.getHasBeenSold()==true){ 
     treesSold.add(t); 
     System.out.println(t.getTreeId() + " " + t.getTreeType()); 
    } 
    else{ 
     System.out.println("Nothing sold in the last 2 weeks"); 
     return; //Exit function 
    } 
} 

}

J'ai personnellement pas broblem avec des pauses, le retour cependant vous oblige à incorporer du code dans plus de fonctions, Wich est toujours bon.

Answer 2

La coupure n'est pas mauvaise par définition - elle peut être mal utilisée mais la façon dont vous l'utilisez est autorisée. Vous pouvez envisager de remplacer la rupture par un retour.

Answer 3

Je n'utilise pas cela break dans votre exemple résultera dans n'importe quel problème. Postant toujours une solution de contournement possible (éviter la rupture):

boolean bContinue = true; 
    int iSize = theTrees.size(); 
    for(int indx=0; indx < iSize && bContinue; indx++){ 
    Tree t = theTrees.get(indx); 
    if(t.getSimpleDateSold().getTime().after(cal.getTime()) 
     && t.getHasBeenSold()==true){ 
     treesSold.add(t); 
     System.out.println(t.getTreeId() + " " + t.getTreeType()); 
    } 
    else{ 
     System.out.println("Nothing sold in the last 2 weeks"); 
     bContinue = false;//This will stop the loop 
    } 
} 

Answer 4

Je suppose que vous ne voulez pas vraiment casser ici de toute façon. Sûrement vous voulez juste continuer avec le reste des arbres - juste parce que vous trouvez un arbre qui ne correspond pas à votre condition (par exemple, il n'est pas vendu), cela signifie-t-il vraiment qu'il ne peut y avoir des correspondants plus tard dans votre collection?

Votre méthode est nettement étrange de toute façon, en termes d'être une méthode get qui ne retour quoi que ce soit, mais ajoute des valeurs à une collection existante. C'est la première chose que je changerais.

Answer 5

Le problème est que vous utilisez une boucle foreach qui ne vous permet pas d'utiliser des paramètres conditionnels. Si vous le convertir en une boucle, vous serez mieux:

boolean loopContinue = true; 

for(int i = 0; i < theTrees.size() && loopContinue; i++) { 
     Tree t = theTrees.get(i); 
     if(t.getSimpleDateSold().getTime().after(cal.getTime()) && t.getHasBeenSold()==true){ 
      treesSold.add(t); 
      System.out.println(t.getTreeId() + " " + t.getTreeType()); 
     } 
     else{ 
      System.out.println("Nothing sold in the last 2 weeks"); 
      loopContinue = false; //Stop the above 
     } 
    } 
} 

EDIT & Précision: je dis « mieux » ce qui signifie que vous aurez plus de contrôle de la boucle. Comme d'autres l'ont dit, l'utilisation de break n'est pas intrinsèquement mauvaise.

Answer 6

Travailler avec un Iterator:

public void getWoodSoldRecently(){ 
Calendar cal = Calendar.getInstance(); 
cal.add(Calendar.WEEK_OF_YEAR, -2); 
Iterator itr = theTrees.iterator(); 
boolen b = true; 
while (itr.hasNext() && b == true) { 
     if(t.getSimpleDateSold().getTime().after(cal.getTime()) && t.getHasBeenSold()==true){ 
      treesSold.add(t); 
      System.out.println(t.getTreeId() + " " + t.getTreeType()); 
     } 
     else{ 
      System.out.println("Nothing sold in the last 2 weeks"); 
      b = false; 
     } 
    } 
}