StackOverflow2013

Note that there are some explanatory texts on larger screens.

plurals

PO
primarykey
Id
19575091
data
AcceptedAnswerId
0
AnswerCount
0
ClosedDate
CommentCount
0
CommunityOwnedDate
CreationDate
2013-10-24T19:40:07.517
FavoriteCount
0
LastActivityDate
2013-10-24T22:43:12.200
LastEditDate
2013-10-24T22:43:12.200
LastEditorUserId
344096
OwnerUserId
344096
ParentId
19574035
PostTypeId
2
Score
0
ViewCount
0
LastEditorDisplayName
text
Body
I see multiple things: First of all your last return statement will make the method return the first time you find an element that equals <code>x</code>. You should remove it. This is one cause of error. I think you wanted to use a while loop here so to find every <code>x</code> on the beginning of <code>start</code>? <pre><code>else if (start.data.equals(x)) { start = start.next; count--; } </code></pre> Now look at what happens if <code>start</code> only consists of one element, and this one element is also equal to <code>x</code>? You will receive an exception here <pre><code>for (ptr = start; ptr.next != null; ptr = ptr.next) { </code></pre> as you would call <code>null.next</code> that doesn't exist. Actually it would be even worse with a while-loop, but fortunately a null test will help. ;) Now to more cosmetic things: This <pre><code>if (start == null) return; else if (start.data.equals(x)) { </code></pre> could be written as <pre><code>if (start == null) return; if (start.data.equals(x)) { </code></pre> by omitting the <code>else</code> it becomes a bit easier to read, even though it will behave the same. <hr> This is what I would write: <pre><code>public void eraseNumber(Object x) { while (start != null && start.data.equals(x) { start = start.next; count--; } Node lastNode = start; Node currentNode = start; while (currentNode != null) { if (currentNode.data.equals(x)) { lastNode.next = currentNode.next; count--; } else { lastNode = currentNode; } currentNode = currentNode.next; } } </code></pre> You may want to substitute <code>Object</code> with something more specific. Note: I had to fix a bug in my implementation missing the first x in the linked list.
Tags
Title
singulars
PostAcceptedAnswerId
1. This table or related slice is empty.
PostParentId
1. POUsing Linked List to remove items
 singulars
 PostTypePostTypeId
 PTQuestion
PostTypePostTypeId
1. PTAnswer
UserLastEditorUserId
1. USTheMorph
UserOwnerUserId
1. USTheMorph
plurals
PostLinksPostIdRelatedPostId
1. This table or related slice is empty.
PostLinksRelatedPostIdPostId
1. This table or related slice is empty.
PostsAcceptedAnswerId
1. This table or related slice is empty.
PostsParentIdCreationDate
1. This table or related slice is empty.
VotesPostIdCreationDate
1. This table or related slice is empty.
CommentsPostId
1. This table or related slice is empty.

Querying!

Guidance

A row detail

Detail views are divided into sections. All the information in the data section comes from columns in the selected row. The other sections display data from other, related rows.

Related data can be related in a to-one or a to-many fashion. Captions of data related in a to-many fashion link to a list view showing a filtered view of the table.

Try moving around until you find a non-empty to-many entry and click on the label to get to one. You can move back to the root by clicking on the database name in the header.