テキストを指定の文字数で切り詰めて表示するプログラムを作りました。このプログラムは正常に動いているように見えることもありますが、実際には重大な誤りを含んでいます。場合によっては、クラッシュしてしまうかもしれません。
どこが間違っているのか分かりますか?
main.c
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
void PrintTruncated(const char *text, size_t length) {
char truncatedText[length];
strncpy(truncatedText, text, length);
printf("length = %zu: %s\n", length, truncatedText);
}
int main(void) {
for (int i=4; i<=8; i++) {
PrintTruncated("Hello!", i);
}
return EXIT_SUCCESS;
}
期待される実行結果
length = 4: Hell
length = 5: Hello
length = 6: Hello!
length = 7: Hello!
length = 8: Hello!
切り詰められた文字列を
strncpy()
で作っているのが分かるかしら?
はい。そこは分かったんですけど……。
質問がありそうね?ただいま受け付け中よ。
それじゃ、遠慮なく。配列の長さを変数で指定してるところがありますよね?
truncatedText
の長さが固定値じゃなくてlength
になってるけど……。
char truncatedText[length];
配列の長さって、固定値じゃなくてもいいんですか?
いいのよ。これは「可変長配列」というやつね。
へぇー!そんな機能があるんですね。
じゃあ、それをふまえて何が問題か考えてみてね。
はーい!
strncpy()
の仕様を確認してみましょう。