[Android]fix jira problems: NMS-5478、NMS-5484

Author: xgfd3

Android/APIExample/app/src/main/java/io/agora/api/example/examples/advanced/LiveStreaming.java

@@ -177,6 +177,7 @@ public void onClick(View v) {
                 engine.stopPreview();
                 engine.leaveChannel();
                 join.setText(getString(R.string.join));
+                remoteUid = 0;
             }
         } else if (v.getId() == R.id.btn_publish) {
             isHost = !isHost;
@@ -236,7 +237,10 @@ public void onClick(View v) {
             }
         } else if (v.getId() == R.id.btn_take_shot) {
             if (remoteUid != 0) {
-                engine.takeSnapshot(remoteUid, "/sdcard/APIExample_snapshot_" + et_channel.getText().toString() + "_" + remoteUid + ".png");
+                int ret = engine.takeSnapshot(remoteUid, "/sdcard/APIExample_snapshot_" + et_channel.getText().toString() + "_" + remoteUid + ".png");
+                if(ret != Constants.ERR_OK){
+                    showLongToast("takeSnapshot error code=" + ret + ",msg=" + RtcEngine.getErrorDescription(ret));
+                }
             } else {
                 showLongToast(getString(R.string.remote_screenshot_tip));
             }
@@ -454,16 +458,18 @@ public void onUserJoined(int uid, int elapsed) {
         public void onUserOffline(int uid, int reason) {
             Log.i(TAG, String.format("user %d offline! reason:%d", uid, reason));
             showLongToast(String.format("user %d offline! reason:%d", uid, reason));
-            if(uid == remoteUid)
-            handler.post(new Runnable() {
-                @Override
-                public void run() {
-                    /**Clear render view
-                     Note: The video will stay at its last frame, to completely remove it you will need to
-                     remove the SurfaceView from its parent*/
-                    engine.setupRemoteVideo(new VideoCanvas(null, RENDER_MODE_HIDDEN, uid));
-                }
-            });
+            if(uid == remoteUid) {
+                remoteUid = 0;
+                runOnUIThread(new Runnable() {
+                    @Override
+                    public void run() {
+                        /**Clear render view
+                         Note: The video will stay at its last frame, to completely remove it you will need to
+                         remove the SurfaceView from its parent*/
+                        engine.setupRemoteVideo(new VideoCanvas(null, RENDER_MODE_HIDDEN, uid));
+                    }
+                });
+            }
         }
 
         /**

Android/APIExample/app/src/main/java/io/agora/api/example/examples/advanced/PlayAudioFiles.java

@@ -41,6 +41,7 @@
 )
 public class PlayAudioFiles extends BaseFragment implements View.OnClickListener, SeekBar.OnSeekBarChangeListener {
     private static final String TAG = PlayAudioFiles.class.getSimpleName();
+    private static final int EFFECT_SOUND_ID = 0;
     private EditText et_channel;
     private Button mute, join, speaker, bgm, effect;
     private SeekBar mixingPublishVolBar, mixingPlayoutVolBar, mixingVolBar;
@@ -143,24 +144,31 @@ private void preloadAudioEffect(){
         // Only mp3, aac, m4a, 3gp, and wav files are supported.
         // You may need to record the sound IDs and their file paths.
         int id = 0;
-        int preloadRet = audioEffectManager.preloadEffect(id++, Constant.EFFECT_FILE_PATH);
+        audioEffectManager.preloadEffect(id++, Constant.EFFECT_FILE_PATH);
+    }
+
+    private void playEffect() {
         /** Plays an audio effect file.
          * Returns
          * 0: Success.
          * < 0: Failure.
          */
         int playRet = audioEffectManager.playEffect(
-                0,  // The sound ID of the audio effect file to be played.
+                EFFECT_SOUND_ID,  // The sound ID of the audio effect file to be played.
                 Constant.EFFECT_FILE_PATH,  // The file path of the audio effect file.
                 -1,   // The number of playback loops. -1 means an infinite loop.
                 1,    // pitch	The pitch of the audio effect. The value ranges between 0.5 and 2. The default value is 1 (no change to the pitch). The lower the value, the lower the pitch.
                 0.0,  // Sets the spatial position of the effect. 0 means the effect shows ahead.
                 100,  // Sets the volume. The value ranges between 0 and 100. 100 is the original volume.
                 true // Sets whether to publish the audio effect.
         );
-        // Pauses all audio effects.
-        int pauseRet = audioEffectManager.pauseAllEffects();
-        Log.i(TAG, "result preloadRet:" + preloadRet + ", playRet:"+ playRet + ", pauseRet:"+ pauseRet);
+        Log.i(TAG, "result playRet:"+ playRet);
+    }
+
+    private void stopEffect() {
+        audioEffectManager.stopEffect(
+                EFFECT_SOUND_ID  // The sound ID of the audio effect file to be played.
+                );
     }
 
     @Override
@@ -266,11 +274,11 @@ else if (v.getId() == R.id.btn_effect)
             effect.setText(!effect.isActivated() ? getString(R.string.effect_on): getString(R.string.effect_off));
             if(effect.isActivated()){
                 // Resumes playing all audio effects.
-                audioEffectManager.resumeAllEffects();
+                playEffect();
             }
             else {
                 // Pauses all audio effects.
-                audioEffectManager.pauseAllEffects();
+                stopEffect();
             }
         }
     }