util.GetCSVReader - Code Metrics - Inspection of "Fix tests" - vvval/go-metadata-scanner - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( afbf8e...641184 )

by Valentin

created 2018-09-19 13:43 UTC

util.GetCSVReader A

↳ Parent: util/files.go

Complexity

Conditions

Size

Total Lines	6
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	5
nop	2
dl	0
loc	6
rs	10
c	0
b	0
f	0

package util

import (
	"encoding/csv"
	"log"
	"os"
	"path/filepath"
	"strings"
)

func MustOpenReadonlyFile(filename string) *os.File {

	file, err := os.OpenFile(filename, os.O_RDONLY, os.ModePerm)
	if err != nil {
		log.Fatalln(err)
	}

	return file
}

func GetCSVReader(file *os.File, sep rune) *csv.Reader {

	reader := csv.NewReader(file)
	reader.Comma = sep
	reader.FieldsPerRecord = -1

	return reader
}

func FileExists(filename string) bool {

	_, err := os.Stat(filename)

	return err == nil
}

func Extension(filename string) string {

	return strings.Trim(filepath.Ext(filename), ".")
}

func Abs(path string) string {

	abs, err := filepath.Abs(path)
	if err == nil {
		return abs
	}

	return path
}

func PathsEqual(p1, p2 string) bool {

	p1 = strings.TrimRight(p1, "/\\")
	p2 = strings.TrimRight(p2, "/\\")

	if p1 == p2 {
		return true
	}

	if filepath.FromSlash(p1) == filepath.FromSlash(p2) {
		return true
	}

	if filepath.ToSlash(p1) == filepath.ToSlash(p2) {
		return true
	}

	return false
}


1			package util
2
3			import (
4			"encoding/csv"
5			"log"
6			"os"
7			"path/filepath"
8			"strings"
9			)
10
11			func MustOpenReadonlyFile(filename string) *os.File {
			1 ignored issue – show introduced 2018-09-19 13:02 UTC by Report Bug Copy Issue Report exported function MustOpenReadonlyFile should have comment or be unexported Loading history...
12			file, err := os.OpenFile(filename, os.O_RDONLY, os.ModePerm)
13			if err != nil {
14			log.Fatalln(err)
15			}
16
17			return file
18			}
19
20			func GetCSVReader(file os.File, sep rune) csv.Reader {
			1 ignored issue – show introduced 2018-09-19 13:02 UTC by Report Bug Copy Issue Report exported function GetCSVReader should have comment or be unexported Loading history...
21			reader := csv.NewReader(file)
22			reader.Comma = sep
23			reader.FieldsPerRecord = -1
24
25			return reader
26			}
27
28			func FileExists(filename string) bool {
			1 ignored issue – show introduced 2018-09-19 13:02 UTC by Report Bug Copy Issue Report exported function FileExists should have comment or be unexported Loading history...
29			_, err := os.Stat(filename)
30
31			return err == nil
32			}
33
34			func Extension(filename string) string {
			1 ignored issue – show introduced 2018-09-19 13:02 UTC by Report Bug Copy Issue Report exported function Extension should have comment or be unexported Loading history...
35			return strings.Trim(filepath.Ext(filename), ".")
36			}
37
38			func Abs(path string) string {
			1 ignored issue – show introduced 2018-09-19 13:02 UTC by Report Bug Copy Issue Report exported function Abs should have comment or be unexported Loading history...
39			abs, err := filepath.Abs(path)
40			if err == nil {
41			return abs
42			}
43
44			return path
45			}
46
47			func PathsEqual(p1, p2 string) bool {
			1 ignored issue – show introduced 2018-09-19 13:46 UTC by Report Bug Copy Issue Report exported function PathsEqual should have comment or be unexported Loading history...
48			p1 = strings.TrimRight(p1, "/\\")
49			p2 = strings.TrimRight(p2, "/\\")
50
51			if p1 == p2 {
52			return true
53			}
54
55			if filepath.FromSlash(p1) == filepath.FromSlash(p2) {
56			return true
57			}
58
59			if filepath.ToSlash(p1) == filepath.ToSlash(p2) {
60			return true
61			}
62
63			return false
64			}
65

vvval / go-metadata-scanner

Push — master ( afbf8e...641184 )

util.GetCSVReader A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like