Visual Studioでscanf_s後にprintfが表示されない問題の原因と対処法を教えてください

Question

実現したいこと

・C言語で構造体の配列を使い、5人分の社員情報（名前・年齢・番号など）を入力し、最後にその情報をprintfで表示したい。

発生している問題・分からないこと

・scanf_sで情報を入力した後、printfで表示しようとしても、何も出力されない。

・プログラムはエラーも警告も出さず、正常終了しているように見える。

・printfの出力がまったくされず、デバッグの手がかりがない。

該当のソースコード

typedef struct employee_info
{
    short num;
    char name[20];
    char department[4];
    short phone;
    short age;
    int salary;
} employee_info;

int main(void)
{
    employee_info person[5];
    employee_info* p = person;
    for (int i = 0; i < 5; i++) {
        printf("%d件目\n", i + 1);
        printf("従業員番号: ");
        scanf_s("%hd", &p[i].num);
        printf("名前: ");
        scanf_s("%s", p[i].name, sizeof(p[i].name));
        printf("部門: ");
        scanf_s("%s", p[i].department, sizeof(p[i].department));
        printf("年齢: ");
        scanf_s("%hd", &p[i].age);
        printf("内線番号: ");
        scanf_s("%hd", &p[i].phone);
        printf("給与: ");
        scanf_s("%d", &p[i].salary);
    }

    for (int i = 0; i < 5; i++) {
        printf("%d人目: %s\n", i + 1, p[i].name);
        printf("年齢: %hd\n", p[i].age);
        printf("従業員番号: %hd\n", p[i].num);
        printf("部門: %s\n", p[i].department);
        printf("内線番号: %hd\n", p[i].phone);
        printf("給与: %d\n", p[i].salary);
    }

    return 0;
}

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

試したこと
scanf_sの第3引数にsizeof(p[i].name)のようにサイズを正しく指定。

数値の読み込みには&をつけてアドレス渡しをしている。

入力の直後にwhile (getchar() != '\n');を入れて、バッファの改行を処理。

fflush(stdout);をprintfの後に書いてみたが効果なし。

fgetsへの切り替えも試したが、入力のタイミングがずれたり改行が残ったりしてうまくいかず断念。

入力・出力の箇所を単純化してみたが、現象は変わらず。

補足

使用環境は Visual Studio。

構造体の定義やループ構造自体にはエラーは出ておらず、入力時に明らかな異常は見られない。

printfが実行されているかのデバッグが非常に難しい状態。

Comments

[zikyu1113yen]

初めてまだ２か月でまだわからないことばかりですが、どなたかご指導よろしくお願いします。

[zikyu1113yen]

ちなみにstructはテキストの指定された名前、指定された大きさで設定しています

[TakaiY]

質問は投稿した後でも編集できますので、追加の情報などは質問を編集して追記したほうがいいですよ。

[TakaiY]

> printfで表示しようとしても、何も出力されない。
何も出力されないとはどういう状況でしょうか？

出力されないというのは、最後の方の「for (int i = 0; i < 5; i++) {」のループの中のことだと思いますが、その中の年齢なども表示されないのでしょうか？

> デバッグの手がかり
とありますが、デバッガでステップ実行して途中の変数の内容を確認したりしていますか？

[zikyu1113yen]

>何も出力されないとは、具体的にどのような状況でしょうか？
そうなんです……本当に何も表示されないんです。

>デバッグの手がかりについて
申し訳ありません、私の説明不足でした。
デバッグして確認したところ、下の for 文で処理している値が、何も入力されていないときに表示される数値とまったく同じでした。

追加情報で申し訳ないのですが、上の「名前」や「部門」の入力欄を削除して実行してみたところ、下の for 文はかなりうまく動作するようなんです……

[jimbe]

scanf_s の戻り値をチェックしてみては如何でしょうか。

[fana]

（言うまでもないような話ですが）私ならとりあえず各 scanf_s の直後に入力されたハズの物をそのまま出力する printf を入れてみたらどうなるのか？　とかを真っ先に試してみるかな，とか思いました．

あとは，何が起こってるのかわからない場合には，私ならとりあえず物事を単純化してみるかな，とか．
「要素を減らす」というか．例えば，

* person[5] じゃなくて person としたプログラム（配列無し：データ１つだけ）でも同様のことが起こるのか？
* struct無しで，単純に num とか name とかいう変数がバラで存在している世界でも同様のことが起こるのか？
* numだけとかnameだけみたいに，データの種類を減らしてみたらどうなるのか？
* sizeofみたいなのはとりあえず「この値になるハズだよね」っていう値をダイレクトに書いてみる
* etc

とにかく色々と簡素化していくとどこかの時点で問題が起きなくなるのであれば，そこで何かヒントが得られるんじゃなかろうか？　っていう話．

[little_street]

改善策になるかはわかりませんが，もし「部門」への入力が半角4文字以上または全角（漢字，かな等）2文字以上なら，そのための領域（char department[4];）を増やす必要がありそうです。全角を入力する場合はエンコーディング方式も関係するので一概には言えませんが，全角1文字4バイト＋半角1文字1バイト＋終端用1バイトで設定してみてください。

[hiroki-o]

Windows 11、Visual Studio 2022で確認しましたが、再現しませんでした。
下記の情報を「質問を編集して」追記してください。
- Windows 11か10か
- Visual Studioのバージョン(2022,2019...)
- ソースファイルの拡張子は、*.cか*.cppか(ソースはC言語ですが、C++でビルドしていないか)
- includeが省略されているが、#include <stdio.h>でよいか
- ビルドは64bitか32bitか
- 入力したテストデータの例

[YT0014]

Windows 11 Pro(64bit、24H2、26100.4061)。VisualStudio2022(17.10.5)で現象を確認
＝＞ごめんなさい、誤認かも。正常動作を確認しました
なお、入力値は、全て半角英数1文字
名前と部門用のscanf_s()で、警告あり

[cametan]

ごめん。バカバカしいたぁ思うけど一応確認を取る。
提示コードは完全コピペ?まさか、とは思うけど#include <stdio.h>を書き忘れた・・・ってこたぁ無いよね?

[YT0014]

＞zikyu1113yen
質問編集にて、テスト時の入力値と、可能であれば実際の画面のスクリーンショット画像も、ご提示ください
また、私がやった誤認ですが、画面スクロールに気づかず、出力表示を、入力時表示と間違えている可能性はないですか？

[cametan]

あと、もう一つ。
バカバカしいたぁ思うけど、「出力されたのに出力されてない」って思い込むケース。
これもあるんだ。
特にこのプログラムのように、入力のプロンプトと出力に大して差がない場合、一瞬で出力が終わっちゃって「出力された」と気付かない。
んで意外とパソコンの、特に端末(DOS窓)のスクロールがカクカクしててスムーズじゃねぇんだよな。だから「出力が増えてても気付かない」って事があり得るんだ。
出力のループの直前に、例えばputs("----------------");とか入れておいて、「これ以降は出力」ってサインを入れておけばどうなのか気づきやすくなる。

Answer 1

これ、難しいのがなぁ。

まず、C11以降で確かにscanf_sってのが定義されている。
一方、それってオプショナルで、事実上、Microsoftの処理系でしか実装されていない(※)。
と言う事はポータブルじゃない、って事になるんだ。

もう一つの問題が「どうやって字数を数えてるのか」。
これって、かなりややこしいんだよ。
基本的に貴方の書いてるコードって「入力がASCIIコード前提」って話になるんだ。っつーかならざるを得ない。
つまり、仮に日本語入力、って前提になると、もう型からして違う、って話になるわけ。
いや、実質上、この辺はコンパイラによって、独自に解決してくれるケースも多いんだけど、その辺実装依存になる、っつーか・・・・・・。
つまり、Microsoftの処理系が「どう解釈するのか」ってのを熟知せなアカン、って話になるんじゃないかなぁ・・・。
多分。
C11以降でUTF-8を扱えるようになってる筈、なんだけど、この辺、甚だ不安定なんだよな。仕様上(苦笑)。
よって、前提としては、(日本と違って世界的には)デファクトスタンダードなC99、あるいはJIS C規格を考慮すると、そもそもwchar_tを使わな「数えられない」と思う。
そしてその辺がかなりややこしいんだ。

生憎、僕はそんなにCが得意じゃない(どころか嫌いだ・笑)んだけど、調べてみて、ポータブルに書く、って前提だと次のようになるんじゃなかろうか。
詳しい話は識者の人(Cが得意な人)が解説してくれれば嬉しいんだけど(人任せ・笑)・・・。

#include <stdio.h>
#include <stdlib.h>
#include <inttypes.h>
#include <wchar.h>
#include <locale.h>

#define N 5

static wchar_t buffer[41];

typedef struct employee_info
{
  int16_t num;
  wchar_t name[21];
  wchar_t department[5];
  int16_t phone;
  int16_t age;
  int32_t salary;
}
  employee_t;

int main(void)
{
  employee_t p[N] = {0};

  setlocale(LC_ALL, "");

  for (size_t i = 0; i < N; i++)
    {
      wprintf(L"%" PRIu64 L"件目\n", (uint64_t)(i + 1));

      wprintf(L"従業員番号: ");
      fgetws(buffer, 5, stdin);
      buffer[wcscspn(buffer, L"\n")] = L'\0';
      p[i].num = (int16_t)wcstol(buffer, NULL, 10);

      wprintf(L"名前: ");
      fgetws(buffer, 21, stdin);
      buffer[wcscspn(buffer, L"\n")] = L'\0';
      wcscpy(p[i].name, buffer);

      wprintf(L"部門: ");
      fgetws(buffer, 5, stdin);
      buffer[wcscspn(buffer, L"\n")] = L'\0';
      wcscpy(p[i].department, buffer);

      wprintf(L"年齢: ");
      fgetws(buffer, 3, stdin);
      buffer[wcscspn(buffer, L"\n")] = L'\0';
      p[i].age = (int16_t)wcstol(buffer, NULL, 10);

      wprintf(L"内線番号: ");
      fgetws(buffer, 5, stdin);
      buffer[wcscspn(buffer, L"\n")] = L'\0';
      p[i].phone = (int16_t)wcstol(buffer, NULL, 10);

      wprintf(L"給与: ");
      fgetws(buffer, 10, stdin);
      buffer[wcscspn(buffer, L"\n")] = L'\0';
      p[i].salary = (int32_t)wcstol(buffer, NULL, 10);
    }

  for (size_t i = 0; i < N; i++)
    {
      wprintf(L"%" PRIo64 L"人目: %ls\n", (uint64_t)(i + 1), p[i].name);
      wprintf(L"年齢: %" PRId16 L"\n", p[i].age);
      wprintf(L"従業員番号: %" PRId16 L"\n", p[i].num);
      wprintf(L"部門: %ls\n", p[i].department);
      wprintf(L"内線番号: %" PRId16 L"\n", p[i].phone);
      wprintf(L"給与: %" PRId32 L"\n", p[i].salary);
    }

  return EXIT_SUCCESS;

}

wchar.hってヘッダで定義された関数を多用するのが一番信頼性が高いんじゃないか。
んで、読み込みはscanfとかwscanfを使うよかfgetwsを使う。バッファbufferを定義してそこに文字列として読み込むんだけど、取り込んだ改行文字(L"\n")を、ワイド文字の終端文字、L'\0'へと変換する。=> L"\n"の位置検索にwcscspnを使用
その後、それを数値データに変換する(wcstol)なり、ワイド文字列とするなり、で構造体のスロットに代入するとかコピー(wcscpy)する。
かなり面倒くさいんだけど、多分これならどのコンパイラでも動いてくれるんじゃないかな。恐らく。多分。

※: 外野から見ると、Microsoftが提案した機能をCの仕様策定委員会が「敢えて」オプションにしたり、または他のグループが提案したブツもまたオプション化して、Microsoftがそれを実装しなかったり、と、策定委員会がMicrosoft vs. 「その他」に分裂してるように見える。
全く、「なにやっとんねん」と言うカンジだ(笑)。何のための標準化委員会なんだかサッパリ分からん(笑)。
いずれにせよ、「その他」にとってはMicrosoftは「怖い」会社なんだろう。

なお、日本国内では公式ではC言語 = C99なんだけど、長い間Microsoftはキャッチアップして来なかった。現在では最新のC言語仕様はC23と呼ばれるモノだが、Microsoftでは、知ってる限り、現在ではC17に対応はしている。

参考: Visual Studio に C11 および C17 サポートをインストールする

また、C/C++のライブラリ関数に付いては、IBMのページが良くまとまっている。

Answer 2

Windows 11、Visual Studio 2022で確認しました。
コメントに書いた通り、こちらの環境では現象は再現しませんでした。

以下、今回の現象と直接関係無いと思いますが、修正したほうがよい箇所を指摘します。

標準入出力ライブラリーの明記

#include <stdio.h>

変数のゼロ初期化

    //employee_info person[5];
    employee_info person[5] = {0};

文字列を格納する領域と、NULL終端の意識

        //scanf_s("%s", p[i].name, sizeof(p[i].name));
        scanf_s("%19s", p[i].name, (unsigned)sizeof(p[i].name));
        rewind(stdin);

        //scanf_s("%s", p[i].department, sizeof(p[i].department));
        scanf_s("%3s", p[i].department, (unsigned)sizeof(p[i].department));
        rewind(stdin);

Answer 3

Visual Studioでscanf_s後にprintfが表示されない問題の原因と対処法を教えてください

原因はあなたの書いたコードの不具合があるため、ですね
もっと勉強して、そういう不具合がない正しいコードが書けるようになりましょう。