아래 코드가 동작은 하는 코드이긴 하지만 가독성이 너무 떨어집니다. 파일 스트림 부분이 나눠서 작성되어 있기도 하고 중복되는 부분 때문인듯 합니다.

제 아이디어로는 파일 스트림부분을 클래스 멤버 변수?(함수?) 로 사용하고 파일이 열수있는지 체크하는 부분을 함수로 만들면 될 것 같긴한데 초보자라 어떻게 손을 대야할지 모르겠습니다.

void RegisterAccount::createAccount() {
    ofstream accountFile("pass.dat", ios::out /*| ios::binary */ | ios::app);   // ID 와 암호화된 비번이 기록될 파일
    if (!accountFile) {
        cerr << "File could not be opened" << endl;
        exit(EXIT_FAILURE);
    } // 파일스트림 부분의 코드를 깔끔하게 정리할 수 있는 방법은 뭘까요..

    if (inputID() == true) {    // getID()에 userID가 들어오는 시점

        string fileNameToUserName = (getID() + ".dat"); 
        ofstream userInfoFile(fileNameToUserName.c_str(), ios::out /*| ios::binary*/ | ios::app); // ID, 이메일, 계정권한, 예약내역이 기록될 파일
        if (!userInfoFile) {
            cerr << "File could not be opened" << endl;
            exit(EXIT_FAILURE);
        }   // 파일스트림 부분의 코드를 깔끔하게 정리할 수 있는 방법은 뭘까요..

        if (inputPW() == true) {
            if (inputEmail() == true) {
                accountFile << getID() << endl;
                accountFile << cipher(getPW()) << endl; // 암호화하여 파일에 입력

                userInfoFile << getID() << endl;
                userInfoFile << getEmail() << endl;
                userInfoFile << getAuthInfo() << endl;
            }
        }
    }
    accountFile.close();
}