2

我有一个程序会询问学生姓名和他/她的 10 个班级。该代码是为了防止重复条目，但每次我运行程序时它都说每个条目都已经存在。我已经经历了一千次这件事，但我一生都无法弄清楚。对我的问题的任何见解将不胜感激。

#include <iostream>

using namespace std;

struct student
{
   string name;
   string classes[10];
};

int main(int argc, char *argv[])
{
  string test_name;
  student entry;

  cout << "Enter the name of the student you wish to insert (string) \n"; 
  cin >> entry.name;

  for(int i = 0; i < 9; i++)
  {
    cout << "Enter class number " << i + 1 << " For " << entry.name << endl; 
    cin >> test_name;

    for(int j = 0; j < 9; j++)
      if(test_name == entry.classes[j])
      {
         cout << "Class already exists for " << entry.name << ". Please try again.\n";
         i -= 1;
      }
      else
      {
          entry.classes[i] = test_name;
      }
  }
  system("PAUSE");
  return EXIT_SUCCESS;
}
4

3 回答 3

2

您的内部for循环测试所有 10 个位置，包括您要插入新类的位置。

您真的只想扫描填充的位置，看看是否有任何匹配，然后在循环之外添加新类（如果它不是重复的）。

伪代码：

for (i = 0; i < 10; i++)
{
    get class 'i';

    bool repeat = false; 

    for (j = 0; j < i; j++)  // Only check previous classes (j < i)
        if (test_name == entry.classes[j])
        {
            repeat = true;
            break;
        }

    if (repeat) 
    {
         output the error
         rewind 'i'
         continue;
    } else
    {
         insert the new entry
    }
}
于 2013-11-02T03:20:34.577 回答
0

虽然这是在 Java 中，但你可以做类似的事情，我认为它满足了忽略类数组中已经存在的任何输入并在达到 10 个类之前添加它的要求：

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.ArrayList;



public class Test {

    public static void main(String[] args) throws IOException {


        ArrayList < String > classes = new ArrayList < String > ();


        System.out.println("Enter the name of the student you wish to insert (string) \n");



        while (classes.size() < 10) {


            BufferedReader bufferRead = new BufferedReader(new InputStreamReader(System. in ));
            String s = bufferRead.readLine();


            for (String classy: classes) {
                if (classy.equals(s)) {
                    System.out.println("This is duplicate class");
                    break;

                }


            }
            classes.add(s);



        }
        System.out.println("Ten Classes have been input");



    }

}
于 2013-11-02T04:59:01.510 回答
0

要真正看到，有什么问题在您的if()陈述之前添加这一行：

cerr << "test_name == \"" << test_name << "\", entry.classes[j] == \"" << entry.classes[j] << "\"\n"

我希望您看到的是 test_name 由于某种原因是空的，而且我知道所有未初始化的字符串都将显示为空字符串。但是，无论有什么问题，上面的行都应该显示出来。

于 2013-11-02T09:43:19.983 回答