Gogumalang / OSSL_Big_Project

0 stars 1 forks source link

login_info Review and Question #12

Open Gogumalang opened 1 year ago

Gogumalang commented 1 year ago

이 내용을 넣은 이유가 무엇인지 궁금합니다! for (int i = 0; i < 20; i++) { if (a[i].ID[0] == '\0' || NULL) { a[i].ID[0] = '.'; a[i].ID[1] = '\0'; a[i].PW[0] = '.'; a[i].PW[1] = '\0'; } }

그리고 get info 를 했을 때 파일에 저장이 잘 되지 않는 오류도 발견했습니다. 또한 모든 파일에 있는 개인정보를 구조체에 저장을 해서 비교를 하게 될 경우, 나중에 사용자가 많아지면, 그에 따라 구조체의 크기도 키워야하는 문제가 발생해서 이 부분은 제가 한번 바꿔봐도 괜찮을까요?

Gogumalang commented 1 year ago

추가적으로 굳이 패스워드는 문자로 받는 것이 아닌, 숫자로 받고 싶은 이유는 문자로 받을 시 사용자가 실수로 스패이스바를 누르게 되면, 그것을 분별하는 기능이 없으면, 사용자의 개인정보를 온전히 받지 못하기 때문입니다. 혹시 의견 주시면 감사하겠습니다 :)

K-Minhyeok commented 1 year ago

' 20 = 유저 수' 라고 생각하면 되고, 그 밑에 작업들은 20개로 자리는 만들어 놓았는데 들어온 정보가 없는 경우에는 그냥 일단 초기화하려고 해놓은 건데, 제가 다시 확인해보니 없어도 되는 부분인 것 같습니다.

get info 에서 어느 케이스에서 저장이 잘 되지 않았나요?

네네 당연히 바꾸셔도 됩니다!! 어떻게 구현하시는 건지만 알려주시면 됩니다..!

K-Minhyeok commented 1 year ago