better compile scripts, fixed some code

arrToBin.c

@@ -21,7 +21,6 @@ int main(){
         bin[i] = b;
     }
     for(int j = 0; j < 16; j++){
-        if(
             printf("%d", bin[j]);
     }
     printf("\n");

compile-all.sh

@@ -1,3 +1,13 @@
 #!/bin/bash
-echo "compiling all files in working directory $(pwd)"
-for file in $(/bin/ls); do ./compile.sh $file; done
+returnCode=0
+echo -e "compiling all files in working directory $(pwd)\n"
+for file in $(/bin/ls *.c); 
+do 
+    ./compile.sh $file; 
+    if [ "$?" -ne 0 ]
+    then
+        echo -e "\nERROR: could not compile $file !\n";
+        returnCode=1;
+    fi
+done
+exit $returnCode

compile-and-run.sh

@@ -1,4 +1,4 @@
 #!/bin/bash
 noext=$(echo "$1" | cut -f 1 -d '.')
-gcc $1 -o bin/$noext
+./compile $1
 ./bin/$noext $2 $3 $4 $5 $6 $7 $8 $9

compile.sh

@@ -1,4 +1,4 @@
 #!/bin/bash
 echo "compiling $1 ..."
 noext=$(echo "$1" | cut -f 1 -d '.')
-gcc $1 -o bin/$noext
+gcc $1 -o bin/$noext -lm

echo.c

@@ -2,7 +2,16 @@
 
 int main() {
 	char s[100];
-	gets(s);
+    //gets(s);
+    /*
+     * Never use gets().  Because it is impossible to tell without knowing the data in advance how many characters gets() will read,  and  be‐
+     * cause gets() will continue to store characters past the end of the buffer, it is extremely dangerous to use.  It has been used to break
+     * computer security.  Use fgets() instead.
+     *
+     * For more information, see CWE-242 (aka "Use of Inherently Dangerous Function") at http://cwe.mitre.org/data/definitions/242.html
+     *
+     */
+    fgets(s, sizeof(s), stdin);
 	printf("%s\n", s);
 	return 0;
 }