improve AppInfo class

hide public member variables and introduce getters and setters for them.

Change-Id: I9108e94516238ef2ff8d4ea18db381d4e7e73bec
Signed-off-by: Tasuku Suzuki <tasuku.suzuki@qt.io>

1 files changed, 124 insertions, 29 deletions

diff --git a/interfaces/src/appframework.cpp b/interfaces/src/appframework.cpp
index 15c57c0..ced809e 100644
--- a/interfaces/src/appframework.cpp
+++ b/interfaces/src/appframework.cpp
@@ -16,8 +16,52 @@
 
 #include "include/appframework.hpp"
 
+#include <QtCore/QJsonObject>
+
+class AppInfo::Private : public QSharedData
+{
+public:
+    Private();
+    Private(const Private &other);
+
+    QString id;
+    QString version;
+    int width;
+    int height;
+    QString name;
+    QString description;
+    QString shortname;
+    QString author;
+    QString iconPath;
+};
+
+AppInfo::Private::Private()
+    : width(-1)
+    , height(-1)
+{
+}
+
+AppInfo::Private::Private(const Private &other)
+    : QSharedData(other)
+    , id(other.id)
+    , version(other.version)
+    , width(other.width)
+    , height(other.height)
+    , name(other.name)
+    , description(other.description)
+    , shortname(other.shortname)
+    , author(other.author)
+    , iconPath(other.iconPath)
+{
+}
 
 AppInfo::AppInfo()
+    : d(new Private)
+{
+}
+
+AppInfo::AppInfo(const AppInfo &other)
+    : d(other.d)
 {
 }
 
@@ -25,48 +69,99 @@ AppInfo::~AppInfo()
 {
 }
 
+AppInfo &AppInfo::operator =(const AppInfo &other)
+{
+    d = other.d;
+    return *this;
+}
+
+QString AppInfo::id() const
+{
+    return d->id;
+}
+
+QString AppInfo::version() const
+{
+    return d->version;
+}
+
+int AppInfo::width() const
+{
+    return d->width;
+}
+
+int AppInfo::height() const
+{
+    return d->height;
+}
+
+QString AppInfo::name() const
+{
+    return d->name;
+}
+
+QString AppInfo::description() const
+{
+    return d->description;
+}
+
+QString AppInfo::shortname() const
+{
+    return d->shortname;
+}
+
+QString AppInfo::author() const
+{
+    return d->author;
+}
+
+QString AppInfo::iconPath() const
+{
+    return d->iconPath;
+}
+
 void AppInfo::read(const QJsonObject &json)
 {
-    id = json["id"].toString();
-    version = json["version"].toString();
-    width = json["width"].toInt();
-    height = json["height"].toInt();
-    name = json["name"].toString();
-    description = json["description"].toString();
-    shortname = json["shortname"].toString();
-    author = json["author"].toString();
-    iconPath = json["iconPath"].toString();
+    d->id = json["id"].toString();
+    d->version = json["version"].toString();
+    d->width = json["width"].toInt();
+    d->height = json["height"].toInt();
+    d->name = json["name"].toString();
+    d->description = json["description"].toString();
+    d->shortname = json["shortname"].toString();
+    d->author = json["author"].toString();
+    d->iconPath = json["iconPath"].toString();
 }
 
-QDBusArgument &operator <<(QDBusArgument &argument, const AppInfo &mAppInfo)
+QDBusArgument &operator <<(QDBusArgument &argument, const AppInfo &appInfo)
 {
     argument.beginStructure();
-    argument << mAppInfo.id;
-    argument << mAppInfo.version;
-    argument << mAppInfo.width;
-    argument << mAppInfo.height;
-    argument << mAppInfo.name;
-    argument << mAppInfo.description;
-    argument << mAppInfo.shortname;
-    argument << mAppInfo.author;
-    argument << mAppInfo.iconPath;
+    argument << appInfo.d->id;
+    argument << appInfo.d->version;
+    argument << appInfo.d->width;
+    argument << appInfo.d->height;
+    argument << appInfo.d->name;
+    argument << appInfo.d->description;
+    argument << appInfo.d->shortname;
+    argument << appInfo.d->author;
+    argument << appInfo.d->iconPath;
     argument.endStructure();
 
     return argument;
 }
 
-const QDBusArgument &operator >>(const QDBusArgument &argument, AppInfo &mAppInfo)
+const QDBusArgument &operator >>(const QDBusArgument &argument, AppInfo &appInfo)
 {
     argument.beginStructure();
-    argument >> mAppInfo.id;
-    argument >> mAppInfo.version;
-    argument >> mAppInfo.width;
-    argument >> mAppInfo.height;
-    argument >> mAppInfo.name;
-    argument >> mAppInfo.description;
-    argument >> mAppInfo.shortname;
-    argument >> mAppInfo.author;
-    argument >> mAppInfo.iconPath;
+    argument >> appInfo.d->id;
+    argument >> appInfo.d->version;
+    argument >> appInfo.d->width;
+    argument >> appInfo.d->height;
+    argument >> appInfo.d->name;
+    argument >> appInfo.d->description;
+    argument >> appInfo.d->shortname;
+    argument >> appInfo.d->author;
+    argument >> appInfo.d->iconPath;
     argument.endStructure();
     return argument;
 }