有没有人可以帮助我？我正在写一个程序（颠倒数组），但它不会运行:(

Question

我正在写一个关于使用3种方法（GenerateNumber，reverse和PrintOut）颠倒数组的程序。

不幸的是， “吨运行。你能帮助我找到错误并解决这些问题？

为什么它不跑？

public class Program 
{ 
    static int[] GenerateNumber() 
    { 
     string a = Console.ReadLine(); 
     int b = Convert.ToInt32(a); 
     int[] number = new int [b]; 
     string[] c = new string[b]; 
     for (int index = 0; index < number.Length; index++) 
     { 
      c[index] = Console.ReadLine(); 
      number [index]= Convert.ToInt32(c[index]); 
     } 
     return number; 
    } 
    static int[] reverse(int[] array) 
    { 

     for (int index =0; index <array.Length; index++) 
     { 
      int c = array[index]; 
      array[index] = array[array.Length - index - 1]; 
      array[array.Length - index - 1]= c; 
     } 
     return array; 
    } 
    static int[] PrintOut (int[] array) 
    { 
     for (int index = 0; index > array.Length; index++) 
      Console.Write(array[index]); 
     return array; 
    } 
      static void Main(string[] args) 
    { 
     int[] number = GenerateNumber(); 
     reverse(number); 
     PrintOut(number); 
     Console.ReadKey(); 
    } 

Answer 1

不当行为的直接原因是在

static int[] PrintOut (int[] array) 
{ 
    for (int index = 0; index > array.Length; index++) // <- wrong condition 
     Console.Write(array[index]); 

的比较应该是<而不是>：

for (int index = 0; index < array.Length; index++) 

一个更好的选择，但是，是foreach循环，而不是for

foreach (var item in array) 
     Console.Write(item); // propably, you want WriteLine not Write 

几点建议：

public class Program { 
    static int[] GenerateNumber() { 
     // You don't want "c" array, but "number" 
     int[] number = new int [Convert.ToInt32(Console.ReadLine())]; 

     for (int index = 0; index < number.Length; index++) 
     number [index] = Convert.ToInt32(Console.ReadLine()); 

     return number; 
    } 

    // Nice implementation, nothing to improve but naming (reverse -> Reverse) 
    static int[] Reverse(int[] array) { 
     for (int index = 0; index <array.Length; index++) { 
      int c = array[index]; 
      array[index] = array[array.Length - index - 1]; 
      array[array.Length - index - 1] = c; 
     } 
     return array; 
    } 

    static int[] PrintOut (int[] array) { 
     // foreach is easier to implement and easier to read 
     foreach (var item in array) 
      Console.WriteLine(item); // <- you, probably, want WriteLine not Write 

     return array; 
    } 

    static void Main(string[] args) { 
     int[] number = GenerateNumber(); 
     Reverse(number); 
     PrintOut(number); 
     Console.ReadKey(); 
    }