do~ While()の書き換えを教えてください。

Question

文字列の中から特定の文字列を検索し、何文字目にターゲットがあるかを表示するプログラムを書きました。
strstr関数の繰り返しとしてwhile文を用いようと思い書いてみましたが、ポインタ変数の初期化と干渉してしまい、
一度も繰り返し処理に入ることができず、処理が終了しました。
そこで以下のコードでは繰り返し文として、do Whileを用いていますが、do whileは条件が後に来るため可読性の意味で、避けたいので、 別の方法で繰り返し処理を行いたいです。
ご査収の程よろしくお願いいたします。
コード

#include <iostream>
#include <string>

int main() {
    char chArray[] = "aabaabbaab";
    char chStringTarget[] = "ab";
    char *pStartAdress;  // 検索開始する位置のアドレス格納用
    char *pFindTargetAdress; // パターンの存在する位置のアドレス用
    int nPositionTarget = 0;

    // テキストの先頭アドレスをtextの先頭にセット
    pStartAdress = chArray;

    // strstrを繰り返し実行
    do {
        pFindTargetAdress = strstr( pStartAdress, chStringTarget );

        if( pFindTargetAdress != NULL ){

            // patternが存在する位置のアドレスと、何文字目にあるかを表示
            nPositionTarget = pFindTargetAdress - chArray;
            printf( "%sはアドレス：%p、%d文字目に存在します。\n", chStringTarget, pFindTargetAdress, nPositionTarget + 1 );

            // ターゲットを見つけた位置の次の文字を、検索再開文字とする。
            pStartAdress = pFindTargetAdress + 1;
        }

    } while( pFindTargetAdress != NULL );

    return 0;
}

Comments

[thkana]

どうでもいいけど、「ご査収」ってこういうときに使う言葉じゃないと思う。

Answer 1

do while自体が可読性が悪いとは私は思わないけれど。単に慣れてないだけだったりしないかしら?
まぁ、それは議論してもしかたないので。

一般論としては...

do{
処理
}while(条件)
は、最初に１回は必ず処理を行うところがミソ。

処理
while(条件){
処理
}
は等価ではあるけれど同じ処理を二回書くのはよろしくない気がするので

bool flag=true
while(flag){
処理
flag=条件
}
とでもしてフラグを立てることで最初の一回は処理を行わせるとか。

で、今回のプログラムの話としては

    do {
        pFindTargetAdress = strstr( pStartAdress, chStringTarget );
        if( pFindTargetAdress != NULL ){//この条件と
        }
    } while( pFindTargetAdress != NULL );//この条件が同じ

ifで処理する条件とループの継続条件が同じ、というほうが私は気になるけれど、

そもそもとして

  pFindStartAdress = chArray;
  while( pFindTargetAdress = strstr( pFindStartAdress, chStringTarget )){
    // patternが存在する位置のアドレスと、何文字目にあるかを表示
    nPositionTarget = pFindTargetAdress - chArray;
    printf( "%sはアドレス：%p、%d文字目に存在します。\n", chStringTarget, pFindTargetAdress, nPositionTarget + 1 );

    // ターゲットを見つけた位置の次の文字を、検索再開文字とする。
    pFindTargetAdress++;
  }

じゃいけないのかなぁ、と思ったり。コーディングルールで条件式には関数を書くな、というのを見たこともあるのでそういう縛りなら仕方ないけれど。

Comments

[don]

do whileについての理解が不十分でした。
たまたま教えていただいた方にdo whileの使用を注意されたことがあったため使用を避けていました。
今考えれば「必要に迫られない場合にはあえて使うものではない」といったニュアンスだったのかもしれません。
関数の戻り値は別の変数に置きなおしたほうがif分の条件式が見やすいかと思い、そうしていたのですがワンクッション置く意味がなかったかもしれません。
”ご査収”適切な表現ではなかったかもしれません。気を付けます！
回答ありがとうございました！

Answer 2

pFindTargetAdress=chArray; // なんでもいい
while(pFindTargetAdress!=NULL){
なんやかや
}

ではどうでしょう

---

あるいは

while(true) {
    pFindTargetAdress = strstr( pStartAdress, chStringTarget );
    if( pFindTargetAdress == NULL ) break;

    なんやかや

}

Comments

[don]

回答ありがとうございます！
簡単なことでした。
ありがとうございました。