From 730c4de722b11932e81d92ff2e883b4b5f20c654 Mon Sep 17 00:00:00 2001
From: liuxiaolong <736321739@qq.com>
Date: 星期二, 04 六月 2019 09:49:46 +0800
Subject: [PATCH] fix server,return when conn closed

---
 server.go |   43 ++++++++++++++++++++++---------------------
 1 files changed, 22 insertions(+), 21 deletions(-)

diff --git a/server.go b/server.go
index 0caff95..a39a6c9 100644
--- a/server.go
+++ b/server.go
@@ -18,8 +18,12 @@
 var port = flag.Int("port", 6000, "The port to listen on; ")
 var sec = flag.Int("sec", 10, "the second for query data. ")
 
+var Level = flag.String("level","ErrorLevel","log level")
+
 func main() {
 	flag.Parse()
+
+	log.SetLogLevel(*Level)
 
 	fmt.Println(*port)
 	src := *addr + ":" + strconv.Itoa(*port)
@@ -39,7 +43,7 @@
 		conn, err := listener.Accept()//
 		connArr = append(connArr,conn)
 		if err != nil {
-			log.Log.Errorf("some connecion error: %s\n", err)
+			log.Log.Infoln("some connecion error: ", err)
 		}
 		go handleConnection(conn,connArr)
 	}
@@ -52,9 +56,11 @@
 	ech := make(chan error)
 	go func(conn net.Conn, ech chan error) {
 		buf := make([]byte, 10)
-		_, err := conn.Read(buf)
-		if err != nil {
+		for {
+			readMsg, err := conn.Read(buf)
+			log.Log.Infoln("Read completed,readMsg:",readMsg,",err:",err)
 			ech <- err
+			break
 		}
 	}(conn, ech)
 	tick := time.NewTicker(10 * time.Second)
@@ -62,38 +68,33 @@
 	for {
 		select {
 		case <-tick.C:
-			handleMessage(conn, connArr)
+			if !handleMessage(conn, connArr){
+				conn.Close()
+				return
+			}
 		case err := <-ech:
-			log.Log.Errorln(err, "remoteAddr ", remoteAddr, " close")
+			log.Log.Infoln(err, "remoteAddr ", remoteAddr, " close")
 			conn.Close()
-			break
+			return
 		}
+
 	}
 
 	log.Log.Infoln("Client at " + remoteAddr + " disconnected.")
 }
 
-func handleMessage(conn net.Conn, connArr []net.Conn) {
+func handleMessage(conn net.Conn, connArr []net.Conn) bool {
 	jsonstring := esutil.PostAction(*sec, *Eurl, *Picurl)
 	if jsonstring == nil {
-		log.Log.Infoln("the data is nil")
-		return
+		log.Log.Infoln("the data is nil,remoteArr:",conn.RemoteAddr())
+		return true
 	}
 	jsonstring = append(jsonstring, []byte("\000")...)
 	log.Log.Infoln("jsonstring len: ", len(jsonstring), "\000 data: ", len("\000"))
 	_, err := conn.Write(jsonstring)
 	if err !=nil{
-		fmt.Println("conn: ",conn)
-		log.Log.Errorln(err)
-		conn.Close()
-		index :=0
-		for ;index < len(connArr); {
-			if connArr[index] == conn{
-				connArr = append(connArr[:index],connArr[index+1:]...)
-				continue
-			}
-			index++
-		}
-		fmt.Println("connArr: ",connArr)
+		log.Log.Infoln("conn.WriteErr:",err)
+		return false
 	}
+	return true
 }

--
Gitblit v1.8.0