J’ai tendance à être de ton avis, mais après zieuté la partie iOS, t’as clairement besoin de doc.
get_root_viewcontroller est soit méchamment bugee, soit plutôt inutile, voire dangereuse. Pour faire simple, iOS présente les alertes dans une nouvelle UIWindow (qui devient donc la key window). Ta fonction renvoie donc un controller d’alerte dans ce cas, là où je pense que tu voudrais renvoyer le UIApplication.shared.delegate.window.rootViewController. Problème c’est qu’on pas la moindre idée de ce que tu voulais renvoyer au final, vu que ton nom de méthode est super générique et que t’as pas comment l’intentention.
Idem sur present_view_controller dont l’implémentation est super chelou (le respondsToSelector, vous targetez vraiment encore iOS 7?), et aussi super dangereux (popoverPresentationController peut être nil, auquel cas ça va te faire bizarre quand tu vas lui assigner une source view). Je passe sur la réutilisation d’un nom de method uikit mais avec une semantique très différente.
Au final, je suis vraiement pas convaincu d’avoir même une vague idée de ce qu’est censée faire cette fonction.
Bref, si tu veux suivre la philosophie "no docs, va falloir sérieusement revoir tes noms de fonctions, ou admettre qu’écrire un peu de doc ne fait pas de mal.
[^] # Re: Merci pour ce partage - mais la doc fouque
Posté par groumly . En réponse au journal Publication de bibliothèques c++ sous licence libre. Évalué à 9.
J’ai tendance à être de ton avis, mais après zieuté la partie iOS, t’as clairement besoin de doc.
get_root_viewcontroller est soit méchamment bugee, soit plutôt inutile, voire dangereuse. Pour faire simple, iOS présente les alertes dans une nouvelle UIWindow (qui devient donc la key window). Ta fonction renvoie donc un controller d’alerte dans ce cas, là où je pense que tu voudrais renvoyer le UIApplication.shared.delegate.window.rootViewController. Problème c’est qu’on pas la moindre idée de ce que tu voulais renvoyer au final, vu que ton nom de méthode est super générique et que t’as pas comment l’intentention.
Idem sur present_view_controller dont l’implémentation est super chelou (le respondsToSelector, vous targetez vraiment encore iOS 7?), et aussi super dangereux (popoverPresentationController peut être nil, auquel cas ça va te faire bizarre quand tu vas lui assigner une source view). Je passe sur la réutilisation d’un nom de method uikit mais avec une semantique très différente.
Au final, je suis vraiement pas convaincu d’avoir même une vague idée de ce qu’est censée faire cette fonction.
Bref, si tu veux suivre la philosophie "no docs, va falloir sérieusement revoir tes noms de fonctions, ou admettre qu’écrire un peu de doc ne fait pas de mal.