检查多个哈希图是否为空

Question

我正在尝试查找多个 HashMap 是否为空。

提供一些上下文。我在这里声明了一个 hashmap。

static Map<Integer, College> tblColleges = new HashMap<Integer, College>();

对于每个大学对象：

Map<Integer, Department> tblDepartments = new HashMap<Integer, Department>();

我正在尝试添加专业。专业只能作为部门的属性存在。

这是我现在拥有的。

int numberofColleges = Databases.tblColleges.size();
int emptyColleges = 0;
for(int key: Databases.tblColleges.keySet()) {
        if(Databases.getTblColleges(key).tblDepartments.isEmpty()) {
                emptyColleges++;
            }
        }
    if(numberofColleges == emptyColleges) {
        System.out.println("Invalid. Requires at least 1 department.");
    }

如果至少有 1 所大学有部门，我应该只能创建专业。

基本上对于 tblColleges 中存在的每个大学对象，我正在检查它的部门哈希图是否为空。如果它是空的，那么我增加空大学的数量。

之后，我将大学对象的数量与找到的空大学对象的数量进行比较，如果它们相等，则打印一个错误。

我想知道是否有更好更有效的方法来做到这一点，也许使用一些我不熟悉的函数而不是使用变量。

Answer 1

问：你能“更有效”地进行检查吗？

A：你可以稍微优化一下：

boolean nonEmptyColleges = false;

for (int key: Databases.tblColleges.keySet()) {
    if (!Databases.getTblColleges(key).tblDepartments.isEmpty()) {
        nonEmptyColleges = true;
        break;
    }
}

一旦找到College 和Department，上述短路。在很多情况下，这将是一个实质性的改进。

那么，假设Databases.tblColleges 是Map：

boolean nonEmptyColleges = false;

for (int college: Databases.tblColleges.values()) {
    if (!college.tblDepartments.isEmpty()) {
        nonEmptyColleges = true;
        break;
    }
}

---

问：你能用更少的代码做检查吗？

答：使用 Java 8 流，您可以将最后一个流编写为：

boolean nonEmptyColleges = Databases.tblColleges.values().stream()
           .anyMatch(c -> !c.tblDepartments.isEmpty());

（我认为...）

---

问：但这是正确的方法吗？

答：IMO，没有。

您似乎打算在每次添加专业时都进行此检查。那没有必要。

专业只能作为部门的属性存在。

您需要检查的关键是您要为其添加专业的部门是否存在。

• 如果部门不存在，则无法将专业添加到其中。
• 如果系确实存在，您可以选择它的专业，无论它目前是否是大学的系¹。

这里更重要的一点是，任何数据模型都会有各种数据完整性规则/约束。但这确实意味着您需要在每次更改模型时明确检查所有这些。您只需要检查变更的前提条件（例如，部门是否存在）以及可能因变更而失效的任何约束。

---

^{1 - “非”情况假设可能有其他方法可以找到Department。它可能是一个单独的 Department 对象表，也可能是您正在创建和构建一个新的 Department 并且尚未将其添加到其 College 中。}