container/gmap/gmap_hash_k_v_map.go:324

• The SetIfNotExist method does not utilize the nil checker mechanism. When the value is nil (according to the custom nil checker), it should not be set in the map. This is inconsistent with other methods like doSetWithLockCheck and GetOrSetFuncLock. Add an isNil check before setting the value to maintain consistency with the PR's goal of filtering typed nil values.

func (m *KVMap[K, V]) SetIfNotExist(key K, value V) bool {
	m.mu.Lock()
	defer m.mu.Unlock()
	if m.data == nil {
		m.data = make(map[K]V)
	}
	if _, ok := m.data[key]; !ok {
		m.data[key] = value
		return true
	}
	return false
}

container/gmap/gmap_hash_k_v_map.go:351

• The SetIfNotExistFuncLock method does not utilize the nil checker mechanism. Similar to SetIfNotExist, it should check if the value returned by f() is nil (according to the custom nil checker) before setting it in the map. This creates inconsistent behavior compared to GetOrSetFuncLock and other methods that were updated in this PR.

func (m *KVMap[K, V]) SetIfNotExistFuncLock(key K, f func() V) bool {
	m.mu.Lock()
	defer m.mu.Unlock()
	if m.data == nil {
		m.data = make(map[K]V)
	}
	if _, ok := m.data[key]; !ok {
		m.data[key] = f()
		return true
	}
	return false
}

container/gmap/gmap_hash_k_v_map.go:333

• The SetIfNotExistFunc method does not check for nil values. While it delegates to SetIfNotExist, which also doesn't check for nil, this creates a gap in the nil checker implementation. Consider evaluating f() and checking isNil before calling SetIfNotExist to ensure consistent behavior with the pattern established in ListKVMap.

func (m *KVMap[K, V]) SetIfNotExistFunc(key K, f func() V) bool {
	if !m.Contains(key) {
		return m.SetIfNotExist(key, f())
	}
	return false
}