2012-03-20 6 views
6

На моей странице загружается, я звоню ReturnStuff() один или три раза? Если я назову это три раза, есть ли более эффективный способ сделать это?Я правильно использую списки?

protected void Page_Load(object sender, EventArgs e) 
{ 
    string thing1 = ReturnStuff(username,password)[0]; 
    string thing2 = ReturnStuff(username, password)[1]; 
    string thing3 = ReturnStuff(username, password)[2]; 
} 

public static List<string> ReturnStuff(string foo, string bar) 
{ 

    // Create a list to contain the attributes 
    List<string> Stuff = new List<string>(); 

    // Some process that determines strings values based on supplied parameters 

    Stuff.Add(fn); 
    Stuff.Add(ln); 
    Stuff.Add(em); 

    return Stuff; 
} 
+1

Вы спрашиваете, трижды вы вызываете метод? –

+1

Да, это и как сделать это более эффективно. – mmcglynn

ответ

13

Вы называете это три раза. Вот более эффективный способ:

protected void Page_Load(object sender, EventArgs e) 
{ 
    var stuff = ReturnStuff(username,password); 
    string thing1 = stuff[0]; 
    string thing2 = stuff[1]; 
    string thing3 = stuff[2]; 
} 

Но более того, если у вас есть имя, фамилия и адрес электронной почты, я хотел бы написать функцию, которая возвращает объект, слагающее имя, фамилия , и e-mail:

public class User 
{ 
    public string LastName {get;set;} 
    public string FirstName {get;set;} 
    public string EMail {get;set;} 
} 

public static User GetUser(string username, string password) 
{ 
    // Some process that determines strings values based on supplied parameters 

    return new User() {FirstName=fn, LastName=ln, EMail=em}; 
} 

protected void Page_Load(object sender, EventArgs e) 
{ 
    var user = GetUser(username,password); 
} 
+0

Dang it, sniped ... – squillman

+0

Ничего себе, это много, чтобы обернуть мою голову n00b, так что спасибо! – mmcglynn

+0

@mmcglynn: Вы можете просто сосредоточиться на части о возврате списка раньше, а затем :) – BoltClock

1

Вы называете это 3 раза. Вызовите его один раз и сохраните результаты в переменной, затем вы можете работать с этой переменной.

Попробуйте это:

var stuff = ReturnStuff(username,password); 
string thing1 = stuff[0]; 
string thing2 = stuff[1]; 
string thing3 = stuff[2]; 
+0

Почему вы используете вариант? какое значение здесь? почему не сильные типы? – Pankaj

+2

@ Pankaj Garg: Erm, "вариант"? – BoltClock

+0

Я использую var, чтобы уменьшить отношение сигнал/шум в моем коде. Это не всегда уместно, и даже не всегда хорошая идея. Прочитайте это. http://stackoverflow.com/questions/41479/use-of-var-keyword-in-c-sharp – squillman

1

3 раза. следующий код поможет вам понять. перейдите на главную функцию и вызовите функцию func().

class howmanytimescallingafunction 
    { 
     public static int i = 0; 
     public List<string> fun() 
     { 
      List<string> list = new List<string> { "A", "B", "C" }; 
      i++; 
      return list; 
     } 
     public void func() 
     { 
      Console.WriteLine(fun()[0]); 
      Console.WriteLine(i); 
      Console.WriteLine(fun()[1]); 
      Console.WriteLine(i); 
      Console.WriteLine(fun()[2]); 
      Console.WriteLine(i); 
     } 
    } 

Вы должны вызвать эту функцию один раз, получить возвращаемое значение в локальном списке <> переменной, а затем получить доступ с помощью переменной. например:

List<string> list = function-that-returns-List<string>(); 
list[0]; //Do whatever with list now. 
Смежные вопросы