다음번에 짤 때에는 선배님들께서 짜 주신 형태로 짜는것이 좋다는 말씀이신가요? -
허아영
다음부터는 저런 형태가 아니라... 위의 코드와 어떤점에서 다른건지... 한번 생각 해 보라는 소리인듯... -
이승한
선배님들 소스가 장난이 아니에요. 사실 간단한 문제라고 생각했었는데, 고정관념이 깨인듯한 느낌이네요. -
허아영
우우우.. 선배님들 소스 보니까 말이 안나오는..ㅎㅎㅎ
내가 너무 아무생각없이 나눈건가..ㅎㅎㅎ 앞으로 나눌때 신경좀 써야겠다는..;;
(짤때 CAUCSE와 5를 상수로 만들어 볼까라는 생각을 해보긴 했지만서도, 전역변수가 이유없이 늘어나는걸 안좋아 하는데..
거기다 지역변수로 하면 각 함수에 넣어버리니 수정하기가 힘들지 않을까 하는 생각에 그냥 해버렸건만..;;)
-
조현태
CAUCSE 문자열도 함수 입력으로 넣어 주면 좀 더 변화하기 쉬운 소스가 되지. 그리고 message나 messages 같은 함수 이름도 좀 더 명확하게 하는 게 좋을 거 같아. 나 같으면 밑에 처럼 풀 거야. 함수로 추출해 낼 때는 의미 단위로 추출하는 게 좋아. 그래야 함수 이름을 지을 때 편하지. 역으로 함수 이름을 짖기가 힘들다면 잘못된 설계가 되는거지. --재동
~cpp
#include <iostream>
using namespace std;
void printMessages(const char* message, int messageLength);
int main()
{
printMessages("CAUCSE LOVE.\n", 5);
return 0;
}
void printMessages(const char* message, int messageLength)
{
for(int i = 0; i < messageLength; i++)
cout << message;
}