항상 코드에 문제가 있을것을 고려해서 코드를 짜야할까요?

Question

using System.Collections.Generic; public class Temp { private List ints = new List(); public void AddValue(int value) { // 1. 여기서 ints 배열에 값의 크기를 제한해서 저장하도록 코드를 설계. if (value < 0) value = 0; if (value > 10) value = 10; ints.Add(value); } public int GetValue(int index) { if (index < 0) { // 대충 처리 } if (index >= ints.Count) { // 대충 처리 } int returnValue = ints[index]; // 2. 그러나 혹시라도 ints 배열에 값의 크기를 벗어나는 경우가 있을 수 있으니 처리. if (returnValue < 0) { // 에러 로그 returnValue = 0; } if (returnValue > 10) { // 에러 로그 returnValue = 10; } return returnValue; } }위와 같이 코드를 짰을 때, 1번 처럼 값이 제한되어 있지만, 혹시라도 저장된 값이 어떻게 되어있을지 모르니 2번처럼 처리하는게 맞을까요?

엡실론 · Answer

이 경우는 아니오. 너무 명백한데다 테스트에서도 저 코드를 실행 시킬 수가 없습니다. 메모리에 있는 리스트가 아니라 데이터베이스에 저장한다면 또 다른 얘기겠지만요.

mirheeoj · Answer

문제가 있을 것을 고려해서 코드를 작성해야 하는 건 맞습니다. 그러나 그렇게 고려한 이후 문제가 없는 것이 확실하다는 결론이 나오면 안해도 되죠. 이 케이스가 바로 그렇습니다. List를 노출/변경하는 다른 메서드나 프라퍼티등이 있지 않는 한에는 저건 중복처리이므로 둘 중 하나만 하시면 됩니다.

AskeTeddy · Answer

값을 변화시키는 함수에서만 검증하면 됩니다보통은 생성자에서만 검증하는 대신에 변경할 수 없는 클래스를 만들긴 해요https://enterprisecraftsmanship.com/posts/specification-pattern-always-valid-domain-model/

집킨 · Answer

DBMS에서도 제약 조건은 write 할때 발생하지, read할때 발생하지 않습니다유효하지 않은 데이터를 가지고 있는 것 자체가 해당 객체의 무결성이 깨진 상황이기 때문에 그 책임은 write하는 쪽에 있는거죠

항상 코드에 문제가 있을것을 고려해서 코드를 짜야할까요?

4개의 답변

#unity 목록