c# Reflection deep copy set Datetime values

Question

I am trying to deep copy a complex object that has some date properties. I am getting "The value '' could not be converted to a valid date" error. I am using below code for copying:-

private static object CloneProcedure(Object obj)
{
    if (type.IsPrimitive || type.IsEnum || type == typeof(string))
    {
        return obj;
    }
    else if (type.IsClass || type.IsValueType)
    {
        object copiedObject = Activator.CreateInstance(obj.GetType());
        // Get all PropertyInfo.
        PropertyInfo[] properties = type.GetProperties(BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance);
        foreach (PropertyInfo property in properties)
        {
            object propertyValue = property.GetValue(obj);
            if (propertyValue != null && property.CanWrite && property.GetSetMethod() != null)
            {
                property.SetValue(copiedObject, CloneProcedure(propertyValue));
            }
        }
    }
}

Am I missing something?

It would help if you provide the CloneProcedure-method. – Christoph Sonntag Nov 08 '17 at 21:18 — Christoph Sonntag, Nov 08 '17 at 21:18
@Compufreak Updated my question – Kavya Shetty Nov 08 '17 at 21:21 — Kavya Shetty, Nov 08 '17 at 21:21

score 0 · Answer 1 · answered Nov 08 '17 at 21:43

You are not returing anything if your if is false.

This code is working in a .Net 4.5.2 Console application :

public class Program
{
    public static void Main()
    {
        ComplexClass t = new ComplexClass()
        {
            x = DateTime.Now
        };
        object t2 = CloneProcedure(t);
        Console.WriteLine(t.x);
        Console.ReadLine();
    }
    private static object CloneProcedure(Object obj)
    {
        var type = obj.GetType();
        if (type.IsPrimitive || type.IsEnum || type == typeof(string))
        {
            return obj;
        }
        else if (type.IsClass || type.IsValueType)
        {
            object copiedObject = Activator.CreateInstance(obj.GetType());
            // Get all PropertyInfo.
            PropertyInfo[] properties = type.GetProperties(BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance);
            foreach (PropertyInfo property in properties)
            {
                object propertyValue = property.GetValue(obj);
                if (propertyValue != null && property.CanWrite && property.GetSetMethod() != null)
                {
                    property.SetValue(copiedObject, CloneProcedure(propertyValue));
                }
            }
        }
        return obj;
    }

    public class ComplexClass
    {
        public DateTime x { get; set; }
    }
}

c# Reflection deep copy set Datetime values

1 Answers1