classes 폴더 내의 주요 클래스에 autoload 적용#1374
Merged
4 commits merged intoMay 27, 2015
Merged
Conversation
Contributor
Author
|
classes 폴더 내의 클래스들 뿐만 아니라 모든 모듈 (컨트롤러, 모델, 뷰, api, wap, mobile 등) 클래스 로딩에도 autoload를 사용할 수 있도록 해보았습니다. |
Contributor
Author
|
DB 핸들러 및 XML 파서에서 무조건 다 인클루드되는 파일들도 autoload로 전환했습니다. 아직도 몇 군데 제 테스트 서버에서는 메인화면 기준 상당히 빠른 서버에 PHP 5.5이기 때문에 딱히 로딩 속도가 빨라지는 것이 느껴지지는 않지만, 서버 사양이 좋지 않거나 PHP 버전이 낮은 경우에는 불필요한 파일 로딩을 줄여서 상당한 성능 개선 효과가 있을지도 모르겠습니다. |
ghost
pushed a commit
that referenced
this pull request
May 27, 2015
classes 폴더 내의 주요 클래스에 autoload 적용
ghost
pushed a commit
that referenced
this pull request
May 28, 2015
- composer autoload 추가 - load time 측정 오류 수정 - 대문자를 포함한 모듈명에 대한 호환성 문제 수정
YJSoft
referenced
this pull request
in YJSoft/xe-core-old
Jun 12, 2015
- composer autoload 추가 - load time 측정 오류 수정 - 대문자를 포함한 모듈명에 대한 호환성 문제 수정
This pull request was closed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
제목 그대로입니다. 상당히 오랫동안 TODO라고 되어 있기에 한번 고쳐보았습니다. 불필요한 클래스 로딩을 조금이라도 줄여서 XE 성능 향상에 도움이 되었으면 합니다.
PHP 5.3 이상에서 autoload 콜백을 정의할 때는 클로져를 활용하는 것이 가장 좋은 방법이지만, PHP 버전이 충분한지 확인하기도 전에 문법 오류를 뿜어서는 곤란하겠기에 클로져를 사용하지 않고 그냥 함수를 선언했습니다.