Linq to object Duplication delete

Question

I have codes like this;

XElement TEST = XElement.Parse(
@"<SettingsBundle>
    <SettingsGroup Id=""QAVerificationSettings"">
        <Setting Id=""RegExRules"">True</Setting>
        <Setting Id=""RegExRules1""><RegExRule><Description>Foo</Description><IgnoreCase>false</IgnoreCase><RegExSource></RegExSource><RegExTarget>[^\s]+@[^\s]+</RegExTarget><RuleCondition>TargetOnly</RuleCondition></RegExRule></Setting>
        <Setting Id=""RegExRules2""><RegExRule><Description>Boo</Description><IgnoreCase>false</IgnoreCase><RegExSource></RegExSource><RegExTarget>\s{2,}</RegExTarget><RuleCondition>TargetOnly</RuleCondition></RegExRule></Setting>
        <Setting Id=""RegExRules2""><RegExRule><Description>Boo</Description><IgnoreCase>false</IgnoreCase><RegExSource></RegExSource><RegExTarget>\s{2,}</RegExTarget><RuleCondition>TargetOnly</RuleCondition></RegExRule></Setting>
    </SettingsGroup>
</SettingsBundle>");
List<XElement> LIST1 = new List<XElement> { };
foreach( XElement x in TEST.Descendants( "RegExRule"))
    LIST1.Add(x);
var LIST2 = LIST1.Distinct();           
var NEW = new XDocument(
    new XElement("SettingsBundle",
        new XElement("SettingsGroup", new XAttribute("Id", "QAVerificationSettings"),
            new XElement("Settings", new XAttribute("Id", "RegExRules"), "True"),
            LIST2.Select((x, i) => new XElement("Setting", new XAttribute("Id", "RegExRules" + i ), x ))
    )));
NEW.Save(Directory.GetCurrentDirectory() + @"\_Texting.xml");

I'd like to delete one of the same item (I need only one "RegExRules2" not two).

But, have failed.

What do I have to do ? (I guess, I am not good at with "Distinct()")

Thanks

Nothing to do with the quesiton, but: Why do you use capitalized variable names? — Markus Weninger, Feb 21 '16 at 13:12
@MarkusWeninger Hmmm. I do not have any special reasons. I just made it. — Jason, Feb 21 '16 at 13:13
Have a look at the [Naming Guidelines](https://msdn.microsoft.com/en-us/library/xzf533w0(v=vs.71).aspx) to make your code more readable for others, just as advice. :) — Markus Weninger, Feb 21 '16 at 13:14
@MarkusWeninger Thanks for the info. I'll check it out with leisure. — Jason, Feb 21 '16 at 13:15

score 2 · Accepted Answer · answered Feb 21 '16 at 13:35

You can do GroupBy and use only the first item in each group to get distinct elements :

XNamespace ns = "http://schemas.datacontract.org/2004/07/Sdl.Verification.QAChecker.RegEx";

var distinctRules = TEST.Descendants(ns + "RegExRule")
                        .GroupBy(o => o.ToString())
                        .Select(o => o.First());
var result = new XDocument(
    new XElement("SettingsBundle",
        new XElement("SettingsGroup", new XAttribute("Id", "QAVerificationsettings"),
            new XElement("Settings", new XAttribute("Id", "RegExRules"), "True"),
            distinctRules.Select((x, i) => new XElement("Setting", new XAttribute("Id", "RegExRules" + i), x))
    )));

result.Save(Directory.GetCurrentDirectory() + @"\_Texting.xml");

Thanks for good answer (sorry, I just deleted all the NameSpace). I will check it out now. — Jason, Feb 21 '16 at 13:37

score 1 · Answer 2 · edited May 23 '17 at 11:50

Another possibility would be to create an IEqualityComparer for XElement.

A really basic example (which you shouldn't treat as a good example for an implementation of GetHashCode. Look here for that) would be:

public class XElementComparer : IEqualityComparer<XElement>
{

    public bool Equals(XElement x, XElement y)
    {
        return x.Parent.Attribute("Id").Value == y.Parent.Attribute("Id").Value;
    }

    public int GetHashCode(XElement obj)
    {
        string val = obj.Parent.Attribute("Id").Value;
        return val.GetHashCode();
    }
}

And in your Distinct operation would afterwards look like this: var LIST2 = LIST1.Distinct(new XElementComparer());

Linq to object Duplication delete

2 Answers2