プログラマ専用SNS ミクプラ

コード:

if (!ptr){ //ptrが0（NULL）であれば下記を実行,そうでなければelse文を実行

コード:

8：stlrenとポインタの計算の考え方が間違っていると思います。ちなみに文字列の終端はNULLではなくNUL文字です。
\0とかヌル文字とか表現するならＯＫですが、NULLはNULLポインタ値と混同するので避けて下さい。

10：無限ループ　→　私も不要だと思います。

12：最初の区切り文字のアドレスをptrに代入①
　→　最初だけに限定しません。

14：ptrが0（NULL）であれば下記を実行,そうでなければelse文を実行
　→　これも見て分かるコメントです。書くならNULLはどういう意味か書くべきです。

16：bufのアドレスがbuf最後の文字のアドレスより大きいまたは同じであれば実行
17：無限ループを抜ける
　→　これも見て分かるコメントです。書くならbreakも含めて抜ける条件を簡潔に書くべきです。

19：ptrにbufの最後の文字のアドレスを代入
　→　これは元プログラムのバグだと思います。と言うかsep処理が継続しないのでプログラム的に変では？
[訂正]間違えました、後で訂正してあります。

21：①で得たアドレスの中をNULLにする。
　→　上と同様にNULLとヌル文字を混同しています。また、元のプログラムも書き方が悪いです。

22：アドレスをひとつずらす（1回目のループでは二個目の区切り文字の所）②
　→　いいえ違います。それと元プログラムに重症なバグがあります。
[訂正]これも問題なかったです。ただ自分の趣味で直してもらいました。

24：$からNULLがあるところまで表示
　→　$からって何処でしょう？

27：②のアドレスをbufに代入
　→　②だけではありません。

コード:

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

 
void main(void )
{
	char *ptr,*pEnd;
char buf[] = "A";
printf("%d\n",buf[0]);
printf("%d\n",buf[1]);
printf("%d\n",strlen(buf));
printf("%p\n",pEnd);
//printf("%p\n",pEnd[0]);
printf("%p\n",ptr);


}

コード:

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

 
void main(void )
{
	char *ptr,*pEnd;
	char buf[] = "A";
	
	printf("%d\n",buf[0]);
	printf("%d\n",buf[1]);
	printf("%d\n",strlen(buf));
	printf("%p\n",pEnd);
	pEnd = buf + strlen(buf); 
	printf("%p\n",pEnd[0]);


}

コード:

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

 
void main(void )
{
	char *ptr,*pEnd;
	char buf[] = "A";
	pEnd = buf + strlen(buf);
	printf("%d\n",buf[0]);
	printf("%d\n",buf[1]);
	printf("%d\n",strlen(buf));
	printf("%p\n",pEnd);
	printf("%p\n",pEnd[0]);


}

コード:

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

void StrTok(char *buf,char *Sep)
{
	char *pEnd = buf + strlen(buf);
	while( buf<pEnd ){
		char *sep_ptr = strpbrk(buf,Sep);
 		if( sep_ptr!=NULL ) {
	        *sep_ptr = '\0';
        }
        printf(">%s \n", buf);
        if( sep_ptr==NULL ) {
			break;
		} else {
			buf = sep_ptr + 1;
		}
	}
}

void main(void )
{
	char buf[] = "$GPGGA,,,,,,0,02,,,,,,,*64";
 	StrTok(buf,","); //「,」で区切る
}

コード:

#include <stdio.h>
#include <string.h>
 
void StrTok(char *buf,char *Sep)
{
    char *pEnd = buf + strlen(buf);//①　←何度も言いますが理解して下さい。
    while( buf<pEnd ){//←コメントが必要です。
        char *sep_ptr = strpbrk(buf,Sep);//分割する文字がない場合は、NULL ポインタを返す。 ←ＮＧパターンしか説明していません。
        if( sep_ptr!=NULL ) {//分割する文字があった場合そこのアドレスの中を\0にする。←で結局どうなるの？と言った感じのコメントです。これは何のために行っていますか？こうすると何処に影響があるんでしょう？
            *sep_ptr = '\0';
        }
        printf(">%s \n", buf);//bufの\0まで表示。
        if( sep_ptr==NULL ) {//分割する文字がなかったらなくなったらループを抜ける
            break;
        } else {
            buf = sep_ptr + 1;//←②sep_ptrが指している値はなんでしょうか？そして、それを+１する意味は？
        }
    }
}

int main(void)
{
    char buf[] = "$GPGGA,,,,,,0,02,,,,,,,*64";
    StrTok(buf,","); //「,」で区切る
    return 0;
}