making consistency of data member in the whole programme without using static keyword

Question

0.00/5 (No votes)

See more:

In the below programme i use one boolean variable named check , which is being accessed inside main function by two objects of Tst1 and Test2 . But the value of check variable is not maintained in the programme . we can use static but i want to know some alternative way ..could anyone give me some hints on it ?
Thanks in advance .

Inside jointdeatils.h

C++

#pragma once
    
    class Jointdetails
    {
    public:
    	Jointdetails(void);
    	~Jointdetails(void);
    	bool check;
    
    
    };

Inside jointdeatils.cpp

C++

#include "Jointdetails.h"
    
    Jointdetails::Jointdetails(void)
    {
    	check = false ;
    }
    
    Jointdetails::~Jointdetails(void)
    {
    }

Inside analyzer.h

C++

#pragma once
   #include "Jointdetails.h"
   class Analyzer
   {
   public:
       Analyzer(void);
       Jointdetails* GetJointDetails();
       Jointdetails* m_ptheCTJointDetails;
       ~Analyzer(void);
   };

Inside analyzer.cpp

C++

#include "Analyzer.h"
    #include "stddef.h"
    Analyzer::Analyzer(void)
    {
    	m_ptheCTJointDetails = new Jointdetails();
    	
    }
    
    Analyzer::~Analyzer(void)
    {
    }
    Jointdetails* Analyzer::GetJointDetails()
    {
    	
    	if(m_ptheCTJointDetails) 
    		return m_ptheCTJointDetails;
    	else
    		return NULL;
    	
    
    }

Inside Test1.h

C++

#pragma once
  #include "Analyzer.h"
  class Tst1
  {
  public:
      Tst1(void);
      Analyzer *analyzer1 ;
  public:
      ~Tst1(void);
  };

Inside Test1.cpp

C++

#include "Tst1.h"
    
    Tst1::Tst1(void)
    {
    	analyzer1 = new Analyzer ;
    }
    
    Tst1::~Tst1(void)
    {
    }

Inside Test2.h

C++

#pragma once
    #include "Analyzer.h"
    class Test2
    {
    public:
    	Test2(void);
    	Analyzer *analyzer2 ;
    public:
    	~Test2(void);
    };

Inside Test2.cpp

C++

#include "Test2.h"
    
    Test2::Test2(void)
    {
    	analyzer2 = new Analyzer ;
    }
    
    Test2::~Test2(void)
    {
    }

Inside main.cpp

C++

#include "Test2.h"
    #include "Tst1.h"
    #include "stdio.h"
    
    int main()
    {
    	Tst1 *test1 = new Tst1 ; //check = false
    	Test2 *test2 = new Test2 ; //check = false
    	test1->analyzer1->GetJointDetails()->check = true ;
    	if(test2->analyzer2->GetJointDetails()->check )
    		printf("Check value is changed");
    	else
    		printf("Check value is not changed");
    		return 0 ;
    }

Posted 2-Sep-12 8:57am

vikuseth

Updated 2-Sep-12 9:00am

v2

Add a Solution

Comments

pasztorpisti 2-Sep-12 16:24pm

Why do you want to avoid the static keyword? If you want to store only a single bool value inside your program than the use of static is reasonable here.

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

pasztorpisti · Accepted Answer · 2012-09-02T09:56:00

1. In C++ when your method doesn't have a parameter list you don't have to write void in place of the parameter list. In plain C you had to do this because there a function without any parameters is a vararg function if you don't put the void there but in C++ its just noise, an empty parameter list in C++ isn't vararg!
2. In C++ its better to keep all member variables private and make public or private set/get methods for them like this:

C++

class Jointdetails
{
public:
    Jointdetails();
    ~Jointdetails();
    bool IsChecked() const
    {
        return Checked;
    }
    void SetChecked(bool checked)
    {
        Checked = checked;
    }
private:
    bool Checked;
};

Another good practice is to give noun member variable names and verb method names like SetXXX(), GetXXX(), DoThis(), DoThat(), the only exception are some bool getter methods that start with "Is" like in my example.
3. If you need the Checked variable in every instances then you are already OK, if you need this Checked variable only onece in your application then you can make it static and you can do it like this:
jointdetails.h:

C++

class Jointdetails
{
public:
    static bool IsChecked();
    static void SetChecked(bool checked);
};

jointdetails.cpp:

C++

#include "Jointdetails.h"

static bool g_Checked = false;    

bool Jointdetails::IsChecked()
{
    return g_Checked;
}

void Jointdetails::SetChecked(bool checked)
{
    g_Checked = checked;
}

This way you can call Jointdetails::IsChecked() anywhere in you application.