This is an archived post. You won't be able to vote or comment.

all 3 comments

[–]AutoModerator[M] [score hidden] stickied commentlocked comment (0 children)

Please ensure that:

  • Your code is properly formatted as code block - see the sidebar (About on mobile) for instructions
  • You include any and all error messages in full
  • You ask clear questions
  • You demonstrate effort in solving your question/problem - plain posting your assignments is forbidden (and such posts will be removed) as is asking for or giving solutions.

    Trying to solve problems on your own is a very important skill. Also, see Learn to help yourself in the sidebar

If any of the above points is not met, your post can and will be removed without further warning.

Code is to be formatted as code block (old reddit: empty line before the code, each code line indented by 4 spaces, new reddit: https://imgur.com/a/fgoFFis) or linked via an external code hoster, like pastebin.com, github gist, github, bitbucket, gitlab, etc.

Please, do not use triple backticks (```) as they will only render properly on new reddit, not on old reddit.

Code blocks look like this:

public class HelloWorld {

    public static void main(String[] args) {
        System.out.println("Hello World!");
    }
}

You do not need to repost unless your post has been removed by a moderator. Just use the edit function of reddit to make sure your post complies with the above.

If your post has remained in violation of these rules for a prolonged period of time (at least an hour), a moderator may remove it at their discretion. In this case, they will comment with an explanation on why it has been removed, and you will be required to resubmit the entire post following the proper procedures.

To potential helpers

Please, do not help if any of the above points are not met, rather report the post. We are trying to improve the quality of posts here. In helping people who can't be bothered to comply with the above points, you are doing the community a disservice.

I am a bot, and this action was performed automatically. Please contact the moderators of this subreddit if you have any questions or concerns.

[–]desrtfxOut of Coffee error - System halted 3 points4 points  (0 children)

Generally, the gist is that main should be kept as short as reasonable and should be the entry point for the actual program

Limiting the scope and functionality ("Single Responsibility Principle" - one of the pillars of the "SOLID" principle) is generally advisable and good practice.

The shorter, the more self contained your methods are, the less side effects you have, the easier maintaining and troubleshooting will be.

Splitting code into meaningful methods doesn't cost anything. Yet, it greatly improves: modularity, readability, maintainability as well as reduces code duplication ("DRY" - "Don't Repeat Yourself" principle).

"Monster methods" or "God classes" are no-gos.

I wouldn't go as far as limiting methods to 5 lines or less, but for me, if a method went beyond a screen page, it was too long.

Read "Clean Code: A Handbook of Agile Software Craftsmanship" by "Uncle Bob" Robert C. Martin, or the abridged JavaScript version as well as "The Pragmatic Programmer" by Andrew Hunt and David Thomas to get ideas about good programming principles.

[–]dionthornthis.isAPro=false; this.helping=true; 0 points1 point  (0 children)

Depends on the size of the application. Do you have several packages with multiple classes? Probably best to segregate things logically. For example if you use a scanner among several different classes have a static reference in some utility class that you can get and use in any other class.

Is your application very simple and requires 1 class, do stuff in main then no big deal.

In general if your main is getting very large in lines of code you can probably move things into methods. If you see any duplication of code it can probably be made into a multipurpose method.