{"action":"created","issue":{"url":"https://api.github.com/repos/QuickRecon/ScotchOARKit/issues/27","repository_url":"https://api.github.com/repos/QuickRecon/ScotchOARKit","labels_url":"https://api.github.com/repos/QuickRecon/ScotchOARKit/issues/27/labels{/name}","comments_url":"https://api.github.com/repos/QuickRecon/ScotchOARKit/issues/27/comments","events_url":"https://api.github.com/repos/QuickRecon/ScotchOARKit/issues/27/events","html_url":"https://github.com/QuickRecon/ScotchOARKit/issues/27","id":152014602,"number":27,"title":"Standardise client-server command formats","user":{"login":"TomW1605","id":17092573,"avatar_url":"https://avatars.githubusercontent.com/u/17092573?v=3","gravatar_id":"","url":"https://api.github.com/users/TomW1605","html_url":"https://github.com/TomW1605","followers_url":"https://api.github.com/users/TomW1605/followers","following_url":"https://api.github.com/users/TomW1605/following{/other_user}","gists_url":"https://api.github.com/users/TomW1605/gists{/gist_id}","starred_url":"https://api.github.com/users/TomW1605/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TomW1605/subscriptions","organizations_url":"https://api.github.com/users/TomW1605/orgs","repos_url":"https://api.github.com/users/TomW1605/repos","events_url":"https://api.github.com/users/TomW1605/events{/privacy}","received_events_url":"https://api.github.com/users/TomW1605/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-04-30T07:01:25Z","updated_at":"2016-05-01T02:31:10Z","closed_at":null,"body":"i have tried to do this to an extent. i don't know how hard it would be to send args as a list (unless u make it a string, send it then split it on the other side) but that would probably be the best option.\r\n\r\nanother way (almost like this now) is:\r\n1. client console splits at \" \" like it does now but only the first time\r\n2. the rest of it is viewed as the arguments but there can only ever be 0 or 1\r\n3. the client adds the necessary flag (eg E, G etc) and sends the data as a string\r\n4. the server receives the data and sorts it by flag\r\n5. then it strips the flag\r\n5. the server then splits the arguments by some other identifier (possibly \"-\", \"_\" or \",\") and acts on them\r\n\r\nany other ideas/comments?"},"comment":{"url":"https://api.github.com/repos/QuickRecon/ScotchOARKit/issues/comments/216009504","html_url":"https://github.com/QuickRecon/ScotchOARKit/issues/27#issuecomment-216009504","issue_url":"https://api.github.com/repos/QuickRecon/ScotchOARKit/issues/27","id":216009504,"user":{"login":"campell1212","id":2208334,"avatar_url":"https://avatars.githubusercontent.com/u/2208334?v=3","gravatar_id":"","url":"https://api.github.com/users/campell1212","html_url":"https://github.com/campell1212","followers_url":"https://api.github.com/users/campell1212/followers","following_url":"https://api.github.com/users/campell1212/following{/other_user}","gists_url":"https://api.github.com/users/campell1212/gists{/gist_id}","starred_url":"https://api.github.com/users/campell1212/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/campell1212/subscriptions","organizations_url":"https://api.github.com/users/campell1212/orgs","repos_url":"https://api.github.com/users/campell1212/repos","events_url":"https://api.github.com/users/campell1212/events{/privacy}","received_events_url":"https://api.github.com/users/campell1212/received_events","type":"User","site_admin":false},"created_at":"2016-05-01T02:31:10Z","updated_at":"2016-05-01T02:31:10Z","body":"I am currently (getting to it soon) redoing the command backend via abstraction so that each command will have its own class and then extends the ACommand class to be run. In that class it then will find what class it needs to off load the processing to (it may do some before hand) and then each class can do its own thing that returns the finalized string that is sent off to the server to get processed as a command. The classes will get initialized at startup via Reflections. I have already made a issue (Issue #11). Cased based is good but does not leave to much room for any changes if one type of command needs it..."}}