diff --git a/src/server/game/player_data.go b/src/server/game/player_data.go
index 7787f504..e67560bb 100644
--- a/src/server/game/player_data.go
+++ b/src/server/game/player_data.go
@@ -641,6 +641,9 @@ func (p *Player) HandleItem(itemList []*item.Item, Label string) error {
 		switch IType {
 		case item.ITEM_TYPE_ENERGY: // 能量
 			err := p.GetPlayerBaseMod().AddEnergy(p, v.Num)
+			if err != nil {
+				return err
+			}
 			p.TeLog("asset_change", map[string]interface{}{
 				"item_id":       v.Id,
 				"change_type":   change_type,
@@ -649,15 +652,13 @@ func (p *Player) HandleItem(itemList []*item.Item, Label string) error {
 				"change_reason": Label,
 			})
 			is_update = true
-			if err != nil {
-				return err
-			}
+
 		case item.ITEM_TYPE_STAR: // 星星
 			err := p.GetPlayerBaseMod().AddStar(p, v.Num)
-			is_update = true
 			if err != nil {
 				return err
 			}
+			is_update = true
 			p.TeLog("asset_change", map[string]interface{}{
 				"item_id":       v.Id,
 				"change_type":   change_type,
@@ -667,10 +668,10 @@ func (p *Player) HandleItem(itemList []*item.Item, Label string) error {
 			})
 		case item.ITEM_TYPE_DIAMOND: // 钻石
 			err := p.GetPlayerBaseMod().AddDiamond(v.Num)
-			is_update = true
 			if err != nil {
 				return err
 			}
+			is_update = true
 			if v.Num < 0 {
 				p.QuestTrigger(&quest.Trigger{Label: quest.TRIGGER_LABEL_DIAMOND, A: []interface{}{-v.Num}})
 			}
@@ -929,6 +930,9 @@ func (p *Player) HandleItem(itemList []*item.Item, Label string) error {
 			BackDataType[item.ITEM_TYPE_AD_CARD] = struct{}{}
 		default:
 			err := ItemMod.AddItem(v.Id, v.Num)
+			if err != nil {
+				return err
+			}
 			p.TeLog("asset_change", map[string]interface{}{
 				"item_id":       v.Id,
 				"change_type":   change_type,
@@ -948,9 +952,6 @@ func (p *Player) HandleItem(itemList []*item.Item, Label string) error {
 					"event_id":   aId,
 				})
 			}
-			if err != nil {
-				return err
-			}
 		}
 	}
 	ResItemPopId := 0