Thread: I got a bug in my program

Here is the link to my github:
https://github.com/jeremyduncanartif...ence-software/

The bug is in the the function thefunctionpartone line 258, it goes to a class called CountingTheFourDirections and a function called direction_value__which_direction_is_lowest__direct ion_value_array_transfer_value_to_new_array__displ aying_which_direction_is_lowest

From there I don't know how to fix it. I have looked at it and decided to come here for some help.

Here is the error:

Code:

Feeler 1 the lowest value is the west direction, so west is blocked.
Array element zero is: 0 value


Values with feeler 1
I tried to take 9 steps in the north direction, but I actual took 8 steps.
I tried to take 9 steps in the south direction, but I actual took 6 steps.
I tried to take 9 steps in the east direction, but I actual took 6 steps.
I tried to take 9 steps in the west direction, but I actual took 5 steps.

step 1.) change feeler in that direction.
step 2.) feel again in that direction
note: the line A5 to A4


Feeler 2 the lowest value is the west direction, so west is blocked.
Array element zero is: 0 value


Values with feeler 2
I tried to take 9 steps in the north direction, but I actual took 6 steps.
I tried to take 9 steps in the south direction, but I actual took 5 steps.
I tried to take 9 steps in the east direction, but I actual took 7 steps.
I tried to take 9 steps in the west direction, but I actual took 4 steps.




The second feeler did something to unblock feeler 1

Step 1 ended = make feeler 1 and 2 unequal - gain control.
note: line A5 to A3

<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<

Feeler 2 the lowest value is the south direction, so south is blocked.
Array element zero is: 0 value

Feeler 2 the lowest value is the south direction, so south is blocked.
Array element zero is: 0 value

Feeler 2 the lowest value is the east direction, so east is blocked.
Array element zero is: 1 value

Feeler 2 the lowest value is the south direction, so south is blocked.
Array element zero is: 0 value

Feeler 2 the lowest value is the north, south direction, so south is blocked.
Array element zero is: 1 value

Feeler 2 the lowest value is the west direction, so west is blocked.
Array element zero is: 1 value

Feeler 2 the lowest value is the south direction, so south is blocked.
Array element zero is: 1 value

Feeler 2 the lowest value is the south direction, so south is blocked.
Array element zero is: 1 value

Feeler 2 the lowest value is the east direction, so east is blocked.
Array element zero is: 1 value

Feeler 2 the lowest value is the west direction, so west is blocked.
Array element zero is: 0 value

Values with feeler 2
I tried to take 9 steps in the north direction, but I actual took 6 steps.
I tried to take 9 steps in the south direction, but I actual took 6 steps.
I tried to take 9 steps in the east direction, but I actual took 7 steps.
I tried to take 9 steps in the west direction, but I actual took 4 steps.

Step 2 ended = make feeler 1 and 2 equal - lose control.
note: from A3, line A4 is closed.
A3 went to A4 9 times before going to A6

<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<

Feeler 3 the lowest value is the west direction, so west is blocked.
Array element zero is: 0 value

Feeler 3 the lowest value is the north, east direction, so north is blocked.
Array element zero is: 0 value

Values with feeler 3
I tried to take 9 steps in the north direction, but I actual took 4 steps.
I tried to take 9 steps in the south direction, but I actual took 6 steps.
I tried to take 9 steps in the east direction, but I actual took 4 steps.
I tried to take 9 steps in the west direction, but I actual took 5 steps.

Step 3 ended = make feeler 1 and 2 unequal - regain control.
note: line A3 to line A6

<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<




Process returned 0 (0x0)   execution time : 0.281 s
Press any key to continue.

west is blocked for the top part, but they should be different directions.

Run the program a few times to get the error it doesn't happen every time. And no I have not debugged the program because I'm not very good at that and thought it would be easier to ask you for help instead.

To say that you want someone else to debug your program for you because you find it difficult is not going to get you very far. You are in the best position to debug it since you know how it works. Try printing out the values of certain variables at strategic locations and see if they have the values you expect.

The bug was I was getting a value and it couldn't change, so if it gave me the wrong value it couldn't change it. So I made a while loop and gave it a condition to change it if its the wrong value. Now the program works fine.

Here is the new code:

Code:

while(1)
    {
        north_count_2 = directions_value.testing_four_directions(north_array_2, left_or_right, 4);
        south_count_2 = directions_value.testing_four_directions(south_array_2, left_or_right, 5);
        east_count_2 = directions_value.testing_four_directions(east_array_2, left_or_right, 6);
        west_count_2 = directions_value.testing_four_directions(west_array_2, left_or_right, 7);

        cout << '\n';

        feeler = 2;
        element = 1;

        directions_value.direction_value__which_direction_is_lowest__direction_value_array_transfer_value_to_new_array__displaying_which_direction_is_lowest(
        lowest_value_feeler_2_try_1,
        lowest_value_array_feeler_2_try_1,
        north_array_2,
        south_array_2,
        east_array_2,
        west_array_2,
        &feeler_2_lowest_value_array_direction,
        &north_count_2,
        &south_count_2,
        &east_count_2,
        &west_count_2,
        &feeler,
        &element,
        &direction_value );
        if(feeler_2_lowest_value_array_direction != *feeler_1_lowest_value_array_direction)
        {
            break;
        }
    }

It's in the thefunctionpartone class. That other class had a different bug I found. I tried to use the codeblocks debugger by putting the red dot by that function in thefunctionpartone and then pressing step into, it gave me some idea on where to look for the eventual fix when I saw what it was doing step by step.

I got lazy to try the debugger, but now I see it's not that difficult I won't be so spooked by it the next time.

Thanks for the advice you 2, problem solved.

Originally Posted by laserlight

direction_value__which_direction_is_lowest__direct ion_value_array_transfer_value_to_new_array__displ aying_which_direction_is_lowest?

I thought I write long function/variable names until I came across this one!