-1

I am currently coding a function, that has to update the value of a boolean.
For some reason though, it does not do this. Let me show you a simplified version:

main.cpp

#include <iostream>
#include "header.h" 


int main(){

bool limit = true;

test(limit);

if (limit == false){cout << "hi";}

}

header.h

void test(bool x);

header.cpp

#include <iostream>
#include "header.h"

void test(bool x){

x = false;
}

I define the bool "limit" in my main function. Then I pass it to the function "test" that is located in header.cpp. In this function, the bool limit is set to false. However, returning to main.cpp the bool is still true. This I see, because the if condition is not called. I suspect that the error has to do with the information being passed between the files. Do you know the real reason and a possible alternative?

MegAmaNeo1
  • 15
  • 6
  • 2
    Check up on "pass by value" and "pass by reference". It's a *very* basic tenet of C++ programming. – Adrian Mole Jul 13 '21 at 20:01
  • 2
    Vital reading: [What's the difference between passing by reference vs. passing by value?](https://stackoverflow.com/questions/373419/whats-the-difference-between-passing-by-reference-vs-passing-by-value) – user4581301 Jul 13 '21 at 20:06

2 Answers2

2

When you declare the function test as

void test(bool x);

the parameter bool x is passed by value. That is, the computer makes a copy of the variable that you pass to test. So when test updates such a variable, you are just modifying a temporary variable, with no effect on the bool limit that you pass to it from main.

If you intend to update x you should pass a reference and declare test as

void test(bool& x);
user4581301
  • 33,082
  • 7
  • 33
  • 54
francesco
  • 7,189
  • 7
  • 22
  • 49
1

In test, x is a copy of the value passed into it. Use a reference instead:

void test(bool &x)
{
    x=false;
}
jkb
  • 2,376
  • 1
  • 9
  • 12