[fin] add_subcription

modules/subscription.go

@@ -46,7 +46,7 @@ func (manager *sub_manager) add_subscription(msg MsgUnit) error {
 
 	//namelist에서 name찾고, 없으면 추가
 	for nameptr != nil {
-		// *  compare함수 구현되기 전 임시로 끼어넣은 것 (이 부분은 secure.go 완성되면 다시 확인)
+		// *  compare함수 구현되기 전 임시로 끼워넣은 것 (이 부분은 secure.go 완성되면 다시 확인)
 		if compare(nameptr.name, name) == 1 {
 			findOk = true
 			break
@@ -73,22 +73,14 @@ func (manager *sub_manager) add_subscription(msg MsgUnit) error {
 
 	// list[name]에 value추가
 	if len(value) == 2 { // 단일 식이라면 (single)
-		valptr := nameptr.list.head
-		for valptr != nil {
-			// * 이 부분도 compare함수 구현하면 다시봐야함
-			if compare(valptr.val, conv(value[0])) == 1 { // value와 같은 val을 갖는 노드가 존재한다면
-				valptr.insert_Sub(value[1], subnumber, true)
-				return nil // add_subscription ok
-			}
-			valptr = valptr.next
+		valptr := nameptr.list.getPos(conv(value[0]))
+		if valptr == nil{
+			nameptr.list.add_ValueNode(value[0], value[1], subnumber, true)
+		} else{
+			valptr.insert_Sub(value[1], subnumber, true)
 		}
-
-		// value와 일치하는 노드가 x -> 추가
-		nameptr.list.add_ValueNode(value[0], value[1], subnumber, true)
-
 		return nil // add_subscription ok
 	} else {
-
 		// rangesub check
 		manager.israngesub[subnumber] = true
 
@@ -110,11 +102,8 @@ func (manager *sub_manager) add_subscription(msg MsgUnit) error {
 			if valptr2 == nil {
 				nameptr.list.add_ValueNode(value[3], value[4], subnumber, false)
 			}
-
 		} else {
-
 			// '||'로 묶여 있다면 두 식 모두 single로 처리가능
-
 			if valptr1 != nil && valptr2 != nil { // val1, val2를 갖는 노드가 둘 다 존재
 				valptr1.insert_Sub(value[1], subnumber, true)
 				valptr2.insert_Sub(value[4], subnumber, true)
@@ -126,7 +115,7 @@ func (manager *sub_manager) add_subscription(msg MsgUnit) error {
 				nameptr.list.add_ValueNode(value[3], value[4], subnumber, true)
 			}
 		}
-		return nil
+		return nil // add_subscription ok
 	}
 	return errors.New("Can't add_subscription")
 }
@@ -146,4 +135,4 @@ func conv(val string) []int64 {
 // * 암호화된 두 value 비교함수 (임시)
 func compare(v1 []int64, v2 []int64) int {
 	return 1
-}
+}