The bottom one deletes the node. The top one doesn't. This doesn't make it "better", it just means it does something the other doesn't. Deleting might even be bad depending on whether or not getFromList is expected to delete the retreived item.
yah, and IMHO a function named "get-something" performs a read-only operation; I'd be annoyed if as a side effect it modified the list.
But this is a homework question, and homework questions are not meant to teach the best programming practices; rather they try to teach specific concepts.