瀏覽代碼

codacy: This for loop can be replaced by a foreach loop

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Andy Scherzinger 6 年之前
父節點
當前提交
f21f4eff22

+ 5 - 6
src/main/java/com/owncloud/android/ui/adapter/OCFileListAdapter.java

@@ -819,13 +819,12 @@ public class OCFileListAdapter extends RecyclerView.Adapter<RecyclerView.ViewHol
             Vector<OCFile> filteredFiles = new Vector<>();
 
             if (!TextUtils.isEmpty(constraint)) {
-                for (int i = 0; i < mFilesAll.size(); i++) {
-                    OCFile currentFile = mFilesAll.get(i);
-                    if (currentFile.getParentRemotePath().equals(currentDirectory.getRemotePath()) &&
-                            currentFile.getFileName().toLowerCase(Locale.getDefault()).contains(
+                for (OCFile file : mFilesAll) {
+                    if (file.getParentRemotePath().equals(currentDirectory.getRemotePath()) &&
+                        file.getFileName().toLowerCase(Locale.getDefault()).contains(
                                     constraint.toString().toLowerCase(Locale.getDefault())) &&
-                            !filteredFiles.contains(currentFile)) {
-                        filteredFiles.add(currentFile);
+                            !filteredFiles.contains(file)) {
+                        filteredFiles.add(file);
                     }
                 }
             }

+ 2 - 2
src/main/java/com/owncloud/android/ui/fragment/EditShareFragment.java

@@ -380,8 +380,8 @@ public class EditShareFragment extends Fragment {
                 }
                 if (canEditCompound.isChecked() && allDisabled) {
                     toggleDisablingListener(canEditCompound);
-                    for (int i = 0; i < sSubordinateCheckBoxIds.length; i++) {
-                        getView().findViewById(sSubordinateCheckBoxIds[i]).setVisibility(View.GONE);
+                    for (int sSubordinateCheckBoxId : sSubordinateCheckBoxIds) {
+                        getView().findViewById(sSubordinateCheckBoxId).setVisibility(View.GONE);
                     }
                 }
             }