ポインタを使って関数内で文字列を操作する方法について

Question

文字列から数字を除去して表示する関数を作成したのですが、
関数内では数字が正しく除去されているのですが、
メインプログラムで関数を呼びだしても数字が除去されません。
どこがおかしいのか教えていただけないでしょうか？

#include <stdio.h>
#include <string.h>
#include <ctype.h>

void digit(char* s){
    char tmp[256];
    int i,j = 0;
    for(i=0; i < strlen(s); i++){
        if (!((*(s+i) >= '0') && (*(s+i) <= '9')))
            tmp[j++] = *(s+i);
    }
    tmp[j+1] = '0';     //末尾にNULL文字を追加
    s = tmp;            //ポインタが配列tmpを指すように変更
    printf("関数内のポインタSが指す文字列は%s\n", s);
 }
int main(void){
    char* str = "abc4de5fg";
    digit(str);
    printf("メイン関数のポインタstrが指す文字列は%s", str);
    return 0;
}

Answer 1

ここが問題です。

s = tmp; //ポインタが配列tmpを指すように変更

ポインタ変数自体を関数内でいくら書き換えようと、呼び出し元には影響しません。
簡単に対応できそうな方法は一つ。

• ポインタのポインタを渡す 非推奨。理由はコメント欄に記載しています。

あるいは、strを文字型配列として確保すれば(註)、次の二つの方法も選べます。

• tmpを使わず、sが指し示す先を直接書き換える
• strncpy

註: 文字列リテラルで確保した領域は読み取り専用だった筈。

---

ついでに、ここも変です。

tmp[j+1] = '0'; //末尾にNULL文字を追加

tmp[j] = '\0' としてください。
'0'はただの数字0であって、NULL文字とは呼べません。

また、jの値がループ後にどのようになっているか確認した方が良いでしょう。
現状のコードではtmp[j]の値が不定です。

Comments

[err0r]

丁寧に解説いただいてありがとうございました。
ポインタを関数に渡す場合でもコピーした値になるんですね。
関数ヘ渡す引数にポインタstrのポインタ(int** pstr)を追加して、
関数内の処理を、*pstr = tmp　にすることで正しく動作するようになりました！NULL文字についての情報もありがとうございました。
ループの最後にtmp[j++] が評価されてjがインクリメントされて不定になるという事ですね。配列は宣言時に初期化するようにします。

[LouiS0616]

一個だけ回答に致命的な抜けがありました。
tmpの生存期間はdigit関数内ですので、そのポインタを利用するならばグローバル変数にするなりstaticにするなりして延ばしてやる必要があります。

[LouiS0616]

ただしこの場合も、複数回関数を呼び出した際の動作はおそらく期待しないものになります。
やはりポインタのポインタを利用するのではなく、strを配列として宣言しその中身を書き換えるか、受け取り用の配列を呼び出し元で用意した方が良いでしょう。

[err0r]

補足ありがとうございました。おっしゃる通りですね。関数で定義した値を指すよりも配列の中身を書き換えたほうがコードの依存性がないので簡潔ですね。勉強になりました。

Answer 2

既に解決済みとなっていますが、こんな感じ?

void digit(char* s)
{
	char *src = s;
	char *dst = s;
	while (*src != '\0') {
		if (!isdigit(*src))	*dst++ = *src;
		src++;
	}
	*dst = '\0';
    printf("関数内のポインタSが指す文字列は%s\n", s);
}

int main(void)
{
    char str[] = "abc4de5fg";	// []で宣言するので、書き換え可
    digit(str);
    printf("メイン関数のポインタstrが指す文字列は%s", str);
    return 0;
}

Comments

[err0r]

ありがとうございます。
同じ処理でもこんなに簡潔に記述できるんですね。。
同じ所を指すポインタを複数作れるのは知りませんでした。
次回から参考にさせていただきます！