need help with a java assignment.

[Z-slasher]

For my java assignment I am supposed to modify a code to add other functions to it. I need to add sin, cos, tan, and x^y buttons on the calculator and make them work.

The problem is I get this error. (I know I probably got ^ wrong):
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 23
at Calculator.<init>(Calculator.java:124)
at Calculator.main(Calculator.java:586)

original code:
#1798262 - Pastie

where I am at so far:
#1799334 - Pastie

As I'm still a beginner java programmer I probably got other things wrong in the code, so any help that I can get I will appreciate.

[Dark]

You're getting an index out of range error. You're trying to call a array slot that doesn't exist.

Your program is hundreds of lines long so I'm not going to go through it all to find where, but you should investigate your arrays. I have a feeling its with your jbnButtons array.

[Z-slasher]

Sweet. fixed that! it was on line 97. thanks for showing that one.

now I see sin, cos, tan, and x^y isn't functioning the way they should.

any where I should put that on the code?

[Fubarable]

now I see sin, cos, tan, and x^y isn't functioning the way they should.

any where I should put that on the code?

Please re-read this comment:

Your program is hundreds of lines long so I'm not going to go through it all to find where, ....

Consider creating and posting an SSCCE to help us be better able to help you. You can find out what this entails by checking my link on this subject below.

[Z-slasher]

my apologies. let me break it down to the problem. There are 2 areas I'm not so sure about I'll start with the first switch statement on the program. I'll start with case 20 to 23 because that is the ones to look at.

(Find this one on line 240-368)

for (int i=0; i<jbnButtons.length; i++)
{
if(e.getSource() == jbnButtons[i])
{
switch(i)
{



case 20: // sin
processOperator("sin");
break;

case 21: // cos
processOperator("cos");
break;

case 22: // sin
processOperator("tan");
break;

case 23: // x^y
processOperator("^");
break;



and I modified the if statement.

(this is found on line 538 to 568 in the pastie)

double processLastOperator() throws DivideByZeroException {
double result = 0;
double numberInDisplay = getNumberInDisplay();

if (lastOperator.equals("/"))
{
if (numberInDisplay == 0)
throw (new DivideByZeroException());

result = lastNumber / numberInDisplay;
}

if (lastOperator.equals("*"))
result = lastNumber * numberInDisplay;

if (lastOperator.equals("-"))
result = lastNumber - numberInDisplay;

if (lastOperator.equals("+"))
result = lastNumber + numberInDisplay;

if (lastOperator.equals("sin"))
result = Math.sin(numberInDisplay);

if (lastOperator.equals("cos"))
result = Math.cos(numberInDisplay);

if (lastOperator.equals("tan"))
result = Math.tan(numberInDisplay);

return result;

These are the codes where I find the +,-,* and divide operators function. What could I be doing wrong for sin, cos, tan, and x^y?

[Fubarable]

For the trig calculations: Are your numbers in radians or degrees?

[Z-slasher]

It is radians. I actually figured this one out by looking at sqrt since that uses the math logic. I'll paste the what I did in C indentation so it is easy to look at.

Changing the switch statements to this gave me the correct number in radians.

here is the code for case 20 to 22:

{
result = Math.sin(getNumberInDisplay());
displayResult(result);
}

so I did that for cos, and tan as well.

now to figure out x^y. how do I do this one using the Math.pow logic?

edit:
{
result = Math.pow(getNumberInDisplay(), getNumberInDisplay());
displayResult(result);
}

This gives x^x without the = sign input. I need x^y.

[Z-slasher]

I figured it out. :) thanks for the help guys. I guess this is pretty simple even for someone who isn't really a java programmer.