Proposal: Add a new flag to generate another type String()

[xiekeyi98]

Is your feature request related to a problem? Please describe.

It is inconvenience when I print logs with String().
%#v it is not readable enough.

Describe the solution you'd like

I suppose we could add a new flag to indicate another String() function.
Such as
-generate_json_string() to indicate thriftgo generate the string() function with "json.MarshalString()"

[xiekeyi98]

thriftgo/generator/golang/templates/struct.go

Lines 96 to 102 in 1412205

	func (p *{{$TypeName}}) String() string {
	if p == nil {
	return "<nil>"
	}
	{{- UseStdLibrary "fmt"}}
	return fmt.Sprintf("{{$TypeName}}(%+v)", *p)
	}

I guess we could change to

func (p *{{$TypeName}}) String() string {
	{{- if Features.NilSafe}}
	if p == nil {
		return ""
	}
       return JSON.MarshalToString()
	{{- end}}

       {{else }}
	if p == nil {
		return "<nil>"
	}
	{{- UseStdLibrary "fmt"}}
	return fmt.Sprintf("{{$TypeName}}(%+v)", *p)
       {{ - end}}
}

[HeyJavaBean]

I agree.