Question

我对C ++很新。我想编写一个程序/函数来检查字符串输入（来自控制台或其他来源，这里不重要），如果已经在数组中。如果不是那么它应该被写入数组。否则什么都不做。我的问题是for循环和if条件。我错过了什么？

#include <iostream>
#include <string>
#include <fstream>
#include <stdio.h>

using namespace std;

typedef struct {
   string id[10];
   string foo1[10];
   string type[10];
   string func[10];
}Device;

int main() {
   Device fooDevice;
   string mystring;
   int i = 0;

   mystring = "foo";

   ofstream temp;
   temp.open("temp.txt", ios::out | ios::app);


   for (fooDevice.id[i]; fooDevice.id[9]; i++) {
      if (fooDevice.id[i] != mystring) {
        fooDevice.id[i] = mystring;
        temp << mystring << endl;
      } else {
        //do nothing
      }
   }
return 0;
}

Answer 1

问题是for循环的结构。我不确定你认为你的情况意味着什么，但这是它的外观：

for (std::size_t i = 0; i < 10; ++i) {

这会将索引值i从0增加到9（包括）。然后，您可以检查fooDevice[i]。

的值

目前，您似乎正在尝试使用新字符串覆盖数组的每个元素。我不知道你怎么知道阵列在任何给定时间有多满。让我们假设当你到达第一个空字符串时停止：

for (std::size_t i = 0; i < 10; ++i) {
    if (myString == fooDevice.id[i]) {
        // already there, stop looping
        break;
    }
    else if (fooDevice.id[i].empty()) {
        // none of the currently set elements matches
        fooDevice.id[i] = myString;
        temp << myString << '\n';
        break;
    }
}

您也可以使用基于范围的for：

替换它

for (auto& deviceId: fooDevice.id) {
    if (myString == deviceId) {
        // already there, stop looping
        break;
    }
    else if (deviceId.empty()) {
        // none of the currently set elements matches
        deviceId = myString;
        temp << myString << '\n';
        break;
    }
}

更好的是，使用来自<algorithm>标题的std::vector std::find。（警告：未经测试的代码）：

struct Device {
    // ...
    std::vector<std::string> id;
    // ...
};

// ...

auto foundId = std::find(fooDevice.id.begin(), fooDevice.id.end(), myString);
if (fooDevice.id.end() == foundId) {
    // not already there
    fooDevice.id.push_back(myString);
    temp << myString << '\n';
}

似乎你对C和C ++之间的区别感到有些困惑：

<stdio.h>的C ++版本是<cstdio>，但您根本不需要它（而且通常不在C ++程序中）。
您不需要typedef C ++中struct的名称，只需执行struct Device { ... };

关于C ++风格，请重新考虑您对通常被视为不良做法的使用：using namespace std;和endl。

Answer 2

for (fooDevice.id[i]; fooDevice.id[9]; i++)似乎没有任何意义。我想你想要：

for(; i<9; ++i)

Answer 3

您不能使用string值作为循环条件。只评估为布尔值的表达式。

此外，在搜索之前，您也没有初始化设备阵列。

尝试更像这样的事情：

#include <iostream>
#include <string>
#include <fstream>

using namespace std;

struct Device {
   string id[10];
   string foo1[10];
   string type[10];
   string func[10];
};

int main() {
   Device fooDevice;
   string mystring;

   // fill fooDevice as needed...

   mystring = "foo";

   ofstream temp;
   temp.open("temp.txt", ios::out | ios::app);    

   bool found = false;
   int idx_available = -1;

   for (int i = 0; i < 10; ++i) {
      if (fooDevice.id[i] == mystring) {
         //do nothing
         found = true;
         break;
      }

      if ((idx_available == -1) && fooDevice.id[i].empty())
         idx_available = i;
   }

   if ((!found) && (idx_available != -1)) {
        fooDevice.id[idx_available] = mystring;
        temp << mystring << endl;
   }

   return 0;
}

通过重写可以更好地处理：

#include <iostream>
#include <string>
#include <fstream>
#include <vector>
#include <algorithm>

struct Device {
   std::string id;
   std::string foo1;
   std::string type;
   std::string func;
};

struct isDeviceId {
   const std::string &m_id;
   isDeviceId(const std::string &id) : m_id(id) {
   }
   bool operator()(const Device &device) {
      return (device.id == m_id);
   }
};

int main() {
   std::vector<Device> devices;
   std::string mystring;

   // fill devices as needed...

   mystring = "foo";

   if (std::find_if(devices.begin(), devices.end(), isDeviceId(mystring)) == devices.end()) {
      Device device;
      device.id = mystring;
      devices.push_back(device);

      std::ofstream temp;
      temp.open("temp.txt", std::ios::out | std::ios::app);    
      temp << mystring << std::endl;
   }

   return 0;
}

或者，在C ++ 11及更高版本中：

#include <iostream>
#include <string>
#include <fstream>
#include <vector>
#include <algorithm>

struct Device {
   std::string id;
   std::string foo1;
   std::string type;
   std::string func;
};

int main() {
   std::vector<Device> devices;
   std::string mystring;

   // fill devices as needed...

   mystring = "foo";

   if (std::find_if(devices.begin(), devices.end(),
      [mystring&](const Device &device) { return (device.id == mystring); }
    ) == devices.end()) {
      devices.emplace_back();
      devices.back().id = mystring;

      // or, in C++17:
      // devices.emplace_back().id = mystring;

      std::ofstream temp;
      temp.open("temp.txt", std::ios::out | std::ios::app);    
      temp << mystring << std::endl;
   }

   return 0;
}

C ++错误C2451条件字符串是非法的

3 个答案: