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
[64235] Re:Re:Re:이건 문제있는 코드일까요? (문자열 처리 관련)
장성호 [nasilso] 1383 읽음    2011-04-25 15:07
기본 strlwr함수를 보니
파라메터로 넘겨주는 값
즉 원본 문자열의 내용을 변경하네요

return값도 원본 char* 그대로 return하구요


용맨소녀 님이 쓰신 글 :
: 답변 감사합니다. 저도 보니까 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:이건 문제있는 코드일까요? (문자열 처리 관련) 용맨소녀 1083 2011/04/25
64235             Re:Re:Re:이건 문제있는 코드일까요? (문자열 처리 관련) 장성호 1383 2011/04/25
Google
Copyright © 1999-2015, borlandforum.com. All right reserved.