output.Saver.Save - Code Metrics - Inspection of "Adding flag to specify where to save scan output &..." - stefanoj3/dirstalk - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#68)

by Stefano

created 2019-08-11 13:03 UTC

output.Saver.Save A

↳ Parent: pkg/scan/output/saver.go

Complexity

Conditions

Size

Total Lines	12
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	8
nop	1
dl	0
loc	12
rs	10
c	0
b	0
f	0

package output

import (
	"fmt"
	"io"
	"os"

	"github.com/pkg/errors"
	"github.com/stefanoj3/dirstalk/pkg/scan"
)

var (
	errNilWriteCloser = errors.New("Saver: writeCloser is nil")
)

func NewFileSaver(path string) (Saver, error) {

	file, err := os.Create(path)
	if err != nil {
		return Saver{}, errors.Wrapf(err, "failed to create file `%s` for output", path)
	}

	return Saver{writeCloser: file}, nil
}

type Saver struct {

	writeCloser io.WriteCloser
}

func (f Saver) Save(r scan.Result) error {

	if f.writeCloser == nil {
		return errNilWriteCloser
	}

	rawResult, err := convertResultToRawData(r)
	if err != nil {
		return errors.Wrap(err, "Saver: failed to convert result")
	}

	_, err = fmt.Fprintln(f.writeCloser, string(rawResult))
	return errors.Wrapf(err, "Saver: failed to write result: %s", rawResult)
}

func (f Saver) Close() error {

	if f.writeCloser == nil {
		return errNilWriteCloser
	}

	return f.writeCloser.Close()
}


1			package output
2
3			import (
4			"fmt"
5			"io"
6			"os"
7
8			"github.com/pkg/errors"
9			"github.com/stefanoj3/dirstalk/pkg/scan"
10			)
11
12			var (
13			errNilWriteCloser = errors.New("Saver: writeCloser is nil")
14			)
15
16			func NewFileSaver(path string) (Saver, error) {
			0 ignored issues – show introduced 2019-08-11 11:30 UTC by Report Bug Copy Issue Report exported function NewFileSaver should have comment or be unexported Loading history...
17			file, err := os.Create(path)
18			if err != nil {
19			return Saver{}, errors.Wrapf(err, "failed to create file `%s` for output", path)
20			}
21
22			return Saver{writeCloser: file}, nil
23			}
24
25			type Saver struct {
			0 ignored issues – show introduced 2019-08-11 11:30 UTC by Report Bug Copy Issue Report exported type Saver should have comment or be unexported Loading history...
26			writeCloser io.WriteCloser
27			}
28
29			func (f Saver) Save(r scan.Result) error {
			0 ignored issues – show introduced 2019-08-11 11:30 UTC by Report Bug Copy Issue Report exported method Saver.Save should have comment or be unexported Loading history...
30			if f.writeCloser == nil {
31			return errNilWriteCloser
32			}
33
34			rawResult, err := convertResultToRawData(r)
35			if err != nil {
36			return errors.Wrap(err, "Saver: failed to convert result")
37			}
38
39			_, err = fmt.Fprintln(f.writeCloser, string(rawResult))
40			return errors.Wrapf(err, "Saver: failed to write result: %s", rawResult)
41			}
42
43			func (f Saver) Close() error {
			0 ignored issues – show introduced 2019-08-11 11:30 UTC by Report Bug Copy Issue Report exported method Saver.Close should have comment or be unexported Loading history...
44			if f.writeCloser == nil {
45			return errNilWriteCloser
46			}
47
48			return f.writeCloser.Close()
49			}
50

stefanoj3 / dirstalk

Pull Request — master (#68)

output.Saver.Save A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like