0
votes

Réduire le nombre d'accessoires

J'ai un composant d'où l'utilisateur peut router pour former. Il y a deux options: créer une nouvelle forme ou éditer l'ancien. J'envoie toutes les données avec un lieu de liaison à ce formulaire composant. Mais parce que j'ai plus de 8 champs, il devient désordonné de vérifier si l'état de l'emplacement existe ou s'il n'y a pas d'état de localisation, utilisez des valeurs par défaut.

Actuellement, j'ai quelque chose comme ça: P>

<Link to={{
  pathname: '/register/create', state: {
    userProfile: item <--comes from mapping
  }
}}>


0 commentaires

3 Réponses :


1
votes

Au moins un point qui est possible pour simplifier: dans emplacement.State. / code> stocke un objet et l'état local peut être rempli directement:

p>

this.state = this.props.location.state 
  ? this.props.location.state.userProfile
  : {
    firstName'',
    familyName: '',
    address: 'Enter your address',
    city: 'Select your city',
    country: 'Select your country',
    field: '',
  };


1 commentaires

Aussi agréable moyen de simplifier ce long état



0
votes

Le "Sélectionnez votre ville" ne devrait pas être dans votre état. Il vaut mieux le laisser vide et le transmettre à la forme comme {state.city || "Sélectionnez votre ville"}

JS: xxx

jsx (vue): xxx


0 commentaires

1
votes

Je préférerais déconstruire l'objet et avoir un objet par défaut dans le cas de toutes les données xxx


1 commentaires

pas de soucis @kertix j'aime faire du code propre et lisible haha