Алгоритм, чтобы найти, пересекаются ли два набора

Что такое 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 ).

13
задан higuaro 5 June 2018 в 13:28
поделиться

7 ответов

Притворитесь, что Вы делаете сортировку с объединением, но не отправляете результаты никуда. Если Вы добираетесь в конец любого источника, нет никакого пересечения. Каждый раз Вы сравниваете следующий элемент каждого, если они равны, существует пересечение.

, Например:

counterA = 0;
counterB = 0;
for(;;) {
    if(counterA == ArrayA.length || counterB == ArrayB.length)
        return false;
    else if(ArrayA[counterA] == ArrayB[counterB])
        return true;
    else if(ArrayA[counterA] < ArrayB[counterB])
        counterA++;
    else if(ArrayA[counterA] > ArrayB[counterB])
        counterB++;
    else
        halt_and_catch_fire();
}
39
ответ дан 1 December 2019 в 17:32
поделиться

Так как кто-то задался вопросом о stl. Out-of-the-box, set_intersection алгоритм сделал бы больше, чем Вы хотите: это нашло бы все общие ценности.

    #include <vector>
    #include <algorithm>
    #include <iterator>
    using namespace std;
//    ...    
      int ArrayA[] = {5, 17, 150, 230, 285};
      int ArrayB[] = {7, 11, 57, 110, 230, 250};
      vector<int> intersection;
      ThrowWhenWritten output_iterator;
        set_intersection(ArrayA, ArrayA + sizeof(ArrayA)/sizeof(int),
                         ArrayB, ArrayB + sizeof(ArrayB)/sizeof(int),
                         back_insert_iterator<vector<int> >(intersection));

        return !intersection.empty();

это выполняет в O (m+n) время, но он требует хранения всех дубликатов и не останавливается, когда он находит первый дубликат

Теперь, изменяя код от гну реализация из stl, мы можем получить более точно, что Вы хотите.

 template<typename InputIterator1, typename InputIterator2>
 bool 
 has_intersection(InputIterator1 first1, InputIterator1 last1,
             InputIterator2 first2, InputIterator2 last2)
    {
       while (first1 != last1 && first2 != last2) 
       {
          if (*first1 < *first2)
             ++first1;
          else if (*first2 < *first1)
             ++first2;
          else
             return true;
       }
       return false;
}
7
ответ дан 1 December 2019 в 17:32
поделиться

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

3
ответ дан 1 December 2019 в 17:32
поделиться

Если Вы используете C# 3.0 затем, почему бы не использовать в своих интересах LINQ здесь?

ArrayA.Intersect(ArrayB).Any()

Не только этот дженерик (работы для любого сопоставимого типа), внедрение под капотом довольно эффективно (использует алгоритм хеширования).

1
ответ дан 1 December 2019 в 17:32
поделиться

Если диапазон значений является маленьким, Вы могли бы создать справочную таблицу для одного из них (стоимость времени = O (N)) и затем проверить, установлен ли бит из другого списка (стоимость времени = O (N)). Если спектр является большим, Вы могли бы сделать что-то похожее с хеш-таблицей.

прием сортировки с объединением от Glomek является еще лучшей идеей.

0
ответ дан 1 December 2019 в 17:32
поделиться

Glomek на правильном пути, но отчасти замял алгоритм.

Запускаются путем сравнения ArrayA[0] с ArrayB[0]. если они равны, Вы сделаны. Если ArrayA[0] является меньше, чем ArrayB[0], то переместитесь в ArrayA[1]. Если ArrayA[0] является больше, чем ArrayB[0], то переместитесь в ArrayB[1].

Хранение, ступающее через, пока Вы не достигаете конца одного массива или находите соответствие.

0
ответ дан 1 December 2019 в 17:32
поделиться

Если один список намного намного короче, чем другой, двоичный поиск является способом пойти. Если бы списки имеют подобную длину, и Вы довольны O (m+n), стандартное "слияние" работало бы. Существуют более необычные алгоритмы, которые более гибки. Одна бумага, с которой я столкнулся в своих собственных поисках:

http://www.cs.uwaterloo.ca/~ajsaling/papers/paper-spire.pdf

4
ответ дан 1 December 2019 в 17:32
поделиться
Другие вопросы по тегам:

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