Merge pull request #2 from netfoundry/bugfix/1-initial-simpleclient-index-out-of-bounds

Fixing argument handling in initial client code

Author: tburtchell

initial/client/src/main/java/com/example/restservice/SimpleClient.java

@@ -13,6 +13,7 @@ public class SimpleClient {
 
   private static void usageAndExit() {
     System.out.println("Usage: SimpleClient <-i identityFile> <-s serviceName>");
+    System.exit(1);
   }
 
   public static void main(String[] args) {
@@ -21,14 +22,14 @@ public static void main(String[] args) {
 
     for(int i = 0; i < args.length; i++) {
       if("-i".equals(args[i])) {
-        if( i < args.length ) {
+        if( i < args.length-1 ) {
           identityFile = args[++i];
         } else {
           usageAndExit();
         }
       }
       if("-s".equals(args[i])) {
-        if( i < args.length) {
+        if( i < args.length-1 ) {
           serviceName = args[++i];
         } else {
           usageAndExit();