It's nice that edge doesn't need to contain references to source and target vertices, but it can be a bit awkward to have to getEdges and then getEndpoints(). This is especially true for situations (e.g. timestamped edges) where there are multiple edges connecting pairs and the implementing class is likely to know a smarter way to get a set of Pairs. If there isn't a problem with this I'd be fine doing this myself, I just thought I'd post it here first in case there are reasons for not wanting this.
Log in to post a comment.