change methods arguments to be nullable

[windmemory]

Align definition with wechaty/grpc#81

Also use null value to clear the corporation remark or description.

[huan]

Could you please explain why do we have a contactPhone set method but have no contactcorporationRemark set method?

[windmemory]

We do have set method for phone, corporationRemark and description, but we don't have get method for corporationRemark and description since we put it into the payload. Do you think we need a method to get it?

[huan]

We have three new payloads: phone, description, and corporationRemark.

You mean 2 of them are in the payload but the other is not?

Should we put them all into the payload, or remove them all out of the payload, I mean, treat them all the same class citizen?

[windmemory]

Yes, the phone is not in the payload now, maybe we should put it also in the payload?

[huan]

I'd like to keep them the same class because they are very similar payload properties, aren't they?

So let's put the phone to the payload, then all those methods will only need to implement the set method.

[windmemory]

Yep, I think you are right, modified the code according to the comment.

[huan]

Please fix the array optionally property then we will be able to merge it.

[huan]

LGTM

[huan]

Thank you very much for the improvement!