Xcos MVC: standardize C++ code 40/15540/3
Clément DAVID [Mon, 17 Nov 2014 15:24:47 +0000 (16:24 +0100)]
 * use m_ prefix for class' members
 * use static members for Singletons
 * use 'const' modifiers for constants

Change-Id: I7b6035f043950836476c2bf1191aa0e5cf80fe0e

scilab/modules/scicos/includes/Controller.hxx
scilab/modules/scicos/includes/model/BaseObject.hxx
scilab/modules/scicos/src/cpp/Controller.cpp
scilab/modules/scicos/src/cpp/LoggerView.cpp
scilab/modules/scicos/src/cpp/Model.cpp
scilab/modules/scicos/src/cpp/model/Port.hxx
scilab/modules/xcos/src/java/org/scilab/modules/xcos/Controller.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/JavaControllerJNI.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/View.java
scilab/modules/xcos/src/jni/JavaController_wrap.cxx

index eb4acfa..8c0264a 100644 (file)
@@ -34,13 +34,10 @@ namespace org_scilab_modules_scicos
 class SCICOS_IMPEXP Controller
 {
 public:
-    static void delete_all_instances();
-
     static void register_view(View* v);
     static void unregister_view(View* v);
 
     Controller();
-    Controller(const Controller& c);
     ~Controller();
 
     ScicosID createObject(kind_t k);
@@ -52,21 +49,21 @@ public:
     template<typename T>
     bool getObjectProperty(ScicosID uid, kind_t k, object_properties_t p, T& v) const
     {
-        return _instance->model.getObjectProperty(uid, k, p, v);
+        return m_instance.model.getObjectProperty(uid, k, p, v);
     };
 
     template<typename T>
     update_status_t setObjectProperty(ScicosID uid, kind_t k, object_properties_t p, T v)
     {
-        update_status_t status = _instance->model.setObjectProperty(uid, k, p, v);
+        update_status_t status = m_instance.model.setObjectProperty(uid, k, p, v);
         if (status == SUCCESS)
         {
-            for (view_set_t::iterator iter = _instance->allViews.begin(); iter != _instance->allViews.end(); ++iter)
+            for (view_set_t::iterator iter = m_instance.allViews.begin(); iter != m_instance.allViews.end(); ++iter)
             {
                 (*iter)->propertyUpdated(uid, k, p);
             }
         }
-        for (view_set_t::iterator iter = _instance->allViews.begin(); iter != _instance->allViews.end(); ++iter)
+        for (view_set_t::iterator iter = m_instance.allViews.begin(); iter != m_instance.allViews.end(); ++iter)
         {
             (*iter)->propertyUpdated(uid, k, p, status);
         }
@@ -94,7 +91,7 @@ private:
      *
      * This will be allocated on-demand be Controller::get_instance()
      */
-    static SharedData* _instance;
+    static SharedData m_instance;
 
     /*
      * Methods
index 145cb0d..91b997c 100644 (file)
@@ -26,47 +26,47 @@ class BaseObject
 {
 public:
     BaseObject(kind_t k) :
-        _id(0), _kind(k)
+        m_id(0), m_kind(k)
     {
     }
     BaseObject(const BaseObject& b) :
-        _id(0), _kind(b._kind)
+        m_id(0), m_kind(b.m_kind)
     {
     }
 
     inline bool operator<(BaseObject o) const
     {
-        return _id < o._id;
+        return m_id < o.m_id;
     }
     inline bool operator==(BaseObject o) const
     {
-        return _id == o._id;
+        return m_id == o.m_id;
     }
 
     inline ScicosID id() const
     {
-        return _id;
+        return m_id;
     }
-    inline void id(ScicosID id)
+    inline void id(ScicosID _id)
     {
-        _id = id;
+        m_id = _id;
     }
 
     inline kind_t kind() const
     {
-        return _kind;
+        return m_kind;
     }
 
 private:
     /**
      * An id is used as a reference to the current object
      */
-    ScicosID _id;
+    ScicosID m_id;
 
     /**
      * Kind of the Object
      */
-    const kind_t _kind;
+    const kind_t m_kind;
 };
 
 /** @defgroup utilities Shared utility classes
@@ -78,26 +78,26 @@ private:
  */
 struct Geometry
 {
-    double x;
-    double y;
-    double width;
-    double height;
+    double m_x;
+    double m_y;
+    double m_width;
+    double m_height;
 
-    Geometry() : x(0), y(0), width(20), height(20) {};
-    Geometry(const Geometry& g) : x(g.x), y(g.y), width(g.width), height(g.height) {};
-    Geometry(const std::vector<double>& v) : x(v[0]), y(v[1]), width(v[2]), height(v[3]) {};
+    Geometry() : m_x(0), m_y(0), m_width(20), m_height(20) {};
+    Geometry(const Geometry& g) : m_x(g.m_x), m_y(g.m_y), m_width(g.m_width), m_height(g.m_height) {};
+    Geometry(const std::vector<double>& v) : m_x(v[0]), m_y(v[1]), m_width(v[2]), m_height(v[3]) {};
 
     void fill(std::vector<double>& v) const
     {
         v.resize(4);
-        v[0] = x;
-        v[1] = y;
-        v[2] = width;
-        v[3] = height;
+        v[0] = m_x;
+        v[1] = m_y;
+        v[2] = m_width;
+        v[3] = m_height;
     }
     bool operator==(const Geometry& g) const
     {
-        return x == g.x && y == g.y && width == g.width && height == g.height;
+        return m_x == g.m_x && m_y == g.m_y && m_width == g.m_width && m_height == g.m_height;
     }
 };
 
@@ -110,28 +110,28 @@ struct Datatype
 {
 public:
     Datatype(const Datatype& d) :
-        refCount(0), datatype_id(d.datatype_id), rows(d.rows), columns(d.columns) {};
+        m_refCount(0), m_datatype_id(d.m_datatype_id), m_rows(d.m_rows), m_columns(d.m_columns) {};
     Datatype(const std::vector<int>& v) :
-        refCount(0), datatype_id(v[2]), rows(v[0]), columns(v[1]) {};
+        m_refCount(0), m_datatype_id(v[2]), m_rows(v[0]), m_columns(v[1]) {};
 
     // reference counter for the flyweight pattern
-    int refCount;
+    int m_refCount;
 
-    const int datatype_id;
-    const int rows;
-    const int columns;
+    const int m_datatype_id;
+    const int m_rows;
+    const int m_columns;
 
     bool operator==(const Datatype& d) const
     {
-        return datatype_id == d.datatype_id && rows == d.rows && columns == d.columns;
+        return m_datatype_id == d.m_datatype_id && m_rows == d.m_rows && m_columns == d.m_columns;
     }
 
     bool operator<(const Datatype& d) const
     {
         // Lexicographical order
-        return datatype_id < d.datatype_id ||
-               (datatype_id == d.datatype_id && rows < d.rows) ||
-               (datatype_id == d.datatype_id && rows == d.rows && columns < d.columns);
+        return m_datatype_id < d.m_datatype_id ||
+               (m_datatype_id == d.m_datatype_id && m_rows < d.m_rows) ||
+               (m_datatype_id == d.m_datatype_id && m_rows == d.m_rows && m_columns < d.m_columns);
     }
 };
 
index dd1081f..7df9061 100644 (file)
@@ -36,59 +36,30 @@ Controller::SharedData::SharedData() :
 
 Controller::SharedData::~SharedData()
 {
-    for (view_set_t::iterator iter = _instance->allViews.begin(); iter != _instance->allViews.end(); ++iter)
+    for (view_set_t::iterator iter = m_instance.allViews.begin(); iter != m_instance.allViews.end(); ++iter)
     {
         delete *iter;
     }
 }
 
-Controller::SharedData* Controller::_instance = 0;
-
-void Controller::delete_all_instances()
-{
-    if (_instance == 0)
-    {
-        return;
-    }
-
-    delete _instance;
-}
+Controller::SharedData Controller::m_instance;
 
 void Controller::register_view(View* v)
 {
-    if (_instance == 0)
-    {
-        _instance = new SharedData();
-    }
-
-    _instance->allViews.push_back(v);
+    m_instance.allViews.push_back(v);
 }
 
 void Controller::unregister_view(View* v)
 {
-    if (_instance == 0)
+    view_set_t::iterator it = std::find(m_instance.allViews.begin(), m_instance.allViews.end(), v);
+    if (it != m_instance.allViews.end())
     {
-        return;
-    }
-
-    view_set_t::iterator it = std::find(_instance->allViews.begin(), _instance->allViews.end(), v);
-    if (it != _instance->allViews.end())
-    {
-        _instance->allViews.erase(it);
+        m_instance.allViews.erase(it);
     }
 }
 
 Controller::Controller()
 {
-    if (_instance == 0)
-    {
-        _instance = new SharedData();
-    }
-}
-
-Controller::Controller(const Controller& /*c*/)
-{
-    // _instance is already initialized
 }
 
 Controller::~Controller()
@@ -97,9 +68,9 @@ Controller::~Controller()
 
 ScicosID Controller::createObject(kind_t k)
 {
-    ScicosID id = _instance->model.createObject(k);
+    ScicosID id = m_instance.model.createObject(k);
 
-    for (view_set_t::iterator iter = _instance->allViews.begin(); iter != _instance->allViews.end(); ++iter)
+    for (view_set_t::iterator iter = m_instance.allViews.begin(); iter != m_instance.allViews.end(); ++iter)
     {
         (*iter)->objectCreated(id, k);
     }
@@ -150,9 +121,9 @@ void Controller::deleteObject(ScicosID uid)
     }
 
     // delete the object
-    _instance->model.deleteObject(uid);
+    m_instance.model.deleteObject(uid);
 
-    for (view_set_t::iterator iter = _instance->allViews.begin(); iter != _instance->allViews.end(); ++iter)
+    for (view_set_t::iterator iter = m_instance.allViews.begin(); iter != m_instance.allViews.end(); ++iter)
     {
         (*iter)->objectDeleted(uid, k);
     }
@@ -340,7 +311,7 @@ ScicosID Controller::cloneObject(ScicosID uid)
 
 std::shared_ptr<model::BaseObject> Controller::getObject(ScicosID uid) const
 {
-    return _instance->model.getObject(uid);
+    return m_instance.model.getObject(uid);
 }
 
 } /* namespace org_scilab_modules_scicos */
index bd7e84f..1bd3e58 100644 (file)
@@ -244,6 +244,8 @@ std::ostream& operator<<(std::ostream& os, object_properties_t p)
         case VERSION_NUMBER:
             os << "VERSION_NUMBER";
             break;
+        default:
+            break;
     }
     return os;
 }
index 6b8ef73..98a3b00 100644 (file)
@@ -144,7 +144,7 @@ model::Datatype* Model::flyweight(const model::Datatype& d)
     datatypes_set_t::iterator iter = std::lower_bound(datatypes.begin(), datatypes.end(), &d, isInferior);
     if (iter != datatypes.end() && !(d < **iter)) // if d is found
     {
-        (*iter)->refCount++;
+        (*iter)->m_refCount++;
         return *iter;
     }
     else
@@ -158,8 +158,8 @@ void Model::erase(model::Datatype* d)
     datatypes_set_t::iterator iter = std::lower_bound(datatypes.begin(), datatypes.end(), d, isInferior);
     if (iter != datatypes.end() && !(*d < **iter)) // if d is found
     {
-        (*iter)->refCount--;
-        if ((*iter)->refCount < 0)
+        (*iter)->m_refCount--;
+        if ((*iter)->m_refCount < 0)
         {
             delete *iter;
             datatypes.erase(iter);
index ce193da..bfa2d47 100644 (file)
@@ -72,9 +72,9 @@ private:
         else
         {
             v.resize(3);
-            v[0] = m_dataType->rows;
-            v[1] = m_dataType->columns;
-            v[2] = m_dataType->datatype_id;
+            v[0] = m_dataType->m_rows;
+            v[1] = m_dataType->m_columns;
+            v[2] = m_dataType->m_datatype_id;
         }
     }
 
index f737197..8494a61 100644 (file)
@@ -36,11 +36,7 @@ public class Controller {
     }
 
     public Controller() {
-        this(JavaControllerJNI.new_Controller__SWIG_0(), true);
-    }
-
-    public Controller(Controller c) {
-        this(JavaControllerJNI.new_Controller__SWIG_1(Controller.getCPtr(c), c), true);
+        this(JavaControllerJNI.new_Controller(), true);
     }
 
     public long createObject(Kind k) {
index 704e389..efd3dfb 100644 (file)
@@ -18,8 +18,7 @@ public class JavaControllerJNI {
     public final static native void View_propertyUpdated__SWIG_1(long jarg1, View jarg1_, long jarg2, int jarg3, int jarg4, int jarg5);
     public final static native void View_director_connect(View obj, long cptr, boolean mem_own, boolean weak_global);
     public final static native void View_change_ownership(View obj, long cptr, boolean take_or_release);
-    public final static native long new_Controller__SWIG_0();
-    public final static native long new_Controller__SWIG_1(long jarg1, Controller jarg1_);
+    public final static native long new_Controller();
     public final static native void delete_Controller(long jarg1);
     public final static native long Controller_createObject(long jarg1, Controller jarg1_, int jarg2);
     public final static native void Controller_deleteObject(long jarg1, Controller jarg1_, long jarg2);
index 0aa8725..1570137 100644 (file)
@@ -71,8 +71,8 @@ public class View {
         JavaControllerJNI.View_propertyUpdated__SWIG_0(swigCPtr, this, uid, k.ordinal(), p.ordinal());
     }
 
-    public void propertyUpdated(long uid, Kind k, ObjectProperties p, UpdateStatus u) {
-        JavaControllerJNI.View_propertyUpdated__SWIG_1(swigCPtr, this, uid, k.ordinal(), p.ordinal(), u.ordinal());
+    public void propertyUpdated(long arg0, Kind arg1, ObjectProperties arg2, UpdateStatus arg3) {
+        JavaControllerJNI.View_propertyUpdated__SWIG_1(swigCPtr, this, arg0, arg1.ordinal(), arg2.ordinal(), arg3.ordinal());
     }
 
 }
index 9937c5b..521001b 100644 (file)
@@ -1002,7 +1002,7 @@ SWIGEXPORT void JNICALL Java_org_scilab_modules_xcos_JavaControllerJNI_View_1cha
 }
 
 
-SWIGEXPORT jlong JNICALL Java_org_scilab_modules_xcos_JavaControllerJNI_new_1Controller_1_1SWIG_10(JNIEnv *jenv, jclass jcls) {
+SWIGEXPORT jlong JNICALL Java_org_scilab_modules_xcos_JavaControllerJNI_new_1Controller(JNIEnv *jenv, jclass jcls) {
   jlong jresult = 0 ;
   org_scilab_modules_scicos::Controller *result = 0 ;
   
@@ -1014,25 +1014,6 @@ SWIGEXPORT jlong JNICALL Java_org_scilab_modules_xcos_JavaControllerJNI_new_1Con
 }
 
 
-SWIGEXPORT jlong JNICALL Java_org_scilab_modules_xcos_JavaControllerJNI_new_1Controller_1_1SWIG_11(JNIEnv *jenv, jclass jcls, jlong jarg1, jobject jarg1_) {
-  jlong jresult = 0 ;
-  org_scilab_modules_scicos::Controller *arg1 = 0 ;
-  org_scilab_modules_scicos::Controller *result = 0 ;
-  
-  (void)jenv;
-  (void)jcls;
-  (void)jarg1_;
-  arg1 = *(org_scilab_modules_scicos::Controller **)&jarg1;
-  if (!arg1) {
-    SWIG_JavaThrowException(jenv, SWIG_JavaNullPointerException, "org_scilab_modules_scicos::Controller const & reference is null");
-    return 0;
-  } 
-  result = (org_scilab_modules_scicos::Controller *)new org_scilab_modules_scicos::Controller((org_scilab_modules_scicos::Controller const &)*arg1);
-  *(org_scilab_modules_scicos::Controller **)&jresult = result; 
-  return jresult;
-}
-
-
 SWIGEXPORT void JNICALL Java_org_scilab_modules_xcos_JavaControllerJNI_delete_1Controller(JNIEnv *jenv, jclass jcls, jlong jarg1) {
   org_scilab_modules_scicos::Controller *arg1 = (org_scilab_modules_scicos::Controller *) 0 ;