Question

当我运行下面的代码时，我收到错误：

school_mark.go:8: invalid array bound s

我的代码：

package main

import "fmt"

func main(){
   var subj float64
   fmt.Println("Enter how much you have subjects inn school: ")
   fmt.Scanf("%f", &subj)
   s := subj
   var mark [s]float64
   var a float64
   for a = 0; a<s; a++{
     fmt.Scanf("%f", &mark)
   }
   var total float64
   var i float64
   for i= 0; i<subj; i++{
       total += mark[i]
   }
       fmt.Println(total/subj)    
}

有什么问题？

Answer 1

Spec: Array types:

长度是数组类型的一部分;它必须通过类型int的值来评估为非负constant。

[s]float64中的长度不是常数。

使用slice而不是数组类型，并且知道必须使用整数类型作为长度，例如：

var mark []float64 = make([]float64, int(s))

或简短：

mark := make([]float64, int(s))

接下来，总是使用整数类型（例如int）作为索引。您可以在for中声明它：

for i := 0; i < len(mark); i++ {
    fmt.Scanf("%f", &mark[i])
}

或者您甚至可以使用for ... range来限制切片的索引：

for i := range mark {
    fmt.Scanf("%f", &mark[i])
}

我还会在任何地方使用int类型：如果它们是非整数，则主题和标记本身没有意义。

同样fmt.Scanf()不使用换行符，后续的fmt.Scanf()调用会立即返回错误。

您还应该检查错误或成功解析的值，这两个值都由fmt.Scan*()函数返回。

此外，您不必将切片循环两次，您可以计算第一个切片的总数（总和）。

更进一步，您甚至不必将标记存储在切片中，您只需询问输入的标记数，即时计算总和并打印平均值。

这样的事情：

var n, total, mark int
fmt.Println("Number of subjects:")
fmt.Scanln(&n)
for i := 0; i < n; i++ {
    fmt.Scanln(&mark)
    total += mark
}
fmt.Println("Average:", float64(total)/float64(n))

如果您要添加所有必需的检查，它可能如下所示：

var n, total, mark int
fmt.Print("Number of subjects: ")
if _, err := fmt.Scanln(&n); err != nil || n <= 0 {
    fmt.Println("Wrong number!")
    return
}
for i := 0; i < n; i++ {
    fmt.Printf("Enter %d. mark: ", i+1)
    if _, err := fmt.Scanln(&mark); err != nil || mark < 1 || mark > 5 {
        fmt.Println("Wrong mark, enter it again!")
        fmt.Scanln()
        i-- // We're gonna re-scan the same mark
    } else {
        total += mark
    }
}
fmt.Println("Average:", float64(total)/float64(n))

Golang-无效数组绑定

1 个答案: