Question

我是Golang的初学者。我曾经做过关于Go频道的练习。我打开并从主goroutine中的文件读取数据，然后将数据传递到第二个goroutine以保存到具有通道的另一个文件。我的代码是流程

  func main() {
   f, err := os.OpenFile("test.go", os.O_RDONLY, 0600)
   ch := make(chan []byte)
   buf := make([]byte, 10)
   bytes_len, err := f.Read(buf)
   fmt.Println("ReadLen:", bytes_len)
   if err != nil {
      fmt.Println("Error: ", err)
      return
   }
   go WriteFile(ch)
   for {
      ch<-buf
      bytes_len, err = f.Read(buf)
      if err != nil {
          fmt.Println("error=", err)
          break
      }
      if bytes_len < 10 {
          ch<-buf[:bytes_len]
          fmt.Println("Finished!")
          break
      }
   }
   time.Sleep(1e9)
   f.Close()
 }

  func WriteFile(ch <-chan []byte) {
    fmt.Println("* begin!")
    f, err := os.OpenFile("/home/GoProgram/test/test.file",  os.O_RDWR|os.O_APPEND|os.O_CREATE, 0660)
    if err != nil {
       fmt.Println("* Error:", err)
       return
    }
    /* Method1:  use the "select" will write to target file OK, but it is too slow!!!
    for {
      select {
         case bytes, ok:= <-ch:
            if ok {
              f.Write(bytes)
            } else {
              fmt.Println("* file closed!")
              break
            }
         default:
            fmt.Println("* waiting data!")
      }
    } \*/
    // Method 2: use "for {if}", this will get messed text in target file, not identical with the source file.
    for {
      if bytes, ok := <-ch; ok {
            f.Write(bytes)
            fmt.Println("* buff=", string(bytes))
            bytes = nil
            ok = false
      } else {
        fmt.Println("** End ", string(bytes), "  ", ok)
        break
      }
    }

    /* Method 3: use "for range", this will get messed text like in method2
    for data:= range ch {
         f.Write(data)
       //fmt.Println("* Data:", string(data))
    }
    \*/
    f.Close()
}

我的问题是为什么Method2和Method3会在目标文件中得到混乱文本？我该如何解决？

Answer 1

方法2和方法3搞乱了文本，因为读者和作者共享的缓冲区上存在争用。

以下是上述程序可能的语句执行顺序：

 R: bytes_len, err = f.Read(buf)  
 R: ch<-buf[:bytes_len]
 W: bytes, ok := <-ch; ok
 R: bytes_len, err = f.Read(buf)  // this writes over buffer
 W: f.Write(bytes)                // writes data from second read

运行您的计划with the race dectector。它将为您标记问题。

解决问题的一种方法是复制数据。例如，从读取的字节创建一个字符串，并将该字符串发送到通道。

另一种选择是将goroutines与io.Pipe连接起来。一个goroutine从源读取并写入管道。另一个goroutine从管道读取并写入目标。管道处理同步问题。

Answer 2

为了在Method2和Method3的评论中使用for循环获取代码段，您需要使用buffered频道。

文本在目标文件中混乱的原因是func main中的循环没有机制在锁定步骤中与WriteFile中的通道上的循环进行同步。

另一方面，发送到缓冲通道仅在缓冲区已满时阻止。缓冲区为空时接收阻止。因此，通过初始化缓冲区长度为1的频道，您可以使用Method1和/或Method2。剩下的就是记住在完成后关闭频道。

func main() {
    f, _ := os.OpenFile("test.txt", os.O_RDONLY, 0600)
    defer f.Close()
    ch := make(chan []byte, 1) // use second argument to make to give buffer length 1
    buf := make([]byte, 10)
    go WriteFile(ch)
    for {
        ch <- buf
        byteLen, err := f.Read(buf)
        if err != nil {
            break
        }
        if byteLen < 10 {
            ch <- buf[:byteLen]
            break
        }
    }
    close(ch) //close the channel when you done
}

func WriteFile(ch <-chan []byte) {
    f, err := os.OpenFile("othertest.txt", os.O_RDWR|os.O_APPEND|os.O_CREATE, 0660)
    defer f.Close()
    if err != nil {
        fmt.Println("* Error:", err)
        return
    }

    //Method 3: use "for range"
    for data := range ch {
        f.Write(data)
    }
}

如何从goroutine的频道连续接收数据

2 个答案: