ObservableCollection, который также наблюдает изменения на элементах в наборе

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

32
задан Naser Asadi 27 July 2013 в 08:38
поделиться

4 ответа

Сделанный быстрой реализацией самостоятельно:

public class ObservableCollectionEx<T> : ObservableCollection<T> where T : INotifyPropertyChanged
{
    protected override void OnCollectionChanged(NotifyCollectionChangedEventArgs e)
    {
        Unsubscribe(e.OldItems);
        Subscribe(e.NewItems);
        base.OnCollectionChanged(e);
    }

    protected override void ClearItems()
    {
        foreach(T element in this)
            element.PropertyChanged -= ContainedElementChanged;

        base.ClearItems();
    }

    private void Subscribe(IList iList)
    {
        if (iList != null)
        {
            foreach (T element in iList)
                element.PropertyChanged += ContainedElementChanged;
        }
    }

    private void Unsubscribe(IList iList)
    {
        if (iList != null)
        {
            foreach (T element in iList)
                element.PropertyChanged -= ContainedElementChanged;
        }
    }

    private void ContainedElementChanged(object sender, PropertyChangedEventArgs e)
    {
        OnPropertyChanged(e);
    }
}

Допущенный, это довольно сбивало бы с толку и было бы вводящим в заблуждение сделать, чтобы PropertyChanged стрелял в набор, когда свойство, которое на самом деле изменилось, находится на содержавшем элементе, но это соответствовало бы моей определенной цели. Это могло быть расширено с помощью нового события, которое запущено вместо этого в мыслях ContainerElementChanged

?

РЕДАКТИРОВАНИЕ: Должен отметить, что BCL ObservableCollection только выставляет интерфейс INotifyPropertyChanged посредством явной реализации, таким образом, необходимо было бы предоставить броску для присоединения к событию как так:

ObservableCollectionEx<Element> collection = new ObservableCollectionEx<Element>();
((INotifyPropertyChanged)collection).PropertyChanged += (x,y) => ReactToChange();

EDIT2: Добавленная обработка ClearItems, Josh

EDIT3 спасибо: Добавленный корректное отказываются от подписки для PropertyChanged, спасибо Mark

EDIT4: Ничего себе, это действительно learn-as-you-go :). KP отметил, что событие было запущено с набором как отправитель а не с элементом, когда содержавший элемент изменяется. Он предложил объявить событие PropertyChanged на классе, отмеченном с новый . Это имело бы несколько проблем, которые я попытаюсь проиллюстрировать с образцом ниже:

  // work on original instance
  ObservableCollection<TestObject> col = new ObservableCollectionEx<TestObject>();
  ((INotifyPropertyChanged)col).PropertyChanged += (s, e) => { Trace.WriteLine("Changed " + e.PropertyName); };

  var test = new TestObject();
  col.Add(test); // no event raised
  test.Info = "NewValue"; //Info property changed raised

  // working on explicit instance
  ObservableCollectionEx<TestObject> col = new ObservableCollectionEx<TestObject>();
  col.PropertyChanged += (s, e) => { Trace.WriteLine("Changed " + e.PropertyName); };

  var test = new TestObject();
  col.Add(test); // Count and Item [] property changed raised
  test.Info = "NewValue"; //no event raised

Вы видите от образца, что 'переопределение' события имеет побочный эффект, что необходимо быть чрезвычайно осторожными, из которого типа переменной Вы используете при подписке на событие, так как это диктует, какие события Вы получаете.

43
ответ дан 27 November 2019 в 20:44
поделиться

Если Вы хотите использовать что-то встроенное в платформу, можно использовать FreezableCollection. Затем Вы захотите слушать , событие .

Changed Происходит, когда Замораживаемое или объект, который оно содержит, изменяются.

Вот небольшая выборка. collection_Changed метод назовут дважды.

public partial class Window1 : Window
{
    public Window1()
    {
        InitializeComponent();

        FreezableCollection<SolidColorBrush> collection = new FreezableCollection<SolidColorBrush>();
        collection.Changed += collection_Changed;
        SolidColorBrush brush = new SolidColorBrush(Colors.Red);
        collection.Add(brush);
        brush.Color = Colors.Blue;
    }

    private void collection_Changed(object sender, EventArgs e)
    {
    }
}
3
ответ дан 27 November 2019 в 20:44
поделиться

Проверьте Библиотека Универсального набора C5 . Все его наборы содержат события, которые можно использовать для присоединения обратных вызовов для того, когда объекты добавлены, удалены, вставлены, очищены, или когда набор изменяется.

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

0
ответ дан 27 November 2019 в 20:44
поделиться

@soren.enemaerke: Я бы сделал этот комментарий к вашему ответу, но не могу (не знаю почему, возможно, потому что у меня мало очков репутации). В любом случае, я просто подумал, что должен упомянуть, что в вашем коде, который вы опубликовали, я не думаю, что отписка будет работать правильно, потому что она создает новую лямбду в строке, а затем пытается удалить обработчик события для нее.

Я бы изменил строки обработчиков событий add/remove на что-то вроде:

element.PropertyChanged += ContainedElementChanged;

и

element.PropertyChanged -= ContainedElementChanged;

А затем изменил бы сигнатуру метода ContainedElementChanged на:

private void ContainedElementChanged(object sender, PropertyChangedEventArgs e)

Это распознало бы, что remove предназначен для того же обработчика, что и add, и удалило бы его правильно. Надеюсь, это кому-нибудь поможет :)

7
ответ дан 27 November 2019 в 20:44
поделиться
Другие вопросы по тегам:

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