|
How to show only unique elements from this double linked list using UniqueList function? I made something, but it removes only the first...
<pre>
#include "iostream"
using namespace std;
void Add(int n);
void UniqueList();
struct Elem
{
int key;
Elem *prev;
Elem *next;
} *start;
int main()
{
int num;
while(cin >> num)
{
Add(num);
}
cout << endl;
UniqueList();
return 0;
}
void Add(int n)
{
Elem *p = start;
start = new Elem;
start->key = n;
start->next = p;
start->prev = NULL;
}
void UniqueList()
{
Elem *p = start;
if(NULL != start) cout << "The list is:" << endl;
else cout << "Empty list.";
while(NULL != p)
{ if(p->prev != p->next)
cout << p->key << endl;
p = p->next;
}
}
</pre>
|
|
|
|
|
one problem is that you're never setting the 'prev' members of the added items to anything but NULL. if you have a double-linked list, you need to do something like this:
void Add(int n)
{
Elem *p = start;
start = new Elem;
start->key = n;
start->next = p;
start->prev = NULL;
p->prev=start;
}
but that will probably blow up the first time through, since start itself hasn't been initialized, and *start is never allocated.
|
|
|
|
|
and what about the other main problem with the duplicates?
|
|
|
|
|
i'm going to guess that that's probably a side effect of:
a) not setting prev to anything but NULL and
b) not initializing 'start' the first time around.
and, maybe i don't understand the task, but why are you doing this:
if(p->prev != p->next)
why compare prev and next pointers ? and why compare prev and next and not p itself:
if(p->key != p->next->key)
|
|
|
|
|
I fixed the prev problem. "if(p->prev != p->next)" is a shot in the dark. Any ideas how to show only uniques 
|
|
|
|
|
you're on the right track. but the key thing that's missing is that the list needs to be sorted before you can be guaranteed that duplicates will show up in p and p->next.
duplicates, but p, p->next will never be the same: 1,4,3,5,3
this will work: 1,3,3,4,5
|
|
|
|
|
it's getting messy. How to sort a list??? (Doubly linked)
|
|
|
|
|
You do not need to actually sort the list, since that would mean you first compare a lot of them and move around some nodes, then remove some (the duplicates). That is too much effort for what is needed.
What you need to do is compare a lot and remove, but never move. In pseudo-code (always solve an algorithmic problem in pseudo-code first):
foreach(Node n1) {
foreach (Node n2 before n1) {
if (n2.value==n1.value) {
remove(n1);
break;
}
}
}
PS: and that could be done on a single-linked list as well, as both loops are enumerating from the start and never need to go backward.
Luc Pattyn [Forum Guidelines] [My Articles] Nil Volentibus Arduum
Please use <PRE> tags for code snippets, they preserve indentation, improve readability, and make me actually look at the code.
|
|
|
|
|
i did not understand anything you said. And whats that code C++?
|
|
|
|
|
if you don't want to sort:
for each node, Cur
for each node, Temp
if Temp!=Cur
if Temp.value==Cur.value, duplicate
Temp=Temp.next
Cur=Cur.next
basically, for each node, you compare its value to every other node. it's inefficient, but it will work.
you can also sort your list when you add:
1. walk the list, find the first node with a value greater than the new value. this is 'Cur'
2. change the next pointer of Cur->prev to point to your new node.
3. set your new node's prev to Cur->prev. now you're all linked in the prev direction.
4. change the prev pointer of Cur to point to your new node.
5. set your new node's cur to point to Cur. now you're all linked in the next direction.
that way, the list is always sorted, and you can just walk it, looking for duplicates.
|
|
|
|
|
OK, I decided to sort the list, but can you tell me why is not working well?
|
|
|
|
|
TCPMem wrote: How to sort a list???
Insert the items in their "sorted" position within the list.
"One man's wage rise is another man's price increase." - Harold Wilson
"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons
"Some people are making such thorough preparation for rainy days that they aren't enjoying today's sunshine." - William Feather
|
|
|
|
|
for(int i = 1; i < N; i++)
for(int j = N - 1; j >= i; j--)
if(p->key > p->next->key)
{
t = p->key;
p->key = p->next->key;
p->next->key = t;
}
Here is the sort but it doesn' t work
|
|
|
|
|
first problem: you're never moving p
|
|
|
|
|
|
p = p->next;
p = p->prev; (forgot you're inserting at the beginning, not at the end of list)
if (p==NULL) bail.
|
|
|
|
|
OK, here is the full code that I' m working on. Can you tell me my mistakes exactly because I can' t understand where to put what. The UniqueList function is important, which now only shows the list and nothing more. The sort does not work...
#include "iostream"
using namespace std;
void Add(int n);
void UniqueList(int N);
struct Elem
{
int key;
Elem *prev;
Elem *next;
} *start;
int main()
{
int num;
int N = 0;
start = NULL;
while(cin >> num)
{
Add(num);
N++;
}
cout << endl;
UniqueList(N);
return 0;
}
void Add(int n)
{
Elem *p = start;
start = new Elem;
start->key = n;
start->next = p;
start->prev = NULL;
if(NULL != p) p->prev = start;
}
void UniqueList(int N)
{
Elem *p = start;
int t;
if(NULL != start) cout << "List:" << endl;
else cout << "Empty list!";
for(int i = 1; i < N; i++)
for(int j = N - 1; j >= i; j--)
{
if(p->key > p->next->key)
{
t = p->key;
p->key = p->next->key;
p->next->key = t;
}
}
while(NULL != p)
{
cout << p->key << endl;
p = p->next;
}
}
|
|
|
|
|
|
maybe it's best to rethink this from the beginning.
if your goal is to have a list with no duplicates, the easiest thing to do is to simply not add duplicates.
so, every time, you go to add a new Elem, walk through your list and see if you already have an Elem with that key. if you do, don't add anything.
to do that, you need a function that can walk through the list and look for an Elem with a given key:
bool haveKey(Elem *p, int k)
{
while (p!=NULL)
{
if (p->key=k) return true;
p = p->next;
}
return false;
}
just call that at the top of your Add. if it returns false, don't add anything.
|
|
|
|
|
No, the assignment is to add duplicates, but when show the list to show it without them. I think the sort thing will do the job but can' t do it properly.
modified on Monday, April 18, 2011 4:57 PM
|
|
|
|
|
ok, then i would recommend sorting the nodes on input. it is far more efficient than sorting the whole list, and it's actually fairly straightforward. (i outlined it above)
unfortunately, i have to leave for the day. so.. good luck!
|
|
|
|
|
I posted a question on the quick questions and answers board and I got some helpful advice. Here's the link: Where-to-look-for-my-project-goal[^]
This will give you the basics of my goal. Any way I created a blog to chronicle my progress toward that goal and would like some input from time to time on my blog to help me along in the learning process. Please bear in mind that I am a novice and am completely self taught through trial an error with mostly error. Now I have decided to stop dabbling and blundering through trying to learn programming and actually learn something useful.
What I need at this point are helpful comments and nudges that will point me in the right direction or let me know if I am getting it. My project is very ambitious to say the least and is probably impossibly high but is my motivation to learn.
Ok enough babbling here's the link to my blog: thegreatproject.wordpress.com[^]
|
|
|
|
|
Hi All,
I am tring to write an application that open a web page and Copy its data to the clipboard.
I know i can use function to httpRead the page into a buffer, but in this case i have all the html's tags
and i do not need them, if i use copy (CTRL+C) i am able to only get the text.
I thought to open an IE process with the url as parameter and then use the SendMessage API with the process handle and the CTRL+A as the message and then CTRL+C as the message (hopefully it will generate a Select All and Copy).
If this works would it be able to work when the user is logout (no visible UI).
Love to hear your thought about it, i am working with C++ or C# on windows.
Thanks a lot for your time,
Ram.
modified on Wednesday, April 20, 2011 2:11 AM
|
|
|
|
|
Ram Shmider wrote: and i do not need them, if i use copy (CTRL+C) i am able to only get the text.
Can you parse the HTML content using some class for ex: HTML Reader C++ Class Library[^]
You talk about Being HUMAN. I have it in my name
AnsHUMAN
|
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
