DefaultRoleManager added.

DefaultRoleManager added.

Removed unnecessary includeFiles.

Made changes in ordered to unorder_map, enum to enum-class and declaring iterators inside for loop.

h to hpp

Author: divy9881

.gitignore

@@ -85,7 +85,7 @@ StyleCopReport.xml
 *.tmp
 *.tmp_proj
 *_wpftmp.csproj
-*.log
+# *.log
 *.vspscc
 *.vssscc
 .builds

src/effect/DefaultEffector.h renamed to src/effect/DefaultEffector.hpp

@@ -1,8 +1,6 @@
-#include <string>
-
-#include "Effect.h"
-#include "Effector.h"
-#include "exception/UnsupportedOperationException.h"
+#include "Effect.hpp"
+#include "Effector.hpp"
+#include "exception/UnsupportedOperationException.hpp"
 
 /**
  * DefaultEffector is default effector for Casbin.
@@ -20,34 +18,34 @@ class DefaultEffector : public Effector{
             if (!expr.compare("some(where (p_eft == allow))")) {
                 result = false;
                 for(unsigned int index = 0 ; index < number_of_effects ; index++){
-                    if (effects[index] == Allow) {
+                    if (effects[index] == Effect::Allow) {
                         result = true;
                         break;
                     }
                 }
             } else if (!expr.compare("!some(where (p_eft == deny))")) {
                 result = true;
                 for(unsigned int index = 0 ; index < number_of_effects ; index++){
-                    if (effects[index] == Deny) {
+                    if (effects[index] == Effect::Deny) {
                         result = false;
                         break;
                     }
                 }
             } else if (!expr.compare("some(where (p_eft == allow)) && !some(where (p_eft == deny))")) {
                 result = false;
                 for(unsigned int index = 0 ; index < number_of_effects ; index++){
-                    if (effects[index] == Allow) {
+                    if (effects[index] == Effect::Allow) {
                         result = true;
-                    } else if (effects[index] == Deny) {
+                    } else if (effects[index] == Effect::Deny) {
                         result = false;
                         break;
                     }
                 }
             } else if (!expr.compare("priority(p_eft) || deny")) {
                 result = false;
                 for(unsigned int index = 0 ; index < number_of_effects ; index++){
-                    if (effects[index] != Indeterminate) {
-                        if (effects[index] == Allow) {
+                    if (effects[index] != Effect::Indeterminate) {
+                        if (effects[index] == Effect::Allow) {
                             result = true;
                         } else {
                             result = false;

src/effect/Effect.h

@@ -0,0 +1,5 @@
+enum class Effect{
+    Allow, Indeterminate, Deny
+};
+
+typedef enum class Effect Effect;

src/effect/Effect.hpp

@@ -1,6 +1,6 @@
 #include <string>
 
-#include "Effect.h"
+#include "Effect.hpp"
 
 /**
  * Effector is the abstract class for Casbin effectors.

src/effect/Effector.h renamed to src/effect/Effector.hpp

@@ -1,13 +1,9 @@
-#include <string>
-
-#include "Logger.h"
-#include "Log.h"
+#include "Logger.hpp"
+#include "Log.hpp"
 
 class DefaultLogger : public Logger{
-    private:
-        bool enable;
-    
     public:
+
         void EnableLog(bool enable) {
             this->enable = enable;
         }

src/exception/IllegalArgumentException.h renamed to src/exception/IllegalArgumentException.hpp

@@ -1,7 +1,4 @@
-#include <string>
-
-#include "DefaultLogger.h"
-#include "Logger.h"
+#include "DefaultLogger.hpp"
 
 class LogUtil{
 	private: