我的任务是创建一个GUI计算器,用于计算平均值,标准偏差,并使用数组,方法和while循环跟踪您输入的等级数,以便您可以重新输入允许的等级数。
我的问题是我认为我已经正确地布置了所有内容,我可能是错的,因为我对Java和编码更新。我的程序提取我的JOptionPane来输入成绩,但是当我输入成绩并尝试输入空白语句或点击取消时我得到一个NullPointerException,我尝试了一些方法来修复它无济于事。
我似乎无法正确退出循环并在文本区域显示我的值。
import javax.swing.JOptionPane;
public class GradeCalculatorForm extends javax.swing.JFrame {
double[] gradeArray = new double [25];
private double avg;
private double stdDeviation;
int i;
int numElem;
double sum;
String input;
public GradeCalculatorForm() {
initComponents();
}
/**
*
* @param gradeArray
* @param numElem
* @return
*/
public double getAverage(double[] gradeArray, int numElem){ //getAverage Method
sum = 0;
for(i=0;i<numElem;i++){
sum=sum+gradeArray[i];
}
return (sum/numElem);
}
/**
*
* @param gradeArray
* @param numElem
* @param avg
* @return
*/
public double getStdDev(double[] gradeArray, int numElem, double avg){ //getStdDev method
sum = 0;
for(i=0;i<numElem;i++){
sum=sum+Math.pow((gradeArray[i]-avg), 2);
}
return Math.sqrt(sum/numElem);
}
我当前的主要活动代码:
GradeCalculatorForm grades = new GradeCalculatorForm();
//initialize while loop
i=0;
while(i<25){
//try for NumberFormateException
try
{
//enter values into array
for(i=0;i<25;i++)
numElem = Integer.parseInt(JOptionPane.showInputDialog("Enter grade:"));
gradeArray[i]=numElem;
avg = grades.getAverage(gradeArray, numElem);
stdDeviation = grades.getStdDev(gradeArray, numElem, avg);
//over 25 values error message
if(i>25)
{
JOptionPane.showMessageDialog(null, "You can only input 25 values!");
}
}
//catch for NumberFormatException
catch(NumberFormatException e)
{
JOptionPane.showMessageDialog(null, "Your input must be numeric!");
}
}
Numbertxt.setText(Double.toString(numElem));
Meantxt.setText(Double.toString(avg));
Stdtxt.setText(Double.toString(stdDeviation));
}
我再一次对Java,方法和数组都很陌生。这可能是完全错误的,但我没有任何语法错误,主要是陷入我自己的循环,异常错误,并没有正确显示我的数据,我取得了一些成功。我知道的很多。我有点不好意思。
编辑:我在我的JOptionPane循环中遇到了无法结束整个程序的情况,因此我无法抓住任何堆栈跟踪错误。编辑*:当我点击取消时,我终于让我的NPE出现了。
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at GradeCalculatorForm.GradesbtnActionPerformed(GradeCalculatorForm.java:168)
at GradeCalculatorForm.access$000(GradeCalculatorForm.java:9)
at GradeCalculatorForm$1.actionPerformed(GradeCalculatorForm.java:76)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2018)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2341)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:252)
at java.awt.Component.processMouseEvent(Component.java:6505)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3321)
at java.awt.Component.processEvent(Component.java:6270)
at java.awt.Container.processEvent(Container.java:2229)
at java.awt.Component.dispatchEventImpl(Component.java:4861)
at java.awt.Container.dispatchEventImpl(Container.java:2287)
at java.awt.Component.dispatchEvent(Component.java:4687)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4832)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4492)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4422)
at java.awt.Container.dispatchEventImpl(Container.java:2273)
at java.awt.Window.dispatchEventImpl(Window.java:2719)
at java.awt.Component.dispatchEvent(Component.java:4687)
at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:735)
at java.awt.EventQueue.access$200(EventQueue.java:103)
at java.awt.EventQueue$3.run(EventQueue.java:694)
at java.awt.EventQueue$3.run(EventQueue.java:692)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:87)
at java.awt.EventQueue$4.run(EventQueue.java:708)
at java.awt.EventQueue$4.run(EventQueue.java:706)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:705)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:242)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:161)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:150)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:146)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:138)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:91)
我仍然遇到一个问题,如果在我的JOptionPane中为空,当它为空时,它会拉出我的NumberFormatException捕获消息。建议?
答案 0 :(得分:2)
取出while循环。
在for循环中添加{}:
for(i=0;i<25;i++) {
numElem = Integer.parseInt(JOptionPane.showInputDialog("Enter grade:"));
gradeArray[i]=numElem;
}
答案 1 :(得分:0)
您正在使用numElem作为输入的等级,以及在getAverage和getStdDev中控制for循环。 gradeArray有25个元素,因此如果输入等级80,那些循环将尝试超出数组的末尾。数组知道它的长度,你可以将它作为gradeArray.length引用。但是,您可以通过从数组切换到List来轻松支持更多成绩。
循环和成绩输入测试没有多大意义。没有理由在for循环周围有一个while循环,if语句永远不会执行,因为i
到达时它将是25。另外,正如M21B8指出的那样,你的for循环需要围绕它。按照目前的情况,JOptionPane
将显示25次,然后在数组中设置条目并计算avg
和stdDeviation
的行将运行一次,{{1} }。
但是,这应该会导致i == 25
,因为ArrayIndexOutOfBoundsException
只有25个元素,索引从0到24.必须有一个单独的问题导致NPE。
答案 2 :(得分:0)
摆脱循环的最简单方法是使用break。你所要做的只是插入“break;”无论何处需要和繁荣,你的循环都会停止。
另外,我相信你应该考虑让你的输入成为一个字符串,这样你就可以测试不存在的和空的答案。
if ((input != null) && (input.length() > 0)) {
最后,由于这些是成绩,请确保没有输入为负数或0。
希望有所帮助。