La manipulation des valeurs null dans protobuffers

Je suis en train de travailler sur quelque chose qui récupère des données à partir de la base de données et les constructions protobuff message. Compte tenu de la possibilité que les valeurs nulles peuvent être récupérées à partir de la base de données pour certains champs , je vais obtenir Null pointer exception, tout en essayant de construire le protobuff message. Arriver à savoir que null n'est pas pris en charge dans protobuffs le fil http://code.google.com/p/protobuf/issues/detail?id=57, je me demande si le seul autre moyen de gérer les NPE se faire jeter consiste à insérer des contrôles manuels dans le fichier java correspondant à la proto comme ci-dessous!

message ProtoPerson{
    optional string firstName = 1;
    optional string lastName = 2;
    optional string address1 = 3;
}

ProtoPerson.Builder builder = ProtoPerson.Builder.newBuilder();
if (p.getFirstName() != null) builder.setFirstName(p.getFirstName());
if (p.getLastName() != null) builder.setLastName(p.getLastName());
if (p.getAddress1() != null) builder.setAddress1(p.getAddress1());
...

Donc, quelqu'un peut-il préciser si il y a de quelque autre manière efficace de gérer les valeurs nulles durant protobuff construction??

Version courte de la même chose: Optional.ofNullable(p.getFirstName()).ifPresent(builder::setFirstName);

InformationsquelleAutor Aarish Ramesh | 2014-01-20

9

Il n'y a pas de solution facile à ce. Je vous recommande juste de traiter avec le nul de contrôles. Mais si vous voulez vraiment vous débarrasser d'eux, voici quelques idées:
- Vous pourriez écrire un générateur de code du plugin qui ajoute setOrClearFoo() méthodes de chaque classe Java. Le générateur de code Java fournit les points d'insertion pour cela (voir la fin de cette page).
- Vous pouvez utiliser Java réflexion pour itérer sur les get*() méthodes de p, appeler, vérifier pour null, et ensuite appeler la set*() méthode de builder si non null. Cela aura l'avantage supplémentaire que vous n'aurez pas à mettre à jour votre copie de code à chaque fois que vous ajoutez un nouveau champ, mais il sera beaucoup plus lente que d'écrire du code que de copies de chaque champ explicitement.
- oh bien merci!!
- Veuillez jeter un oeil à cette question stackoverflow.com/questions/21474024/...
- Quelqu'un a réellement créé un générateur de code plugin qui fait la setOrClearFoo() ou de fournir un exemple?
InformationsquelleAutor Kenton Varda
43

Avertissement: la Réponse d'un Googleurs à l'aide de protobufs sur une base quotidienne. Je ne suis pas représentant de Google de toute façon.
1. Nom de votre proto Person au lieu de PersonProto ou ProtoPerson. Compilé protobufs sont juste des définitions de classe spécifiée par la langue que vous utilisez, avec quelques améliorations. L'ajout de "Proto" est extra verbosité.
2. Utilisation YourMessage.hasYourField() au lieu de YourMessage.getYourField() != null. La valeur par défaut pour protobuf chaîne est une chaîne vide, ce qui ne PAS égale à null. Alors que, peu importe si votre domaine n'est pas défini ou désactivée ou une chaîne vide, .hasYourField() renvoie toujours false. Voir les valeurs par défaut pour la commune de protobuf types de champ.
3. Vous avez probablement connu, mais je veux leur dire explicitement: Ne pas définir par programmation un protobuf champ de null. Même à l'extérieur de la protobuf, null provoque toutes sortes de problèmes. Utilisation .clearYourField() à la place.
4. Person.Builder classe ne PAS ont un .newBuilder() méthode. Person classe. Comprendre les Générateur de Modèle comme ceci: Vous créez un nouveau constructeur que si vous ne le possédez pas encore.
Une réécriture de votre protobuf:
```
message Person {
  optional firstName = 1;
  optional lastName = 2;
  optional address1 = 3;
}
```
Une réécriture de votre logique:
```
Person thatPerson = Person.newBuilder()
    .setFirstName("Aaa")
    .setLastName("Bbb")
    .setAddress1("Ccc")
    .build();

Person.Builder thisPersonBuilder = Person.newBuilder()

if (thatPerson.hasFirstName()) {
  thisPersonBuilder.setFirstName(thatPerson.getFirstName());
}

if (thatPerson.hasLastName()) {
  thisPersonBuilder.setLastName(thatPerson.getLastName());
}

if (thatPerson.hasAddress1()) {
  thisPersonBuilder.setAddress1(thatPerson.getAddress1());
}

Person thisPerson = thisPersonBuilder.build();
```
Et si thatPerson est un objet personne que vous avez créé qui a les valeurs d'attribut qui peut être une chaîne vide, des espaces vides ou nulles, alors je vous recommande d'utiliser La goyave est Chaînes de caractères de la bibliothèque:
```
import static com.google.common.base.Strings.nullToEmpty;

Person.Builder thisPersonBuilder = Person.newBuilder()

if (!nullToEmpty(thatPerson.getFirstName()).trim().isEmpty()) {
  thisPersonBuilder.setFirstName(thatPerson.getFirstName());
}

if (!nullToEmpty(thatPerson.hasLastName()).trim().isEmpty()) {
  thisPersonBuilder.setLastName(thatPerson.getLastName());
}

if (!nullToEmpty(thatPerson.hasAddress1()).trim().isEmpty()) {
  thisPersonBuilder.setAddress1(thatPerson.getAddress1());
}

Person thisPerson = thisPersonBuilder.build();
```
- Votre post est peut-être plein de bons conseils...mais je ne suis pas sûr d'avoir bien compris sa question? Par exemple, ses données n'est pas en provenance d'un autre protobuf objet; c'est à venir à partir d'une base de données ou d'une autre source, et peut avoir des valeurs null. Il voulait savoir comment traiter avec les valeurs null, essentiellement.
- Bon point Erhannis, merci. J'ai mis à jour la réponse par l'ajout d'une section pour les cas de non-protocole-tampon objets a vide de la chaîne de valeurs, une succession de valeurs de l'espace, et les valeurs null.
- 'provoque toutes sortes de problèmes de lien est mort, et j'aimerais vraiment en savoir plus
- Correction du lien. La goyave a été migré de Google Code pour GitHub, donc le nouveau lien maintenant. Merci de me laisser savoir!
- Je vais le nom de mon proto comme PersonMessage, de distinguer entre ma classe Personne (représenter une personne) et PersonMessage classe (représentent message à propos de la personne). Je ne vois rien de mal avec sa nomination PersonProto, ou ProtoPerson. Peut-être que c'est juste mon opinion personnelle.
- Comment devrait être thématiques avec des valeurs NULL à l'aide de proto3? Il n'existe pas de "Votre_message.hasYourField()" Méthodes...
- N'est-ce pas répondre la même chose que ce qui est suggéré dans la question - il suffit d'utiliser la valeur null chèques? La partie pertinente de la partie qui répond à la question est seulement dans le dernier paragraphe. Point de prêter à l'autre partie de la réponse est de répondre à des questions qui n'ont pas été posées.
- Concernant 1) la raison pour laquelle les gens ont tendance à avoir des classes nommées pour indiquer qu'il est un protobuf objet, c'est que pour la plupart des cas, l'entité que vous êtes le transfert est déjà un objet de domaine. En utilisant le même nom de classe pour le proto de la classe que l'objet de domaine peut avoir du sens, mais devient très déroutant si vous avez ces deux classes de référence dans une autre classe. En général, je ne voudrais pas la faire précéder de proto, mais à la place, ajouter DTO pour indiquer son but de mieux: en.wikipedia.org/wiki/Data_transfer_object
InformationsquelleAutor Michael Xin Sun

Vous devez vous connecter pour publier un commentaire.