时间戳

首页 > 解决方案 > C++ - 使用结构数据类型将单词中的单个小写字符更改为大写，反之亦然

c++ - C++ - 使用结构数据类型将单词中的单个小写字符更改为大写，反之亦然

问题描述

正如您从标题中看到的那样，我想将 word 中的小写字符更改为大写，反之亦然。我还需要使用结构对象（在我的例子中）。当我将字符从小写更改为大写时，我遇到了一个问题，它只在第一个单词中发生变化，而不是在第二个、第三个等等中发生变化。我也在从文件中读取单词

这是输入文件

Aayak Audi 
Ahmed Golf7 
Samed Golf5

这是代码

#include <iostream>
#include <fstream>
#include <string>
#include <algorithm>
#include <cctype>
using namespace std;

struct pismaStr
{
    string ime;
    string objekat;
};


void malaVelikaSlova (string name)
{
    for (int i = 0; i < name.length()-1; i++)
    {
            if (name.at(i) >= 'A' && name.at(i) <= 'Z')
            name.at(i) += 32;
            else if (name.at(i) >= 'a' && name.at(i) <= 'z')
            name.at(i) -= 32;
            cout << name;
            break;
    }
}
int main()
{
    ifstream pismo;
    pismo.open("pismo.txt");
    ofstream novoPismo;
    novoPismo.open("novaSlova.txt");
    pismaStr stvari[200];
    int brojStvari = 0;

    while(pismo >> stvari[brojStvari].ime >> stvari[brojStvari].objekat)
    {
        brojStvari++;
    }
for (int i = 0; i < brojStvari; i++)
    {
       vector <pismaStr> vec = {pismaStr{stvari[i].ime}};
       for (auto obj : vec)
       {
        malaVelikaSlova (obj.ime);
       }
}

这是输出：

aayak
ahmed
samed

它是：

Aayak 
ahmed 
samed

我希望它看起来像这样

aAYAK
sAMED
aHMED

我怎样才能解决这个问题？有小费吗？

标签： c++arraysstructlowercase

解决方案

切线，

但这将是一个问题，这条线是

for (int i = 0; i < name.length()-1; i++)

这将从 循环name[0]到name[name.length() - 2]。返回可用字符的std::string::length数量。它不包括空终止符，所以你不需要减去 1。它应该是

for (int i = 0; i < name.length(); i++)

你更大的问题

是break循环末尾的语句（为清楚起见添加了缩进）

for (int i = 0; i < name.length()-1; i++)
{
    if (name.at(i) >= 'A' && name.at(i) <= 'Z')
        name.at(i) += 32;
    else if (name.at(i) >= 'a' && name.at(i) <= 'z')
        name.at(i) -= 32;
    cout << name;
    break;          // <--- this exits the loop entirely
}

你break;告诉程序立即退出循环。不再执行循环的进一步迭代。您的cout陈述也在循环中。一旦您为每次迭代运行循环，您将输出转换的每个步骤。只输出一次（最后）你把它放在循环之外。如果你想遍历每个字符（你确实这样做了），你的最终代码看起来像

void malaVelikaSlova (string name)
{
    for (int i = 0; i < name.length() - 1; i++)
    {
        if (name.at(i) >= 'A' && name.at(i) <= 'Z')
            name.at(i) += 32;
        else if (name.at(i) >= 'a' && name.at(i) <= 'z')
            name.at(i) -= 32;
    }
    cout << name;
}

您可以更改的其他内容

您不需要对string索引进行边界检查，因为您是根据string长度循环的，而且它没有改变，所以您不需要额外的std::string::at. 您可以只使用索引运算符：

// name.at(i); // <-- no need to do this anywhere
name[i]        // <-- slightly faster

由于您正在对容器（字符串）中的每个元素（字符）应用一些操作，因此这是库中std::transform,的绝佳候选者<algorithm>。我在这里也使用了一个 lambda 表达式，这对于 C++ 来说也很适合熟悉。

https://en.cppreference.com/w/cpp/algorithm/transform
https://en.cppreference.com/w/cpp/language/lambda

void malaVelikaSlova (string name)
{
    std::transform(
        name.begin(),
        name.end(),
        [](char c) -> char
        {
            if (c >= 'A' && c <= 'Z')
                return c + 32;
            if (c >= 'a' && c <= 'z')
                return c - 32;
            return c; // <-- for all other characters
        }
    );
    
    std::cout << name << "\n";
}

您甚至可以利用std::isupper、std::islower、std::toupper和std::tolower函数使您的代码更加明确。请注意，它是（其值类型为）std::string的别名， and函数对s 进行操作，因此您需要将s 转换为s：std::basic_string<char>charupperlowerunsigned charcharunsigned char

https://en.cppreference.com/w/cpp/string/basic_string
https://en.cppreference.com/w/cpp/string/byte/tolower
https://en.cppreference.com/w/cpp/字符串/字节/toupper
https://en.cppreference.com/w/cpp/string/byte/isupper
https://en.cppreference.com/w/cpp/string/byte/islower

void malaVelikaSlova (string name)
{
    std::transform(
        name.begin(),
        name.end(),
        [](unsigned char c) -> unsigned char // <-- must convert to unsigned to be safe with upper/lower functions
        {
            if std::isupper(c) return std::tolower(c);
            if std::islower(c) return std::toupper(c);

            return c; // <-- for all other characters
        }
    );
    
    std::cout << name << "\n";
}

推荐阅读