Refactor loop implementation to avoid logical operators

- Added comments to clarify the approach and potential pitfalls of using multiplication instead of logical operators.

Author: ohkimur

chapter_2/exercise_2_02/loop.c

@@ -2,30 +2,32 @@
 
 #define MAXLINE 1000
 
+/*
+ * K&R Exercise 2-2: Write a loop equivalent to:
+ *   for (i = 0; i < lim - 1 && (c = getchar()) != '\n' && c != EOF; ++i)
+ *       s[i] = c;
+ * without using && or ||.
+ *
+ * Note: Using * instead of && is unsafe because it doesn't short-circuit,
+ * causing getchar() to be called even when the buffer is full or EOF is
+ * reached.
+ */
+
 int main(void) {
     char s[MAXLINE];
     int c;
     int i = 0;
 
-    // int i;
-    // int c;
-    // for (i = 0; (i < MAXLINE - 1) * ((c = getchar()) != '\n') * (c != EOF);
-    // ++i)
-    // {
-    //   s[i] = c;
-    // }
-
     while (1) {
         if (i >= MAXLINE - 1) {
             break;
         }
 
         c = getchar();
-
+        
         if (c == '\n') {
             break;
         } else if (c == EOF) {
-            printf("\n");
             break;
         } else {
             s[i++] = c;
@@ -38,10 +40,3 @@ int main(void) {
 
     return 0;
 }
-
-// NOTE: The multiplication operation could work in this case because each
-// expression is evaluated as a 1 or 0 (true or false), and a multiplication
-// between expressions can have the value 1 only if all the expressions are
-// true. However, the order of multiplication is not guaranteed to be sequenced
-// as with logical operations. So, this is could cause serious problems
-// depending on how the compiler deals with multiplication.