Каков наилучший способ вернуть два значения из метода?
Когда мне приходится писать методы, которые возвращают два значения, я обычно об этом говорю, как в следующем коде, который возвращает List<string>
. Или, если я должен вернуться, например. a id и строкой, тогда я возвращаю List<object>
, а затем выделяю их с помощью index и recast значения.
Эта переработка и ссылка по индексу кажется неэлегантной, поэтому я хочу разработать новую привычку для методов, возвращающих два значения. Каков наилучший шаблон для этого?
using System;
using System.Collections.Generic;
using System.Linq;
namespace MultipleReturns
{
class Program
{
static void Main(string[] args)
{
string extension = "txt";
{
List<string> entries = GetIdCodeAndFileName("first.txt", extension);
Console.WriteLine("{0}, {1}", entries[0], entries[1]);
}
{
List<string> entries = GetIdCodeAndFileName("first", extension);
Console.WriteLine("{0}, {1}", entries[0], entries[1]);
}
Console.ReadLine();
}
/// <summary>
/// gets "first.txt", "txt" and returns "first", "first.txt"
/// gets "first", "txt" and returns "first", "first.txt"
/// it is assumed that extensions will always match
/// </summary>
/// <param name="line"></param>
public static List<string> GetIdCodeAndFileName(string line, string extension)
{
if (line.Contains("."))
{
List<string> parts = line.BreakIntoParts(".");
List<string> returnItems = new List<string>();
returnItems.Add(parts[0]);
returnItems.Add(line);
return returnItems;
}
else
{
List<string> returnItems = new List<string>();
returnItems.Add(line);
returnItems.Add(line + "." + extension);
return returnItems;
}
}
}
public static class StringHelpers
{
public static List<string> BreakIntoParts(this string line, string separator)
{
if (String.IsNullOrEmpty(line))
return null;
else
{
return line.Split(new string[] { separator }, StringSplitOptions.None).Select(p => p.Trim()).ToList();
}
}
}
}
Добавлено:
Хорошо, спасибо всем, мне нравится, что " возвращает пользовательский класс" отвечает лучше всего, никогда не думал, что out
было таким простым для чтения, кажется, что он взломан я возвращаю первую переменную в одну сторону и вторую, здесь мой рефакторинг возвращает пользовательский класс:
using System;
using System.Collections.Generic;
using System.Linq;
namespace MultipleReturns
{
class Program
{
static void Main(string[] args)
{
string extension = "txt";
{
IdCodeFileNamePair pair = GetIdCodeAndFileName("first.txt", extension);
Console.WriteLine("{0}, {1}", pair.IdCode, pair.FileName);
}
{
IdCodeFileNamePair pair = GetIdCodeAndFileName("first", extension);
Console.WriteLine("{0}, {1}", pair.IdCode, pair.FileName);
}
Console.ReadLine();
}
/// <summary>
/// gets "first.txt", "txt" and returns "first", "first.txt"
/// gets "first", "txt" and returns "first", "first.txt"
/// it is assumed that extensions will always match
/// </summary>
/// <param name="line"></param>
public static IdCodeFileNamePair GetIdCodeAndFileName(string line, string extension)
{
if (line.Contains("."))
{
List<string> parts = line.BreakIntoParts(".");
List<string> returnItems = new List<string>();
return new IdCodeFileNamePair { IdCode = parts[0], FileName = line };
}
else
{
List<string> returnItems = new List<string>();
return new IdCodeFileNamePair { IdCode = line, FileName = line + "." + extension };
}
}
}
public static class StringHelpers
{
public static List<string> BreakIntoParts(this string line, string separator)
{
if (String.IsNullOrEmpty(line))
return null;
else
{
return line.Split(new string[] { separator }, StringSplitOptions.None).Select(p => p.Trim()).ToList();
}
}
}
public class IdCodeFileNamePair
{
public string IdCode { get; set; }
public string FileName { get; set; }
}
}
Ответы
Ответ 1
Я предпочитаю либо создавать легкий класс с двумя свойствами (см. ниже), либо использовать кортеж (теперь доступный для использования в среде .NET 4, но не сложно написать свой собственный)
class MyReturnValue
{
public string Id { get; set; }
public string Name { get; set; }
}
Ответ 2
Вы можете вернуть tuple, начиная с 4.0.
Ответ 3
Использовать ключевое слово out
http://msdn.microsoft.com/en-us/library/ee332485.aspx
Это лучше, чем приведение определенных элементов в список объектов.
Ответ 4
Другой вариант - вернуть KeyValuePair<int, string>
.
Ответ 5
Почему бы не public static void GetIdCodeAndFileName(string line, string extension, out string id, out string fileName)
?
Ответ 6
Я либо использую params, либо создаю структуру со свойствами (для синтаксиса инициализатора свойства) и возвращаю это. Создание настраиваемой структуры/класса имеет преимущество имен переменных, которые могут соответствовать передаваемым данным. Это делает код более удобочитаемым.
IdAndString GetIDAndString()
{
return new IdAndString()
{
ID = 1,
Str = "123"
};
}
struct IdAndString
{
public int ID { get; set; }
public string Str { get; set; }
}
Ответ 7
Я бы рекомендовал использовать объект Lightweight, такой как Mark. Но есть и другие модели.
Другой простой подход - использовать свойство call by reference. Например, вызывающий объект отправляет пустой массив в качестве параметра, который будет заполнен функцией.
Ответ 8
Как насчет шаблона Pair<T,V>
? Разумеется, для этого можно создавать пользовательские типы данных, если одни и те же будут передаваться, но не как общие.
(Или, может быть, С# позволяет генерировать классы "на лету"? update: ok игнорировать этот бит)
Ответ 9
Просто просто не элегантный способ вернуть два значения в С#. Хотя я определенно считаю, что использование параметров out
лучше, чем возвращение List
, которое не очень удобно или легко читается. Разработчики С# ожидают out
, они напрямую поддерживаются языком, и он хорошо понимает, что он делает.
Начиная с версии 4.0, вы можете использовать Tuple
.