Créer un utilisateur et modifier utilisateur - DRY ou séparation, ce qui est plus important

Question

Je me demande ce que les développeurs plus expérimentés pensent de la façon de gérer des processus presque identiques, comme un utilisateur créant un profil et un utilisateur éditant son profil. Pour le moment, j'ai une seule méthode de contrôleur et une vue unique qui gère la distinction entre un nouvel utilisateur et un utilisateur existant/éditant, juste en passant un drapeau d'édition $, donc je sais gérer les légères différences entre les deux. Avoir ces conditions à travers ma méthode et vue de contrôleur semble un peu en désordre, mais alors il se sent aussi comme beaucoup de duplication pour créer une vue distincte et une méthode de contrôleur pour chaque situation. Qu'est-ce que les gens ont tendance à faire dans cette situation?

Answer 1

Lorsque je configure un système qui peut créer de nouveaux utilisateurs, j'ai tendance à n'avoir besoin que de très peu d'informations sur l'enregistrement - courrier électronique et mot de passe par exemple. Ensuite, la section "profile" de votre site peut ensuite gérer toutes les modifications des détails de l'utilisateur restant, sans avoir à s'inquiéter de savoir s'il s'agit d'un utilisateur nouveau ou existant. D'après mon expérience, les gens ont tendance à se décourager lorsqu'ils s'inscrivent à quelque chose si trop de questions sont posées dès le début! Retardez ce qui n'est pas vraiment nécessaire et recueillez cette information plus tard!

Answer 2

Je pense que vous devriez toujours considérer les aspects à long terme des décisions de conception.

Il peut être plus propre de séparer ces fonctions, mais vous avez deux fonctions de nettoyage. Donc, en termes d'esthétique, vous pourriez argumenter de toute façon. Il y a l'ancien compromis entre le code dupliqué propre et la maintenabilité. Toutefois, à long terme, d'autres considérations entrent en jeu et vous commencez à penser aux risques. Le risque de duplication du code est que les deux parties du code deviennent désynchronisées, potentiellement de manière subtile et difficile à détecter (par exemple, en mesure d'ajouter des données rejetées lors de la mise à jour). Cela peut être bien pire si vous n'êtes pas la personne qui maintient le code, ou si vous n'avez pas touché le code depuis longtemps.

Donc, à mon humble avis garder le drapeau d'édition. Vous avez tout en un seul endroit, même si c'est désordonné.