Вопрос

Я все еще новичок в C #, и я боролся с различными проблемами, связанными с массивами.У меня есть массив объектов метаданных (пары имя-значение), и я хотел бы знать, как создать только то количество объектов "InputProperty", которое мне действительно нужно.В этом цикле я произвольно установил количество элементов равным 20 и пытаюсь выйти из системы, когда запись становится нулевой, но веб-службе на принимающей стороне этого не нравятся никакие переданные ей нулевые элементы:

private Update BuildMetaData(MetaData[] nvPairs)
{
    Update update = new Update();
    InputProperty[] ip = new InputProperty[20];  // how to make this "dynamic"
    int i;
    for (i = 0; i < nvPairs.Length; i++)
    {
        if (nvPairs[i] == null) break;
        ip[i] = new InputProperty();
        ip[i].Name = "udf:" + nvPairs[i].Name;
        ip[i].Val = nvPairs[i].Value;
    }
    update.Items = ip;
    return update;
}

Подводя итог, скажем, у меня есть только 3 пары namevalue в приведенном выше входном массиве?Вместо того, чтобы выделять 20 элементов для массива с именем ip, как можно закодировать это так, чтобы ip был только такого размера, каким он должен быть?Объект update передается через другой веб-сервис, поэтому важна сериализация (т.е.Я не могу использовать namevaluecollection и т.д.).

p.s.Это единственный способ следить за опубликованным вопросом через функцию "добавить комментарии"?

Это было полезно?

Решение

Если вы не хотите использовать List, ArrayList, или другую коллекцию динамического размера, а затем преобразовать в массив (кстати, я бы рекомендовал именно этот метод), затем вам нужно будет выделить массив максимально возможного размера, отслеживать, сколько элементов вы в него поместили, а затем создать новый массив только с этими элементами в нем:

private Update BuildMetaData(MetaData[] nvPairs)
{
    Update update = new Update();
    InputProperty[] ip = new InputProperty[20];  // how to make this "dynamic"
    int i;
    for (i = 0; i < nvPairs.Length; i++)
    {
        if (nvPairs[i] == null) break;
        ip[i] = new InputProperty(); 
        ip[i].Name = "udf:" + nvPairs[i].Name;
        ip[i].Val = nvPairs[i].Value;
    }
    if (i < nvPairs.Length)
    {
        // Create new, smaller, array to hold the items we processed.
        update.Items = new InputProperty[i];
        Array.Copy(ip, update.Items, i);
    }
    else
    {
        update.Items = ip;
    }
    return update;
}

Альтернативным методом было бы всегда назначать update.Items = ip; а затем при необходимости измените размер:

update.Items = ip;
if (i < nvPairs.Length)
{
    Array.Resize(update.Items, i);
}

Это меньше кода, но, скорее всего, в конечном итоге выполнит тот же объем работы (т.е.создание нового массива и копирование старых элементов).

Другие советы

InputProperty[] ip = new InputProperty[nvPairs.Length]; 

Или вы можете использовать список следующим образом:

List<InputProperty> list = new List<InputProperty>();
InputProperty ip = new (..);
list.Add(ip);
update.items = list.ToArray();

Еще одна вещь, на которую я хотел бы обратить внимание, в C # вы можете ограничить использование переменной int в цикле for прямо внутри цикла:

for(int i = 0; i<nvPairs.Length;i++
{
.
.
}

И просто потому, что я в настроении, вот более чистый способ сделать этот метод, IMO:

private Update BuildMetaData(MetaData[] nvPairs)
{
        Update update = new Update();
        var ip = new List<InputProperty>();

        foreach(var nvPair in nvPairs)
        {
            if (nvPair == null) break;
            var inputProp = new InputProperty
            {
               Name = "udf:" + nvPair.Name,
               Val = nvPair.Value
            };
            ip.Add(inputProp);
        }
        update.Items = ip.ToArray();
        return update;
}

Должен ли is быть массивом?Если вы используете ArrayList или один из других объектов, доступных в C #, у вас не будет этого ограничения на контент.Хэш-таблица, идентификатор, IList и т.д..все они допускают динамическое количество элементов.

Используй это:

 Array.Resize(ref myArr, myArr.Length + 5);

Вы могли бы использовать List внутри метода и преобразовать его в массив в конце.Но я думаю, что если мы говорим о максимальном значении 20, то ваш код будет быстрее.

    private Update BuildMetaData(MetaData[] nvPairs)
    {
        Update update = new Update();
        List<InputProperty> ip = new List<InputProperty>();
        for (int i = 0; i < nvPairs.Length; i++)
        {
            if (nvPairs[i] == null) break;
            ip[i] = new InputProperty();
            ip[i].Name = "udf:" + nvPairs[i].Name;
            ip[i].Val = nvPairs[i].Value;
        }
        update.Items = ip.ToArray();
        return update;
    }

Или в C # 3.0 с использованием System.Linq вы можете пропустить промежуточный список:

private Update BuildMetaData(MetaData[] nvPairs)
{
        Update update = new Update();
        var ip = from nv in nvPairs
                 select new InputProperty()
                 {
                     Name = "udf:" + nv.Name,
                     Val = nv.Value
                 };
        update.Items = ip.ToArray();
        return update;
}

Использование Array.CreateInstance для динамического создания массива.

    private Update BuildMetaData(MetaData[] nvPairs)
    {
        Update update = new Update();
        InputProperty[] ip = Array.CreateInstance(typeof(InputProperty), nvPairs.Count()) as InputProperty[];
        int i;
        for (i = 0; i < nvPairs.Length; i++)
        {
            if (nvPairs[i] == null) break;
            ip[i] = new InputProperty();
            ip[i].Name = "udf:" + nvPairs[i].Name;
            ip[i].Val = nvPairs[i].Value;
        }
        update.Items = ip;
        return update;
    }

Как правило, массивам требуются константы для инициализации их размера.Вы могли бы один раз просмотреть nvPairs, чтобы получить длину, затем "динамически" создать массив, используя переменную для длины, подобную этой.

InputProperty[] ip = (InputProperty[])Array.CreateInstance(typeof(InputProperty), length);

Хотя я бы не рекомендовал этого делать.Просто придерживайтесь

List<InputProperty> ip = ...
...
update.Items = ip.ToArray();

решение.Это не намного менее производительно и намного лучше выглядит.

Вы можете создать массив динамически таким образом:

 static void Main()
    {
        // Create a string array 2 elements in length:
        int arrayLength = 2;
        Array dynamicArray = Array.CreateInstance(typeof(int), arrayLength);
        dynamicArray.SetValue(234, 0);                              //  → a[0] = 234;
        dynamicArray.SetValue(444, 1);                              //  → a[1] = 444;
        int number = (int)dynamicArray.GetValue(0);                      //  → number = a[0];


        int[] cSharpArray = (int[])dynamicArray;
        int s2 = cSharpArray[0];

    }
Лицензировано под: CC-BY-SA с атрибуция
Не связан с StackOverflow
scroll top