Question

我在从另一个类调用的方法中获取视图以使其正常运行时遇到一些困难。

我的应用包含主要活动（名为Overlay）中的体验栏，可使用进度条跟踪用户进度。我想要做的是在另一个类中有一个单独的方法来处理与体验栏相关的任何代码。

我已经为类创建了一个实例，并且正确调用了该方法，但是某些与android / view相关的部分会导致严重问题。 System.out.println将显示文本到调试器中，但是Toast和其他视图的使用不会。以下是两个类文件和相关错误。

public class OverlayActivity extends AppCompatActivity
    implements NavigationView.OnNavigationItemSelectedListener {

NavigationView navigationView = null;
Toolbar toolbar = null;
int level = 0;
int addXP = 110;
ProgressBar experienceBar;
Button buttonXP;
TextView experienceLevel;
ExperienceManager exp;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_overlay);

    //Set the Overlay Fragment

    OverlayFragment fragment = new OverlayFragment();
    android.support.v4.app.FragmentTransaction fragmentTransaction =
            getSupportFragmentManager().beginTransaction();
    fragmentTransaction.replace(R.id.fragment_container, fragment);
    fragmentTransaction.commit();

    toolbar = (Toolbar) findViewById(R.id.toolbar);
    setSupportActionBar(toolbar);

    FloatingActionButton fab = (FloatingActionButton) findViewById(R.id.fab);
    fab.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            Snackbar.make(view, "Hay! Don't touch me!", Snackbar.LENGTH_LONG)
                    .setAction("Action", null).show();
        }
    });

    DrawerLayout drawer = (DrawerLayout) findViewById(R.id.drawer_layout);
    ActionBarDrawerToggle toggle = new ActionBarDrawerToggle(
            this, drawer, toolbar, R.string.navigation_drawer_open, R.string.navigation_drawer_close);
    drawer.setDrawerListener(toggle);
    toggle.syncState();

    navigationView = (NavigationView) findViewById(R.id.nav_view);
    navigationView.setNavigationItemSelectedListener(this);

    experienceBar = (ProgressBar) findViewById(R.id.experienceBar);
    experienceBar.setProgress(12);

    experienceLevel = (TextView) findViewById(R.id.ds_appBar_level);

    buttonXP = (Button) findViewById(R.id.buttonXP);
    buttonXP.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View View) {
            exp = new ExperienceManager();
            exp.addExperience(level, addXP);
        }
    });
}

public class ExperienceManager extends AppCompatActivity {

ProgressBar experienceBar;

public int addExperience(int level, int addXP) {
    experienceBar = (ProgressBar) findViewById(R.id.experienceBar);

    int addLevel = 0;
    while (addXP >= experienceBar.getMax()) {
        addXP -= experienceBar.getMax();
        addLevel += 1;
    }
    if (addXP >= experienceBar.getMax()-experienceBar.getProgress()) {

        experienceBar.setProgress(experienceBar.getProgress() + addXP - experienceBar.getMax());
        addLevel += 1;
    } else {
        experienceBar.setProgress(experienceBar.getProgress() + addXP);
    }
    if (addLevel >= 1) {
        level += addLevel;
        Toast toast = Toast.makeText(getBaseContext(), "Level up!", Toast.LENGTH_SHORT);
        toast.show();
        //experienceLevel.setText("Level " + level);
    }
    return level;
}

E/AndroidRuntime: FATAL EXCEPTION: main
              Process: com.teamcirelios.deepstone, PID: 4138
              java.lang.NullPointerException: Attempt to invoke virtual method 'android.view.Window$Callback android.view.Window.getCallback()' on a null object reference
                  at android.support.v7.app.AppCompatDelegateImplBase.<init>(AppCompatDelegateImplBase.java:116)
                  at android.support.v7.app.AppCompatDelegateImplV9.<init>(AppCompatDelegateImplV9.java:147)
                  at android.support.v7.app.AppCompatDelegateImplV11.<init>(AppCompatDelegateImplV11.java:27)
                  at android.support.v7.app.AppCompatDelegateImplV14.<init>(AppCompatDelegateImplV14.java:50)
                  at android.support.v7.app.AppCompatDelegateImplV23.<init>(AppCompatDelegateImplV23.java:29)
                  at android.support.v7.app.AppCompatDelegate.create(AppCompatDelegate.java:199)
                  at android.support.v7.app.AppCompatDelegate.create(AppCompatDelegate.java:181)
                  at android.support.v7.app.AppCompatActivity.getDelegate(AppCompatActivity.java:521)
                  at android.support.v7.app.AppCompatActivity.findViewById(AppCompatActivity.java:190)
                  at com.teamcirelios.deepstone.ExperienceManager.addExperience(ExperienceManager.java:15)
                  at com.teamcirelios.deepstone.OverlayActivity$2.onClick(OverlayActivity.java:77)
                  at android.view.View.performClick(View.java:5697)
                  at android.widget.TextView.performClick(TextView.java:10826)
                  at android.view.View$PerformClick.run(View.java:22526)
                  at android.os.Handler.handleCallback(Handler.java:739)
                  at android.os.Handler.dispatchMessage(Handler.java:95)
                  at android.os.Looper.loop(Looper.java:158)
                  at android.app.ActivityThread.main(ActivityThread.java:7225)
                  at java.lang.reflect.Method.invoke(Native Method)
                  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1230)
                  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1120)

同样，尝试在片段中注册按钮单击时会产生类似的错误。体验栏本身显示Overlay acitvity，但是为了使app能够按预期工作，它需要接收一个输入，以便从片段内的按钮（或者更准确，一个复选框）调用该类。

我能看到的唯一解决方案是使用experienceBar.getMax（）和类似的方法并将它们作为参数包含在内。但是，我需要返回级别和当前xp，据我所知，这是不可能的。

基本上总结一切，我有三个主要问题。

这个错误的非技术性解释是什么？
修复代码以实现所需功能的最佳（且相对简单）方法是什么？
如何在片段中注册事件？

谢谢！

Answer 1

首先，您的实用程序类，即体验管理器不应扩展AppCompatActivity ...您的实用程序类不是活动！ PS你无法在Experience Manager课程中找到你的进度栏，因为ProgressBar不属于你的经验活动，它是你的OverlayActivity的一部分。

如果您希望创建静态实用程序方法以增加体验和管理它，您应该以这种方式编写代码。

public class ExperienceManager{

public static int addExperience(ProgressBar experienceBar, int level, int addXP){

    int addLevel = 0;
    while (addXP >= experienceBar.getMax()) {
        addXP -= experienceBar.getMax();
        addLevel += 1;
    }
    if (addXP >= experienceBar.getMax()-experienceBar.getProgress()) {

        experienceBar.setProgress(experienceBar.getProgress() + addXP - experienceBar.getMax());
        addLevel += 1;
    } else {
        experienceBar.setProgress(experienceBar.getProgress() + addXP);
    }
    if (addLevel >= 1) {
        level += addLevel;
        Toast toast = Toast.makeText(getBaseContext(), "Level up!", Toast.LENGTH_SHORT);
        toast.show();
        //experienceLevel.setText("Level " + level);
    }
    return level;
}
}

你可以用这种方式调用这个方法：

buttonXP.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View View) {
            // Calling the static utility method over here...
            ExperienceManager.addExperience(experienceBar, level, addXP);

        }
});

你需要做的就是：）

我建议在尝试Android之前熟悉Java设计模式和最佳编程实践。

Android Studio - 使用带有视图的外部类的方法

1 个答案: