Android ChipGroup重复标签限制实现优化求助

先指出原代码的问题 #

你的现有逻辑存在bug：当检测到重复标签时，仅执行break跳出循环，但后续的addChip依然会执行，导致重复标签还是会被添加到ChipGroup中。下面提供两种更优的解决方案：

方案一：用HashSet维护已存在标签（推荐） #

HashSet的查找操作时间复杂度为O(1)，比遍历子View的O(n)更高效，代码也更简洁易维护。

1. 在类中声明一个HashSet成员变量，用于存储已添加的标签文本：

private val existingTags = HashSet<String>()

2. 修改标签添加逻辑：

val inputTag = input_video_tag.text.toString().trim()
if (inputTag.isNotEmpty()) {
    if (existingTags.contains(inputTag)) {
        Toast.makeText(this, "不能添加重复标签", Toast.LENGTH_SHORT).show()
    } else {
        addChip(inputTag)
        existingTags.add(inputTag)
        input_video_tag.setText("")
    }
}

3. 若支持删除Chip，记得同步更新HashSet（示例）：

private fun addChip(tagText: String) {
    val chip = Chip(this).apply {
        text = tagText
        isCloseIconVisible = true
        setOnCloseIconClickListener {
            chip_group.removeView(this)
            existingTags.remove(tagText)
        }
    }
    chip_group.addView(chip)
}

方案二：优化原遍历逻辑（无需额外集合） #

如果不想额外维护集合，可修复原代码的逻辑漏洞，确保检测到重复时不执行添加操作：

val inputTag = input_video_tag.text.toString().trim()
if (inputTag.isNotEmpty()) {
    var isDuplicate = false
    for (i in 0 until chip_group.childCount) {
        val chip = chip_group.getChildAt(i) as Chip
        if (chip.text.toString() == inputTag) {
            isDuplicate = true
            break
        }
    }
    if (isDuplicate) {
        Toast.makeText(this, "不能添加重复标签", Toast.LENGTH_SHORT).show()
    } else {
        addChip(inputTag)
        input_video_tag.setText("")
    }
}

方案对比 #

• HashSet方案：查找效率更高，适合Chip数量较多的场景，代码简洁易维护，优先推荐。
• 遍历方案：无需额外维护集合，适合Chip数量极少的简单场景，但每次检测都要遍历所有子View，效率偏低。

内容的提问来源于stack exchange，提问作者Andrey