Объект является нулевым, так как второй вызов метода

Question

Объект является нулевым, так как второй вызов метода

1

С помощью следующего кода скрипт входит в блок if (mySet! = null) только при первом вызове метода.

Почему со второго вызова mySet результаты mySet null?

Map<Integer, Set<String, String>> mMap;

// ... code to populate the map with correct elements

public String myMethod(int idName)
{
    String html = "";

    Set<String> mySet = mMap.get(idName);
    // Set<String> mySet = new HashSet<>(mMap.get(idName));
    if(mySet != null)
    {
        // the script enters here only at first call?!?!?

        for(String name : mySet)
        {
            html += ", " + name;
        }
        mySet.clear();
    }

    return html;
}

myMethod(1); // return some html
myMethod(1); // return empty html...

Если я заменю эту строку:

Set<String> mySet = mMap.get(idName);

с этой строкой:

Set<String> mySet = new HashSet<>(mMap.get(idName));

это работает при каждом вызове.

Элементы map установлены правильно, каждый вызов выполняется без изменения активности, без вращения устройства и других действий.

user2342558 17 апр. 2019, в 15:11

Источник

Теги:

java

android

android-studio

reference

variable-assignment

3 ответа

1

Поскольку Set является объектом типа Reference, то впервые, когда он соответствует условию, а затем очищает набор, теперь набор становится пустым, и поэтому значение, соответствующее 1 становится [] пустым набором. Предположим, что myMap имеет значение: 1, ["1", "2", "3"]

Теперь, когда вы выполняете myMethod(1) в первый раз, он очищает набор, соответствующий значению 1

Теперь значение карты становится: 1, []

Таким образом, он снова удовлетворяет вашему условию if(mySet != null) но ничего не добавляет в переменную html поскольку набор mySet пуст. Итак, он возвращает пустую переменную html.

Bishal Jaiswal 17 апр. 2019, в 10:49

0

Начиная со второго вызова, оно не оценивает как true выражение if(mySet != null) .
user2342558 17 апр. 2019, в 12:58
0

Во втором вызове ваш набор пуст [], но не null ([]! = Null дает true), поэтому он входит в условие if, но в вашем цикле for вы обращаетесь к (String name: mySet) как mySet пустой, он ничего не добавляет к переменной html и возвращает пустую переменную html
Bishal Jaiswal 17 апр. 2019, в 13:05
0

из документации ( docs.oracle.com/javase/7/docs/api/java/util/… ) говорится, что метод get метода Map вернет null если ключ отсутствует на карте. Пожалуйста, объясни мне больше.
user2342558 17 апр. 2019, в 13:08
0

В сценарии, упомянутом в вопросе, вы не удаляете ключ из hashmap, но обращаетесь к значению определенного ключа и очищаете значение определенного набора в [], но ключ все еще существует в существующей карте, поэтому, когда Вы вызываете метод снова, у карты все еще есть ключ 1 , поэтому он дает значение, соответствующее 1 т. е. [], которое не равно нулю, так что теперь ваше значение из hashmap пусто, т. е. [], поэтому условие if доволен.
Bishal Jaiswal 17 апр. 2019, в 13:16
0

Я понимаю, спасибо +1
user2342558 17 апр. 2019, в 13:17

Показать ещё 3 комментария

0

Я думаю, что вы можете исправить это с помощью клона mymap.get(idName) в myset

    public String myMethod(int idName)
{
    String html = "";

    Set<String> mySet = (Set<String>) ((HashSet<String>) mymap(idName)).clone();

    if(mySet != null)
    {
        // the script enters here only at first call?!?!?

        for(String name : mySet)
        {
            html += ", " + name;
        }
        mySet.clear();
    }

    return html;
}

Wishwa Gayan 17 апр. 2019, в 11:47

0

change mySet variable as HashSet Я думаю, что это бесполезно; вместо этого это сделает код менее понятным, так как поддерживая его как Set, я могу изменить класс реализации при необходимости, без изменения типа переменной.
user2342558 17 апр. 2019, в 13:41
0

Спасибо! Я изменил код
Wishwa Gayan 17 апр. 2019, в 13:49
0

Почему я должен разыграть дважды карту?
user2342558 17 апр. 2019, в 13:53
0

Кроме того, мне не нужно клонировать карту целиком, а добавить в набор только набор, привязанный к ключу, переданному в методе.
user2342558 17 апр. 2019, в 13:57

Показать ещё 2 комментария

Ещё вопросы

Начиная со второго вызова, оно не оценивает как true выражение if(mySet != null) .
Во втором вызове ваш набор пуст [], но не null ([]! = Null дает true), поэтому он входит в условие if, но в вашем цикле for вы обращаетесь к (String name: mySet) как mySet пустой, он ничего не добавляет к переменной html и возвращает пустую переменную html
из документации ( docs.oracle.com/javase/7/docs/api/java/util/… ) говорится, что метод get метода Map вернет null если ключ отсутствует на карте. Пожалуйста, объясни мне больше.
В сценарии, упомянутом в вопросе, вы не удаляете ключ из hashmap, но обращаетесь к значению определенного ключа и очищаете значение определенного набора в [], но ключ все еще существует в существующей карте, поэтому, когда Вы вызываете метод снова, у карты все еще есть ключ 1 , поэтому он дает значение, соответствующее 1 т. е. [], которое не равно нулю, так что теперь ваше значение из hashmap пусто, т. е. [], поэтому условие if доволен.
change mySet variable as HashSet Я думаю, что это бесполезно; вместо этого это сделает код менее понятным, так как поддерживая его как Set, я могу изменить класс реализации при необходимости, без изменения типа переменной.
Почему я должен разыграть дважды карту?
Кроме того, мне не нужно клонировать карту целиком, а добавить в набор только набор, привязанный к ключу, переданному в методе.

GhostCat · Accepted Answer · 2019-04-17T11-06-00.000Z

Это здесь:

Set<String> mySet = new HashSet<>(mMap.get(idName));

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

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