Java: “Break” en boucle à partir de la méthode appelée?

Il y a un certain nombre de choses que vous pouvez faire ici. En fin de compte ce que vous faites doit tout dépend de votre style de codage et de ce que vous essayez d'accomplir.

L'Option 1 serait une variation de:

for (;;)
{
   int number = /* ... */;
   myMethod(number);
   if (number == -10)
      break;
}

Vous pourriez dire que, plutôt de manière subjective et selon les circonstances, que ce qui est mauvais, parce que la connaissance de la résiliation condition est contenue dans la boucle plutôt que de la méthode de faire le "vrai travail". Peut-être pour votre boucle c'est OK. Peut-être dans d'autres circonstances (ou peut-être avec d'autres programmeurs? C'est beaucoup une question de goût.) vous voudrez peut-être faire myMethod prendre cette décision. En général mon propre goût personnel généralement se penche vers ne pas avoir de scénario connaissances seront distribués tout au long des différentes méthodes dans la source, mais dans un seul endroit.

De sorte que la plupart de ce que je vais écrire de la façon de faire myMethod prendre la décision quant à savoir si ou de ne pas résilier.

Option 2 - myMethod renvoie un booléen qui indique que nous devrions mettre fin à:

for (;;)
{
   int number = /* ... */;
   if (myMethod(number))
      break;
}

boolean myMethod(int number)
{
   //TODO - do stuff

   return number == -10;
}

Mais vous pourriez dire que myMethod veut déjà revenir un autre type. Je viens très souvent du C de fond de sorte que le langage dont je suis le plus utilisé pour serait le "paramètre de sortie". Ce qui m'a option 3:

Option 3 paramètre vous permet de décider de l'appelant à mettre fin à:

public class CancelIndicator
{
   public boolean shouldCancel;
};

CancelIndicator cancel = new CancelIndicator();

while (!cancel.shouldCancel)
{
   int number = /* ... */;
   myMethod(number, cancel);
}

int myMethod(int number, CancelIndicator cancel)
{
   //TODO - do stuff.

   cancel.shouldCancel = (number == -10);

   return /* ... */;
}

Ou peut-être vous êtes de plus en plus fan des exceptions:

Option 3:

public class CancellationException extends Exception
{
}

try
{
   for (;;)
   {
      int number = /* ... */;
      myMethod(numberl);
   }
}
catch (CancellationException ex)
{
}

void myMethod(int number) throws CancellationException
{
   //TODO - do stuff.

   if (number == -10)
      throw new CancellationException();
}

Comme vous pouvez le voir, il ya un certain nombre d'options. Je suis sûr que l'on pourrait passer une journée entière à parler de différentes façons de le faire. Voici mon exemple de phrases idiomatiques j'ai vu - je vous préviens qu'il a été un certain temps depuis que je l'ai beaucoup fait en Java donc je ne pourrais pas écrire les plus idiomatiques code ici. 🙂

OriginalL'auteur asveikau

Attendre, alors je n'ai obtenir ce droit? Vous avez une boucle qui demande un nombre, fait quelque chose si le nombre n'est pas -10, et sinon les pauses?

Si si, regardez ceci:

while(true) {
    number=Integer.parseInt( JOptionPane.showInputDialog( "bla bla" ) );
    if(number == -10) {
        break;
    }
    themethod(number);
}

Autrement, si ce n'est pas le cas, et que vous voulez couper dans les deux cas, l'-10 cas et la méthode est fausse cas, vous pouvez le faire:

Changer votre méthode pour renvoyer une valeur de type boolean. Si c'est vrai, alors ce n'est pas le casser. Si c'est faux alors il veut rompre, alors ce faire:

while(true) {
    number=Integer.parseInt( JOptionPane.showInputDialog( "bla bla" ) );
    if(number == -10) {
        break;
    }
    if(!themethod(number)) break;
}

Si vraiment vous ne pouvez pas modifier la boucle, puis il suffit de lancer une exception de la méthode! Qui va quitter l'ensemble de votre programme. Je ne vois pas d'autre façon possible de le faire, cependant.

OriginalL'auteur eboix