C++字符串解析分享，求优化,该如何处理

我没有检查你的程式正确与否，只是做了一些小小的改动

#include <iostream>
#include <string>
#include <vector>

//const std::string const_str = "#C255 244 233 22##H5##a1#"; //try your best to avoid global parameters, most of the times global params are evils


//检测##的位置以及数目
std::vector<int> GetTagPos(const std::string& str ,const std::string& tag)
{
    //check the tag and str are empty or not first, this is one of the technique which called "lazy evaluation"
    if(tag.empty()  
[解决办法]
 str.empty()){
        return std::vector<int>();
    }

    std::vector<int> iPosList; 
 
    size_t tag_pos = 0 ;
    size_t search_pos = 0;
    size_t search_length = str.length();

    //iPosList.clear(); //don't need to clear it since it do not have any data at all
    iPosList.reserve(32);
    while(search_pos <= search_length)   //检测tag位置
    {
        if((tag_pos = str.find(tag,search_pos)) != std::string::npos )
        {
            iPosList.push_back(tag_pos);
            search_pos = tag_pos + 1 ;
        }
        else
        {
            break;
        }
    }

    if(iPosList.size() % 2 != 0 )   //如果不配对，返回空
    {
        return std::vector<int>();
    }

    return iPosList; //remove goto, this is c++, not c, with the helps of RAII,we rarely need goto
}


void GetResult(const std::string& str , const std::string& tag )
{
     //check the str is empty or not first, this is one of the technique which called "lazy evaluation"
    if(true == str.empty()){
        return;
    }

    std::vector<int> itagpos = GetTagPos(str, tag); //this way the codes are shorter and compiler has more chances to optimize your codes
    if(itagpos.empty()){
        return ;
    }

    //size_t ipos = 0 ; //don't need to declare your local paorameter at here, you could declare it later on
    size_t word_pos = 0 ; 
 
    std::string strNum;
    auto end = std::end(itagpos); //you could use auto to save you some typing trouble
    for(auto it = std::begin(itagpos); it != end ; ++it)    //首先检测字母
    {
        for(auto ipos = (*it) + 1 ; ipos != *(++it) + 1 ; ++ipos)
        {
            char const c = str[ipos];
            if((c >= 'a' && c <= 'z') 
[解决办法]
 (c >= 'A' && c <= 'Z'))
            {
                std::cout << c << std::endl;
                word_pos = ipos ;
                break;
            }
            /*else
            {
                continue ;
            }*/
        }

        strNum.clear();
        //strNum.erase(); //1 : if you don't want to release the memory but clear the data, theyn you don't need to call erase at here
                          //2 : this is not the correct ways to call erase, please google the proper api or check your c++ primer 5
        for(auto ipos = word_pos ; ipos != *it+1 ; ++ipos)//检测数字
        {
            char const c = str[ipos]; 
 
            if( c >= '0'  && c <= '9')
            {
                strNum += c ;
            }
            else if( (c == ' ' 
[解决办法]
  ipos == *it ) && !strNum.empty())
            {
                std::cout << strNum << "\n"<< std::endl;   // 输出了字符，可以转换为数字
                strNum.clear();
                continue ;
            }
            /*else
            {
                continue ;
            }*/
        }
    }
}

int main()
{    
    std::cout << "string parse test\n"<< std::endl;

    GetResult("#C255 244 233 22##H5##a1#","#");


    return 0;
}

以下是进阶题材，可进修也可忽略
如果你要设计出很好的API，可以参考boost spirit，这个library利用expression template
的威力，在c++内设计出"domain specific language",提供了非常high level，容易维护,而且
效率极高的api（大部分情况下比c的API还要更加快）

其他的还有matrix template library 4,利用expression template消除temporary parameters
运算效率极高，比许多c程序员写的代码都来得更高

再提醒一次，这些是进阶题材，除非你有志设计出非常高质量的libraries(boost, Qt, MPL4等)，否则
你大概一辈子都不用知道什么是expression template