|
기본 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이라는 함수가 없더군요.. 못찾아서 그런가..;;)
|