V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
kirara
V2EX  ›  程序员

C++的 Vector 容器中 string 类型元素可否用一种指针替代它?

  •  
  •   kirara · 2019-09-20 00:20:02 +08:00 · 2395 次点击
    这是一个创建于 1873 天前的主题,其中的信息可能已经有所发展或是发生改变。

    代码目的是把迭代器第一个 string 类型元素的第一段不包括空格的字符全改为大写。
    期望输出:SOME string oh
    实际输出:SOME string oh
    代码输出确实是正确的,但是总觉得代码里太多解引用(*it)写起来很繁琐,不知道大佬们怎么解决。如题。

    #include <iostream>
    #include <string>
    #include <vector>
    using namespace std;
    int main()
    {
    	vector<string> s{ "some string"," oh" };
    	for (auto it = s.begin(); it != s.end() && !it->empty(); ++it) {
    		if (it == s.begin()) { //想把(*it)改成字符串直接使用;
    			for (decltype((*it).size()) index = 0; index != (*it).size() && !isspace((*it)[index]); index++) {
    				(*it)[index] = toupper((*it)[index]);
    			}
    		}
    		cout << (*it) << " ";
    	}
    	return 0;
    }
    
    10 条回复    2019-09-20 08:51:14 +08:00
    littlespider89
        1
    littlespider89  
       2019-09-20 00:23:30 +08:00
    为啥不用指针写法 it->size()
    mind3x
        2
    mind3x  
       2019-09-20 00:27:13 +08:00 via Android
    既然是只处理第一个 string,没看懂外面一圈 for 是拿来干嘛的
    kirara
        3
    kirara  
    OP
       2019-09-20 00:44:35 +08:00 via Android
    @mind3x 写着玩的,别在意
    inhzus
        4
    inhzus  
       2019-09-20 00:51:58 +08:00 via Android
    这个代码的可读性真的震惊到我了…果然对 c++来说 一千个人心中有一千个哈姆雷特。
    1. 既然只处理第一个,判断不为空,直接取 s[0] 不久好了吗。
    2. 第二个循环既然和 index 的位置无关,用 ranged-based for 不也会让代码简洁很多吗。
    3. 大家写 c++ 也会像楼主这样把 size_type 用 decltype 进行 type deduction 吗?这个写法虽然没问题,不过感觉不符合我的习惯…
    aijam
        5
    aijam  
       2019-09-20 00:52:45 +08:00
    for (auto &str: s) {
    for (auto it = str.begin(); it != str.end() && !isspace(*it); it++) {
    *it = toupper(*it);
    }
    cout << str << " ";
    }
    wbing
        6
    wbing  
       2019-09-20 01:00:33 +08:00
    ```
    if(s.size() == 0) return -1;
    std::stringstream ss(s[0]);
    std::string str;
    if(ss >> str)
    {
    auto index = s[0].find_first_of(str);
    std::transform(str.begin(),str.end(),str.begin(),::toupper);
    s[0].replace(index,str.size(),str);
    }
    ```
    lishunan246
        7
    lishunan246  
       2019-09-20 01:14:03 +08:00 via Android   ❤️ 3
    这里就两个 for 循环,一个用 iterator,一个用 index ;一个用++i,一个用 i++。
    size_t 写成 decltype((*it).size())。
    《如何把你写 c++的朋友气到脑中风》
    tianshilei1992
        8
    tianshilei1992  
       2019-09-20 01:47:53 +08:00
    这样如何?
    ```C++
    #include <iostream>
    #include <string>
    #include <vector>

    using namespace std;

    inline void func(vector<string> &vec) {
    if (vec.empty()) {
    return;
    }

    auto& str = vec.front();

    for (auto itr = str.begin(); *itr != ' ' && itr != str.end(); ++itr) {
    *itr = toupper(*itr);
    }
    }

    int main(int argc, char *argv[]) {
    vector<string> vec = {"some string", " oh"};
    func(vec);

    for (const auto& str : vec) {
    cout << str << ' ';
    }

    cout << endl;

    return 0;
    }
    ```
    tianshilei1992
        9
    tianshilei1992  
       2019-09-20 01:48:50 +08:00
    @tianshilei1992 Hot fix:
    #include <iostream>
    #include <string>
    #include <vector>

    using namespace std;

    inline void func(vector<string> &vec) {
    if (vec.empty()) {
    return;
    }

    auto& str = vec.front();

    for (auto itr = str.begin(); itr != str.end() && *itr != ' '; ++itr) {
    *itr = toupper(*itr);
    }
    }

    int main(int argc, char *argv[]) {
    vector<string> vec = {"some string", " oh"};
    func(vec);

    for (const auto& str : vec) {
    cout << str << ' ';
    }

    cout << endl;

    return 0;
    }
    jmc891205
        10
    jmc891205  
       2019-09-20 08:51:14 +08:00
    你可能想要 range-based for loop
    for (auto str: s) {cout << str << endl;}
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   3107 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 24ms · UTC 13:58 · PVG 21:58 · LAX 05:58 · JFK 08:58
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.