ST: Write to static field from instance method (ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD)

This instance method writes to a static field. This is tricky to get correct if multiple instances are being manipulated, and generally bad practice.

FindBugs 플러그인으로 안드로이드 앱을 체크해보니, 앱 사용중 재생시킨 음악을 다른 액티비티, 재생중 전화가 올때를 대비한 SystemService 관련 코드에서 사용중인 재생Flag를 static으로 처리했더니 다수의 인스턴스일때 문제가 되며, 일반적으로 성능이 좋지 않다고 marked됨.

-> Android앱에서는 하나의 액티비티가 여러개의 instance가 생기지는 않겠지만 수정하기로 함
-> Static 변수를 제거하고 해당 플래그는 영속성을 가지는 sharedpreference를 사용하기로 함.
-> 더이상 marked되지 않는다 

저작자 표시
신고
크리에이티브 커먼즈 라이선스
Creative Commons License
Posted by 영겁회귀

티스토리 툴바