วิธีแทนที่รายการด้วยวิธีที่ดีที่สุด

if (listofelements.Contains(valueFieldValue.ToString()))
{
    listofelements[listofelements.IndexOf(valueFieldValue.ToString())] = value.ToString();
}

ฉันได้แทนที่เหมือนข้างบน มีวิธีอื่นใดที่จะเปรียบเทียบได้ดีกว่าทางนี้

ใช้ Lambda เพื่อค้นหาดัชนีในรายการและใช้ดัชนีนี้เพื่อแทนที่รายการ

List<string> listOfStrings = new List<string> {"abc", "123", "ghi"};
listOfStrings[listOfStrings.FindIndex(ind=>ind.Equals("123"))] =  "def";

คุณสามารถทำให้อ่านง่ายขึ้นและมีประสิทธิภาพมากขึ้น:

string oldValue = valueFieldValue.ToString();
string newValue = value.ToString();
int index = listofelements.IndexOf(oldValue);
if(index != -1)
    listofelements[index] = newValue;

สิ่งนี้ถามเพียงครั้งเดียวสำหรับดัชนี แนวทางของคุณใช้Containsก่อนซึ่งจำเป็นต้องวนซ้ำรายการทั้งหมด (ในกรณีที่เลวร้ายที่สุด) จากนั้นคุณกำลังใช้IndexOfสิ่งที่ต้องระบุรายการอีกครั้ง

คุณกำลังเข้าถึงรายการของคุณสองครั้งเพื่อแทนที่องค์ประกอบหนึ่ง ฉันคิดว่าการforวนซ้ำแบบง่ายควรจะเพียงพอ:

var key = valueFieldValue.ToString();
for (int i = 0; i < listofelements.Count; i++)
{
    if (listofelements[i] == key)
    {
        listofelements[i] = value.ToString();
        break;
    }
}

ทำไมไม่ใช้วิธีการขยาย?

พิจารณารหัสต่อไปนี้:

        var intArray = new int[] { 0, 1, 1, 2, 3, 4 };
        // Replaces the first occurance and returns the index
        var index = intArray.Replace(1, 0);
        // {0, 0, 1, 2, 3, 4}; index=1

        var stringList = new List<string> { "a", "a", "c", "d"};
        stringList.ReplaceAll("a", "b");
        // {"b", "b", "c", "d"};

        var intEnum = intArray.Select(x => x);
        intEnum = intEnum.Replace(0, 1);
        // {0, 0, 1, 2, 3, 4} => {1, 1, 1, 2, 3, 4}

• ไม่มีการทำซ้ำรหัส
• ไม่จำเป็นต้องพิมพ์นิพจน์ linq ยาว ๆ
• ไม่จำเป็นต้องมีการใช้งานเพิ่มเติม

ซอร์สโค้ด:

namespace System.Collections.Generic
{
    public static class Extensions
    {
        public static int Replace<T>(this IList<T> source, T oldValue, T newValue)
        {
            if (source == null)
                throw new ArgumentNullException(nameof(source));

            var index = source.IndexOf(oldValue);
            if (index != -1)
                source[index] = newValue;
            return index;
        }

        public static void ReplaceAll<T>(this IList<T> source, T oldValue, T newValue)
        {
            if (source == null)
                throw new ArgumentNullException(nameof(source));

            int index = -1;
            do
            {
                index = source.IndexOf(oldValue);
                if (index != -1)
                    source[index] = newValue;
            } while (index != -1);
        }


        public static IEnumerable<T> Replace<T>(this IEnumerable<T> source, T oldValue, T newValue)
        {
            if (source == null)
                throw new ArgumentNullException(nameof(source));

            return source.Select(x => EqualityComparer<T>.Default.Equals(x, oldValue) ? newValue : x);
        }
    }
}

มีการเพิ่มสองวิธีแรกเพื่อเปลี่ยนอ็อบเจ็กต์ของประเภทการอ้างอิงในตำแหน่ง แน่นอนคุณสามารถใช้เพียงวิธีที่สามสำหรับทุกประเภท

ปล. ขอบคุณการสังเกตของไมค์ฉันได้เพิ่มเมธอด ReplaceAll

ตามคำตอบของ rokkuchan อัปเกรดเพียงเล็กน้อย:

List<string> listOfStrings = new List<string> {"abc", "123", "ghi"};

int index = listOfStrings.FindIndex(ind => ind.Equals("123"));
if (index > -1)
    listOfStrings[index] =  "def";

ใช้FindIndexและแลมบ์ดาเพื่อค้นหาและแทนที่ค่าของคุณ:

int j = listofelements.FindIndex(i => i.Contains(valueFieldValue.ToString())); //Finds the item index

lstString[j] = lstString[j].Replace(valueFieldValue.ToString(), value.ToString()); //Replaces the item by new value

คุณสามารถใช้ส่วนขยายถัดไปซึ่งเป็นไปตามเงื่อนไขเพรดิเคต:

    /// <summary>
    /// Find an index of a first element that satisfies <paramref name="match"/>
    /// </summary>
    /// <typeparam name="T">Type of elements in the source collection</typeparam>
    /// <param name="this">This</param>
    /// <param name="match">Match predicate</param>
    /// <returns>Zero based index of an element. -1 if there is not such matches</returns>
    public static int IndexOf<T>(this IList<T> @this, Predicate<T> match)
    {
        @this.ThrowIfArgumentIsNull();
        match.ThrowIfArgumentIsNull();

        for (int i = 0; i < @this.Count; ++i)
            if (match(@this[i]))
                return i;

        return -1;
    }

    /// <summary>
    /// Replace the first occurance of an oldValue which satisfies the <paramref name="removeByCondition"/> by a newValue
    /// </summary>
    /// <typeparam name="T">Type of elements of a target list</typeparam>
    /// <param name="this">Source collection</param>
    /// <param name="removeByCondition">A condition which decides is a value should be replaced or not</param>
    /// <param name="newValue">A new value instead of replaced</param>
    /// <returns>This</returns>
    public static IList<T> Replace<T>(this IList<T> @this, Predicate<T> replaceByCondition, T newValue)
    {
        @this.ThrowIfArgumentIsNull();
        removeByCondition.ThrowIfArgumentIsNull();

        int index = @this.IndexOf(replaceByCondition);
        if (index != -1)
            @this[index] = newValue;

        return @this;
    }

    /// <summary>
    /// Replace all occurance of values which satisfy the <paramref name="removeByCondition"/> by a newValue
    /// </summary>
    /// <typeparam name="T">Type of elements of a target list</typeparam>
    /// <param name="this">Source collection</param>
    /// <param name="removeByCondition">A condition which decides is a value should be replaced or not</param>
    /// <param name="newValue">A new value instead of replaced</param>
    /// <returns>This</returns>
    public static IList<T> ReplaceAll<T>(this IList<T> @this, Predicate<T> replaceByCondition, T newValue)
    {
        @this.ThrowIfArgumentIsNull();
        removeByCondition.ThrowIfArgumentIsNull();

        for (int i = 0; i < @this.Count; ++i)
            if (replaceByCondition(@this[i]))
                @this[i] = newValue;

        return @this;
    }

หมายเหตุ: - แทนที่จะเป็นส่วนขยาย ThrowIfArgumentIsNull คุณสามารถใช้วิธีการทั่วไปเช่น:

if (argName == null) throw new ArgumentNullException(nameof(argName));

ดังนั้นกรณีของคุณที่มีส่วนขยายเหล่านี้สามารถแก้ไขได้ดังนี้:

string targetString = valueFieldValue.ToString();
listofelements.Replace(x => x.Equals(targetString), value.ToString());

ฉันไม่รู้ว่ามันดีที่สุดหรือไม่ แต่คุณสามารถใช้มันได้เช่นกัน

List<string> data = new List<string>
(new string[]   { "Computer", "A", "B", "Computer", "B", "A" });
int[] indexes = Enumerable.Range(0, data.Count).Where
                 (i => data[i] == "Computer").ToArray();
Array.ForEach(indexes, i => data[i] = "Calculator");

หรือสร้างตามคำแนะนำของ Rusian L. หากรายการที่คุณกำลังค้นหาสามารถอยู่ในรายการมากกว่าหนึ่งครั้ง ::

[Extension()]
public void ReplaceAll<T>(List<T> input, T search, T replace)
{
    int i = 0;
    do {
        i = input.FindIndex(i, s => EqualityComparer<T>.Default.Equals(s, search));

        if (i > -1) {
            FileSystem.input(i) = replace;
            continue;
        }

        break;  
    } while (true);
}

คุณสามารถใช้แลมบ์ดานิพจน์เช่นนี้

int index = listOfElements.FindIndex(item => item.Id == id);  
if (index != -1) 
{
    listOfElements[index] = newValue;
}

ฉันคิดว่าดีที่สุดสำหรับการทำอย่างรวดเร็วและเรียบง่าย

1. ค้นหารายการของคุณในรายการ

   var d = Details.Where(x => x.ProductID == selectedProduct.ID).SingleOrDefault();
   

2. สร้างโคลนจากปัจจุบัน

   OrderDetail dd = d;
   

3. อัปเดตโคลนของคุณ

   dd.Quantity++;
   

4. ค้นหาดัชนีในรายการ

   int idx = Details.IndexOf(d);
   

5. ลบรายการที่ก่อตั้งใน (1)

     Details.Remove(d);
   

6. แทรก

    if (idx > -1)
         Details.Insert(idx, dd);
     else
         Details.Insert(Details.Count, dd);