文字列についていい方法はありますか？

Question

マイコンでプログラムを作っています。
配列aに文字を入力し、それを７セグに表示したいです。
表示自体は出来ています。
なので、文字列を配列に入力する方法を教えてください。
（数字以外を表示します。）

まず、現在は
char *a[6];
a[0]="12345";
seg(*a);//７セグ表示用関数

このようなソースを書きました。これは希望通りの動作してます。
しかし、例えば
a[0] = "12";
a[2] = "345";
や、
a[0] = "12" + "345"
のように、文字列どうしをくっつけるような使い方が出来ないものかと悩んでいます。

最悪の場合
char a[5];
a[0] = '1';
a[1] = '2';
a[2] = '3';
a[3] = '4';
a[4] = '5';
と、このような方法を使おうと思ってますが、あまり好きじゃないです。
出来ないのは無知なせいなのか、方法は有るが最後の方法が一番マシなのか、もっといい方法があるなら何を使えばいいのか、等アドバイスをお願いします。

よろしくお願いします。

shuyamakawa · Accepted Answer

#4,#5です。とりとめなくいろいろ書きます。

> その危険と言うのが
> strcpy( a, "1234567" );
> となる事があるようにプログラムを書いてしまった時のみの危険なのか、
こっちです。

文字数が分かっているのならstrcpyよりもmemcpyを使用した方が速いかも
しれません。
> memcpy( a, "12345", 5 );

一般的にライブラリ関数は、エラーチェック等のさまざまなオーバヘッドが
含まれるものなので、
スピードを気にするならエラーチェック無しの自作のシンプルな代替関数を
作って速度比較すると良いかもしれません。

#2さん回答のお礼部分を見ると、
「char a[ 6 ];」と「char *a[ 6 ];」の違いが分かっていないような気がします。
二者は明らかにデータ構造が違います。
> 俺も良く分からないんですが、例えば
> a[0] = "12";
> a[2] = "345";
> printf("%s",*a);
> とやると実行結果は
> 12
> となってしまい、345が表示されません。
>
> マイコンの場合は
> 12???
> と、？の部分には何が入るか分からないような状態になってしまいます。（とりあえず今やったの
> でした。）
>
> たぶんですが、a[0]に"12"のアドレスを渡しただけなので、seg()が受け取った配列にはa[0]の先
> スから５文字分が入力されていて、"345"が行方不明になってるんじゃないかと思います。
これは「char *a[6];」の場合で、「printf( "%s", a[2] );」とすると"345"が表示されると思います。
a[0] → "12"
a[1] → 不明
a[2] → "345"
a[3] → 不明
a[4] → 不明
a[5] → 不明
です。
やはり意図しているのは、「char a[6];」のほうで、
a[0] → '1'
a[1] → '2'
a[2] → '3'
a[3] → '4'
a[4] → '5'
a[5] → '\0'
こういう構造を望んでいるんじゃないかという気がします。

shuyamakawa · Answer

#4,#5,#8,#11です。本題からそれちゃってすみません。

推測ですが、組み込みのmemcpyは以下のような方法(の組み合わせ)で
コンパイラが最適化(高速化)しているのではないかと思います。

(1)関数のインライン展開
  関数呼び出し(call)せずに、呼び出し部分に処理を直接展開することで、
  関数呼び出しによるオーバヘッドが無くなります。(あまり差は出ないですが)

(2)ループ展開
  ループ回数が少なく、かつ固定回数の場合、ループ展開してしまうことが
  出来ます。memcpyの場合、以下のようにコンパイラが展開してしまうことで
  高速化できます。
  (これは、ループ回数を、大きい数にしたり、変数で与えたりすることで
  無効になるかもしれません。)

|memcpy( a, "12345", 5 );
↓
|a[0] = '1';
|a[1] = '2';
|a[2] = '3';
|a[3] = '4';
|a[4] = '5';

(3)アセンブラのメモリ転送命令に置き換え
  memcpyに対応するアセンブラのメモリ転送命令があるのかもしれません。

shuyamakawa · Answer

#4,#5,#8です。

> ＞「char a[ 6 ];」と「char *a[ 6 ];」の違いが分かっていないような気がします。
> 違いが分かっていませんでした。
> と言う事は、「char *a[6];」でやってた事は「char *a;」でも同じ動作をしてたと言う事ですね。
そうです。

#8で書いた、
> 一般的にライブラリ関数は、エラーチェック等のさまざまなオーバヘッドが
> 含まれるものなので、
> スピードを気にするならエラーチェック無しの自作のシンプルな代替関数を
> 作って速度比較すると良いかもしれません。
ですが、
ちょっと作ってみました。

void my_memcpy( char* dst, char* src, int count ) {

    while ( --count >= 0 ) {
        *dst++ = *src++; 
    }
}

int main( void ){ 
    char *a = "0123456789";
    char b[ 10 ];

    my_memcpy( b, &a[ 2 ], 3 ); 

    printf( "a=%.10s
", a );
    printf( "b=%.10s
", b );

    return 0;
}

sis2006jp · Answer

#7です。
＞初期化しておくと何かメリットがあるんでしょうか？
char *a[6];
は文字列６個のアドレスを確保したという意味ですから。文字列領域を確保したわけではありません。宣言と同時に初期化をすると、その文字数で領域を確保してくれます。strcpyは確保されている文字数を超えてのコピーは保障してません。エラーの原因となります。
いっそのこと、char a[6][10];とでもしておけば、9文字(1文字は文字列の終了'\0'で使用）の文字列を6個確保したことになります。各文字列はa[0],a[2],・・・a[5]で使えますので、strcpyなりstrcatなり9文字以内なら自由に使えます。他の方も指摘してますがポインタ操作は気をつけないとエラーの原因となります。以上です。

wohaokele#2 · Answer

こんばんわ。
私も質問者さんは、ポインタと配列の事、少々誤解されているように思いますが。それはさておき...

もし、表示される文字列がコンパイル時で定数なら、何も a を介す必要は無いのではと思います。seg("12"); や seg("345"); で良いじゃないでしょうか? これなら コンパイル時の演算と言うことで seg("12"  "345"); とすば、"12345" と結合もしてくれますよ。

もし、実行時に動的に文字列を構築したいなら、他の方も仰せの通り、strXXX()系か、Xprintf()系の関数を使うのが簡単ですが、質問者さんも仰せの通り、組み込み関係では Xprintf()系自体実装されてない場合もあるようですね。

で、そうなると自分で文字列を構築する関数を用意すると言うのが多分妥当かと思うのですが、真面目に Xprintf() 系の実装なんかはやってられないと思います。

ですが、もしも構築される文字列が数字列に限定されるならこんな安直な実装は如何でしょうか?
--
char *to_str(char *dest /* 6byte 以上あること */, int val)
{
　char *p;
　int d, n;
　for (p = dest, d = 10000 /* 5桁限定 */ ; d > 0; d /= 10)
　{
　　n = val / d;
　　*(p++) = (char)n + '0';
　　val -= n * d;
　}
　*dest = '\0';
　return dest;
}
--
こうすれば、char a[6]; とでもしておいて、seg(to_str(a, 123)); とかで、 "123" が得られると思います。

sis2006jp · Answer

初期化で６つの文字列が代入できます。最大文字数で仮に入れておいたらどうですか。 #include #include int main() { char *a[6]={"12","345"}; printf("%s ",a[0]); printf("%s ",a[1]); //連結 strcat(a[0],a[1]); printf("%s ",a[0]); return 0; }

noname#22058 · Answer

#2です。
文字列の連結には、sprintf関数を使います。

#include <stdio.h>

int main(void)
{
char *a[] = { "12", "", "345" };
char str[20];

sprintf(str, "%s%s", a[0], a[2]);
printf("%s
", str);
return 0;
}

shuyamakawa · Answer

#4です。補足です。

「char a[6];」で正しいとしたら、ソースはこのようになります。
> char a[6];
> strcpy( a, "12345" );
> seg(a);//７セグ表示用関数

厳密にいうと「strcpy」は、配列境界を越えてしまう危険があり、
それを避けるためには、このように置き換えた方が良いです。
> strncpy( a, "12345", sizeof( a ) );
> a[ sizeof( a ) - 1 ] = '\0';

shuyamakawa · Answer

> まず、現在は
> char *a[6];
> a[0]="12345";
> seg(*a);//７セグ表示用関数
> このようなソースを書きました。これは希望通りの動作してます。

「char *a[6];」では、ポインタの配列を宣言していますがそれは意図どおりですか？
(char型変数を指すポインタ変数が、配列で6個宣言されています。 )

もしかしたら、こうじゃないでしょうか？
> char a[6];
(char型変数が、配列で6個宣言されます。)
このようにすれば、質問の後半部分も、普通に質問者さんの意図通りに行くんじゃないでしょうか？

chirubou · Answer

私だったら、

void segall( char *str ) {
    while( str != NULL ) seg( str++ );
}

として、

char *mojiretu[] = {
"hoge",
"moge",
NULL
};

segall( mojiretu );

とします。これで "hogemoge" というように表示されるハズです。

こういうことがやりたかったのかな？

文字列についていい方法はありますか？

#4,#5です。

#4,#5,#8,#11です。

#4,#5,#8です。

#7です。

こんばんわ。

初期化で６つの文字列が代入できます。

#2です。

#4です。

> まず、現在は

私だったら、

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング