General 클래스를 General, GeneralLite로 분리 #236

2023-08-03T19:11:40+09:00

Hide_D commented

2023-08-03 19:11:40 +09:00

내부 코드 사용례에서 General 테이블의 column을 일부만 가져오는 경우가 다수 있습니다.

다만 General 클래스에서 각종 iAction에 해당하는 트리거 함수 호출을 위해서는 모든 column을 다 가져와야 정상 동작이 가능한 만큼, iAction interface를 구현한 General 클래스와, 구현하지 않은 GeneralLite 클래스 둘로 나눕니다.

또한 General 클래스에서는 필요한 column을 다 가져오도록 변경합니다.

내부 코드 사용례에서 General 테이블의 column을 일부만 가져오는 경우가 다수 있습니다. 다만 General 클래스에서 각종 iAction에 해당하는 트리거 함수 호출을 위해서는 모든 column을 다 가져와야 정상 동작이 가능한 만큼, iAction interface를 구현한 General 클래스와, 구현하지 않은 GeneralLite 클래스 둘로 나눕니다. 또한 General 클래스에서는 필요한 column을 다 가져오도록 변경합니다.

Hide_D added 19 commits 2023-08-03 19:11:41 +09:00

refac, wip: GeneralLite 055e5765ae

refac, wip: GeneralLite, General 이원화 d72c3e4c41

wip: 버그 수정중 e960446638

fix: 준비중 ac7c3c25b3

refac, wip: GeneralLite에는 RankVar가 읽기전용으로 필요 af8a417dc1

wip: 값 이동 07035b5a12

misc: phan 경고 수준을 CRITICAL로 982ce86ad8

- General 클래스 호환용도로는 이정도로 충분

fix: Phan Critical 경고 해결 ed035ab75f

refac: General 클래스 관련 적용 0c53d77e61

misc: phan setting f7e5a93fb9

refac: GeneralBase, GeneralLite, General 822b5d3125

- GeneralLite -> General 구조는 PHP에서 경고를 안띄웟서 포기
- GeneralBase에서 각각 요청하는 형태로 변경
- 여전히 GeneralBase를 인자로 받는 함수에서
  GeneralLite를 요구하거나, General을 요구하는 함수를 추가로 호출할 때
  경고가 없으므로 특히 주의해야 함.

refac: DummyGeneralLite 6b91a800fa

refac,wip: GeneralQueryMode 분리 29273d7a8b

- GeneralQueryMode - General 생성용
- GeneralLiteQueryMode - GeneralLite 생성용
- 이를 통해 확실한 에러 메시지를 내줄 것으로..

refac: General create 함수명 변경 55fca2337d

refac: General 생성 방식 변경 ad23398592

- 변경한 GeneralQueryMode에 따라 Lite가 불가능
- Lite여도 된다면 GeneralLite 클래스로 변경
- 그렇지 않다면 General 클래스로 변경
- 내부에서 어떠한 연산을 할 지 알 수 없는데,
  구현 시점에서 편의에 따라 column 여부가 차이나면
  이후에 문제가 될 수 있음
- 엔진이 살짝 무거워 질 것으로 보임

refac: LazyVarAndAuxUpdater 8aeb4883ff

- LazyVarUpdater에다가 aux var 대응 분리

fix: General 생성용 인자 수정 a6ff79d148

- General 클래스는 항상 모든걸 요구함
- 장기적으로 바람직 하지 않음
  - new General 호출이 그냥 가능해선 안 될 것

fix: 누락분 수정 8990436e2d

misc: Phan Severity 복구 3d5f249c17

Hide_D merged commit 3edfff146e into devel

2023-08-03 21:32:02 +09:00

Hide_D referenced this issue from a commit

2023-08-03 21:32:02 +09:00

General 클래스를 General, GeneralLite로 분리 (#236)

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: devsam/core#236