From 40ba019edd0d0ad6928645e0f5033b120816ba51 Mon Sep 17 00:00:00 2001
From: Daisuke MAKIUCHI <makiuchi-d@klab.com>
Date: Mon, 1 Apr 2024 21:17:54 +0900
Subject: [PATCH] =?UTF-8?q?regexp=E3=82=84=E3=82=81=E3=82=8B?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 server/lobby/service/api.go | 16 ++++------------
 1 file changed, 4 insertions(+), 12 deletions(-)

diff --git a/server/lobby/service/api.go b/server/lobby/service/api.go
index e7ca881..320358c 100644
--- a/server/lobby/service/api.go
+++ b/server/lobby/service/api.go
@@ -248,9 +248,7 @@ func (sv *LobbyService) handleCreateRoom(w http.ResponseWriter, r *http.Request)
 }
 
 var (
-	idRegexp  = regexp.MustCompile("^[0-9a-f]+$")
-	numRegexp = regexp.MustCompile("^[0-9]+$")
-	grpRegexp = regexp.MustCompile("^[0-9]+$")
+	idRegexp = regexp.MustCompile("^[0-9a-f]+$")
 )
 
 type JoinVars struct {
@@ -263,25 +261,19 @@ func NewJoinVars(r *http.Request) JoinVars {
 
 func (vars JoinVars) roomId() (string, bool) {
 	id := vars.r.PathValue("roomId")
-	return id, idRegexp.Match([]byte(id))
+	return id, idRegexp.MatchString(id)
 }
 
 func (vars JoinVars) roomNumber() (int32, bool) {
 	v := vars.r.PathValue("roomNumber")
-	if !numRegexp.Match([]byte(v)) {
-		return 0, false
-	}
 	n, err := strconv.ParseInt(v, 10, 32)
-	return int32(n), err == nil && n != 0
+	return int32(n), err == nil && n > 0
 }
 
 func (vars JoinVars) searchGroup() (uint32, bool) {
 	v := vars.r.PathValue("searchGroup")
-	if !grpRegexp.Match([]byte(v)) {
-		return 0, false
-	}
 	n, err := strconv.ParseInt(v, 10, 32)
-	return uint32(n), err == nil
+	return uint32(n), err == nil && n >= 0
 }
 
 func (sv *LobbyService) handleJoinRoom(w http.ResponseWriter, r *http.Request) {