개발초보가 기능 분리하려는데 잘 안됩니다

지금 굉장히 소스가 지저분해서 감당이 안되는데 지금부터 추가흐는 기능이라도 분리하려고 합니다.

MainActivity에 쓰일 함수를 만들려고 하거든요

그래서 MainComp.java 파일을 만들었습니다.

이안에 기능을 구성해서 메인액티비티에서 쓰고 싶은데

딱 이부분을 따로 구성하고 싶습니다.

ImageView country = (ImageView)findViewById(R.id.country_img); //국기이미지
		if(str_national.equals("대한민국")){
			BitmapDrawable img_country = (BitmapDrawable)getResources().getDrawable(R.drawable.korea);
			country.setImageDrawable(img_country);
		}



public class MainComp {
	
	
	
	
	Spinner country;
	public void country(ImageView country, String str_national){
		country = (ImageView)findViewById(R.id.country_img); //국기이미지
		if(str_national.equals("대한민국")){
			BitmapDrawable img_country = (BitmapDrawable)getResources().getDrawable(R.drawable.korea);
			country.setImageDrawable(img_country);
		}
	}

이렇게 떼어놨는데 findViewById 여기와 getResources여기서 에러가 나고 있어요

각각

The method findViewById(int) is undefined for the type FaceonComp

The method getResources() is undefined for the type FaceonComp

이렇게 에러가 나고있는데 정의를 어떻게 해줘야하죠?

메인액티비티에서는 그냥 잘되가지고 어떻게 해야할지 모르곘습니다.

브루스웨인 (8,580 포인트) 님이 2013년 12월 5일 질문

4개의 답변

서브 쓰레드에선 UI업데이트 권한이 없어요.. 그래서 생기는 문제 입니다. ㅋ.ㅋ

country.setImageDrawable(img_country);

이부분을 메인쓰레드에서 실행되도록 수정하시면 됩니다..

서브에서 메인으로 접근하는 방법은... 여러가지가 있습니다. 펍에도 이런 질문은과 답변은 많으니 서브쓰레드 에서 UI수정방법으로 검색해보시길 바랍니다.

익명사용자 님이 2013년 12월 5일 답변

코딩스타일에 답은 없습니다만..

제가 볼때 님께서 하려고 하시는 방법은 그닥 좋지 않을거 같아요.

클래스를 분리하신것은 좋은 방법이긴 합니다만..

메인Activity에서 UI조작을 해주시는게 훨씬 좋을거에요.

지금 하시려는 방법으로 본다면 엑티비티에서 ImageView를 생성하고 레퍼런스변수를 MainComp클래스의 country메서드로 넘겨서 거기서 이미지뷰의 이미지교체 작업이 이루어지는데요.

이 이미지 교체작업 자체가 UI 변경 작업이거든요.

이런 UI변경 작업은 왠만하면 Activity단에서 해주는게 더 좋긴해요.

리스트뷰를 구현해서 어댑터에서 각각의 뷰들마다 스크롤될때마다 동적으로 바뀌어야하는 작업이 아니라면,

위의 방법보다는 제 개인적인 의견으로는

public class MainComp { //기능분리 클래스

//입력된 값에 의거하여 해당되는 Drawable객체를 얻는다.

public static BitmapDrawable getCountryBitmapDrawble(Context context, String str_national){

BitmapDrawable img_country = null;

if(str_national.equals("대한민국")){

img_country = (BitmapDrawable)getResources().getDrawable(R.drawable.korea);

}

return img_country;

}

MainActivity.java //UI단 메인 클래스

ImageView country = (ImageView)findViewById(R.id.country_img);

BitmapDrawable bitDrawable = MainComp.getCountryBitmapDrawble(MainActivity.this, "대한민국"); //BitmapDrawable 을 얻어온다.

country.setImageDrawable(bitDrawable); //세팅한다.

이렇게가 더 낫지 않을까 싶어요.

그리고 왠만하면 UI단에서 UI 갱신관련 작업을 하시라고 말씀드리는건

만약 님이 지금 하시려고 하신것처럼 다른 클래스 메서드에서 UI갱신 작업을 하게끔 설계를 하고 만들어 놓았는데

만약 님께서 나중에 해당 메서드를 UI단이 아닌 다른 쓰레드 내에서 쓰게 되면 Exception이 발생하게 됩니다.

그땐 또 거기에 땜빵으로 Handler처리를 해줘야되구요.

갸아악 (21,260 포인트) 님이 2013년 12월 5일 답변
갸아악님이 2013년 12월 5일 수정

위의 static을 쓰셔도 되고, 아니면 싱글톤 방식으로 구성 하셔도 됩니다.