bool User::Set_User_Info()
{
string user_name;
cout << "Please write new user's name : ";
while(cin >> user_name)
{
if(user_name != "")
{
User::name = user_name;
return true;
}
else
return false;
}
}
함수 개좆만 O
string 씀 O
간결하고 보기 좋음 (__)
뭐임? 이거를 어떻게 고칠까요?
네? 네? 네~!!!
ㄴㄴ ㄱㅅ - dc App
ㄴ 들여쓰기있는데 왜 디씨에 넣으면 저러는지 모르겠음 VSCODE쓴다 - dc App
들여쓰기 텝으로 했으면 텝 스페이스로 바꿔넣는 옵션 켜라.
객체 지향적인 코딩이 어렵긴 하지만 대표적으로 저렇게 매소드를 작성하면 안되는 케이스를 잘보여주신것
객체는 스스로 초기화할수 있는부분은 초기화하고 나머지 데이터는 외부로부터 받아와야함
만약에 저게 GUI 환경이었으면 , 혹은 서버환경이었으면, 데이터를 저렇게 초기화하는게 가능할까?
ㄴ 나도 그렇게 셍각했음 거마움 다시 작성래서 올릴게 - dc App
bool User::Set_User_Info( const string& username ){ name = username; } 이렇게 바꾸는게 좋고. 객체 내부에서 cin을 통해 입력받은것은 하지 않는게 좋음
객체지향은 다른거 없고 '객체를 프로그램의 독립적인 부품'으로 잘만드는것이 핵심임 User는 User가 할일만 만들어 놓으면 되고, 위에 네이노옴님 처럼 절차대로 해야할 프로그램 뼈대는 관리자 클래스같은것을 만들어서 거기서 처리하는것이 맞음
아~!!! 씨발 형님들 이해했음 고마움 - dc App