Thread: I dont understand how this can bugged

If you don't mind everyone seeing the whole code (it's not propretary or anything), then I would suggest.

- delete the contents of Project\Debug\* and Project\Release\*
- Create a zip file of Project
- Rename it to say project.zip.txt (so the board will accept it)
- Upload as an attachment.

This doesn't even COMPILE, let alone run

Code:

gcc -ggdb3 -Wall -pedantic --std=c99 AlgorithmHandler.c GeneticAlgorithm.c Helpers.c InputHandler.c SimAnAlgorithm.c main.c -o proj
AlgorithmHandler.c: In function ‘AlgorithmHandler_verifyComparator’:
AlgorithmHandler.c:70: warning: control reaches end of non-void function
AlgorithmHandler.c: In function ‘AlgorithmHandler_genCritter’:
AlgorithmHandler.c:87: error: too many arguments to function ‘AlgorithmHandler_genNucleotide’
AlgorithmHandler.c: At top level:
AlgorithmHandler.c:103: error: conflicting types for ‘AlgorithmHandler_genNucleotide’
AlgorithmHandler.h:90: error: previous declaration of ‘AlgorithmHandler_genNucleotide’ was here
AlgorithmHandler.c: In function ‘AlgorithmHandler_setFittnes’:
AlgorithmHandler.c:376: warning: unused variable ‘f’
AlgorithmHandler.c:375: warning: unused variable ‘j’
AlgorithmHandler.c: In function ‘AlgorithmHandler_doMutationNucleotide’:
AlgorithmHandler.c:590: warning: ‘return’ with a value, in function returning void
AlgorithmHandler.c:531: warning: unused variable ‘j’
AlgorithmHandler.c:531: warning: unused variable ‘i’
GeneticAlgorithm.c: In function ‘GeneticAlgorithm_defaultPool’:
GeneticAlgorithm.c:32: warning: unused variable ‘genome’
GeneticAlgorithm.c: In function ‘GeneticAlgorithm_doCrossover’:
GeneticAlgorithm.c:101: warning: unused variable ‘tmp’
GeneticAlgorithm.c: In function ‘GeneticAlgorithm_selectNextGeneration’:
GeneticAlgorithm.c:228: warning: unused variable ‘j’
GeneticAlgorithm.c: In function ‘GeneticAlgorithm_printPool’:
GeneticAlgorithm.c:276: warning: passing argument 1 of ‘AlgorithmHandler_CritterToString’ from incompatible pointer type
main.c: In function ‘runProgram’:
main.c:50: warning: unused variable ‘InvariantRows’
main.c: In function ‘test’:
main.c:131: warning: unused variable ‘fitt’

show us the include file definition of the function.

and how are you determining the arguments have random values? with the debugger or printf's?

one approach is to start by commenting all code out of the function and just printing the arguments. if they are wrong at that point then the bug is in the caller. if the arguments are ok then something in the function itself is corrupting the arguments, probably via the stack. add code back little by little until the error occurs to at least find what is causing the problem.

Well it seems to me that you freed something, then tried to make use of it.

Code:

#0  0x00402f94 in AlgorithmHandler_copyCritter (to=0x3960e0, from=0x395878) at C:\Users\sc\Downloads\project AD3\project AD3\project AD3\AlgorithmHandler.c:154
#1  0x00402458 in SimAnAlgorithm_genNeighbour (settings=0x393378, parent=0x395878) at C:\Users\sc\Downloads\project AD3\project AD3\project AD3\SimAnAlgorithm.c:49
#2  0x00402342 in SimAnAlgorithm_run (settings=0x393378, maxRuns=1000, fittnessLimit=18, restartPropability=5) at C:\Users\sc\Downloads\project AD3\project AD3\project AD3\SimAnAlgorithm.c:16
#3  0x004026d6 in runProgram () at C:\Users\sc\Downloads\project AD3\project AD3\project AD3\main.c:113
#4  0x00402514 in main () at C:\Users\sc\Downloads\project AD3\project AD3\project AD3\main.c:31
149	
150		// allocate memory for genome
151		to->genome=(struct AlgorithmHandler_Genome * )malloc(sizeof(struct AlgorithmHandler_Genome));
152	
153		// copy genome attributes
154		to->genome->length=from->genome->length;
155	
156		// allocate memory for root nucleotide
157		to->genome->root=(struct AlgorithmHandler_Nucleotide *)malloc(sizeof(struct AlgorithmHandler_Nucleotide));
158	
$3 = (struct AlgorithmHandler_Critter *) 0x395878
$4 = {
  age = -1426050767, 
  fittness = -1.21979282e-012, 
  diff = -1.14094669e+038, 
  genome = 0xfeeefeee
}

The last output is from "print *from"
Notice the value in genome - this is supposed to be a pointer.
The feeefeee in this case indicates that this block of memory had been freed.

@Salem:
I trimmed the source so thet I don't call any function that frees any memory before trying to call SimAnAlgorithm_run() & it didn't solve the problem.


@dmh2000:
SimAnAlgorithm.c has this as first line:

Code:

#include "SimAnAlgorithm.h"

SimAnAlgorithm.h contents:

Code:

#ifndef SIMANALGORITHM_DEFINED
#define SIMANALGORITHM_DEFINED

#include "AlgorithmHandler.h"


struct AlgorithmHandler_Critter * SimAnAlgorithm_run(
											struct AlgorithmHandler_Settings * settings, 
											int maxRuns, 
											float fittnessLimit, 
											int restartPropability);
struct AlgorithmHandler_Critter * SimAnAlgorithm_genNeighbour(
											struct AlgorithmHandler_Settings * settings,
											struct AlgorithmHandler_Critter * parent);

int SimAnAlgorithm_getTransitionPropability(float eC, 
											float eBuur, 
											float temp);

float SimAnAlgorithm_getTemp(				float progress);
float SimAnAlgorithm_getE(					float fittness);

#endif

how are you determining the arguments have random values? with the debugger or printf's?

I use the debugger.

@rags_to_riches:
I compile with the build in VS 2010 compiler







EDIT:
I've been trying to compile with gcc -std=c99 -lm. Seems like VS 2010 was missing some bugs. After working out most obvious ones, I'm left with this:

Code:

main.o: In function `runProgram':
main.c:(.text+0x225): undefined reference to `SimAnAlgorithm_run'
collect2: ld returned 1 exit status
make: *** [test] Error 1

I guess something must be amiss with my function definition. I've been trying to find out what, but no success so far.

EDIT 2:
I've uploaded the updated project source:

Code:

http://www.opaalkust.com/Download/project%20AD3.rar