ポインタ変数を利用したwhileの終了のさせかた

whileをポインタで回す際に範囲外にアクセスしたらループを終了させたいのですがNULLとしても終了してくれませんどのようにするとよいでしょうか？

C
1int luhn(char *ptr){
2    char *head_ptr=ptr;
3    int i,current_number;
4    int index=1;
5    int sum=0;
6
7    while(*ptr !='\0')ptr++;
8
9    ptr--;
10    printf("ptr=%c",*ptr);
11    do{
12        current_number=*ptr-'\0';
13        if(index%2==0){
14            current_number=current_number*2;
15            if(current_number>10){
16                current_number=current_number-9;
17            }
18        }
19        sum+=current_number;
20        index++;
21    }while(ptr--!=NULL);ここの部分がおかしい
22    if(sum%10==0){
23        return 1;
24    }
25    else{
26        return 0;
27    }
28}

kazuma-s

2021/08/08 07:28

current_number=*ptr-'\0'; の '\0' は 0 です。 '0' の間違いではありませんか？

linkinpark

2021/08/08 09:07

すいません'0'ですねご指摘ありがとうございます。

行動規範の内容に同意します

回答4件

} while (--ptr >= head_ptr);
これはほとんどすべての処理系でコンパイルエラーにもならず、
実行もできますが、規格上は未定義動作です。

ポインタを増減させるということは、そのポインタは配列の要素を指しています。
ポインタの値が配列の末尾の次の値になることは許されていますが、
配列の先頭の前を指すことは許されていません。

次のように書くと規格違反になりません。

C
1#include <stdio.h>
2
3int luhn(char *ptr)
4{
5    char *head_ptr=ptr;
6    int i, current_number;
7    int index = 1;
8    int sum = 0;
9
10    while (*ptr !='\0') ptr++;
11
12    do {
13        current_number = *--ptr - '0';  // ★ ここで --ptr
14        if (index%2 == 0){
15            current_number = current_number*2;
16            if (current_number > 10){
17                current_number = current_number - 9;
18            }
19        }
20        sum += current_number;
21    	printf("*ptr = '%c', current_number = %d, sum = %d\n",
22			*ptr, current_number, sum);
23        index++;
24    } while (ptr != head_ptr);  // ★ まだ配列の先頭ではないか
25    if (sum%10 == 0) {
26        return 1;
27    }
28    else {
29        return 0;
30    }
31}
32
33int main(void)
34{
35	char d[] = "9324";
36	printf("%d\n", luhn(d));
37}