Elegie0
commented
4 years ago 처음에는 문자열 사이에 숫자가 들어있는 인자도 받을 수 있도록 했습니다. 그런데 아무래도 그런 경우도 에러로 처리해야 할 것 같아서 나중에 "유효한 인자 없음" 에러도 추가했습니다. 그 과정에서 숫자 뒤에 문자열이 나오는 경우를 처리했어야 했는데, 그부분 처리가 미흡했던 것 같습니다ㅠㅠㅠ 말씀 안해주셨으면 몰랐을 것 같아요ㅋㅋ
리뷰 감사합니다~!!
sjh@SJH:/mnt/c/Users/SJH/Desktop/42Seoul-master$ ./factorization 4294967296 Error : Too large number. (Max = 4294967295)
sjh@SJH:/mnt/c/Users/SJH/Desktop/42Seoul-master$ ./factorization 4294967296a Error : Too large number. (Max = 4294967295)
sjh@SJH:/mnt/c/Users/SJH/Desktop/42Seoul-master$ ./factorization 429496729a Error : No available arguments.
인자를 검사하는 알고리즘에 약간의 수정이 필요할 것 같습니다! 4294967296a 같은경우에는 10자리 정수가 이미 MAX값을 넘어가서 너무 큰 변수라고 출력하는것 같은데 ,적어도 429496729a 처럼 정수가 10자리를 넘어가지 않는 경우와 통일성은 갖추는게 좋을것 같아요! 엄밀히 따지면 No available arguments 를 출력해야 될 것 같습니다!