• Post Reply Bookmark Topic Watch Topic
  • New Topic

rate my first program  RSS feed

 
Greenhorn
Posts: 24
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
I think I made it a lot more complicated than it has to be...
its just a calculator

if anyone has a post I can look that shows you how to make code neater I'll take a look at it

IntArray.java


Math.java



CalcFrame.java
 
Rancher
Posts: 2240
28
  • Likes 1
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
My comments: 
Several names do not follow standard java naming conventions.  Variables and methods should start with lowercase letter.
The formatting of the source text is inconsistent.  Much of the indentations are not proper.
Too much use of static. 
 
Jeremy Hutto
Greenhorn
Posts: 24
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
thank you for your input!
I'll try to figure out what static means and does, as I was programming it just kept telling me it should be static so I let it correct it.
 
Marshal
Posts: 56600
172
  • Likes 1
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
Welcome to the Ranch
Jeremy Hutto wrote:. . . I'll try to figure out what static means . . . .
That is a misleading error message; even using the term non‑static is bad. It tempts people to make things static because they cannot otherwise be accessed, when the correct solution is to move the access out of a static context.

I am afraid the long lines so difficult to read, that I am going to have to bounce the post back to you and ask you to shorten all those long lines.
 
  • Post Reply Bookmark Topic Watch Topic
  • New Topic
Boost this thread!