最後に確実に"\0"を足しています？ sの最後まで行ってしまった場合にしか、"\0"がbufferに付かないように見えます。そのせいではないかと。

あ・・・・すいません。あの後に
buffer[i] = '\0';
strcpy(r,buffer);
がつきます。

私はＣ言語しかわからないのでＣ＋＋はイマイチよくわからないのですが、
関数内で宣言されたローカル変数は何が入っているかわからないってことになっていたと思います。
私なんかは文字定数レベルで文字列バッファに格納する場合は必ず
memset(buffer,0x00,sizeof(buffer));
とか書いて初期化してから使うことにしています。
たぶん、それじゃないかな？

>この初期化は文字列を扱うときに必要不可欠な物なのでしょうか？
絶対必要ではないと思いますが、やっておくに越したことはないと思いますよ。
特に文字列と文字定数の扱いがごっちゃになっているときは。

buffer[i-h]='\0'ではないのですか？

一つ気になったのですが、r に返す文字列は b の文字は含まれないのかな？
なんか入らないような気がするですけど。
見間違いならごめんなさい

>char buffer[256];
>for(int i=0;i<257;i++) buffer[i]=0;
死にますよ、これでは・・・。
あと、初期化は別に必要ないです。

flag はどこで宣言しているのですが？初期化していますか？

えげつない組み方をすれば、こんなもんでどうでしょ。

void 自作関数(char *s,char *r,char b,char e)
{
char *_b, *_e;

if((_b = strchr(s, b)) == NULL) return;
if((_e = strrchr(s, e)) == NULL) return;

strncpy(r, _b, _e - _b + 1);
r[_e - _b + 1] = '\0';
}

ロクにチェックしてない（笑）

おお、こんなにレスが。ありがとうございます。
＞関数内で宣言されたローカル変数は何が入っているかわからないってことになっていたと
これですが、その通りです。下の方でcout << bufferとなっていますが、これは
あらかじめbufferに何が入っているのかを調べる物です。
これで調べると、むちゃくちゃな値が入っていました。
それを見ると、最終的に初期化の必要はないかと思ったのですが
初期化をしないとうまく動かない場合もあり、とても疑問です。

＞buffer[i-h]='\0'ではないのですか？

ですが、最終文字にヒットした場合iの値がその添え字なので
そこにヌル文字を入れています。hは最初の文字がヒットした場所です。
違いますかね？・・・

＞一つ気になったのですが、r に返す文字列は b の文字は含まれないのかな？

あっ、これですが含まれず、でokです。bとeはあくまで目印なので
そこから始める、というわけでなくその次から、というわけです。
本文おかしいですね・・・すいません。

＞死にますよ、これでは・・・。

あぅ・・・その通りですね・・・変えてきます。
アドバイスありがとうございます。

＞HIRさん
おお、このような組み方が（笑）
参考にさせていただきます。φ(．． )メモ

結果ですが、やはり初期化しないとうまくいかないみたいです。
最後の方で文字化けしている模様・・・cout << bufferで見ると
./test.cgi(π・@
このような感じになっており、うまくiの後に\0が入っていないようです。

>>初期化しないとうまくいかない

bufferに'\0'が本当に入っているかどうかデバッガ(gdbなど)で、
適当なところにブレークポイントを設定して一行づつステップ実行して
確かめるのが一番じゃないかなぁ〜。


>>あと、他のプログラムで見たのですが
>>int GetForm( FormTerm*& term )

私は見たことないです(^^;;; ポインタ値を参照で渡す? わからん(^^;

こういう話題がでてくると
「あぁ　どる仲って　寄席じゃないんだなー」と　関心したり（ぉ

＞どる仲って　寄席じゃないんだなー」と　
えっ！違うの？！Σ（￣□￣ ）ガビーン！

＞えっ！違うの？！
違います、笑点です

> 違います、笑点です
ご褒美は？

>>int GetForm( FormTerm*& term )

ポインタ変数自体を参照渡しするって意味です。
ポインタのポインタと、おんなじようなものですが、
こちらを使ったほうがちょっとスッキリしますね。

//------------------------------------------------------------------------------
#include <string>
#include <stdio.h>

extern std::string g_strX("global string");

void funcRP(char *& rp)
{
rp = const_cast<char*>(g_strX.c_str());
}

void funcPP(char ** pp)
{
*pp = const_cast<char*>(g_strX.c_str());
}

int main(int argc, char* argv[])
{
char *p1=0, *p2=0;
funcRP(p1);
funcPP(&p2);
printf("p1 = %s (0x%p)\np2 = %s (0x%p)\n", p1, p1, p2, p2);
return 0;
}

>ご褒美は？
おーい山田君座布団２枚やってくれっ

>>ポインタのポインタと、おんなじようなものですが、
>>こちらを使ったほうがちょっとスッキリしますね。

あ〜、なるほどー。今度からそう書こっと(^^;;;

COBOL74しか知らないんで参加できない(ゴミ
しかも8割がた忘れてるし(更ゴミ
#条件分岐(evaliate？スペルうろ覚え)使って先輩にどやされた記憶が(^^;;

>>ポインタのポインタと、おんなじようなものですが、
>>こちらを使ったほうがちょっとスッキリしますね。

あ、なるほど。確かに(^^ゞ
evaliate・・・フランス語ですか？（ぉ
BASICと最近Ｃ言語、むかーーーしアセンブラとマシン語やった位なので
言語はまだまだレベルが低いです(T^T)
頑張らないと