C++Builder Programming Forum
C++Builder  |  Delphi  |  FireMonkey  |  C/C++  |  Free Pascal  |  Firebird
볼랜드포럼 BorlandForum
 경고! 게시물 작성자의 사전 허락없는 메일주소 추출행위 절대 금지
C++빌더 포럼
Q & A
FAQ
팁&트릭
강좌/문서
자료실
컴포넌트/라이브러리
메신저 프로젝트
볼랜드포럼 홈
헤드라인 뉴스
IT 뉴스
공지사항
자유게시판
해피 브레이크
공동 프로젝트
구인/구직
회원 장터
건의사항
운영진 게시판
회원 메뉴
북마크
볼랜드포럼 광고 모집

C++빌더 Q&A
C++Builder Programming Q&A
[64233] Re:Re:이건 문제있는 코드일까요? (문자열 처리 관련)
용맨소녀 [doyongid] 1082 읽음    2011-04-25 13:14
답변 감사합니다. 저도 보니까 strcpy는 저기다 왜 넣었는지 모르겠네요.. ;;

리턴하자마자 다른 변수에 복사하면 괜찮을 것 같다는 혼자만의 생각으로 했는데, 역시 문제가 생기네요..

인자로 넘어온 pString을 그냥 소문자로 편집해서 pString 자체를 리턴하면 될까요? 그럼 원본 문자열의 내용이 뒤바뀔 것 같고.. 따로 리턴 인자를 하나 추가해야 하는거면.. 또 거시기해지네요..

방법이 없다면 그렇게라도 해야겠네요..;;

장성호 님이 쓰신 글 :
: 문제있는 코드입니다.
:
:
: 1. 첫째
:
: char TempString[1024] 는 지역변수로써 strlwr 함수를 빠져나가면 자동으로 사라지게 되겠죠
:
: 다음과 같이 코딩하고
: lpRetString    = TempString;    
: return lpRetString;
:
: 위에서 lpRetString은 TempString입니다.
: strlwr 함수 밖에서 retrun받은 값을 사용하지 않는다면 모를까 사용하게 된다면..
: 문제가 심각하지요
:
:
: 2. 둘째
:    TempString을 1024 크기로 잡았습니다.
:    pString으로 넘어오는 문자열의 크기가 1024보다 작아야 겠죠
:    만약 1024보다 크면 Buffer overflow 에러가 나겠죠?
:
:
: 3. 셋째
:    문제는 아니지만.. 쓸데없는 코드가 많네요
:    3.1) TempString을 바로 return하면 될텐데.. 왜 lpRetString에 담아서 return하는지...
:           (TempString을 return하는것 자체가 근본적으로 문제이긴 합니다만..)
:    3.2)  strcpy에서 문자열을 한번 copy하면서 전체 scan하구
:           strlen에서 길이계산하면서 전체scan 하구
:           for문 돌면서 또한번 전체 scan하구..
:        
:          반복 코딩이 많네요
:          루프한번 돌면서 처리할수도 있을텐데...
:
: 그럼..
:
:  
: 용맨소녀 님이 쓰신 글 :
: : char *strlwr(char *pString)
: : {
: :     char    TempString[1024], *lpRetString;
: :     int        Length    = strlen(pString);
: :    
: :     strcpy(TempString, pString);
: :    
: :     for(int i = 0; i < Length; i++)
: :     {
: :         if((BYTE)TempString[i] >= 0x80)
: :             i++;
: :         else
: :         {
: :             if(TempString[i] >= 65 && TempString[i] <= 90)
: :                 TempString[i] += 32;
: :         }
: :     }
: :    
: :     lpRetString    = TempString;
: :    
: :     return lpRetString;
: : }
: :
: : char VoiceName[128];
: : sprintf(VoiceName, "%s.mp3", strlwr(UnitName[Thema][lpSelectUnit->Index]));
: :
: : "banana.mp3"라고 한다면... 잘된 코드였는데.. 요즘, 버전업 작업중인데 지금은  "bana.mp3만 찍힙니다."
: :
: : 문자열이 4자로 줄어들어요..
: :
: : 그래서 이렇게 고치니까 잘됩니다. 
: :
: : strcpy(VoiceName, strlwr(UnitName[Thema][lpSelectUnit->Index]));
: : strcat(VoiceName, ".mp3");
: :
: : 좀 미심쩍은 부분이 있긴한데, 정확히 왜 그럴까요.. (XCode에는 strlwr이라는 함수가 없더군요.. 못찾아서 그런가..;;)

+ -

관련 글 리스트
64231 이건 문제있는 코드일까요? (문자열 처리 관련) 용맨소녀 1450 2011/04/25
64232     Re:이건 문제있는 코드일까요? (문자열 처리 관련) 장성호 1103 2011/04/25
64233         Re:Re:이건 문제있는 코드일까요? (문자열 처리 관련) 용맨소녀 1082 2011/04/25
64235             Re:Re:Re:이건 문제있는 코드일까요? (문자열 처리 관련) 장성호 1383 2011/04/25
Google
Copyright © 1999-2015, borlandforum.com. All right reserved.