strconv.ParseInt дает 0
Я пытаюсь разобрать файл этой формы:
12
0,1,2,3,1,2,3,4,2,3,4,5
1,0,1,2,2,1,2,3,3,2,3,4
2,1,0,1,3,2,1,2,4,3,2,3
3,2,1,0,4,3,2,1,5,4,3,2
1,2,3,4,0,1,2,3,1,2,3,4
2,1,2,3,1,0,1,2,2,1,2,3
3,2,1,2,2,1,0,1,3,2,1,2
4,3,2,1,3,2,1,0,4,3,2,1
2,3,4,5,1,2,3,4,0,1,2,3
3,2,3,4,2,1,2,3,1,0,1,2
4,3,2,3,3,2,1,2,2,1,0,1
5,4,3,2,4,3,2,1,3,2,1,0
0,5,2,4,1,0,0,6,2,1,1,1
5,0,3,0,2,2,2,0,4,5,0,0
2,3,0,0,0,0,0,5,5,2,2,2
4,0,0,0,5,2,2,10,0,0,5,5
1,2,0,5,0,10,0,0,0,5,1,1
0,2,0,2,10,0,5,1,1,5,4,0
0,2,0,2,0,5,0,10,5,2,3,3
6,0,5,10,0,1,10,0,0,0,5,0
2,4,5,0,0,1,5,0,0,0,10,10
1,5,2,0,5,5,2,0,0,0,5,0
1,0,2,5,1,4,3,5,10,5,0,2
1,0,2,5,1,0,3,0,10,0,2,0
Где первая строка дает нам размер матрицы: n x nСледующие n строк - это матрица D. Тогда следующие n строк - это матрица W. 2n + 1 линий.
Вот код для его анализа и помещения в переменные.
func readFile(path string) (int64, Matrix, Matrix) {
// open the file
f, _ := os.Open(path)
defer f.Close()
// init the new reader on the opened file
r := bufio.NewReader(f)
// we get the n value
line, _ := r.ReadString('\n')
splitedLine := strings.Fields(line)
tmp, _ := strconv.ParseInt(splitedLine[0], 10, 64)
n := int64(tmp)
// we init the matrix W and D
D := Matrix{}
D.matrix = make([][]int64, n)
for i, _ := range D.matrix {
D.matrix[i] = make([]int64, n)
}
W := Matrix{}
W.matrix = make([][]int64, n)
for i, _ := range W.matrix {
W.matrix[i] = make([]int64, n)
}
// loop on the n first element
iter := int64(0)
for iter < n {
// we get the n following elements
line, _ = r.ReadString('\n')
for index, ele := range strings.Split(line, ",") {
D.matrix[iter][index], _ = strconv.ParseInt(ele, 10, 64)
}
iter++
}
iter = 0
for iter < n {
// we get the n following elements
line, _ = r.ReadString('\n')
for index, ele := range strings.Split(line, ",") {
W.matrix[iter][index], _ = strconv.ParseInt(ele, 10, 64)
}
iter++
}
return n, W, D
}
Результат для W должен быть таким:
[ 0., 1., 2., 3., 1., 2., 3., 4., 2., 3., 4., 5.],
[ 1., 0., 1., 2., 2., 1., 2., 3., 3., 2., 3., 4.],
[ 2., 1., 0., 1., 3., 2., 1., 2., 4., 3., 2., 3.],
[ 3., 2., 1., 0., 4., 3., 2., 1., 5., 4., 3., 2.],
[ 1., 2., 3., 4., 0., 1., 2., 3., 1., 2., 3., 4.],
[ 2., 1., 2., 3., 1., 0., 1., 2., 2., 1., 2., 3.],
[ 3., 2., 1., 2., 2., 1., 0., 1., 3., 2., 1., 2.],
[ 4., 3., 2., 1., 3., 2., 1., 0., 4., 3., 2., 1.],
[ 2., 3., 4., 5., 1., 2., 3., 4., 0., 1., 2., 3.],
[ 3., 2., 3., 4., 2., 1., 2., 3., 1., 0., 1., 2.],
[ 4., 3., 2., 3., 3., 2., 1., 2., 2., 1., 0., 1.],
[ 5., 4., 3., 2., 4., 3., 2., 1., 3., 2., 1., 0.]
Но когда я печатаю это; это дает мне:
[0 1 2 3 1 2 3 4 2 3 4 0]
[1 0 1 2 2 1 2 3 3 2 3 0]
[2 1 0 1 3 2 1 2 4 3 2 0]
[3 2 1 0 4 3 2 1 5 4 3 0]
[1 2 3 4 0 1 2 3 1 2 3 0]
[2 1 2 3 1 0 1 2 2 1 2 0]
[3 2 1 2 2 1 0 1 3 2 1 0]
[4 3 2 1 3 2 1 0 4 3 2 0]
[2 3 4 5 1 2 3 4 0 1 2 0]
[3 2 3 4 2 1 2 3 1 0 1 0]
[4 3 2 3 3 2 1 2 2 1 0 0]
[5 4 3 2 4 3 2 1 3 2 1 0]
Я не знаю почему. Спасибо за ваш ответ.
1 ответ
Знай это Reader.ReadString() возвращает вам string значение с разделителем. Так r.ReadString('\n') вернет вам строку с последующим символом новой строки.
И когда вы разбиваете строку на запятые, последняя часть будет содержать символ новой строки, и попытка разобрать номер по ней не удастся, как показано в этом примере:
line := "0,1,2\n"
for _, ele := range strings.Split(line, ",") {
fmt.Println(strconv.ParseInt(ele, 10, 64))
}
Вывод (попробуйте на Go Playground):
0 <nil>
1 <nil>
0 strconv.ParseInt: parsing "2\n": invalid syntax
Вы религиозно пропускаете проверки ошибок, не делайте этого! Как вы можете видеть в приведенном выше примере, проверка и печать ошибки, возвращенной strconv.ParseInt() сразу обнаружил ошибку и что не так.
Что вы можете сделать, это вырезать символ новой строки перед тем, как пытаться разобрать любое число из него, например:
strconv.ParseInt(strings.TrimSpace(ele), 10, 64)
Или используйте bufio.Scanner который может сканировать строки, и он автоматически обрезает завершающие символы новой строки.