while 루프를 do while 루프로 변환

Question

다음은 특정 단어 (문자열)의 모든 인스턴스를 부모 문자열의 다른 단어 (strng)로 대체하는 함수입니다.

void clean(std::string &s,const std::string oldVal,const std::string newVal){ 
    std::string::size_type pos = 0; 

    while((pos = s.find(oldVal,pos)) != std::string::npos){ 
     s.replace(pos,oldVal.size(),newVal); 
     pos += newVal.size(); 
    } 
} 

저는 C++을 처음 접했고 약간의 이해가 어려웠습니다. 그래서이 코드를 더 읽기 쉽게 만들 생각입니다. 나는 do while 루프로 만들려고했다. 그러나 프로그램이 중단되었습니다. out_of_range 예외를 Throw합니다.
내 코드에 어떤 문제가 있습니까? 같은 문자열을 사용하여 두 기능을 모두 확인했습니다.

void clean2(std::string &s,const std::string oldVal,const std::string newVal){ 
    std::string::size_type pos = 0; 
    do{ 
     pos = s.find(oldVal,pos); 
     s.replace(pos,oldVal.size(),newVal); 
     pos += newVal.size(); 
    }while(pos != std::string::npos); 
} 

Answer 1

당신이

pos = s.find(oldVal,pos); 

그렇지 않으면 당신은 POS의 유효하지 않은 값을 사용할 수 있습니다 문 다음에 확인해야이 조건

pos != std::string::npos 

. 루프. :)

대신 당신이 루프를 사용하여 함수를 다시 작성할 수있는 루프 DO-동안의 while 루프를 대체하면서

그래서이 경우 while 루프가 좋아 보이는 것은 다음 않습니다. 예를 들어,

Answer 2

while 루프와 do-while 루프가 존재하며 가독성을 위해서만 사용되는 이유가 있습니다.
가장 큰 차이점은 조건을 확인하는 시간입니다.
이전 버전은 find -> test -> replace 순서로 작동합니다.
버전은 find -> replace -> test 순서로 작동합니다.

바꾸기를 시도하기 전에 동일한 루프 조건을 검사하는 바꾸기 전에 if를 추가하는 것이 좋습니다. 그러나 그것은 원래 IMO보다 덜 효율적이고 판독하기가 쉽지 않습니다.

Answer 3

당신이 모두 필요합니다

문자열이 발견되지 않은 경우와 "하지 문자열이 발견되지 않는 경우 대체 전화" "POS에 newVal.size()를 추가하지". 같은에

while(true) 
{ 
    pos = s.find(oldVal,pos); 
    if (pos != std::string::npos) 
    { 
     s.replace(pos,oldVal.size(),newVal); 
     pos += newVal.size(); 
    } 
    else 
    { 
     break; 
    } 
} 

또는 다른 많은 변종 :이 같은 것을 할 수있는, 또는

void clean2(std::string &s,const std::string oldVal,const std::string newVal){ 
    std::string::size_type pos = 0; 
    do{ 
     pos = s.find(oldVal,pos); 
     if (pos != std::string::npos) 
     { 
      s.replace(pos,oldVal.size(),newVal); 
      pos += newVal.size(); 
     } 
    }while(pos != std::string::npos); 
} 

: 그래서 당신은 DO-동안 루프 즉

내부의 경우 다른 필요 테마.