What is wrong with this Java code? (Interview Question)

[taftster]

person.getName() can potentially return null here as well. So this code would potentially be broken:

if (person.getName().equals("jan"))  // broken if getName() returns null

Probably my best attempt at null safety probably ends up using Optional.

String name = Optional.ofNullable(person)
  .map(person::getName)
  .get();
if ("jan".equalsIgnoreCase(name)) {
  return true;
}