调用TreeSet时发生ClassCastException <Long> .contains（Long.valueOf（someLongValue））

Question

我很沮丧 我这样宣布我的集合：

    private Set<Long> applicationIds;

然后我像这样填充它：

public void setApplicationIds( Set<Long> applicationIds ) {
    this.applicationIds = new TreeSet<Long>( applicationIds );
    this.applications = null;
}

然后，我尝试使用它：

public List<Application> getApplications() {
    if ( applications == null ) {
        applications = new ArrayList<Application>();
        if ( applicationIds != null ) {
            for ( Application application : availableApplications ) {
                if ( applicationIds.contains( Long.valueOf( application.getId() ) ) ) {
                    applications.add( application );
                }
            }
        }
    }
    return applications;
}

我最终得到了：

java.lang.ClassCastException: java.lang.String cannot be cast to java.lang.Long
    at java.lang.Long.compareTo(Long.java:50)
    at java.util.TreeMap.getEntry(TreeMap.java:346)
    at java.util.TreeMap.containsKey(TreeMap.java:227)
    at java.util.TreeSet.contains(TreeSet.java:234)
    at org.mitre.asias.pf.pnp.viewmodel.Subscription.getApplications(Subscription.java:84)

导致异常的行（堆栈跟踪中的第84行）是这一行：

                if ( applicationIds.contains( Long.valueOf( application.getId() ) ) ) {

也许我错过了一些东西，但是如果声明是Set<Long>并且我正在调用contains一个Long.valueOf值的contains方法，那么我该如何获取此异常？

这是用于JSF应用程序的模型bean。 我正在使用Java 6，Tomcat 6.0.32，mojarra 2.1.14，但是这些都不应该很重要，因为泛型应该防止这种问题的编译时间...

--------------编辑-----------------

它实际上是JSF ...我用这个设置器组合了一个超级简化的示例：

public void setSelectedValues(Set<Long> selectedValues) {
    this.selectedValues = selectedValues;
    if (logger.isTraceEnabled()) {
        StringBuilder message = new StringBuilder("Selected values:");
        for (Object value : selectedValues) {
            message.append("\n\t'").append(value.getClass().getName())
                    .append("': '").append(value.toString()).append("'");
        }
        logger.trace(message.toString());
    }
    this.selections = null;
}

绑定到此组件：

<p:selectManyCheckbox id="numbers"
   value="#{controller.selectedValues}" layout="pageDirection">
  <f:selectItems value="#{controller.availableValues}" />
</p:selectManyCheckbox>

将其写入日志：

15:45:16.887 [http-bio-8080-exec-9] TRACE com.pastdev.learn.debug.Controller - Selected values:
    'java.lang.String': '1'
    'java.lang.String': '5'

因此，简单的答案就是正确的答案（感谢@PaulTomblin强调这一点）。 设置器获取调用一个Set包含String秒。 那么，现在最好的转换过程是什么？ 我是否需要遍历整个列表，将每个值都转换为Long？

附带说明一下，我使用Java 7在Tomcat 7上对此进行了测试，但ClassCastException消失了，但是， contains方法总是返回false ，这与预期的一样。

--------------编辑2 -----------------

我发现我的组件绑定正确的方式回答这里 。

--------------编辑3 -----------------

这是对该问题的更好解释。

Answer 1

也许我缺少了一些东西，但是如果声明是Set，并且我正在调用传递Long.valueOf值的contains方法，那么我该如何获取此异常？

请注意，在Java中，泛型类型注释仅是编译器的提示，在运行时不起作用，因此有可能在运行时违反这些约束（但在某处会出现编译器警告）。

看起来您的Set<Long>实际上包含至少一个String。 那套东西从哪里来？

因为泛型应该避免这种问题的编译时间

是的，您的代码中应该有一个警告，指出缺少通用类型或未经检查的强制类型转换。 这只是一个警告，因为泛型是可选的。 在您确实使用过的地方，那将是一个错误。

Answer 2

好的，@ Lucas我只是尝试使用快速和肮脏的代码，使用您在问题中发布的代码，它工作正常。 可能是您需要再次检查Application类。

package javaapplication2;
import java.util.ArrayList;
import java.util.List;
import java.util.Set;
import java.util.TreeSet;

public class JavaApplication2 {


private static Set<Long> applicationIds;

private List<Application> applications;

private static final List<Application> availableApplications = new ArrayList<>();

public static void main(String[] args) {

    JavaApplication2 proj = new JavaApplication2();
    for (int i = 1; i <= 11; i++) {
        Application application = new Application();
        application.setId(i);
        availableApplications.add(application);
    }
    Set<Long> applicationIds1 = new TreeSet<>();
    applicationIds1.add(11L);
    applicationIds1.add(12L);
    applicationIds1.add(13L);

    proj.setApplicationIds(applicationIds1);
    for (Application appl : proj.getApplications()) {
        System.out.println(appl.getId());
    }
}

public void setApplicationIds(Set<Long> applicationIds) {
    this.applicationIds = new TreeSet<Long>(applicationIds);
    this.applications = null;
}

public List<Application> getApplications() {
    if (applications == null) {
        applications = new ArrayList<Application>();
        if (applicationIds != null) {

            for (Application application : availableApplications) {
                if (applicationIds.contains(Long.valueOf(application.getId()))) {
                    applications.add(application);
                }
            }
        }
    }
    return applications;
}

}

和：

package javaapplication2;

class Application {
private int id;

public int getId() {
    return id;
}

public void setId(int id) {
    this.id = id;
}

}