Strange output

[unique]

I have written some programs in c lang yet but today I get confused
with output i get.
I have function educate(..) which i call in main program this way:

J = educate(no_laye rs, no_neurons, input, output, lay, weights, 0.1,0);
printf("J= %f\n",J);

In the educate function i count J and i return it (but before I return
it I output it with printf):

float educate (int no_layers, int no_neurons[], float input[], float
output[], float* lay[], float* weights[], float gama, int debug) {
....
printf("J= %f\n",J);
return(J);
}

This is what i get:

J= 0.304447
J= 1050402944.0000 00

Is there any syntax prob? Why the values arent the same? :(

[Barry Schwarz]

On 9 Oct 2005 06:39:15 -0700, "unique" <ju*****@gmail. com> wrote:

So the compiler must assume a default of educate() returning an int,

which of course it doesn't.

Thank you very much Skarmander, you are completely right, i solved it
with header file.

In addition to solving it this time, you should up the warning level
of your compiler so that it tells anytime you invoke a function for
which a prototype is not in scope.
<<Remove the del for email>>

[Emmanuel Delahaye]

unique a écrit :

I was investigating the problem a bit and the problem seems to be that
I call in file educate.c function educate(...) from the other file
neuron.c. I dont use any include, so code is here:
***test1.c***
float educate() {
float a=2.54f;
printf("a= %f", a);
return a;
}
***test2.c***
int main(int argc, char **argv) {
float b=educate();
printf("b= %f", b);
}

After compiling it with: cc test1.c test2.c -lm -o test2 and running it
the output is: a= 2.540000b= 1076006784.0000 00

This works fine to me (single compile unit)

#include <stdio.h>

float educate(void)
{
float a=2.54f;
printf("a= %f\n", a);
return a;
}

int main(void)
{
float b=educate();
printf("b= %f\n", b);
}

Now, if you are gooing to use 2 separated compile units (CUs), you must
define a protoype in a common file called a header. This is the proper
way of doing it:

/* educate.h */
#ifndef H_EDUCATE
#define H_EDUCATE

/* function prototypes */
float educate(void);

#endif /* guard */

Then you must include this header in the definition CU (for consistency)
and in all user CU:

/* educate.c */
#include <stdio.h>
#include "educate.h"
float educate(void)
{
float a=2.54f;
printf("a= %f\n", a);
return a;
}

/* main.c */
#include <stdio.h>
#include "educate.h"

int main(void)
{
float b=educate();
printf("b= %f\n", b);
return 0;
}

[Emmanuel Delahaye]

unique a écrit :

I was investigating the problem a bit and the problem seems to be that
I call in file educate.c function educate(...) from the other file
neuron.c. I dont use any include, so code is here:
***test1.c***
float educate() {
float a=2.54f;
printf("a= %f", a);
return a;
}
***test2.c***
int main(int argc, char **argv) {
float b=educate();
printf("b= %f", b);
}

After compiling it with: cc test1.c test2.c -lm -o test2 and running it
the output is: a= 2.540000b= 1076006784.0000 00

This works fine to me (single compile unit)

#include <stdio.h>

float educate(void)
{
float a=2.54f;
printf("a= %f\n", a);
return a;
}

int main(void)
{
float b=educate();
printf("b= %f\n", b);
}

Now, if you are going to use two separated compile units (CUs), you must
define a protoype in a common file called a header. This is the proper
way of doing it (the guard protects agains multiple inclusions in a
single CU):

/* educate.h */
#ifndef H_EDUCATE
#define H_EDUCATE

/* function prototypes */
float educate(void);

#endif /* guard */

Then you must include this header in the definition CU (for consistency)
and in all user CU:

/* educate.c */
#include <stdio.h>
#include "educate.h"
float educate(void)
{
float a=2.54f;
printf("a= %f\n", a);
return a;
}

/* main.c */
#include <stdio.h>
#include "educate.h"

int main(void)
{
float b=educate();
printf("b= %f\n", b);
return 0;
}