Как выйти из цикла, когда одно условие истинно? [Дубликат]

Что такое NullPointerException?

Хорошим местом для начала является JavaDocs . Они охватывают это:

Брошено, когда приложение пытается использовать null в случае, когда требуется объект. К ним относятся:

  • Вызов метода экземпляра нулевого объекта.
  • Доступ или изменение поля нулевого объекта.
  • Выполнение длины null, как если бы это был массив.
  • Доступ или изменение слотов с нулевым значением, как если бы это был массив.
  • Бросать нуль, как если бы это было значение Throwable.

Приложения должны бросать экземпляры этого класса для указания других незаконных видов использования нулевого объекта.

blockquote>

Также, если вы попытаетесь использовать нулевую ссылку с synchronized, который также выдаст это исключение, за JLS :

SynchronizedStatement:
    synchronized ( Expression ) Block
  • В противном случае, если значение выражения равно null, NullPointerException.
blockquote>

Как это исправить?

Итак, у вас есть NullPointerException. Как вы это исправите? Возьмем простой пример, который выдает NullPointerException:

public class Printer {
    private String name;

    public void setName(String name) {
        this.name = name;
    }

    public void print() {
        printString(name);
    }

    private void printString(String s) {
        System.out.println(s + " (" + s.length() + ")");
    }

    public static void main(String[] args) {
        Printer printer = new Printer();
        printer.print();
    }
}

Идентифицирует нулевые значения

. Первый шаг - точно определить , значения которого вызывают исключение . Для этого нам нужно выполнить некоторую отладку. Важно научиться читать stacktrace . Это покажет вам, где было выбрано исключение:

Exception in thread "main" java.lang.NullPointerException
    at Printer.printString(Printer.java:13)
    at Printer.print(Printer.java:9)
    at Printer.main(Printer.java:19)

Здесь мы видим, что исключение выбрано в строке 13 (в методе printString). Посмотрите на строку и проверьте, какие значения равны нулю, добавив протоколирующие операторы или используя отладчик . Мы обнаруживаем, что s имеет значение null, а вызов метода length на него вызывает исключение. Мы видим, что программа перестает бросать исключение, когда s.length() удаляется из метода.

Трассировка, где эти значения взяты из

Затем проверьте, откуда это значение. Следуя вызовам метода, мы видим, что s передается с printString(name) в методе print(), а this.name - null.

Трассировка, где эти значения должны быть установлены

Где установлен this.name? В методе setName(String). С некоторой дополнительной отладкой мы видим, что этот метод вообще не вызывается. Если этот метод был вызван, обязательно проверьте порядок , что эти методы вызывают, а метод set не будет называться после методом печати. ​​

Этого достаточно, чтобы дать нам решение: добавить вызов printer.setName() перед вызовом printer.print().

Другие исправления

Переменная может иметь значение по умолчанию setName может помешать ему установить значение null):

private String name = "";

Либо метод print, либо printString может проверить значение null например:

printString((name == null) ? "" : name);

Или вы можете создать класс, чтобы name всегда имел ненулевое значение :

public class Printer {
    private final String name;

    public Printer(String name) {
        this.name = Objects.requireNonNull(name);
    }

    public void print() {
        printString(name);
    }

    private void printString(String s) {
        System.out.println(s + " (" + s.length() + ")");
    }

    public static void main(String[] args) {
        Printer printer = new Printer("123");
        printer.print();
    }
}

См. также:

Я все еще не могу найти проблему

Если вы попытались отладить проблему и до сих пор не имеете решения, вы можете отправить вопрос для получения дополнительной справки, но не забудьте включить то, что вы пробовали до сих пор. Как минимум, включите stacktrace в вопрос и отметьте важные номера строк в коде. Также попробуйте сначала упростить код (см. SSCCE ).

49
задан Peter Mortensen 7 August 2014 в 17:01
поделиться

7 ответов

Используйте break; для выхода из первого цикла:

if (s.Contains("mp4:production/CATCHUP/"))
{
   RemoveEXELog();
   Process p = new Process();
   p.StartInfo.WorkingDirectory = "dump";
   p.StartInfo.FileName = "test.exe"; 
   p.StartInfo.Arguments = s; 
   p.Start();
   break;
}

Если вы хотите также избежать второго цикла, вам может понадобиться использовать флаг и проверить защиту от цикла:

        boolean breakFlag = false;
        while (!breakFlag)
        {
            Thread.Sleep(5000);
            if (!System.IO.File.Exists("Command.bat")) continue;
            using (System.IO.StreamReader sr = System.IO.File.OpenText("Command.bat"))
            {
                string s = "";
                while ((s = sr.ReadLine()) != null)
                {
                    if (s.Contains("mp4:production/CATCHUP/"))
                    {

                        RemoveEXELog();

                        Process p = new Process();
                        p.StartInfo.WorkingDirectory = "dump";
                        p.StartInfo.FileName = "test.exe"; 
                        p.StartInfo.Arguments = s; 
                        p.Start();

                        breakFlag = true;
                        break;
                    }
                }
            }

Или, если вы хотите полностью выйти из функции изнутри вложенного цикла, поставьте вместо break; значение return;.

Но на самом деле это не считается лучшей практикой. Вы должны найти способ добавить в логику while необходимую логическую логику.

61
ответ дан Dirk 19 August 2018 в 05:52
поделиться

Если вам нужно продолжить использование дополнительной логики ...

break;

или если у вас есть значение для возврата ...

return my_value_to_be_returned;

Однако, глядя на ваш код, я считаю, что вы будете контролировать цикл с приведенным ниже примером без использования перерыва или возврата ...

private void CheckLog()
        {
            bool continueLoop = true;
            while (continueLoop)
            {
                Thread.Sleep(5000);
                if (!System.IO.File.Exists("Command.bat")) continue;
                using (System.IO.StreamReader sr = System.IO.File.OpenText("Command.bat"))
                {
                    string s = "";
                    while (continueLoop && (s = sr.ReadLine()) != null)
                    {
                        if (s.Contains("mp4:production/CATCHUP/"))
                        {
                            RemoveEXELog();

                            Process p = new Process();
                            p.StartInfo.WorkingDirectory = "dump";
                            p.StartInfo.FileName = "test.exe"; 
                            p.StartInfo.Arguments = s; 
                            p.Start();
                            continueLoop = false;
                        }
                    }
                }
            }
        }
4
ответ дан Alban 19 August 2018 в 05:52
поделиться
6
ответ дан Bobby 19 August 2018 в 05:52
поделиться

«break» - это команда, которая вырывается из «ближайшего» цикла.

. Хотя существует много хороших применений для разрыва, вы не должны использовать его, если вам не нужно - это можно рассматривать как еще один способ использования goto, который считается плохим.

Например, почему нет:

while (!(the condition you're using to break))
        {
         //Your code here.
        }

Если причина, по которой вы используете «break», заключается в том, что вы не хотите продолжать выполнение этой итерации цикла, вы можете использовать ключевое слово «continue», которое сразу же переходит к следующей итерации цикла, независимо от того, находится ли оно во время или для.

while (!condition) {
   //Some code
   if (condition) continue;
   //More code that will be skipped over if the condition was true
}
1
ответ дан Jeremy 19 August 2018 в 05:52
поделиться
  • 1
    Догматики считаются плохими. Все ключевые слова могут быть использованы плохо. Все ключевые слова могут использоваться хорошо. – David Heffernan 16 July 2011 в 20:26
  • 2
    Мой ответ уже сказал, что есть много хороших применений для разрыва. – Jeremy 16 July 2011 в 20:56

Извините за necro-add, но есть кое-что, что я действительно хотел вставить, что отсутствует в существующих ответах (для тех, кому нравится, как я наткнулся на этот вопрос через google): реорганизуйте свой код. Это не только облегчит чтение, но и часто будет устранять эти типы проблем с управлением.

Вот что я бы наклонил, если бы мне пришлось программировать функцию выше:

private const string CatchupLineToIndicateLogDump = "mp4:production/CATCHUP/";
private const string BatchFileLocation = "Command.bat";

private void CheckLog()
{
    while (true)
    {
        Thread.Sleep(5000);
        if (System.IO.File.Exists(BatchFileLocation))
        {
            if (doesFileContainStr(BatchFileLocation, CatchupLineToIndicateLogDump))
            {
                RemoveLogAndDump();
                return;
            }
        }
    }
}

private bool doesFileContainStr(string FileLoc, string StrToCheckFor)
{
  // ... code for checking the existing of a string within a file
  // (and returning back whether the string was found.)
}

private void RemoveLogAndDump()
{
  // ... your code to call RemoveEXELog and kick off test.exe
}
0
ответ дан Kevin 19 August 2018 в 05:52
поделиться

Какой цикл вы пытаетесь выйти? Простой break; выйдет из внутреннего цикла. Для внешнего цикла вы можете использовать переменную с ограниченным окружением (например, boolean exit = false;), которая установлена ​​в true только перед тем, как вы нарушите внутренний цикл. После того, как внутренний цикл цикла проверяет значение выхода и если true, снова используйте break;.

2
ответ дан tomfumb 19 August 2018 в 05:52
поделиться
  • 1
    +1 для рассмотрения обеих петель (и не упоминание 4-буквенного ключевого слова). – Joel Rondeau 16 July 2011 в 20:22

break или goto

while ( true ) {
  if ( conditional ) {
    break;
  }
  if ( other conditional ) {
    goto EndWhile;
  }
}
EndWhile:
7
ответ дан zellio 19 August 2018 в 05:52
поделиться
  • 1
    Goto - впечатляющая вещь. – Dementic 22 June 2014 в 16:25
  • 2
    Не используйте goto, но решение с разрывом хорошо – Florian Neiss 19 March 2015 в 15:39
  • 3
    @Dementic - Не в каждом случае. Если он используется с умом, он может повысить читаемость, может помочь выйти из глубоко вложенного цикла, передать управление на конкретную метку корпуса коммутатора и т. Д. Как показано в приведенном выше примере zellio, он помогает выйти из системы без использования какой-либо другой переменной и т. Д. Из msdn.microsoft.com/en-us/library/13940fs2 (v = vs.71) .aspx «Общее использование goto - это передача управления на конкретную метку ключа коммутатора или метку по умолчанию в оператор switch. Операция goto также полезна для выхода из глубоко вложенных циклов. & Quot; – sandiejat 10 March 2017 в 02:12
  • 4
    Это не работает, когда мы используем ключевое слово «switch» в цикле «while», поэтому просто добавьте еще один тестовый conditon к показу, например. bool bTurnOff = false; while (true & amp; bTurnOff == false) {...} – TomeeNS 12 July 2017 в 22:12
Другие вопросы по тегам:

Похожие вопросы: