Ta version ne me paraît pas difficile à lire. Perso je mettrais les return sur la même ligne que les if et je virerais les accolades pour que ça soit un peu moins verbeux, mais c'est plus une question de goût qu'autre chose.

Par contre tel que c'est écrit, tu as le même chemin de code (enfin y'en a 2, mais ils sont identiques) pour une situation normale (l'utilisateur annule) et une situation anormale (la sauvegarde échoue). Peut-être que tu gères l'échec de la sauvegarde directement dans save(), mais sans le contexte, ça apparaît suspect.