rkuenzi-epfl/Wnder

File GuessLocationActivity.java has 455 lines of code (exceeds 250 allowed). Consider refactoring.

Open

package com.github.wnder.guessLocation;

import android.Manifest;
import android.animation.ValueAnimator;
import android.annotation.SuppressLint;

File TakePictureFragment.java has 316 lines of code (exceeds 250 allowed). Consider refactoring.

Open

package com.github.wnder;

import android.Manifest;
import android.content.ContentValues;
import android.content.Context;

GuessLocationActivity has 29 methods (exceeds 20 allowed). Consider refactoring.

Open

@AndroidEntryPoint
public class GuessLocationActivity extends AppCompatActivity implements OnMapReadyCallback, MapboxMap.OnMapClickListener, MapboxMap.OnCameraMoveListener {

    //Define all necessary and recurrent strings
    public static final String EXTRA_GUESS_MODE = "guess_mode";

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    public static class ViewHolder extends RecyclerView.ViewHolder {
        private final ImageView historyImageView;
        private final TextView yourScoreView;
        private final TextView historyDistanceView;
        private final LinearLayout toMapView;

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    public static class ViewHolder extends RecyclerView.ViewHolder {
        private final ImageView firstPicImageView;
        private final TextView nameTextView;
        private final TextView picCountTextView;
        private final TextView distanceTextView;

• Read up
  Read up

File FirebasePicturesDatabase.java has 251 lines of code (exceeds 250 allowed). Consider refactoring.

Open

package com.github.wnder.picture;

import android.content.Context;
import android.graphics.Bitmap;
import android.graphics.BitmapFactory;

Method onCreate has 37 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        //Initialisation of class variables (MapBox creation)

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    private CompletableFuture<Void> uploadTourPics(String tourUniqueId, List<String> tourPics){
        CompletableFuture<Void> uploadTourPicsFuture = new CompletableFuture<>();

        Map<String, List<String>> tourPicsMap = new HashMap<>();
        tourPicsMap.put("tourPics", tourPics);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    private CompletableFuture<Void> uploadTourName(String tourUniqueId, String tourName){
        CompletableFuture<Void> uploadTourNameFuture = new CompletableFuture<>();

        Map<String, String> tourNameMap = new HashMap<>();
        tourNameMap.put("tourName", tourName);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    @Override
    public CompletableFuture<Double> getTourLength(String tourUniqueId){
        CompletableFuture<Double> tourLengthFuture = new CompletableFuture<>();

        tourCollection.document(tourUniqueId).get().

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    @Override
    public CompletableFuture<String> getTourName(String tourUniqueId) {
        CompletableFuture<String> tourNameFuture = new CompletableFuture<>();

        tourCollection.document(tourUniqueId).get().

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

        try (BufferedReader reader = new BufferedReader(inputStreamReader)) {
            String line = reader.readLine();
            while (line != null) {
                stringBuilder.append(line);
                line = reader.readLine();

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

            try (BufferedReader reader = new BufferedReader(inputStreamReader)) {
                String line = reader.readLine();
                while (line != null) {
                    stringBuilder.append(line);
                    line = reader.readLine();

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        holder.getToScoreboardView().setOnClickListener(scoreField -> {
            Intent intent = new Intent(context, ScoreboardActivity.class);
            intent.putExtra(ScoreboardActivity.EXTRA_PICTURE_ID, pictureId);
            context.startActivity(intent);
        });

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    @Override
    public CompletableFuture<Location> getLocation(String uniqueId) {
        if (isOnline()) {
            return remoteDatabase.getLocation(uniqueId);
        }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        holder.getToMapView().setOnClickListener(distanceField -> {
            Intent intent = new Intent(context, HistoryMapActivity.class);
            intent.putExtra(HistoryMapActivity.EXTRA_PICTURE_ID, pictureId);
            context.startActivity(intent);
        });

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    @Override
    public CompletableFuture<Map<String, Double>> getScoreboard(String uniqueId) {
        if (isOnline()) {
            return remoteDatabase.getScoreboard(uniqueId);
        }

• Read up
  Read up