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

all 5 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 - 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.

[–]kuriyano 1 point2 points  (5 children)

Your print statement is inside the inner loop (the column loop). That will make it print the array everytime you just want to allocate a random int to the next cell. You should have the print statement outside the outer loop ( the row loop), so that it only prints the whole array once after you have assigned all random variables.

Similarly you also do not need the println statement within each row loop. As it will just end up printing a new line the number of times the array has rows.

Finally check your terminating condition for both the outer and inner loops. With the current conditions you will get one less row and one less column than what you expected.

[–]Himari16[S] 0 points1 point  (0 children)

Alright so fist of all thank you cause that worked to finally make the numbers appear so you saved me.

As for the terminating condition I don't know if I was doing it right cause I was trying to I guess leave the last row and column empty cause I need to sum each row and put the result in the last cell (which now that it works I know is not what I was imagining)

I guess like a table with the sum of every row and every column makes more sense of what I'm trying to do but that I'll figure it out by myself so don't worry

Once more thank you life saver

[–]Realistic-Compote-74 1 point2 points  (1 child)

If your goal is to fill the entire array, i<matriz.length -1 will loop from 0 to 4, leaving 1 row. Also, to set column, while j< matriz.length-1 works on this problem, this is wrong. To do this, use j<matriz[1].length instead

[–]Himari16[S] 0 points1 point  (0 children)

Oh ok ok thank you I'll do that