C sprintfの使い方について

Question

前提・実現したいこと

C言語についての質問です。
sprintfを使用して、1×10の＊を出力するプログラムを作成しているのですが、正しく表示させることが出来ず困って今います。
sprintfを使用して、char型text3の中身を作成したいです。

発生している問題・エラーメッセージ

＊を1*10に表示したいが、コンソール画面に＊一つのみが表示され正しく表示されない。

該当のソースコード

#include <stdio.h>

int main(){
    int x, y;
    char text1[256];
    char text2[256];
    char text3[256];
    char  str1 = '*';
    char *str2= "\n";
    
    for(x=0;x<1; x++){
        for(y=0;y<10; y++){
           sprintf(text1,  "%c", str1);
        }
        sprintf(text2, "%s", str2);
    }
    sprintf(text3, "%c%s", text1, text2);
    printf(text3);

    return 0;
}

『表示したいもの』
＊＊＊＊＊＊＊＊＊＊

試したこと

私の書式が間違っているため、検索等してみましたが解決できませんでした。

補足情報（FW/ツールのバージョンなど）

MacOS Big Sur 
terminal zsh バージョン2.11  
gcc -v Homebrew GCC 11.1.0_1 
vs code for Mac 1.57.1

Answer 1

sprintf を使うという条件なんですね。
元のコードをできるだけ活かしたものにしてみました。

#include <stdio.h>

int main(void)
{
    int x, y;
    char text1[256];
    char text2[256];
    char text3[256];
    char str1 = '*';
    char *str2 = "\n";

    for (x = 0; x < 1; x++) {
        for (y = 0; y < 10; y++) {
            sprintf(text1 + y, "%c", str1);  // ★ '*' を入れる場所を変更
        }
        sprintf(text2, "%s", str2);
		sprintf(text3, "%s%s", text1, text2);  // ★ text1 は文字列だから "%s"
		printf(text3);
    }  // ★ for の範囲を変更。x < 3 にすると 3x10 の '*' になる

    return 0;
}

追記
sprintf にこだわらなければ簡単です。

1文字ずつ出力すると、

#include <stdio.h>   // putchar

void draw(int x, int y)
{
	for (int i = 0; i < x; i++) {
		for (int j = 0; j < y; j++) putchar('*');
		putchar('\n');
	}
}

int main(void) { draw(3, 10); }

行単位で出力すると、

#include <stdio.h>   // puts
#include <string.h>  // memset

void draw(int x, int y)
{
    char text[256]; // 可変長配列が使えるバージョンの C なら、char text[y+1];
	memset(text, '*', y); // for (int i = 0; i < y; i++) text[i] = '*';
	text[y] = '\0';
	while (--x >= 0) puts(text);  // printf("%s\n", text)
}

int main(void) { draw(3, 10); }

Comments

[退会済みユーザー]

こんばんは
ご回答が有難うございます。
c言語初学者なため、自分自身でお題を作って勉強しています。
そのため、的外れな問題だったかもしれません。
そもそも今回、私がやりたかった事に、sprintf縛りというのは無理があったのでしょうか？

[kazuma-s]

回答のコードは試していないのですか？
sprintf縛りで出来ているでしょう。
効率は良くないけど、動くはずです。

[退会済みユーザー]

もちろん実行させて頂きました。
その上で、sprintf縛りだとかなりコードが長くなってしまう印象を受けたので、今回私がやりたかった事には適切なコードではないと思い、質問させて頂きました。
〉効率は良くないけど、動くはず。
まさにその通りだと思います。丁寧な回答ありがとうございました。

[dodox86]

@質問者さん

質問を読むと、前提として「sprintf縛り」に思えますよ。だから回答者の皆さん、その縛りで回答されています。

[fana]

配列の要素をふつーにいじくらずにあえて sprintf の %c で1文字ずつ突っ込むコード！
なんてのを見せられたら，そりゃ誰だって「わざと/あえて」やっていると受け取りますよね．

[jimbe]

「'*'を10個並べて表示するCプログラム選手権」を募集したらどれくらい来ますかね ^^

Answer 2

sprintf で追加の効果を得るのでしたら

sprintf(text1, "%s%c", text1, str1);

というふうに、最初に自身を書くことになるかと思います。

---

単に '*' を 10 個並べたものを表示する、ということであれば、もちろん簡単なのは

printf("**********");

ですよね。
いや、任意の数だけ並べる感じで…となれば

for(int i=0;i<10;i++) printf("*");

となります。
いやいや、変数に10個並べたのを用意してそれを表示するようなのが…でしたら

for(int i=0;i<10;i++) text1[i]='*';
text1[10]='\0';
printf(text1);

ポインタで

char *p=text1;
for(int i=0;i<10;i++) *p++='*';
*p='\0';
printf(text1);

いっそ関数で

memset(text1,'*',10);
text1[10]='\0';
printf(text1);

もあり得ます。
文字列を追加していく風？なら

text1[0]='\0';
for(int i=0;i<10;i++) strcat(text1,"*");
printf(text1);

というのも有りでしょうか。
実装依存かもしれませんが

strcpy(text1,"*");
strncat(text1,text1,10-1);
printf(text1);

も動くかもしれません。

Comments

[退会済みユーザー]

こんばんは
ご回答有難うございます。
丁寧かつ、複数のパターンを出して頂き、大変勉強になりました。
有り難うございました。

Answer 3

sprintf を使って * を並べる……最も明快な方法は，
sprintf( text3, "%c%c%c%s", str1, str1, str1, str2 );
みたいな形でしょう．
この例だと3個になりますが，10個にするならそうなるように %c と str1 を頑張って書き並べればよいですね．

↓

質問文記載のコードを見ると text1, text2 という２つのバッファが存在しているので，これを利用すれば，ほんの少しだけマシ(?)になるかもしれません．

sprintf( text1, "%c%c\0", str1, str1 );  //2個
sprintf( text2, "%s%s", text1, text1 );  //4個
sprintf( text3, "%s%s%s%s", text2, text2, text1, str2 );  //10個+改行

Comments

[fana]

「%c を複数並べるのはダメです」とかいう場合には，「２個＝１個＋１個」という操作で２個を作れば良いでしょう．
（目的の個数を半分ずつにでも分割していくようなのを作ってみれば良いんじゃないですかね．）

[退会済みユーザー]

こんばんは
ご回答有難うございます。
大変勉強になりました。

Answer 4

char* p = text1;
for(y = 0; y < 10; y++){
    p += sprintf(p, "%c", str1);
}

スマートではないですが、こういうのもあります。

Comments

[thkana]

printf系の関数は出力した文字数(それともバイト数だっけ?)を返している、というのを知らないと何をしているのか理解できないかもですね。

[退会済みユーザー]

こんばんは
ご回答有難うございます。
自分の中で落とし込むまでに時間がかかりそうですが、大変勉強になりました。
有り難うございました。

Answer 5

私の書式が間違っているため、検索等してみましたが解決できませんでした。

検索すると言うより、考える話です。考えましょう。

        for(y=0;y<10; y++){
           sprintf(text1,  "%c", str1);
        }

text1に*1文字だけ入れるという同じ事を10回繰り返しているだけなので、1回実行しようが100回実行しようが、結果は同じで、1文字だけ入ります。

       sprintf(text3, "%c%s", text1, text2);

text1は文字列なので、%cじゃなくて%sですね。

text1の中に*を10個入れるのであれば、

        for(y=0;y<10; y++){
           text1[y] = str1;
        }
        text[y] = '\0';

でしょうか。

Comments

[退会済みユーザー]

こんばんは
ご回答が有難うございます。
仮に１０×１０の＊を配置する場合どのように考えればいいのでしょうか。
text１の中に１００個の＊を入れると考えるのか
追加の質問になってしまいますが、ご教授いただきたいです。
よろしく御願い致します。

[otn]

どっちでも良いです。
・text1の中に、「10の*と改行」の10回繰り返しを入れて、それを1回出力する
・text1の中に、「10の*と改行」を1つ入れて、それを10回出力する

[退会済みユーザー]

有り難うございました。

Answer 6

sprintfを使うのが間違ってます

        for(y=0;y<10; y++){
           text1[y]=str1;
        }
        text1[y]='\0';

Comments

[退会済みユーザー]

こんばんは
ご回答有難うございます。
大変勉強になりました。