回答編集履歴

N_WORDSマクロに関する追記

2021/04/05 23:54

投稿

rubato6809

スコア1380

test CHANGED Viewed

@@ -277,3 +277,27 @@
 もう少し言うと、元コードにあったグローバル変数 int bangou; はユーザが意識するようなデータではありません。「今、"hello" をタイプさせられているからには bangou は１になってるはずだ」なんて考えません、bangou の存在さえ意識しないでしょう。bangou は処理内部のための、隠れた存在です。
 一方、 配列 word はユーザが否応なく目にするデータ（"hello"などの文字列）を抱えているので、プログラムの外観＝外部仕様に関わるデータと言えます。その分**他の変数よりエライ**、グローバルにするなら bangou より word が相応しい・・・みたいな気持ちです。
+```C
+#define N_WORDS     (sizeof(word) / sizeof(word[0]))
+char *word[] = {
+    "hello",    "world",    "impossible",  "knowledge",  "capital",
+```
+このマクロは配列の要素数をコンパイラに計算させます。一要素分のところに型名を書くこともあります。
+「#define N_WORDS (sizeof(word) / sizeof(char*))」みたいに。
+元のプログラムでは文字列を10個用意しますが、ずっと10個とは限らない事があるものです。文字列を増やしたい（減らしたいこともあるか）時、配列の初期値として並べて書いてる文字列＝**初期値を書き足すだけで良い**、またそうするために「char *word[10] = { ...」とも「char *word[N_WORDS] = { ...」とも書かず、要素数を空欄にするわけ。
+N_WORDS = (配列全体のバイト数 ÷ 一要素分のサイズ) 。32bit機のポインタ変数は4byte長、64bit機のポインタ変数は8byte長です。64bit機で配列word全体が80byteだから、N_WORDS = 80 ÷ 8 = 10 ・・・でも問題は、word がポインタの配列であることとか、ポインタの姿を正確にイメージできていないとか、ポインタにまつわる文法だとか、その辺りがあやふやみたいですね、ありがちだけど。でもそれは別の質問にしてもらったほうがよいかなあ。

質問への回答追加

2021/04/05 23:54

投稿

rubato6809

スコア1380

test CHANGED Viewed

@@ -267,3 +267,13 @@
 次の課題は関数に分割することかな。ということで残念ながらツッコミどころ満載のコードでした。まだ先は長いですね。
 Enjoy!
+---
+配列 word[] をグローバル変数にした件ですが、強い理由はありません。普通ならローカル変数にするものです。修正を始めた時に関数の外に配列を書いたのですが、見た感じ**おさまりがよかった**のでそのままにしました。
+もう少し言うと、元コードにあったグローバル変数 int bangou; はユーザが意識するようなデータではありません。「今、"hello" をタイプさせられているからには bangou は１になってるはずだ」なんて考えません、bangou の存在さえ意識しないでしょう。bangou は処理内部のための、隠れた存在です。
+一方、 配列 word はユーザが否応なく目にするデータ（"hello"などの文字列）を抱えているので、プログラムの外観＝外部仕様に関わるデータと言えます。その分**他の変数よりエライ**、グローバルにするなら bangou より word が相応しい・・・みたいな気持ちです。

少し修正

2021/04/03 00:01

投稿

rubato6809

スコア1380

test CHANGED Viewed

@@ -68,7 +68,7 @@
 「for (int i = 0; i < 10; ++i) 」と書けませんか？
-それと関係して、変数はできるだけ**変数のスコープを意識**したほうが良いです。使う場所の近くで変数定義する、使う場所の近くで初期値を代入する（score = 0; と score += 10; をできるだけ近づける等）といったことです。
+それと関係して、変数はできるだけ**変数のスコープを意識**したほうが良いです。その変数が必要なブロック内で変数定義する・使う場所の近くで変数定義する、使う場所の近くで値を代入する（score = 0; と score += 10; をできるだけ近づける等）といったことです。
@@ -90,7 +90,7 @@
 ```
-という構造は見直すべきです。こうすれば良い。
+という構造・break 文の位置は見直すべきです。こうすれば良い。
 ```
@@ -134,7 +134,7 @@
         for (int i = 5; i > 0; --i) {
-            printf("(((((------> %d <------)))))\n", i);
+            printf("(((((------> %d <------)))))\r", i);
             WAIT_SECONDS(1);                // １秒間隔で繰り返す
@@ -144,13 +144,25 @@
-・・・といった辺りで、関数に分割することはしてないし、変数名などもう少しブラッシュアップしたいなど心残りがあるけど、書き直してみました。
+・・・といった辺りで、関数に分割することはしてないし、マジックナンバを整理したいし、変数名などもう少しブラッシュアップしたいなど心残りがあるけど、書き直してみました。N_WORDS はタイプさせる文字列の数を増やしたい場合もあるかなという、よくある工夫です。
-```
+```
+#include <stdio.h>
+#include <stdlib.h>
+#include <time.h>
+#include <string.h>
+#include <unistd.h>
-#define WAIT_SECONDS(n)        usleep((n) * 1000000)
+#define WAIT_SECONDS(n)    usleep((n) * 1000000)
 #define N_WORDS     (sizeof(word) / sizeof(word[0]))
@@ -158,7 +170,7 @@
     "hello",    "world",    "impossible",  "knowledge",  "capital",
-    "overcome", "analysis", "station",     "linux",      "misunderstanding",
+    "overcome", "analysis", "station",     "linux",  "misunderstanding",
 };
@@ -198,7 +210,7 @@
         for (int i = 5; i > 0; --i) {
-            printf("(((((------> %d <------)))))\n", i);
+            printf("(((((------> %d <------)))))\r", i);
             WAIT_SECONDS(1);                   // １秒間隔で繰り返し