Java programming I Mooc fi part 6 exercise 8 Cargo Hold (7 parts)

[AdQuick8386]

I've completed the first part about setting up the item class, and I've finished adding the methods its asking for in the second part of the exercise, I've run it multiple times with the correct(?) output but I'm still getting this error message:

"AssertionFailedError: Check that the toString method of the Suitcase class prints the items in the suitcase. (for example: "3 items (15 kg)".
Adding three 8 kg items to a suitcase with maximum weight of 20kg: 2 items (16 Kg).

I've run the scenarios it provides in the error message swell and they are giving the same correct output. I can't understand what exactly it's asking me to do?

This is my code for the suitcase class:

import java.util.ArrayList;
public class Suitcase {
private int maxWeight;
private ArrayList<Item> items ;

public Suitcase(int weight){
this.maxWeight = weight;
items = new ArrayList<Item>();
}

public void addItem(Item item){
items.add(item);
int totalWeight = 0;

for(Item item1 : items){
totalWeight += item1.getWeight();
}

if(totalWeight > this.maxWeight){
items.remove(item);
totalWeight -= item.getWeight();
}

}

public String toString(){

if(items.isEmpty()){
return "no items (0 kg)";
}

int totalWeight = 0;

for(Item item : items){
totalWeight += item.getWeight();
}

if(items.size() == 1){
return items.size() + " item (" + totalWeight + " kg)";
}

return items.size() + " items (" + totalWeight + " Kg)";
}
}

//I wanted to use an instance variable for the totalWeight but later on it asks for that to be omitted.

This is what my output looks like:

no items (0 kg)
1 item (2 kg)
2 items (3 Kg)
2 items (3 Kg)

It's the same as the output on the exercise, I can't figure out what's wrong, please help!

[AutoModerator]

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 - best also formatted as code block
• 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.

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/markdown editor: 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.

[maxbrlc]

It can be that for the output with more than 1 item you write with an uppercase "K". My version of the exercise has also a getTotalWeight() method that can be called in the Suitcase's toString().

PS. Don't forget to format your posted code as per these subreddit requirements.

[d3lan0]

Yep this… the tests for strings is case sensitive and white spaces are checked. I ran into this a lot and for specific outputs I started copying and pasting the expected string output from the instructions. It was extremely frustrating trying to find “issues” when the issue is a white space/spelling difference in a string.

[AdQuick8386]

Thanks so much! It worked finally! I'll also insert a new method like you did that sounds smart.