File MapActivity.java has 571 lines of code (exceeds 250 allowed). Consider refactoring.

Open

package sdp.moneyrun.ui.map;

import android.Manifest;
import android.content.pm.PackageManager;
import android.graphics.Color;

MapActivity has 53 methods (exceeds 20 allowed). Consider refactoring.

Open

@SuppressWarnings({"CanBeFinal", "FieldCanBeLocal"})
public class MapActivity extends TrackedMap implements OnMapReadyCallback {
    public static final int THRESHOLD_DISTANCE = 5;
    public static final float DISTANCE_CHANGE_BEFORE_UPDATE = (float) 2;
    private static final long MINIMUM_TIME_BEFORE_UPDATE = 2000;

File Game.java has 434 lines of code (exceeds 250 allowed). Consider refactoring.

Open

package sdp.moneyrun.game;


import android.app.Activity;
import android.content.Intent;

Game has 40 methods (exceeds 20 allowed). Consider refactoring.

Open

@SuppressWarnings("FieldCanBeLocal")
public class Game {

    private final String DATABASE_GAME = "games";
    private final String DATABASE_PLAYER = "players";

GameDbData has 30 methods (exceeds 20 allowed). Consider refactoring.

Open

public final class GameDbData {
    boolean isVisible;
    boolean isDeleted;
    boolean started;
    boolean ended;

User has 26 methods (exceeds 20 allowed). Consider refactoring.

Open

public class User implements Serializable {

    @Nullable
    private String userId;
    @Nullable

GameLobbyActivity has 23 methods (exceeds 20 allowed). Consider refactoring.

Open

@SuppressWarnings("FieldCanBeLocal")
public class GameLobbyActivity extends AppCompatActivity {
    private final String TAG = GameLobbyActivity.class.getSimpleName();
    private final String DB_IS_DELETED = "isDeleted";
    private final String DB_PLAYERS = "players";

File GameDbData.java has 261 lines of code (exceeds 250 allowed). Consider refactoring.

Open

package sdp.moneyrun.database.game;

import android.location.Location;

import androidx.annotation.NonNull;

MenuActivity has 21 methods (exceeds 20 allowed). Consider refactoring.

Open

@SuppressWarnings({"CanBeFinal", "FieldCanBeLocal"})
public class MenuActivity extends AppCompatActivity implements NavigationView.OnNavigationItemSelectedListener {
    //In meters
    public static final float DISTANCE_CHANGE_BEFORE_UPDATE = (float) 100.0;
    private static final long MINIMUM_TIME_BEFORE_UPDATE = 10000;

Method onDestroy has a Cognitive Complexity of 16 (exceeds 5 allowed). Consider refactoring.

Open

    @Override
    protected void onDestroy() {
        super.onDestroy();
        DatabaseProxy.removeOfflineListener();
        if (playerListListener != null)

• Read up
  Read up

File GameLobbyActivity.java has 252 lines of code (exceeds 250 allowed). Consider refactoring.

Open

package sdp.moneyrun.ui.game;

import android.content.Intent;
import android.os.Bundle;
import android.util.Log;

Method getGameFromTaskSnapshot has 47 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    @Nullable
    public Game getGameFromTaskSnapshot(@Nullable Task<DataSnapshot> task) {
        if (task == null)
            throw new IllegalArgumentException("task should not be null.");

Method getGameFromTaskSnapshot has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    @Nullable
    public Game getGameFromTaskSnapshot(@Nullable Task<DataSnapshot> task) {
        if (task == null)
            throw new IllegalArgumentException("task should not be null.");

• Read up
  Read up

Method getRawForecast has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    @Nullable
    private String getRawForecast(@NonNull LocationRepresentation location) throws IOException {
        String queryUrl = API_ENDPOINT +
                "?lat=" + location.getLatitude() +
                "&lon=" + location.getLongitude() +

Method equals has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    @Override
    public boolean equals(@Nullable Object o) {
        if (this == o) return true;
        if (o == null || getClass() != o.getClass()) return false;
        Riddle riddle = (Riddle) o;

• Read up
  Read up

Method onNavigationItemSelected has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    @SuppressLint("NonConstantResourceId")
    @Override
    public boolean onNavigationItemSelected(@NonNull MenuItem item) {
        MediaPlayer.create(this, R.raw.button_press).start();
        switch (item.getItemId()) {

Method deleteOlderMaps has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private void deleteOlderMaps() {
        offlineManager.listOfflineRegions(new OfflineManager.ListOfflineRegionsCallback() {
            @Override
            public void onList(@NonNull OfflineRegion[] offlineRegions) {

Method listenToStarted has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

    private void listenToStarted() {
        if (!game.getHost().equals(player)) {
            isStartedListener = new ValueEventListener() {
                @Override
                public void onDataChange(@NonNull DataSnapshot snapshot) {

• Read up
  Read up

Method endGame has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public static void endGame(int numberOfCollectedCoins, int score, String playerId, @NonNull List<Player> players, @NonNull Activity currentActivity, boolean hasDied) {

Method postNewGame has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public void postNewGame(String name, int maxPlayerCount, int numCoins, double gameRadius, double gameDuration, @NonNull LocationRepresentation location) {