Question

我有以下代码，我将更改为并发程序。

$(element)

我决定尝试同时使printState函数。所以我把它改成了：

// Stefan Nilsson 2013-02-27

// This program creates pictures of Julia sets (en.wikipedia.org/wiki/Julia_set).
package main

import (
    "image"
    "image/color"
    "image/png"
    "log"
    "math/cmplx"
    "os"
    "strconv"
)

type ComplexFunc func(complex128) complex128

var Funcs []ComplexFunc = []ComplexFunc{
    func(z complex128) complex128 { return z*z - 0.61803398875 },
    func(z complex128) complex128 { return z*z + complex(0, 1) },
}

func main() {
    for n, fn := range Funcs {
        err := CreatePng("picture-"+strconv.Itoa(n)+".png", fn, 1024)
        if err != nil {
            log.Fatal(err)
        }
    }
}

// CreatePng creates a PNG picture file with a Julia image of size n x n.
func CreatePng(filename string, f ComplexFunc, n int) (err error) {
    file, err := os.Create(filename)
    if err != nil {
        return
    }
    defer file.Close()
    err = png.Encode(file, Julia(f, n))
    return
}

// Julia returns an image of size n x n of the Julia set for f.
func Julia(f ComplexFunc, n int) image.Image {
    bounds := image.Rect(-n/2, -n/2, n/2, n/2)
    img := image.NewRGBA(bounds)
    s := float64(n / 4)
    for i := bounds.Min.X; i < bounds.Max.X; i++ {
        for j := bounds.Min.Y; j < bounds.Max.Y; j++ {
            n := Iterate(f, complex(float64(i)/s, float64(j)/s), 256)
            r := uint8(0)
            g := uint8(0)
            b := uint8(n % 32 * 8)
            img.Set(i, j, color.RGBA{r, g, b, 255})
        }
    }
    return img
}

// Iterate sets z_0 = z, and repeatedly computes z_n = f(z_{n-1}), n â‰¥ 1,
// until |z_n| > 2  or n = max and returns this n.
func Iterate(f ComplexFunc, z complex128, max int) (n int) {
    for ; n < max; n++ {
        if real(z)*real(z)+imag(z)*imag(z) > 4 {
            break
        }
        z = f(z)
    }
    return
}

此更改会导致图像看起来非常不同。图案基本相同，但有很多白色像素以前没有。

这里发生了什么？

Answer 1

有两个问题：

你实际上并没有等到你的goroutines完成。
你没有将i and j传递给goroutine，所以他们几乎总是最后的i和j。

您的功能应该类似于：

func Julia(f ComplexFunc, n int) image.Image {
    var wg sync.WaitGroup
    bounds := image.Rect(-n/2, -n/2, n/2, n/2)
    img := image.NewRGBA(bounds)
    s := float64(n / 4)
    for i := bounds.Min.X; i < bounds.Max.X; i++ {
        for j := bounds.Min.Y; j < bounds.Max.Y; j++ {
            wg.Add(1)
            go func(i, j int) {
                n := Iterate(f, complex(float64(i)/s, float64(j)/s), 256)
                r := uint8(0)
                g := uint8(0)
                b := uint8(n % 32 * 8)
                img.Set(i, j, color.RGBA{r, g, b, 255})
                wg.Done()
            }(i, j)
        }
    }
    wg.Wait()
    return img
}

奖励提示，在深入了解并发时，使用race detector尝试使用代码通常是个好主意。

您可能必须使用互斥锁拨打img.Set，但我不太确定，我无法测试atm。

朱莉娅设置了由并发破坏的图像渲染

1 个答案: