2013-05-11 88 views
1

我是C#的新手,我无法理解为什么这段代码不起作用。交换数组中2个元素的函数不起作用

using System; 
using System.Collections.Generic; 
using System.Linq; 
using System.Text; 
using System.Threading.Tasks; 

namespace ConsoleApplication1 
{ 
    class Program 
    { 
     static void Main(string[] args) 
     { 
      char[] sw = "ab".ToCharArray(); 
      swap(sw[0], sw[1]); 
      string end = new string(sw); 
      Console.Write(end); 
     } 

     static void swap(char a, char b) 
     { 
      char temp = a; 
      a = b; 
      b = temp; 
     } 
    } 
} 

我在控制台上的预期是“ba”,但我得到“ab”。我能够找到解决这个问题的不同方法,但我想知道的是这个代码中的错误。 感谢您的帮助!

回答

0

您按照价值传递您的论点ab
有关更多信息,请参阅What's the difference between passing by reference vs. passing by value?

以下是解决您的问题的两种解决方案。

//Pass by value and return the values 
static Tuple<char, char> swap2(char a, char b) 
{ 
    char temp = a; 
    a = b; 
    b = temp; 
    return new Tuple<char, char>(a, b); 
} 

//Pass by reference 
static void swap3(ref char a, ref char b) 
{ 
    char temp = a; 
    a = b; 
    b = temp; 
} 

public static void Main(string[] args) 
{ 
    char[] sw2 = "ab".ToCharArray(); 
    var chars2 = swap2(sw2[0], sw2[1]); 
    sw2[0] = chars2.Item1; 
    sw2[1] = chars2.Item2; 
    //Will print "ba" 
    Console.WriteLine(sw2); 

    char[] sw3 = "ab".ToCharArray(); 
    swap3(ref sw3[0], ref sw3[1]); 
    //Will print "ba" 
    Console.WriteLine(sw3); 
} 

下面是关于是否应该使用或尝试避免ref关键字的问题。除了最简单的用途之外,通常建议尽可能避免参考。交换属于“最简单的用法”类别,但我建议您尽量避免在大多数实际情况下使用ref。
When is using the C# ref keyword ever a good idea?

+0

感谢,认为C#做它总是由参,问题溶胶VED。 – user2373458 2013-05-11 18:24:58

8

的问题是,swap方法实际上只是操纵的ab本地副本。您需要通过引用传递参数。所以,你会定义swap方法是这样的:

static void swap(ref char a, ref char b) 
    { 
     char temp = a; 
     a = b; 
     b = temp; 
    } 

,并调用它是这样的:

swap(ref sw[0], ref sw[1]); 
+0

这可能是要走的路! – 2013-05-11 18:26:42

+0

@AlexBell我同意这可能是交换的方式,因为它非常简单。但我通常尽量避免使用ref关键字,因为它往往会导致更复杂的API IMO。 – 2013-05-11 18:41:22

+0

这正是我所要做的,但使'Swap'通用,因此它适用于任何数组,然后将其放置在某些实用程序类中。 – Lukazoid 2017-04-18 11:38:05

0

你交换走的是两条值类型和变量之间交换值。没有什么会修改原始数组。你会需要修改你的交换方法是这样的:)像

static void Swap(char[] array, int a, int b) 
{ 
    char temp = array[a]; 
    array[a] = array[b]; 
    array[b] = temp; 
} 

然后,您可以从主(称之为:

Swap(array, 0, 1); 
1

应该像下面的(注进行修改:在这个例子中,ref char[] arr具有前缀ref主要用于教导目的:阵列将ref传递默认)

namespace ConsoleApplication1 
{ 

     class Program 
     { 
      static void Main(string[] args) 
      { 
       char[] sw = "ab".ToCharArray(); 
       swap(0, 1, ref sw); 
       string end = new string(sw); 
       Console.Write(end); 
      } 

      static void swap(int indexA, int indexB, ref char[] arr) 
      { 
       char temp = arr[indexA]; 
       arr[indexA] = arr[indexB]; 
       arr[indexB] =temp; 
      } 
     } 
    }