循环用于防止重复输入到一个数组

Question

因此，对于我们的C＃类中，我们必须创建5个对象的数组，有4个属性

作业数量
作业名
一个字符串一个int工作描述的字符串
工作时间的两倍。

我需要在我的循环中防止用户输入重复的工作号码：这是我的代码到目前为止。此代码有效，但会允许复制作业编号;

namespace JobDemo2 
{ 
class Program 
{ 
    static void Main(string[] args) 
    { 
     Job[] jobbies = new Job[5]; 
     int x; 
     int jobNum; 
     string customerName; 
     string description; 
     double hours; 
     const double RATE = 45.00; 

     for (x = 0; x < jobbies.Length; ++x)// creates array 
     { 
      GetJobData(out jobNum, out customerName, out description, out hours, jobbies); 
      jobbies[x] = new Job(jobNum, customerName, description, hours);    
     } 

     //Array.Sort(jobbies); 
     Console.WriteLine("The jobs, sorted, are: "); 
     for (x = 0; x < jobbies.Length; ++x) // prints the array values 
     { 
      DisplayJobs(jobbies[x]); 
     } 

     double totalRevenue = (jobbies[0].Hours + jobbies[1].Hours + 
       jobbies[2].Hours + jobbies[3].Hours + jobbies[4].Hours) * RATE; 
     Console.WriteLine(); 
     Console.WriteLine("The total revenue projected is {0}", totalRevenue); 

     Console.ReadKey(); 
    } 

    static void GetJobData(out int jobNum, 
     out string customerName, out string description, out double hours, 
     Job[] jobbies) 
    {     
     string inString; 

     Console.Write("Please enter a job number >> "); 
     inString = Console.ReadLine(); 
     int.TryParse(inString, out jobNum); 

     Console.Write("Please enter the customer's name for this job >> "); 
     customerName = Console.ReadLine(); 

     Console.Write("Please enter the job's description >> "); 
     description = Console.ReadLine(); 

     Console.Write("Please enter the projected hours for the job >> "); 
     inString = Console.ReadLine(); 
     double.TryParse(inString, out hours); 
     Console.WriteLine(); 
    } 

    static void DisplayJobs(Job jobbies) 
    { 
     Console.WriteLine("{0, 5} {1, -10} {2, 6} {3, 8}", 
      jobbies.JobNumber, jobbies.Customer, jobbies.Description, jobbies.Hours); 
    } 
} 

class Job //object 
{ 
    private double hours; 
    private double price; 
    public const double RATE = 45.00; 
    public Job(int num, string cust, string desc, double hrs) 
    { 
     JobNumber = num; 
     Customer = cust; 
     Description = desc; 
     Hours = hrs; 
    } 
    public int JobNumber { get; set; } 
    public string Customer { get; set; } 
    public string Description { get; set; } 
    public double Hours 
    { 
     get 
     { 
      return hours; 
     } 
     set 
     { 
      hours = value; 
      price = hours * RATE; 
     } 
    } 
    public double Price 
    { 
     get 
     { 
      return price; 
     } 
    } 
    public override string ToString() 
    { 
     return (GetType() + " " + JobNumber + " " + Customer + " " + 
      Description + " " + Hours + " hours @" + RATE.ToString("C") + 
      " per hour. Total price is " + Price.ToString("C")); 
    } 
    public override bool Equals(Object e) 
    { 
     bool equal; 
     Job temp = (Job)e; 
     if (JobNumber == temp.JobNumber) 
      equal = true; 
     else 
      equal = false; 
     return equal; 
    } 
    public override int GetHashCode() 
    { 
     return JobNumber; 
    } 
} 
} 

老师正在向班级建议我们在这里比较对象的另一个循环。那个for循环是什么样的？

这里是她的电子邮件：

帮助的循环创建一个布尔变量。
用于循环访问数组，以要求用户输入信息并将bool变量设置为true。

另一个for循环内部调用类中的equals方法，该方法将比较刚刚输入到数组中的每个对象的作业。这是大多数人搞乱的地方，因为你必须将对象与对象进行比较，而不是将作业号码的整数与整个对象进行比较。如果对象相等则将bool设置为false。

而布尔是假的，你想告诉他们他们输入了错误的数字并再次输入。在这里将bool设置为true，然后再次循环进行比较。只要数字保持不变，用户就会陷入这个while循环中。当他们输入一个正确的号码时，它会爆发。

Answer 1

这是家庭作业，所以我只能给你一些指点：

1. 不要用手存储作业GetJobData阵列（jobbies）英寸这种方法应该只有一个问题：获得工作数据。确定数据是否有重复的Id不是它的问题。

2. 编写一个检查重复项的帮助程序方法。它需要什么？它需要所有以前的工作，有多少工作，以及需要验证的新工作。以下签名看起来是正确的：

   private static bool CheckIfNewJobIsValid(Job newJob, 
                Job[] jobs, 
                int jobsValidatedCount) 
   

   此方法必须做什么？那么，它只需要遍历jobs中的第一个jobsValidatedCount作业并检查newJob是否等于它们中的任何一个。如果是这样，请退回false。如果循环结束，您可以返回true，找不到匹配项。

3. jobsValidatedCount不需要是一个新的计数器，也许一些其他已经存在的代码变量可以给你这个信息。

祝你好运！

P.D.因为这是已经被你的老师流传下来的，我会解决了Equals方法一点点：

public override bool Equals(Object e) 
{ 
    bool equal = false; 
    Job temp = e as Job; 

    if (Job != null && JobNumber == temp.JobNumber) 
    { 
     equal = true; 
    } 

    return equal; 
} 

P.D.正如Alexei Levenkov在评论中指出的那样，使用Equals ony代替JobNumber comparisson似乎是一个坏主意。无论所有其他属性的价值是否相同，JobNumber的任何两个工作都相等？可以说至少可以说是令人困惑的。更好的方法是直接检查CheckIfNewJobIsValid内部的JobNumber，而不是使用Equals，但我想它的使用是由于学术原因。