Style update for more concise error handling

bar_chart.go

@@ -180,8 +180,7 @@ func (b *barChart) render(result *defaultRenderResult, seriesList SeriesList) (B
 		})
 	}
 	// the largest and smallest mark point
-	err := doRender(rendererList...)
-	if err != nil {
+	if err := doRender(rendererList...); err != nil {
 		return BoxZero, err
 	}
 

chart_option.go

@@ -393,8 +393,7 @@ func TableOptionRender(opt TableChartOption) (*Painter, error) {
 	if err != nil {
 		return nil, err
 	}
-	_, err = NewTableChart(p, opt).renderWithInfo(info)
-	if err != nil {
+	if _, err = NewTableChart(p, opt).renderWithInfo(info); err != nil {
 		return nil, err
 	}
 	return p, nil

charts.go

@@ -60,8 +60,7 @@ func (rh *renderHandler) Add(fn func() error) {
 
 func (rh *renderHandler) Do() error {
 	for _, fn := range rh.list {
-		err := fn()
-		if err != nil {
+		if err := fn(); err != nil {
 			return err
 		}
 	}
@@ -232,8 +231,7 @@ func defaultRender(p *Painter, opt defaultRenderOption) (*defaultRenderResult, e
 		Left:  rangeWidthLeft,
 		Right: rangeWidthRight,
 	})), opt.XAxis)
-	_, err := xAxis.Render()
-	if err != nil {
+	if _, err := xAxis.Render(); err != nil {
 		return nil, err
 	}
 
@@ -247,8 +245,7 @@ func defaultRender(p *Painter, opt defaultRenderOption) (*defaultRenderResult, e
 
 func doRender(renderers ...Renderer) error {
 	for _, r := range renderers {
-		_, err := r.Render()
-		if err != nil {
+		if _, err := r.Render(); err != nil {
 			return err
 		}
 	}
@@ -423,11 +420,10 @@ func Render(opt ChartOption, opts ...OptionFunc) (*Painter, error) {
 		})
 	}
 
-	err = handler.Do()
-
-	if err != nil {
+	if err = handler.Do(); err != nil {
 		return nil, err
 	}
+
 	for _, item := range opt.Children {
 		item.Parent = p
 		if item.Theme == "" {
@@ -436,8 +432,7 @@ func Render(opt ChartOption, opts ...OptionFunc) (*Painter, error) {
 		if item.FontFamily == "" {
 			item.FontFamily = opt.FontFamily
 		}
-		_, err = Render(item)
-		if err != nil {
+		if _, err = Render(item); err != nil {
 			return nil, err
 		}
 	}

echarts.go

@@ -94,8 +94,7 @@ func (es *EChartsSeriesData) UnmarshalJSON(data []byte) error {
 		return nil
 	}
 	v := _EChartsSeriesData{}
-	err := json.Unmarshal(data, &v)
-	if err != nil {
+	if err := json.Unmarshal(data, &v); err != nil {
 		return err
 	}
 	es.Name = v.Name
@@ -158,8 +157,7 @@ func (eb *EChartsPadding) UnmarshalJSON(data []byte) error {
 		return nil
 	}
 	arr := make([]int, 0)
-	err := json.Unmarshal(data, &arr)
-	if err != nil {
+	if err := json.Unmarshal(data, &arr); err != nil {
 		return err
 	}
 	if len(arr) == 0 {
@@ -225,8 +223,7 @@ func (emd *EChartsMarkData) UnmarshalJSON(data []byte) error {
 	}
 	data = convertToArray(data)
 	ds := make([]*_EChartsMarkData, 0)
-	err := json.Unmarshal(data, &ds)
-	if err != nil {
+	if err := json.Unmarshal(data, &ds); err != nil {
 		return err
 	}
 	for _, d := range ds {
@@ -282,7 +279,7 @@ type EChartsSeries struct {
 	YAxisIndex int                 `json:"yAxisIndex"`
 	ItemStyle  EChartStyle         `json:"itemStyle"`
 	// label configuration
-  Label     EChartsLabelOption `json:"label"`
+	Label     EChartsLabelOption `json:"label"`
 	MarkPoint EChartsMarkPoint   `json:"markPoint"`
 	MarkLine  EChartsMarkLine    `json:"markLine"`
 	Max       *float64           `json:"max"`
@@ -483,17 +480,16 @@ func (eo *EChartsOption) ToOption() ChartOption {
 
 func renderEcharts(options, outputType string) ([]byte, error) {
 	o := EChartsOption{}
-	err := json.Unmarshal([]byte(options), &o)
-	if err != nil {
+	if err := json.Unmarshal([]byte(options), &o); err != nil {
 		return nil, err
 	}
 	opt := o.ToOption()
 	opt.Type = outputType
-	d, err := Render(opt)
-	if err != nil {
+	if d, err := Render(opt); err != nil {
 		return nil, err
+	} else {
+		return d.Bytes()
 	}
-	return d.Bytes()
 }
 
 func RenderEChartsToPNG(options string) ([]byte, error) {

examples/horizontal_bar_chart/main.go

@@ -1,7 +1,6 @@
 package main
 
 import (
-	"io/ioutil"
 	"os"
 	"path/filepath"
 
@@ -10,17 +9,12 @@ import (
 
 func writeFile(buf []byte) error {
 	tmpPath := "./tmp"
-	err := os.MkdirAll(tmpPath, 0700)
-	if err != nil {
+	if err := os.MkdirAll(tmpPath, 0700); err != nil {
 		return err
 	}
 
 	file := filepath.Join(tmpPath, "horizontal-bar-chart.png")
-	err = ioutil.WriteFile(file, buf, 0600)
-	if err != nil {
-		return err
-	}
-	return nil
+	return os.WriteFile(file, buf, 0600)
 }
 
 func main() {
@@ -71,12 +65,9 @@ func main() {
 		panic(err)
 	}
 
-	buf, err := p.Bytes()
-	if err != nil {
+	if buf, err := p.Bytes(); err != nil {
 		panic(err)
-	}
-	err = writeFile(buf)
-	if err != nil {
+	} else if err = writeFile(buf); err != nil {
 		panic(err)
 	}
 }

examples/pie_chart/main.go

@@ -1,7 +1,6 @@
 package main
 
 import (
-	"io/ioutil"
 	"os"
 	"path/filepath"
 
@@ -10,17 +9,12 @@ import (
 
 func writeFile(buf []byte) error {
 	tmpPath := "./tmp"
-	err := os.MkdirAll(tmpPath, 0700)
-	if err != nil {
+	if err := os.MkdirAll(tmpPath, 0700); err != nil {
 		return err
 	}
 
 	file := filepath.Join(tmpPath, "pie-chart.png")
-	err = ioutil.WriteFile(file, buf, 0600)
-	if err != nil {
-		return err
-	}
-	return nil
+	return os.WriteFile(file, buf, 0600)
 }
 
 func main() {
@@ -61,12 +55,9 @@ func main() {
 		panic(err)
 	}
 
-	buf, err := p.Bytes()
-	if err != nil {
+	if buf, err := p.Bytes(); err != nil {
 		panic(err)
-	}
-	err = writeFile(buf)
-	if err != nil {
+	} else if err = writeFile(buf); err != nil {
 		panic(err)
 	}
 }

examples/radar_chart/main.go

@@ -1,7 +1,6 @@
 package main
 
 import (
-	"io/ioutil"
 	"os"
 	"path/filepath"
 
@@ -10,17 +9,12 @@ import (
 
 func writeFile(buf []byte) error {
 	tmpPath := "./tmp"
-	err := os.MkdirAll(tmpPath, 0700)
-	if err != nil {
+	if err := os.MkdirAll(tmpPath, 0700); err != nil {
 		return err
 	}
 
 	file := filepath.Join(tmpPath, "radar-chart.png")
-	err = ioutil.WriteFile(file, buf, 0600)
-	if err != nil {
-		return err
-	}
-	return nil
+	return os.WriteFile(file, buf, 0600)
 }
 
 func main() {
@@ -69,12 +63,9 @@ func main() {
 		panic(err)
 	}
 
-	buf, err := p.Bytes()
-	if err != nil {
+	if buf, err := p.Bytes(); err != nil {
 		panic(err)
-	}
-	err = writeFile(buf)
-	if err != nil {
+	} else if err = writeFile(buf); err != nil {
 		panic(err)
 	}
 }

examples/time_line_chart/main.go

@@ -3,7 +3,6 @@ package main
 import (
 	"crypto/rand"
 	"fmt"
-	"io/ioutil"
 	"math/big"
 	"os"
 	"path/filepath"
@@ -14,17 +13,12 @@ import (
 
 func writeFile(buf []byte) error {
 	tmpPath := "./tmp"
-	err := os.MkdirAll(tmpPath, 0700)
-	if err != nil {
+	if err := os.MkdirAll(tmpPath, 0700); err != nil {
 		return err
 	}
 
 	file := filepath.Join(tmpPath, "time-line-chart.png")
-	err = ioutil.WriteFile(file, buf, 0600)
-	if err != nil {
-		return err
-	}
-	return nil
+	return os.WriteFile(file, buf, 0600)
 }
 
 func main() {
@@ -66,17 +60,13 @@ func main() {
 			}
 		},
 	)
-
 	if err != nil {
 		panic(err)
 	}
 
-	buf, err := p.Bytes()
-	if err != nil {
+	if buf, err := p.Bytes(); err != nil {
 		panic(err)
-	}
-	err = writeFile(buf)
-	if err != nil {
+	} else if err = writeFile(buf); err != nil {
 		panic(err)
 	}
 }

horizontal_bar_chart.go

@@ -148,8 +148,7 @@ func (h *horizontalBarChart) render(result *defaultRenderResult, seriesList Seri
 			labelPainter.Add(labelValue)
 		}
 	}
-	err := doRender(rendererList...)
-	if err != nil {
+	if err := doRender(rendererList...); err != nil {
 		return BoxZero, err
 	}
 	return p.box, nil

line_chart.go

@@ -122,14 +122,14 @@ func (l *lineChart) render(result *defaultRenderResult, seriesList SeriesList) (
 			points = append(points, p)
 
 			// if the label does not need to be displayed, return
-      if labelPainter == nil {
+			if labelPainter == nil {
 				continue
 			}
 			labelPainter.Add(LabelValue{
-				Index: index,
-				Value: item.Value,
-				X:     p.X,
-				Y:     p.Y,
+				Index:    index,
+				Value:    item.Value,
+				X:        p.X,
+				Y:        p.Y,
 				FontSize: series.Label.FontSize,
 			})
 		}
@@ -185,8 +185,7 @@ func (l *lineChart) render(result *defaultRenderResult, seriesList SeriesList) (
 		})
 	}
 	// the largest and smallest mark point
-	err := doRender(rendererList...)
-	if err != nil {
+	if err := doRender(rendererList...); err != nil {
 		return BoxZero, err
 	}