文字列の左右のスペースを削除する関数を作っています。
そこで引数の文字列と同じ領域のメモリを確保し、そこで一時的にスペースを削除する作業をしています。
作業が終わったら、引数のポインタに文字列をコピーし、作業領域を解放したいと思っています。
しかし、確保した領域を解放しようとすると、「セグメンテーション違反」なるエラーが出て、解放できません。どなたか分かる方ご教授お願いします。
void trim(char *str)
{
int i;
int len;
char *tmp_p, *save_p;
tmp_p = (char *)malloc(strlen(str) + 1);
strcpy(tmp_p, str);
save_p = tmp_p;
len = strlen(tmp_p);
// 後方スペース削除
// ポインタを最後まで進める
for(i = 0; i < len - 1; i++)
{
tmp_p++;
}
// 文字だった場合'\0'でターミネート
while(tmp_p != save_p)
{
if(!isspace(*tmp_p))
{
tmp_p++;
*tmp_p = '\0';
break;
}
tmp_p--;
}
// 前方スペース削除
// ポインタを最初に戻す
tmp_p = save_p;
// 文字が出てくるまでポインタを進める
while(isspace(*tmp_p)) tmp_p++;
strcpy(str, tmp_p);
//free(tmp_p); // <- セグメンテーション違反
}
No.3ベストアンサー
- 回答日時:
free(save_p);
とすればいいですが、基本的にこういう書き方は紛らわしいですよね。
以下のようにするのが間違いが少ない書き方だと思います。
(save_pは取得値を保持し、tmp_pはワーク用とする)
char *tmp_p, *save_p;
save_p = (char *)malloc(strlen(str) + 1);
strcpy(save_p, str);
tmp_p=save_p;
(以下、そのまま)
free(save_p);
そうですね。確かに紛らわしいです。
ご指摘の通り修正したほうがいいですね。
早速修正したいと思います。
ご回答ありがとうございました。
No.2
- 回答日時:
while(isspace(*tmp_p)) tmp_p++;
が犯人です
tmp_p = save_p;
free(tmp_p);
とすれば大丈夫ですけど・・・
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
似たような質問が見つかりました
- C言語・C++・C# C言語プログラム変更 2 2022/12/21 15:03
- Visual Basic(VBA) vba 重複データ合算 5 2023/07/05 18:55
- C言語・C++・C# 質問です 下記のコードを分かりやすく解説お願いします 初心者です #include ‹stdio.h 3 2022/05/26 22:03
- UNIX・Linux cronの@rebootでのdateコマンドの実行につきまして 2 2023/06/11 16:23
- Excel(エクセル) エクセルVBAでオブジェクトが必要です 2 2022/09/10 16:37
- C言語・C++・C# プログラミングの授業の課題です 1 2023/01/17 22:15
- Excel(エクセル) エクセルのマクロについて教えてください。 1 2023/02/11 11:05
- Visual Basic(VBA) VBA GetAttrについて教えてください 2 2022/12/22 15:25
- Excel(エクセル) マクロのコードを、少しでも削って短くしたい 3 2022/08/30 07:46
- Visual Basic(VBA) VBA シート上にドロップダウンリストを作り、予め指定値をセットしたいのですが 1 2023/03/25 15:15
関連するカテゴリからQ&Aを探す
おすすめ情報
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
C言語のintとcharの違いってな...
-
fgetc( )の戻り値はなぜ整数??
-
char型にint型の数値を代入する。
-
小数点入りの文字列をfloat型に...
-
文字の色
-
C言語のプログラムについてです
-
文字列の途中から途中までを抽出
-
メモリの解放について
-
【速いブラインドタッチ】手を...
-
VBAにてcolorindexを変数に格納...
-
日付チェック関数について
-
C言語 配列の長さの上限
-
関数から配列を返すには?
-
擬似コード 長さがmの配列でな...
-
CImage GetBitsメソッドについて
-
配列の要素数に変数を入れたい...
-
ヘッダファイルと構造体
-
callocの処理速度
-
mallocについて
-
【エクセルVBA】Functionの引数...
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
char*を初期化したいのですが
-
CStringからchar*への型変換に...
-
C言語のintとcharの違いってな...
-
C言語にて構造体のメンバがNULL...
-
小数点入りの文字列をfloat型に...
-
strcat関数を自作したいです
-
C言語のプログラムについてです
-
const char* s1とただのchar s1...
-
DWORDとcharの変換
-
char型にint型の数値を代入する。
-
文字列の途中から途中までを抽出
-
new charとnew char[N]の違いは?
-
文字列内の数字削除
-
csvファイルをfscanfで読み込む...
-
fgetc( )の戻り値はなぜ整数??
-
char 文字列型 の表現範囲が-12...
-
fstream型オブジェクトを関数の...
-
エクセルのMID関数は、C言語では?
-
ポインタを使って回文かどうか...
-
ポインタ配列
おすすめ情報