0

我正在编写一个简短的 Java 程序，它应该接收 5 个整数的列表，并将每个整数分配给 6 个成员之一。我想最大化在他们的价值min和价值之间获得价值的成员数量max。
无论如何，出于某种我不知道的原因，我的私人member课程似乎正在抛出 NullPointerException，但我完全不知道为什么。任何建议，将不胜感激。

import java.util.*;
public class assign {
    private class member {
        private String name;
        private int min;
        private int max;
        public member(String name, int min, int max) {
            this.name = name;
            this.min = min;
            this.max = max;
        }
    }

    public member[] members = {
        new member("Phil", 1, 20),
        new member("Molly", 1, 20),
        new member("Connor", 21, 40),
        new member("Sam", 21, 40),
        new member("Dan", 41, 60),
        new member("Theresa", 41, 60)
    };

    public void main(String[] args) {
        System.out.println("test");
        String s = StdIn.readLine();
        String[] temp = s.split(" ");
        ArrayList<Integer> nums = new ArrayList<Integer>(5);
        for (int i = 0; i < 5; i++) {
            nums.add(Integer.parseInt(temp[i]));
        }
        ArrayList<member> mems = new ArrayList<member>(Arrays.asList(members));

        for (int n : nums) {
            for (member m : mems) {
                if (n > m.min && n < m.max) {
                    System.out.println(m.name + " : " + n);
                    nums.remove(n);
                    mems.remove(m);
                }
            }
        }
        for (int n : nums) {
            member m = mems.remove(0);
            System.out.println(m.name + " : " + n);
        }
    }
}

从OP的评论：

java.lang.NullPointerException 
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) 
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.jav‌​a:25) 
at java.lang.reflect.Method.invoke(Method.java:597) 
at edu.rice.cs.drjava.model.compiler.JavacCompiler.runCommand(JavacCompiler.java:27‌​2)
4

2 回答 2

3

我敢打赌，您的错误是您当前的主要方法不是静态的：

// this should be a static method
public void main(String[] args) {
  // ...
}

解决方案：使 main 方法应为静态。

public static void main(String[] args) {
  // ...
}

解决此问题后，您将看到其他错误，主要是您尝试以静态方式访问非静态事物。要解决此问题，您需要为您的 assign 类（请重命名此 Assign 以便它以大写字母开头）提供公共方法来调用，然后让您的 main 方法创建一个 Assign 实例并调用这些方法。

例如这里：

ArrayList<member> mems = new ArrayList<member>(Arrays.asList(members));

您正在尝试在静态上下文中使用非静态成员数组。您将只想在您的 Assign 类中，在其公共方法中处理它。要么，要么使members静态。

于 2013-11-03T03:35:03.730 回答
0

像每个人一样的第一个大问题，您的代码在没有静态 main 的情况下如何运行？

您的代码有很多流程，请重新审视逻辑并尝试针对您的代码运行不同的测试用例（有效的测试用例，然后是无效的测试用例）。这是一个很好的问题陈述。

例如

for (int n : nums) {
            for (member m : mems) {
                if (n > m.min && n < m.max) {
                    System.out.println(m.name + " : " + n);
                    nums.remove(n);
                    mems.remove(m);
                }
            }
        }

您有ConcurrentModificationException等待发生。

你确定你想要这行：nums.remove(n);
IndexOutOfBoundsException。

于 2013-11-03T05:13:54.593 回答