Code reviewMediumcr-g589

Subject Tcp socket framingLevel Mid–Senior~20 minCommon in Networking & APIs interviewsIndustries Software development, Telecom

Question

Review this Go TCP client snippet. It reads a length-prefixed message from a server.

What a strong answer looks like

Separate real bugs from style. Rank issues by severity, point at the root cause rather than the symptom, and suggest a concrete fix — specific and kind.

Learn the concepts

Talk through your review

Code to reviewgo

1func readMessage(conn net.Conn) ([]byte, error) {2	var header [4]byte3	if _, err := conn.Read(header[:]); err != nil {4		return nil, err5	}6	n := binary.BigEndian.Uint32(header[:])7	buf := make([]byte, n)8	if _, err := conn.Read(buf); err != nil {9		return nil, err10	}11	return buf, nil12}

Run or narrate your approach, then ask the coach.