Passing a value from member function to non member function

[SSDEEZ]

I'm trying to write a program that manipulates the bits of integers in a vector.

Down below you can see the GetSet() function and the ToBinary() function which are memeber and non member functions respectively. the return value of getset needs to be passed to tobinary so that tobinary can print out the set (which is an integer in the vector) in a binary format.

I'm confused on how i can pass the value to tobinary when it is a non member function and getset is a member function.

Any ideas would be greatly appreciated.

#include <iostream>
#include <vector>
#include <string>
#include <stdio.h>
#include <stdlib.h>
#include <cmath>

using namespace std;

//1 in the 0th bit = 1
//1 in the 1st bit = 2
                   //4
                   //8
                   //16
                   //32
                   //64
                   //128     

// << left shift
// >> right shfit
// & and
// | or 
// ^ xor

const int N_BITS = sizeof(int) * 8;

class BITSET
{
    private:
        vector <int> bits;
         
    public:
        BITSET();
        bool Test(int i) const; //testing bits
        void Set(int idx); //set to 1
        void Clear(int idx); //set to 0
        int GetNumSets() const; //get amt of sets.
        int GetSet(int idx) const; //bits.at()

};

string ToBinary(int value, int spacing); //func def down below.


void UserInterface(){
    cout << "t <index_of_bit> to test a bit" << endl;
    cout << "s <bit> to set a bit" << endl;
    cout << "c <bit> to clear a bit" << endl;
    cout << "g <set> to print the binary representation of desired set" << endl;
    cout << "n to view the number of sets in the bitset" << endl;
    cout << "q to quit :(" << endl;
}


BITSET::BITSET() { 
    
    bits.resize(1, 0);
    
    //CONSTRUCTOR
    //INITIALIZE VECTOR TO SIZE 1 WITH VALUE 0
}

int main(){
   
    BITSET set;
    int value;
    int spacing;
    char option;
    int idx;
    
    while(option != 'q'){
        cout << endl;
        cout << "Welcome to the User Interface please review our options: " << endl;
        cout << endl;
        UserInterface();
        cout << endl;
        cout << "Select your desired option: ";
    
    cin >> option;
    cout << endl;
    switch (option){
        case 't':
            set.Test(idx);
            cout << endl;
            break;
        case 'g': 
            set.GetSet(idx);
            ToBinary(value, spacing);
            break;
        case 's': 
            set.Set(idx);
            break;
        case 'n':
            set.GetNumSets();
            break;
        case 'c':
            set.Clear(idx);
            break;
        case 'f':
            set.GetSet(idx);
            break;
    }   
}
    return 0;

    
}

int BITSET::GetSet(int idx) const
{
    cout << "Enter the set you'd like to view: ";
    cin >> idx;
    return bits.at(idx);
}

void BITSET::Clear(int idx){
    int x = 1;
    cout << "Enter the index of the bit you would like to clear: ";
    cin >> idx;
    bits.at(idx / N_BITS) ^= (x << (idx - 1));
    cout << endl;
    cout << "Integer representation of the number after clearing the bit: " << bits.at(idx / N_BITS);
    cout << endl;
}
bool BITSET::Test(int idx) const{
    // &, >>

    cout << "Enter the index of the integer you'd like to test: ";
    cin >> idx;
    
    if (((bits[idx & 32]) | (idx >> 5)) > 0) {
     printf("1");
     return true;
    }   
    else {
     printf("0");
     return false;
    }

    
}

void BITSET::Set(int idx){
    // <<, |
    
    
    //getnumsets * 32 if greater than index then you're fine 
    //otherwise resize vector

    int x = 1;
    cout << "Enter the index of the bit you would like to set: ";
    cin >> idx;
    cout << endl;

    if ((GetNumSets() * N_BITS) > idx){
        bits.at(idx / N_BITS) |= (x << (idx - 1));
        cout << "Integer representation of number after the set: " << bits.at(idx / N_BITS);
        cout << endl;
    }

    else{
        bits.resize(bits.size() + (idx % N_BITS));
    }
    
}

int BITSET::GetNumSets() const {
    
    cout << bits.size();
    cout << endl;
    return bits.size();
}

string ToBinary(int value, int spacing){
    
    unsigned int mask = pow(2, (N_BITS - 1)); //Mask defined at beginning of program, thanks Woz
    spacing = 4;
    cout << endl;
    int idx = 0;
    while (mask > 0)
    {
        if((value & mask) != 0)
            cout << "1";
        else
            cout << "0";
        mask = (mask >> 1);

        idx++;

        if(idx == spacing)
        {    
            cout << " ";
            idx = 0;
        }
    }
    return "string";
}

[dutch]

I don't understand the problem. Just call the function like usual.

BTW, you should use unsigned int to hold your bits in the vector.

void ToBinary(unsigned value, unsigned spacing = 4)
{
    for (unsigned i = 0, mask = 1u << (N_BITS - 1); mask; mask >>= 1)
    {
        cout << (value & mask ? '1' : '0');
        if (++i % spacing == 0) cout << ' ';
    }
}

[SSDEEZ]

The problem is that ToBinary() can't accept values from GetSet() because it's not included in the class. I can set a bit and then GetSet() will give us the index in the vector after it's been edited. then i need to pass that index into ToBinary() so it'll print out that specific index in the vector when I call ToBinary() with said index.

[dutch]

The problem is that ToBinary() can't accept values from GetSet() because it's not included in the class.

Of course it can.
Maybe something like:

ToBinary(set.GetSet(idx));

BTW, you shouldn't pass 'spacing' in if you are just going to treat it like a local variable of the function.
Look at my rewrite above.
And don't use pow to calculate an integer power of 2. Use left-shift instead.

[SSDEEZ]

Thank you @dutch got it working