Problems with Linked list in C++

Question

I'm completely new to C++ and am breaking my head over a simple problem. I am trying to implement a simple linked list with three nodes. Here is my code:

#include<iostream> using namespace std; struct node(){ int data; struct node* next; }; struct node* BuildOneTwoThree() { struct node* head = NULL; struct node* second = NULL; struct node* third = NULL; head = new node; second = new node; third = new node; head->data = 1; head->next = second; second->data = 2; second->next = third; third->data = 3; third->next = NULL; return head; };

The question obviously being, why does it not compile? :(

Thank you in advance for any help!

If you're "completely new", then please expunge abusing namespace std; from your mind right now while you still have a chance, and never use it again. — Kerrek SB
– Kerrek SB, Commented Feb 9, 2012 at 19:10
Here is a copy of the error message(s): chopapp.com/#fq7vcb86 — Andrej Žukov-Gregorič
– Andrej Žukov-Gregorič, Commented Feb 9, 2012 at 19:10
@user1200428: I don't think that's the first error message you got. — Mooing Duck
– Mooing Duck, Commented Feb 9, 2012 at 19:17
If you can, use the standard library's std::list instead of creating your own. — Zyx 2000
– Zyx 2000, Commented Feb 9, 2012 at 19:19

cli_hlt · Accepted Answer · 2012-02-09 19:10:12Z

6

Remove the "()" from the struct declarations. Your compiler should have told you that.

answered Feb 9, 2012 at 19:10

cli_hlt

7,1842 gold badges29 silver badges23 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

cli_hlt Over a year ago

Oh well yes - glad to have helped, and please take Kerrek SB's advice seriously and remove the "using namespace std" while you still have the chance ;-)

Luchian Grigore · Accepted Answer · 2012-02-09 19:10:06Z

Replace

struct node(){ int data; struct node* next; };

with

struct node{ int data; struct node* next; };

The extra paranthesis after the struct declaration are causing the error. The latter is the proper way of declaring a struct or a class in C++.

Dietmar Kühl · Accepted Answer · 2012-02-09 19:10:34Z

The declaration of your struct has an excess pair of parenthesis. When you remove them this this should be OK:

struct node{ int data; struct node* next; };

Anthony · Accepted Answer · 2012-07-26 02:00:17Z

0

() should be written after functions instead of struct/class name.

edited Jul 26, 2012 at 2:00

Anthony

12.5k10 gold badges72 silver badges107 bronze badges

answered Feb 9, 2012 at 19:32

T.C.ai

411 gold badge1 silver badge3 bronze badges

Collectives™ on Stack Overflow

Problems with Linked list in C++

4 Answers 4

1 Comment

Comments

Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

Comments

Comments

Related