Как вернуть iterator?

Question

[Александр Одинцов]

Добрый день.
В задаче есть метод, который должен возвращать Iterator. Метод ищет в HashMap необходимую строку.

public Iterator<String> getStringsContaining(String chars) {
		Iterator<String> iterator = this.stringSet.iterator();
		if((chars!=null)||(chars!=""))
		{
		HashSet<String> stringSetTemp = new HashSet<String>();
		while (iterator.hasNext()) {
		
			if(!iterator.next().contains(chars))
			{
				stringSetTemp.add(iterator.next());
			}
		}
		iterator = stringSetTemp.iterator();
		}
		return iterator;		
	}

Такой способ не работает, на тесте выдает NullPointExeption. Можно ли в принципе возвращать таким способом HashMap? Заранее спасибо за любую помощь.

Comments

[Александр Одинцов]

На сервере учебного центра. Там методы тестируются на корректность возвращаемого значения и на обработку некорректных значений, насколько я понимаю.

Answer 1

[Alexey_gr]

эксепшен вылезает, наверное потому что итератор пустой, добавь проверку hasNext
Вобще я не уверен, что это хорошая практика, вот так вот вернуть итератор неизвестно из чего. Лучше уже вернуть просто сет.
Ну или если есть возможность использовать java 8

public Stream<String> filterStringSet(final String filter) {
	if (filter == null || filter.isEmpty()) 
		return Stream.empty(); // можно конечно и null вернуть, но тогда это придется обрабатывать

	return stringSet.stream()
	.filter(str -> str.contains(filter));
}

// ну и потом 
filterStringSet("фильтр").forEach(System.out::println)

Comments

[Александр Одинцов]

Спасибо. К сожалению, в интерфейсе этот метод написан именно с возвращением Iterator, менять в интерфейсе ничего нельзя.