ส่งวัตถุไปที่ T

ฉันกำลังแยกวิเคราะห์ไฟล์ XML กับXmlReaderคลาสใน. NET และฉันคิดว่าการเขียนฟังก์ชันการแยกวิเคราะห์ทั่วไปเพื่ออ่านแอตทริบิวต์ต่างๆโดยทั่วไปจะเป็นการดี ฉันมาพร้อมกับฟังก์ชั่นต่อไปนี้:

private static T ReadData<T>(XmlReader reader, string value)
{
    reader.MoveToAttribute(value);
    object readData = reader.ReadContentAsObject();
    return (T)readData;
}

เมื่อฉันตระหนักได้ว่าสิ่งนี้ไม่ได้ผลทั้งหมดตามที่ฉันวางแผนไว้ มันแสดงข้อผิดพลาดกับประเภทดั้งเดิมเช่นintหรือdoubleเนื่องจากนักแสดงไม่สามารถแปลงจากประเภทstringเป็นตัวเลขเป็นประเภทตัวเลขได้ มีวิธีใดบ้างที่ฟังก์ชันของฉันจะเหนือกว่าในรูปแบบที่แก้ไข

ก่อนอื่นให้ตรวจสอบว่าสามารถแคสได้หรือไม่

if (readData is T) {
    return (T)readData;
} 
try {
   return (T)Convert.ChangeType(readData, typeof(T));
} 
catch (InvalidCastException) {
   return default(T);
}

คุณลองConvert.ChangeType แล้วหรือยัง?

หากเมธอดส่งคืนสตริงเสมอซึ่งฉันคิดว่าคี่ แต่นั่นเป็นสิ่งที่นอกเหนือจากจุดนั้นบางทีโค้ดที่เปลี่ยนแปลงนี้อาจทำในสิ่งที่คุณต้องการ:

private static T ReadData<T>(XmlReader reader, string value)
{
    reader.MoveToAttribute(value);
    object readData = reader.ReadContentAsObject();
    return (T)Convert.ChangeType(readData, typeof(T));
}

ลอง

if (readData is T)
    return (T)(object)readData;

คุณอาจกำหนดให้ประเภทเป็นประเภทอ้างอิง:

 private static T ReadData<T>(XmlReader reader, string value) where T : class
 {
     reader.MoveToAttribute(value);
     object readData = reader.ReadContentAsObject();
     return (T)readData;
 }

จากนั้นทำอย่างอื่นที่ใช้ประเภทค่าและ TryParse ...

 private static T ReadDataV<T>(XmlReader reader, string value) where T : struct
 {
     reader.MoveToAttribute(value);
     object readData = reader.ReadContentAsObject();
     int outInt;
     if(int.TryParse(readData, out outInt))
        return outInt
     //...
 }

จริงๆแล้วปัญหาคือการใช้ ReadContentAsObject น่าเสียดายที่วิธีนี้ไม่เป็นไปตามความคาดหวัง ในขณะที่ควรตรวจจับประเภทที่เหมาะสมที่สุดสำหรับค่า แต่ก็ส่งคืนสตริงไม่ว่าจะเป็นอะไรก็ตาม (สามารถตรวจสอบได้โดยใช้ตัวสะท้อนแสง)

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

ลองใช้ ReadContentAs แทนซึ่งเป็นสิ่งที่คุณต้องการจริงๆ

private static T ReadData<T>(XmlReader reader, string value)
{
    reader.MoveToAttribute(value);
    object readData = reader.ReadContentAs(typeof(T), null);
    return (T)readData;
}

คุณสามารถพาสอินเป็นพารามิเตอร์ตัวแทนซึ่งจะแปลงจากสตริงเป็น T.

เพิ่มข้อ จำกัด 'คลาส' (หรือมีรายละเอียดมากขึ้นเช่นคลาสพื้นฐานหรืออินเทอร์เฟซของอ็อบเจ็กต์ T exepected):

private static T ReadData<T>(XmlReader reader, string value) where T : class
{
    reader.MoveToAttribute(value);
    object readData = reader.ReadContentAsObject();
    return (T)readData;
}

หรือwhere T : IMyInterfaceหรือwhere T : new()ฯลฯ

อันที่จริงคำตอบทำให้เกิดคำถามที่น่าสนใจซึ่งเป็นสิ่งที่คุณต้องการให้ฟังก์ชันของคุณทำในกรณีที่เกิดข้อผิดพลาด

บางทีมันอาจจะสมเหตุสมผลกว่าที่จะสร้างมันในรูปแบบของเมธอด TryParse ที่พยายามอ่านเป็น T แต่จะส่งคืนเท็จหากไม่สามารถทำได้?

    private static bool ReadData<T>(XmlReader reader, string value, out T data)
    {
        bool result = false;
        try
        {
            reader.MoveToAttribute(value);
            object readData = reader.ReadContentAsObject();
            data = readData as T;
            if (data == null)
            {
                // see if we can convert to the requested type
                data = (T)Convert.ChangeType(readData, typeof(T));
            }
            result = (data != null);
        }
        catch (InvalidCastException) { }
        catch (Exception ex)
        {
            // add in any other exception handling here, invalid xml or whatnot
        }
        // make sure data is set to a default value
        data = (result) ? data : default(T);
        return result;
    }

แก้ไข: ตอนนี้ฉันคิดเกี่ยวกับเรื่องนี้ฉันจำเป็นต้องทำการทดสอบ convert.changetype หรือไม่ as line แล้วไม่ลองทำงั้นเหรอ? ฉันไม่แน่ใจว่าการเรียก changetype เพิ่มเติมนั้นจะทำให้สำเร็จได้ทุกอย่าง อันที่จริงมันอาจเพิ่มค่าใช้จ่ายในการประมวลผลโดยการสร้างข้อยกเว้น หากใครรู้ถึงความแตกต่างที่ทำให้คุ้มค่าโปรดโพสต์!