แทนที่องค์ประกอบสตริงหลายรายการใน C #

มีวิธีที่ดีกว่านี้ไหม ...

MyString.Trim().Replace("&", "and").Replace(",", "").Replace("  ", " ")
         .Replace(" ", "-").Replace("'", "").Replace("/", "").ToLower();

ฉันได้ขยายคลาสสตริงเพื่อให้มันเหลือเพียงงานเดียว แต่มีวิธีที่เร็วกว่านี้ไหม

public static class StringExtension
{
    public static string clean(this string s)
    {
        return s.Replace("&", "and").Replace(",", "").Replace("  ", " ")
                .Replace(" ", "-").Replace("'", "").Replace(".", "")
                .Replace("eacute;", "é").ToLower();
    }
}

เพียงเพื่อความสนุกสนาน (และเพื่อหยุดการโต้แย้งในความคิดเห็น) ฉันได้เพิ่มส่วนสำคัญในการเปรียบเทียบตัวอย่างต่างๆด้านล่าง

https://gist.github.com/ChrisMcKee/5937656

ตัวเลือก regex ทำคะแนนได้แย่มาก ตัวเลือกพจนานุกรมเกิดขึ้นเร็วที่สุด การเปลี่ยนสตริงบิลด์เดอร์รุ่นยาวจะเร็วกว่ามือสั้นเล็กน้อย

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

หากคุณคาดหวังว่าวิธีนี้จะถูกเรียกใช้อย่างจริงจังโดยStringsมีความยาวที่มีนัยสำคัญหลายอย่างการ "ย้าย" การนำไปใช้งานไปยังStringBuilderคลาสนั้นอาจเป็นการดีกว่า ด้วยการปรับเปลี่ยนใด ๆ จะดำเนินการโดยตรงกับอินสแตนซ์นั้นดังนั้นคุณจึงต้องสำรองการทำสำเนาที่ไม่จำเป็น

public static class StringExtention
{
    public static string clean(this string s)
    {
        StringBuilder sb = new StringBuilder (s);

        sb.Replace("&", "and");
        sb.Replace(",", "");
        sb.Replace("  ", " ");
        sb.Replace(" ", "-");
        sb.Replace("'", "");
        sb.Replace(".", "");
        sb.Replace("eacute;", "é");

        return sb.ToString().ToLower();
    }
}

หากคุณเป็นเพียงวิธีแก้ปัญหาที่สวยงามและไม่จำเป็นต้องประหยัดเพียงไม่กี่นาโนวินาทีน้ำตาล LINQ ล่ะ?

var input = "test1test2test3";
var replacements = new Dictionary<string, string> { { "1", "*" }, { "2", "_" }, { "3", "&" } };

var output = replacements.Aggregate(input, (current, replacement) => current.Replace(replacement.Key, replacement.Value));

สิ่งนี้จะมีประสิทธิภาพมากขึ้น:

public static class StringExtension
{
    public static string clean(this string s)
    {
        return new StringBuilder(s)
              .Replace("&", "and")
              .Replace(",", "")
              .Replace("  ", " ")
              .Replace(" ", "-")
              .Replace("'", "")
              .Replace(".", "")
              .Replace("eacute;", "é")
              .ToString()
              .ToLower();
    }
}

อาจจะอ่านได้มากกว่านี้?

    public static class StringExtension {

        private static Dictionary<string, string> _replacements = new Dictionary<string, string>();

        static StringExtension() {
            _replacements["&"] = "and";
            _replacements[","] = "";
            _replacements["  "] = " ";
            // etc...
        }

        public static string clean(this string s) {
            foreach (string to_replace in _replacements.Keys) {
                s = s.Replace(to_replace, _replacements[to_replace]);
            }
            return s;
        }
    }

เพิ่มคำแนะนำของ New In Town เกี่ยวกับ StringBuilder ...

มีสิ่งหนึ่งที่อาจได้รับการปรับให้เหมาะสมในโซลูชันที่แนะนำ การมีการเรียกหลายครั้งเพื่อให้Replace()รหัสทำการส่งผ่านหลายสายในสตริงเดียวกัน ด้วยสตริงที่ยาวมากการแก้ปัญหาอาจช้าเนื่องจากความจุแคชของ CPU ขาดหายไป อาจจะเป็นหนึ่งควรพิจารณาเปลี่ยนสายหลายในบัตรเดียว

อีกทางเลือกหนึ่งที่ใช้ linq คือ

[TestMethod]
public void Test()
{
  var input = "it's worth a lot of money, if you can find a buyer.";
  var expected = "its worth a lot of money if you can find a buyer";
  var removeList = new string[] { ".", ",", "'" };
  var result = input;

  removeList.ToList().ForEach(o => result = result.Replace(o, string.Empty));

  Assert.AreEqual(expected, result);
}

ฉันกำลังทำบางอย่างที่คล้ายกัน แต่ในกรณีของฉันฉันกำลังทำ Serialization / De-serialization ดังนั้นฉันต้องสามารถไปทั้งสองทิศทางได้ ฉันพบว่าการใช้สตริง [] [] ทำงานได้เกือบจะเหมือนกันกับพจนานุกรมรวมถึงการเริ่มต้น แต่คุณสามารถไปในทิศทางอื่นได้เช่นกันโดยคืนค่าสิ่งทดแทนกลับเป็นค่าดั้งเดิมซึ่งเป็นสิ่งที่พจนานุกรมไม่ได้ตั้งค่าให้ทำ

แก้ไข: คุณสามารถใช้Dictionary<Key,List<Values>>เพื่อให้ได้ผลลัพธ์เช่นเดียวกับสตริง [] []

string input = "it's worth a lot of money, if you can find a buyer.";
for (dynamic i = 0, repl = new string[,] { { "'", "''" }, { "money", "$" }, { "find", "locate" } }; i < repl.Length / 2; i++) {
    input = input.Replace(repl[i, 0], repl[i, 1]);
}