|
아래는 앱과의 통신프로토콜 페이지 입니다.
앱에서 빠른시간안에 두번을 아래페이지를 호출하는 에러가 있어서 begintrans을 select 구문위에 걸어 한번만 실행 되도록 할려고 하는데
계속 두번씩 insert 가 되고 있습니다.
select 구문위에 저렇게 사용하게는게 맞는지 고수님들 조언 부탁합니다.
On Error Resume Next
oConn.BeginTrans
sSQL = "SELECT id FROM members WITH(XLOCK) WHERE id='" & userid & "' "
oRs.Open sSQL, oConn
If Not oRs.BOF Then
ret = "DUP"
Else
sSQL = "SELECT newid() "
Set sRs = oConn.Execute(sSQL)
userguid = sRs(0)
Set sRs = Nothing
userguid = Replace(userguid,"{","")
userguid = Replace(userguid,"}","")
If onenew="Y" Then
sSQL = "INSERT INTO members (member_guid, id, pwd, coin, app_id, join_device, join_key, join_ver, join_date, adult_flag, flag) VALUES ("
sSQL = sSQL & " '" & userguid & "' "
sSQL = sSQL & ",'" & userid & "' "
sSQL = sSQL & ",'" & Replace(userpw,"'","''") & "' "
sSQL = sSQL & ",'0' "
sSQL = sSQL & ",'" & appid & "' "
sSQL = sSQL & ",'" & model & "' "
sSQL = sSQL & ",'" & devicekey & "' "
sSQL = sSQL & ",'" & ver & "' "
sSQL = sSQL & ",getdate() "
sSQL = sSQL & ",'" & adult & "' "
sSQL = sSQL & ",'ONENEW' "
sSQL = sSQL & ") "
LogDev "SQL", sSQL
oConn.Execute sSQL
Else
sSQL = "INSERT INTO members (member_guid, id, pwd, coin, app_id, join_device, join_key, join_ver, join_date, adult_flag) VALUES ("
sSQL = sSQL & " '" & userguid & "' "
sSQL = sSQL & ",'" & userid & "' "
sSQL = sSQL & ",'" & Replace(userpw,"'","''") & "' "
sSQL = sSQL & ",'0' "
sSQL = sSQL & ",'" & appid & "' "
sSQL = sSQL & ",'" & model & "' "
sSQL = sSQL & ",'" & devicekey & "' "
sSQL = sSQL & ",'" & ver & "' "
sSQL = sSQL & ",getdate() "
sSQL = sSQL & ",'" & adult & "' "
sSQL = sSQL & ") "
LogDev "SQL", sSQL
oConn.Execute sSQL
End If
ret = "OK"
End If
oRs.Close
If Err.Number<>0 Then
LogWrite "RegMember Error", Err.Description
ret = "ERROR"
Else
oConn.CommitTrans
End If
| 
등록안내
등록안내
그런데 위 언어는 try ~ catch가 아닌 그냥 성공, 실패여부가 return되나 보네요.
암튼 해당 문제는 트랜젝션으로 해결하는게 아닙니다.
해당 앱에서 2번 호출을 막는게 베스트 이겠으나, 그게 불가능 한경우로 보이는데 그렇다면 insert 문을 보건데
members 테이블의 pk는 member_guid 이지만 논리적 pk는 id 인것으로 보입니다.
따라서 SELECT newid() 를 통해서 member_guid 를 바로가져오지 말고 그전에 id가 이미 있는지를 확인후
없는경우에만 member_guid 를 가져와 insert를 하도록 해줘야 합니다.
또는 newid(id) 와 같이 펑션에 id를 파라메터로 받게하여 id가 없는 경우에만 member_guid 를 리턴하게 해주면 됩니다.
추가로 member_guid에 이미 유니크인덱스가 걸려있을텐데 사용자가 id가 있느냐 없느냐를 빠르게 판단해야할테니 id에 인덱스를 추가로 걸어줘야겠지요.