[THRIFT-4722] Netcore union "data" should be strongly-typed

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.12.0
• Fix Version/s: 0.13.0
• Component/s: netcore - Compiler
• Patch Info: Patch Available
• Assignee: Jens Geyer
• Reporter: J W
• Created: 05/Jan/19 03:58
• Updated: 16/Oct/19 22:27
• Resolved: 24/Jan/19 17:29

Description

Following thrift:
```thrift
struct PlayMsg

{ 1: string url, }

union RequestMsg

{ 1: PlayMsg Play, }

```

Generates:

public abstract partial class RequestMsg : TAbstractBase {
public abstract void Write(TProtocol protocol);
public readonly bool Isset;
public abstract object Data { get; }
protected RequestMsg(bool isset) {
Isset = isset;
}

// SNIP

public class Play : RequestMsg {
private PlayMsg _data;
public override object Data { get { return _data; } }
public Play(PlayMsg data) : base(true) {
this._data = data;
}
//SNIP

Usage:

// RequestMsg message = ...
switch (message)
{
case RequestMsg.Play msg:
    // Need a cast here T_T
    PlayMsg play = (PlayMsg)msg.Data;

If we get rid of that and instead generate a strongly-type getter we don't need to cast `Data`:

public class Play : RequestMsg{ 
    public PlayMsg Data { get; private set; }
    public Play(PlayMsg data) : base(true)
    { this.Data = data; }
//SNIP

 

Issue Links

• Dependency: THRIFT-1786 C# Union Typing
• relates to: THRIFT-4715 C# union "data" should be strongly-typed
• links to: GitHub Pull Request #1705