findByid와 findByname의 리턴을 다르게 작성한 이유가 있나요?

질문 1 : findByName
@Override
public Optional<Member> findById(long Id) {
    return Optional.ofNullable(store.get(Id));

}

@Override
public Optional<Member> findByName(String Name) {
    return store.values().stream().filter(member -> member.getName().equals(Name)).findAny();
}

여기서 findByName은 필요가 없지 않나요?
MemoryMemberRepository 클레스 에서 hashmap을 사용하면 나중에 키 값으로 해당 이름을 가져오겠다는 뜻으로 보이는데
그러면 키값이 null이면 자동으로 value값도 null이니 굳이 작성하지 않아도 되는 코드 아닌가 라는 생각이 들었습니다.
아! 혹시 value값으로 key값을 찾기 위하여 필요한 코드인가요?

질문 2: 왜 findByid와 findByName 코드가 다른가요? 둘다 동일하게 null인경우 optional을 통해서 null값을 출력하는 과정인데
두 코드가 다를 필요가 없지 않나요?

store.values().stream().filter