Fix implementation of 2-4 and 2-3

semester1/pset13/2-3-ArrayEX.c

@@ -1,38 +1,33 @@
 #include <stdio.h>
-#include <stdlib.h>
 #include <string.h>
 #define MAX 100001
 
 void sort(char **arr, int n);
-void swap(char **arr1, char **arr2);
 
 int main(void) {
+  char str[MAX];
   int n;
   scanf("%d", &n);
+  char *arr[n];
   getchar();
-  // 2d array
-  char **str = malloc(sizeof(char *) * n);
-  // Buffer array
-  char *buffer = malloc(sizeof(char) * MAX);
 
+  // Get input
+  char ch;
+  int loc = 0;
   for (int i = 0; i < n; i++) {
-    fgets(buffer, MAX, stdin);
+    arr[i] = &str[loc];
-    str[i] = malloc(strlen(buffer) + 1);
+    while ((ch = getc(stdin)) != '\n') {
-    strcpy(str[i], buffer);
+      str[loc++] = ch;
+    }
+    str[loc++] = '\0';
   }
 
-  sort(str, n);
+  sort(arr, n);
 
   for (int i = 0; i < n; i++) {
-    printf("%s", str[i]);
+    printf("%s\n", arr[i]);
   }
 
-  // Free up memory
-  free(buffer);
-  for (int i = 0; i < n; i++) {
-    free(str[i]);
-  }
-  free(str);
   return 0;
 }
 
@@ -40,7 +35,6 @@ void sort(char **arr, int n) {
   for (int i = 0; i < n; i++) {
     for (int j = 0; j < n - 1; j++) {
       if (strcmp(arr[j], arr[j + 1]) > 0) {
-				// Why I can't use swap() here?
         char *tmp = arr[j];
         arr[j] = arr[j + 1];
         arr[j + 1] = tmp;

semester1/pset13/2-4-ArrayEX.c

@@ -1 +0,0 @@
-2-3-ArrayEX.c

semester1/pset13/2-4-ArrayEX.c

@@ -0,0 +1,50 @@
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#define MAX 1001
+
+void sort(char **arr, int n);
+void swap(char **arr1, char **arr2);
+
+int main(void) {
+  int n;
+  scanf("%d", &n);
+  getchar();
+  // 2d array
+  char **str = malloc(sizeof(char *) * n);
+  // Buffer array
+  char *buffer = malloc(sizeof(char) * MAX);
+
+  for (int i = 0; i < n; i++) {
+    fgets(buffer, MAX, stdin);
+    str[i] = malloc(strlen(buffer) + 1);
+    strcpy(str[i], buffer);
+  }
+
+  sort(str, n);
+
+  for (int i = 0; i < n; i++) {
+    printf("%s", str[i]);
+  }
+
+  // Free up memory
+  free(buffer);
+  for (int i = 0; i < n; i++) {
+    free(str[i]);
+  }
+  free(str);
+  return 0;
+}
+
+void sort(char **arr, int n) {
+  for (int i = 0; i < n; i++) {
+    for (int j = 0; j < n - 1; j++) {
+      if (strcmp(arr[j], arr[j + 1]) > 0) {
+        // Why I can't use swap() here?
+        char *tmp = arr[j];
+        arr[j] = arr[j + 1];
+        arr[j + 1] = tmp;
+      }
+    }
+  }
+}