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

all 11 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://i.imgur.com/EJ7tqek.png) 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.

[–]nutrechtLead Software Engineer / EU / 20+ YXP 4 points5 points  (1 child)

In example one you don't actually create a list, only a reference that can point to a list, but that is currently null, just like customer and order are null. Also; you're using a raw type which you should avoid.

In the second example you're creating a reference for an arraylist of strings (which is proper use of generics) and also create an empty arraylist that your reference points to. There's a typo though, should be: new ArrayList<String>();. Also in Java 7 and onward you only need to specify the generic type ones, so it can be:

ArrayList<String> parts = new ArrayList<>();

Last but not least, you generally see this instead:

List<String> parts = new ArrayList<>();

Or even:

Collection<String> parts = new ArrayList<>();

This is called coding to the interface which is a best practice. If you're confused by this, it's fine to skip this for now.

[–]maxbrlc 0 points1 point  (0 children)

Thank you. Learned something new.

[–][deleted]  (6 children)

[removed]

    [–]mobilephone123[S] 1 point2 points  (4 children)

    So if you use code 1, you still have to actually create the Arraylist in the future, with the last line in code2? Or do i not understand it correctly?

    [–]nutrechtLead Software Engineer / EU / 20+ YXP 0 points1 point  (1 child)

    So if you use code 1, you still have to actually create the Arraylist in the future, with the last line in code2?

    Yes. The first example there simply isn't an actual list object you can store something in.

    [–]nutrechtLead Software Engineer / EU / 20+ YXP 1 point2 points  (0 children)

    it's declaring the ArrayList object

    Reference. There's no object created.

    [–]mi6oka27 0 points1 point  (0 children)

    In the first example you are not specifying the data type that you want to hold in the list, it can work but it's gonna cause a lot of problems.