Блог

Я изучаю golang и пытаюсь закончить tour of go. Я застрял в упражнении для срезов. Скопируйте вопрос и мое решение здесь. Может ли кто-нибудь критиковать это и сказать мне, что я здесь делаю неправильно?

Вопрос:

 Implement Pic. It should return a slice of length dy, each element of which is a slice of dx 8-bit 
unsigned integers. When you run the program, it will display your picture,
interpreting the integers as grayscale (well, bluescale) values.

The choice of image is up to you. Interesting functions include (x y)/2, x*y, and x^y.

(You need to use a loop to allocate each []uint8 inside the [][]uint8.)

(Use uint8(intValue) to convert between types.)
 

Мое решение:

 package main

import "golang.org/x/tour/pic"

func Pic(dx, dy int) [][]uint8 {
    ans := make([][]uint8, dy)
    for i:=0; i< dy; i   {
        slice := make([]uint8, dx)
        for j := 0; j<dx;j  {
            slice = append(slice, uint8((i j)/2))
        }
        ans = append(ans,slice)
    }
    return ans
}

func main() {
    pic.Show(Pic)
}

 

При запуске я получаю сообщение об ошибке:

паника: ошибка времени выполнения: индекс выходит за пределы диапазона [0] с длиной 0

Я не уверен, что я здесь делаю не так. Кроме того, почему в упражнении передается функция? Это предназначено?

Хорошо, я понял. Как я уже сказал в своем комментарии, вы должны заменить свои вызовы append на slice[j] = uint((i j)/2) and ans[i] = slice .

Упражнение вызывает вашу функцию с 256×256. Вы создаете фрагмент длиной 256, а затем добавляете другие фрагменты 256 раз, в результате чего получается фрагмент длиной 512 ans . Первые 256 записей пусты, поскольку append добавляется slice в конце. Поэтому, когда библиотека pic выполняет итерацию ваших данных, она пытается получить доступ к пустому фрагменту.

Обновление: еще один способ исправить алгоритм — инициализировать срезы длиной 0. Итак, редактирование

ans := make([][]uint8, 0) и slice := make([]uint8, 0)

также должно давать правильные результаты.