Microsoft MVP성태의 닷넷 이야기
리플렉션 성능 관련 질문 드립니다. [링크 복사], [링크+제목 복사],
조회: 21946
글쓴 사람
popo
홈페이지
첨부 파일
 

안녕하세요 다음과 같이 DataTable을 특정 모델 클래스의 List<T>타입으로 Convert해서 사용하고 있습니다.

그런데 성능상 데이터가 많은 편이 아닌데도 느린 부분이 있어 혹시 어디 부분에서 성능이 떨어지는지 도움 요청드립니다.

소스는 다음과 같고, 모델 클래스의 각각 Property에 DataTable의 해더명을 어트리뷰트로 정의해서 자동으로 Property의 타입에 맞게 캐스팅해서 속성을 채우는 방식입니다.

테스트시 DataTable에 약 50개 정도의 Row를 List<T>형식으로 변환 했을때 1s ~ 2s사이 정도 시간이 지연 됬습니다.

소스에 문제가 되는 부분이 있는지 도움 요청 드립니다.

public static List<T> DataTableToList<T>(DataTable table) where T : class, new()
        {
            if (table == null) return null;

            try
            {
                List<T> list = new List<T>();

                foreach (DataRow row in table.AsEnumerable())
                {
                    T obj = new T();

                    foreach (PropertyInfo prop in obj.GetType().GetProperties())
                    {
                        try
                        {
                            string headerCaption = null;

                            // 모델 클래스(T) 속성에 설정되어 있는 어트리뷰트 Get
                            object[] atts = prop.GetCustomAttributes(typeof(ExtensionEnumAttribute), false);
                            if (atts != null && atts.Length <= 0)
                            {
                                continue;
                            }

                            ExtensionEnumAttribute att = (ExtensionEnumAttribute)atts[0];
                            if (att == null || att.Objs == null || att.Objs.Length < 0 || string.IsNullOrWhiteSpace(att.Objs[0].ToString()))
                            {
                                headerCaption = prop.Name;
                            }
                            else
                            {
                                // 속성에 설정되어 있는 컬럼 헤더 타이틀 어리트뷰트 값
                                headerCaption = att.Objs[0].ToString();
                            }

                            PropertyInfo propertyInfo = obj.GetType().GetProperty(prop.Name);

                            if (propertyInfo.PropertyType.IsEnum)
                            {
                                object enumVal = Enum.Parse(propertyInfo.PropertyType, row[headerCaption].ToString());
                                propertyInfo.SetValue(obj, enumVal, null);
                            }
                            else
                            {
                                Type t = Nullable.GetUnderlyingType(propertyInfo.PropertyType) ?? propertyInfo.PropertyType;

                                if (row[headerCaption] != DBNull.Value)
                                {
                                    propertyInfo.SetValue(obj, Convert.ChangeType(row[headerCaption], t), null);
                                }
                                else
                                {
                                    propertyInfo.SetValue(obj, null, null);
                                }
                            }
                        }
                        catch(Exception ex)
                        {
                            // 데이터 테이블에 없는 모델 클래스(T) 속성
                            // 예] IsSelectCheck 속성 등..
                            continue;
                        }
                    }

                    list.Add(obj);
                }

                return list;
            }
            catch(Exception ex)
            {
                throw new Exception("데이터 테이블 -> 모델 변환 오류");
            }
        }








[최초 등록일: ]
[최종 수정일: 5/26/2017]


비밀번호

댓글 작성자
 



2017-05-26 02시32분
직접 재보시는 것이 더 빠르지 않을까요? ^^ Stopwatch 클래스를 이용해 어느 부분에서 시간이 더 걸리는 것인지 밝혀낸 다음 왜 그런지 물어보시면 더 좋을 것 같습니다.

참고로, .NET Reflection은 원래 느립니다. (느리다는 말이 매우 상대적인 기준이긴 하지만.)

아울러 다음의 글들도 한번 읽어보시고.

C# 4.0 - dynamic 키워드
; http://www.sysnet.pe.kr/2/0/766

C# - Reflection의 박싱 없이 값 형식을 다루는 방법
; http://www.sysnet.pe.kr/2/0/10866
정성태
2017-05-26 08시47분
일단 리플렉션을 메서드 사용을 최소화하는 게 중요해 보입니다.
GetProperties() 나 GetCustomAttributes() 같은 것들을 캐쉬 처리할 수 있으면 좋을 것 같습니다.
그리고 정성태님이 열거해주신 링크 중 Reflection의 박싱 없이 값 형식을 다루는 방법...에 나오는 LCG 기법이면 박싱도 해결될테구요.
어떤 방식으로 처리하는지 실제 사례를 확인해보시고 싶으시면 github의 dapper를 fork 하셔서 ormapping 하는 코드를 뒤져보시면 도움이 되지 않을까 싶습니다.
Beren Ko

... 91  92  93  94  [95]  96  97 
NoWriterDateCnt.TitleFile(s)
78임은선1/26/200513623        답변글 [답변]: [답변]: 질문이 있는데요....
72최정희1/19/200513073또 접니다.
73정성태1/19/200512875    답변글 [답변]: 또 접니다. [2]
74최정희1/19/200512229        답변글 [답변]: [답변]: 당연히 도움이 되지요.
75정성태1/19/200512769            답변글 [답변]: [답변]: [답변]: 당연히 도움이 되지요.
69최정희10/13/200412216저 기억하실런지? 그때 두리로IE툴바 만들어 주셨죠.
62엄길웅10/1/200413391매번 물어보기만 하는군요..
63정성태10/2/200413152    답변글 [답변]: 매번 물어보기만 하는군요..
64keal...10/5/200414813        답변글 [답변]: [답변]: 매번 물어보기만 하는군요..
65정성태10/6/200414650            답변글 [답변]: [답변]: [답변]: 매번 물어보기만 하는군요..
68keal...10/11/200412365                답변글 [답변]: [답변]: [답변]: [답변]: 매번 물어보기만 하는군요..
71정성태10/31/200412426                    답변글 [답변]: [답변]: [답변]: [답변]: [답변]: 매번 물어보기만 하는군요..
58엄길웅9/30/200412569안녕하세요.. 추석은 잘 보내셨는지요?
59정성태9/30/200412695    답변글 [답변]: 안녕하세요.. 추석은 잘 보내셨는지요?
60엄길웅10/1/200412558        답변글 [답변]: [답변]: 안녕하세요.. 추석은 잘 보내셨는지요?
56엄길웅9/23/200412895ATL을 이용해서 함수를 하나 만들었습니다.
57정성태9/23/200413921    답변글 [답변]: ATL을 이용해서 함수를 하나 만들었습니다.
61엄길웅10/1/200412583        답변글 [답변]: [답변]: ATL을 이용해서 함수를 하나 만들었습니다.
54차광석8/26/200412767잘 지내시죠?
55정성태8/27/200411853    답변글 [답변]: 잘 지내시죠?
52김명원8/3/200412697질문이요..
53정성태8/19/200412543    답변글 [답변]: 질문이요..
50김상표7/5/200412395질문요...
51정성태7/6/200412252    답변글 [답변]: 질문요...
45송야5/29/200412396웹페이지 자동로그인..
46정성태5/31/200413310    답변글 [답변]: 웹페이지 자동로그인..
... 91  92  93  94  [95]  96  97