Merge pull request #164 from divyagar/master

fix: Removed repeated section in loadModel and loadModelFromText in Model.java

Author: hsluoyz

src/main/java/org/casbin/jcasbin/model/Model.java

@@ -107,6 +107,20 @@ private void loadSection(Model model, Config cfg, String sec) {
         }
     }
 
+    /*
+     * Helper function for loadModel and loadModelFromText
+     * 
+     * @param config the configuration parser
+     */
+    private void loadSections(Config cfg) {
+        loadSection(this, cfg, "r");
+        loadSection(this, cfg, "p");
+        loadSection(this, cfg, "e");
+        loadSection(this, cfg, "m");
+
+        loadSection(this, cfg, "g");
+    }
+    
     /**
      * loadModel loads the model from model CONF file.
      *
@@ -115,12 +129,7 @@ private void loadSection(Model model, Config cfg, String sec) {
     public void loadModel(String path) {
         Config cfg = Config.newConfig(path);
 
-        loadSection(this, cfg, "r");
-        loadSection(this, cfg, "p");
-        loadSection(this, cfg, "e");
-        loadSection(this, cfg, "m");
-
-        loadSection(this, cfg, "g");
+        loadSections(cfg);
     }
 
     /**
@@ -131,12 +140,7 @@ public void loadModel(String path) {
     public void loadModelFromText(String text) {
         Config cfg = Config.newConfigFromText(text);
 
-        loadSection(this, cfg, "r");
-        loadSection(this, cfg, "p");
-        loadSection(this, cfg, "e");
-        loadSection(this, cfg, "m");
-
-        loadSection(this, cfg, "g");
+        loadSections(cfg);
     }
 
     /**