C linked list with an array inside

  • Thread starter baddin
  • Start date
  • #1
24
0

Main Question or Discussion Point

I have pasted my code here : http://pastebin.com/nMNarH9i
It is in language C.
Basically, I am reading in 5 numbers at a time.
Putting these numbers in array one_vote_array.
Then I need to put this array in a structure vote_node.
I also then change my next pointer in the structure to head and then make head point to this new structure.
I want to to keep on making structures while I keep reading in 5 numbers.
The problem is my program is only printing the last number i inputted in it.
 

Answers and Replies

  • #2
11,667
5,239
Clearly a case of voter fraud.

so the way your program is written it will print the voting backward right? last voter to first voter.

but in your case it prints the last voter and stops so that implies that the temp->next is null.

Id go back to the first for loop and use your print test to see what voter->node is beiing set to as you iterate thru your voters.

Do you have a debugger that you can run your program thru?

it would allow you to review the values of variables step by step.
 
  • #3
D H
Staff Emeritus
Science Advisor
Insights Author
15,393
683
You are reusing the same array of five integers for each node. You need to allocate a new array of five integers for each node.
 
  • Like
Likes 1 person
  • #4
11,667
5,239
You are reusing the same array of five integers for each node. You need to allocate a new array of five integers for each node.
Good catch, but I thought we were supposed to lead the OP to the answer.
 
  • #5
24
0
But my function read_one_vote changes the contents of the array one_vote_array everytime it is called, so shouldn't that array be different for each node?
 
  • #6
D H
Staff Emeritus
Science Advisor
Insights Author
15,393
683
How can it be different for each node? Your code makes exactly the opposite be true. It's the same array for each node.

If you need a different array for each node, you need to make a different array for each node.
 
  • Like
Likes 1 person
  • #7
24
0
Ok, I did what you said, and it works now. Thank you very much.
 
  • #8
24
0
I still am finding it confusing though, can't I use the same array with different value and set it to each node?
 
  • #9
24
0
Because the values of one_vote_array change each time the function read_one_vote gets called
 
  • #10
184
3
I still am finding it confusing though, can't I use the same array with different value and set it to each node?
Think about it at the hardware level: If you need to keep track of say, 9 different arrays, how many arrays would you need to allocate in memory? Each block of memory can only hold one value at a time, so in order for your computer to "remember" what the content of each array is, it needs to allocate a separate memory block for each one.

What your asking is possible though. If you want, you can have a buffer in which you construct each array you want. Then when it is ready, you can copy it to another newly allocated array, and reset the original array. However, this still requires you to allocate a new array each time.
 

Related Threads on C linked list with an array inside

  • Last Post
Replies
4
Views
2K
  • Last Post
Replies
6
Views
2K
Replies
6
Views
6K
  • Last Post
Replies
3
Views
2K
  • Last Post
Replies
12
Views
2K
  • Last Post
Replies
9
Views
4K
Replies
3
Views
591
  • Last Post
Replies
1
Views
3K
  • Last Post
Replies
1
Views
2K
  • Last Post
Replies
20
Views
3K
Top