Where's the mistake???

[Nikola]

compiler says: function undeclared how come???
help!
#include<stdio. h>
#include<stdlib .h>
#include<time.h >

struct lista{
int element;
struct lista *next;
}*pocetak;

main()
{
struct lista *q, *nova;
int i;
pocetak=NULL;
srand(time(NULL ));
for(i=0;i<5;i++ )
{
q=(struct lista*) mallloc(sizeof( struct lista));
q->element=rand() %100;
pocetak=q;
q->next=NULL;
}
q=pocetak;
printf("Nasumic ni brojevi:\n");
while(q!=NULL)
{
printf("\n%d",q->element);
q=q->next;
}
printf("\nNaopa ko:\n");

return 0;
}

[Nikola]

Heh, been there, done that several times myself, unfortunately.

I don't see it ... :-(

[Nikola]

Oops, I guess my finger kinda stuck ...
sorry for that ...

all well ...

[Nikola]

.... and thanks

[Dario (drinking coffee in the office…)]

One of your mistakes is in the following line:

nova=(struct lista*) mallloc(sizeof( struct lista));

- Dario (the compiler and/or linker already told that?)

[Nikola]

I need to write a program that generates 5 random numbers and puts them into
a linked list. (Print the list) From that list it forms another list in a
way if the arrangement of elements in the first one was 5 8 3 1 in the new
one it should be
1 3 8 5 (print the new list)

where's the error now ??

#include<stdio. h>
#include<stdlib .h>
#include<time.h >

struct lista{
int element;
struct lista *next;
}*pocetak;

main()
{
struct lista *q, *nova;
int i;
pocetak=NULL;
srand(time(NULL ));
for(i=0;i<5;i++ )
{
q=(struct lista*) malloc(sizeof(s truct lista));
q->element=rand() %100;
pocetak=q;
q->next=NULL;
}
q=pocetak;
printf("Nasumic ni brojevi:\n");
while(q!=NULL)
{
printf("\n%d",q->element);
q=q->next;
}

q=pocetak;
for(i=0;i<5;i++ )
{
nova=(struct lista*) malloc(sizeof(s truct lista));
nova->element=poceta k+(5-i)*(sizeof(stru ct lista));
pocetak=nova;
nova->next=NULL;
}
nova=pocetak;
printf("\nNaopa ko:\n");
while(nova!=NUL L)
{
printf("\n%d",n ova->element);
nova=nova->next;
}
system("pause") ;
return 0;
}

[Martin Ambuhl]

Nikola wrote:

compiler says: function undeclared how come??? q=(struct lista*) mallloc(sizeof( struct lista));

^^^^^^^
spelling error

[Eric Sosman]

Nikola wrote:

I need to write a program that generates 5 random numbers and puts them into
a linked list. (Print the list) From that list it forms another list in a
way if the arrangement of elements in the first one was 5 8 3 1 in the new
one it should be
1 3 8 5 (print the new list)

where's the error now ??
[...]
q=(struct lista*) malloc(sizeof(s truct lista));
q->element=rand() %100;
pocetak=q;
q->next=NULL;

Here.

An excellent way to teach yourself how to manipulate
links in data structures is to draw "before and after"
diagrams of the operation you want to perform. Study the
diagrams to see which links must change, and to what.

--
Er*********@sun .com

[Nikola]

> > q=(struct lista*) malloc(sizeof(s truct lista));

q->element=rand() %100;
pocetak=q;
q->next=NULL;

Here.

ok I rewrote the code, no mistakes but it still doesn't work ...
#include<stdio. h>
#include<stdlib .h>
#include<time.h >

struct lista{
int element;
struct lista *next;
}*pocetak1,*poc etak2;

main()
{
struct lista *q, *nova;
int i;
pocetak1=NULL;
srand(time(NULL ));
for(i=0;i<5;i++ )
{
q=(struct lista*) malloc(sizeof(s truct lista));
q->element=rand() %100;
q->next=pocetak 1;
pocetak1=q;
}
q=pocetak1;
printf("Random numbers:\n");
while(q!=NULL)
{
printf("\n%d",q->element);
q=q->next;
}
printf("\nOther way round:\n");
q=pocetak1;
for(i=0;i<5;i++ )
{
nova=(struct lista*) malloc(sizeof(s truct lista));
nova->element=(q+( 5-i)*sizeof(struc t lista))->element;
q=q->next;
nova=nova->next;
nova->next=pocetak 2;
pocetak2=nova;
}
nova=pocetak2;
while(nova!=NUL L)
{
printf("\n%d",n ova->element);
nova=nova->next;
}
system("pause") ;
return 0;
}

[Default User]

Nikola wrote:

I took a wild shot guessing dtat you have a compiler :-)

For the most part, I don't go and run every piece of code posted here.
Most often, when the person asking the question gives us complete
information, the problem can be solved by inspection.

Had you pointed to the correct error line, we could have pointed out
that you mispelled the function name. No need to run it.


Brian Rodenborn

[Jens.Toerring]

Nikola <az*****@inet.h r> wrote:

ok I rewrote the code, no mistakes but it still doesn't work ... #include<stdio. h>
#include<stdlib .h>
#include<time.h > struct lista{
int element;
struct lista *next;
}*pocetak1,*poc etak2;
There doesn't seem to be any good reason to make these variables
globals, so why don't you define them within main()?
main()
Better make that

int main()

or

int main(void)

in order to be prepared for C99-compliant compilers that don't
allow functions without an explicit return type anymore.
{
struct lista *q, *nova;
int i;
pocetak1=NULL;
srand(time(NULL ));
for(i=0;i<5;i++ )
{
q=(struct lista*) malloc(sizeof(s truct lista));
There isn't much of a good reason (well a few of the regulars here
disagree but...) for casting the return value of malloc()...
q->element=rand() %100;
I hope you're aware that this isn't a good way to restrict the range
of the random generator - the lower bits it returns usually have less
"randomness ", so its preferable to use something like

q->element = ( int ) ( 100.0 * rand( ) / ( RAND_MAX + 1.0 ) );
q->next=pocetak 1;
pocetak1=q;
}
q=pocetak1;
printf("Random numbers:\n");
while(q!=NULL)
{
printf("\n%d",q->element);
q=q->next;
}
printf("\nOther way round:\n");
q=pocetak1;
for(i=0;i<5;i++ )
{
nova=(struct lista*) malloc(sizeof(s truct lista));
nova->element=(q+( 5-i)*sizeof(struc t lista))->element;

You seem to be assuming that the elements of the first linked list
somehow are located in memory in a densely packed fashion, one after
another, with the last allocated list element being the one at the
highest memory address. That's definitely not a valid assumption
and you will probably end up with pointing to memory you don't even
own. (And even if your assumption would be correct you would need
"(4 - i)" instead of "(5 - i)"...) A linked list is _not_ an array
and malloc() is perfectly within its rights when it gives you some
memory at a lower address, possibly several megabytes away, than
it returned in the previous call.
Regards, Jens
--
\ Jens Thoms Toerring ___ Je***********@p hysik.fu-berlin.de
\______________ ____________ http://www.toerring.de