Better error handling in lpmln binary

src/Makefile.am

@@ -37,4 +37,5 @@ lpmln_LDADD = -luuid
 
 lpmln_SOURCES = \
 		lpexec_src/lpexec.cpp \
-		lpexec_src/FileConfig.cpp
+		lpexec_src/FileConfig.cpp \
+		lpexec_src/NoLanguageSelectedException.cpp

src/includes/exceptions/NoLanguageSelectedException.h

@@ -0,0 +1,19 @@
+#include <string>
+#include <exception>
+
+#pragma once
+
+class NoLanguageSelectedException : public std::exception{
+	public:
+		NoLanguageSelectedException(std::string);
+		NoLanguageSelectedException();
+		~NoLanguageSelectedException();
+
+		const char* what() const throw(){
+			return str.c_str();
+		}
+
+	private:
+		std::string str; 
+};
+

src/lpexec_src/FileConfig.cpp

@@ -9,13 +9,16 @@
 #include <sys/types.h>
 #include <pwd.h>
 
+#include "exceptions/NoLanguageSelectedException.h"
+
 using namespace std;
 
 FileConfig::FileConfig(){
 	map<string,string> options;
 	ifstream configFile;
 
 	const char *homedir;
+	bool fileCouldNotBeOpened = false;
 
 	if ((homedir = getenv("HOME")) == NULL) {
 	    homedir = getpwuid(getuid())->pw_dir;
@@ -29,9 +32,10 @@ FileConfig::FileConfig(){
 		#ifdef DEBUG
 		cout<<"Cannot read from config file\n";
 		#endif
-		ofstream file;
-		file.open(filename);
-		file.close();
+		// ofstream file;
+		// file.open(filename);
+		// file.close();
+		fileCouldNotBeOpened = true;
 	}
 
 	// Set defaults
@@ -184,8 +188,8 @@ FileConfig::FileConfig(){
 		}
 	}
 
-	if(!execute_alch && !execute_cli){
-		throw;
+	if(!execute_alch && !execute_cli || fileCouldNotBeOpened){
+		throw NoLanguageSelectedException("No language selected from config File. Check to see if config file has the right syntax.");
 	}
 
 	//Parsing done

src/lpexec_src/NoLanguageSelectedException.cpp

@@ -0,0 +1,9 @@
+#include "exceptions/NoLanguageSelectedException.h"
+
+NoLanguageSelectedException::~NoLanguageSelectedException(){}
+
+NoLanguageSelectedException::NoLanguageSelectedException(std::string _str){
+	str = _str;
+}
+
+NoLanguageSelectedException::NoLanguageSelectedException(){}

src/lpexec_src/lpexec.cpp

@@ -10,6 +10,7 @@
 #include <fstream>
 
 #include "FileConfig.h"
+#include "exceptions/NoLanguageSelectedException.h"
 
 #define SPACE " "
 
@@ -129,9 +130,21 @@ int main(int argc, char **argv)
       execute_cli = cf.getExecuteCli();
       execute_tuf = false;
    }
-   catch(...){
+   catch(NoLanguageSelectedException ex){
       // No solver is specified in the config file. Need to pick it up from command line.
+
+      //Default for lpmln: Solver is MLN, Input is MVSM
+      loptions.insert(pair<string,string>("input","mvsm"));
+      //Compile for Clingo
+      loptions.insert(pair<string,string>("solver","clingo"));
+      //max number of solutions for clingo
+      coptions.insert(pair<string,string>("solutions","0"));
+
       pickUpFromCommandLine = true;
+
+      execute_cli = true;
+
+
    }
 
    uuid_t uuid;
@@ -181,6 +194,7 @@ int main(int argc, char **argv)
          execute_cli = false;
          // loptions
          loptions["solver"] = "alchemy";
+         pickUpFromCommandLine = false;
          continue;
       }
 
@@ -190,6 +204,7 @@ int main(int argc, char **argv)
          execute_alch = false;
          // loptions.insert(pair<string,string>("solver","clingo"));
          loptions["solver"] = "clingo";
+         pickUpFromCommandLine = false;
          continue;
       }
 
@@ -302,6 +317,10 @@ int main(int argc, char **argv)
 
    lpmln += inputFileName;
 
+   if(pickUpFromCommandLine){
+      std::cout<<"No input language and solver specified in command line and config file.\nUsing ASP Solver.\n";
+   }
+
 
    for (auto it=aoptions.begin(); it!=aoptions.end(); ++it){
       string str = it->first;
@@ -330,7 +349,6 @@ int main(int argc, char **argv)
    lpmln += SPACE + string(">") + SPACE + string("/tmp/") + fileName;
    std::cout<<"Lpmln Executed with Command:\n"+lpmln+"\n";
    runProcess(lpmln);
-   // std::cout<<lpmln;
 
    if(!isEvidenceUsed && execute_alch == true){
       //Make empty db