File FakeDatabase.java has 630 lines of code (exceeds 250 allowed). Consider refactoring.

Wontfix

package ch.epfl.sweng.favors.database;

import android.app.Activity;
import android.os.Handler;
import android.os.HandlerThread;

Method createBasicDatabase has 269 lines of code (exceeds 25 allowed). Consider refactoring.

Wontfix

    public void createBasicDatabase(){
        getInstance().cleanUp();

        User u0 = new User(FakeAuthentication.UID);
        User u1 = new User("U1");

Method check has a Cognitive Complexity of 46 (exceeds 5 allowed). Consider refactoring.

Wontfix

    private boolean check(CheckType type, Map.Entry<DatabaseField, Object>  entry, DatabaseEntity entity){
        if(entry == null || entity == null) return false;

        if(entry.getKey() instanceof DatabaseStringField) {
            String temp = entity.get((DatabaseStringField) entry.getKey());

• Read up
  Read up

FakeDatabase has 22 methods (exceeds 20 allowed). Consider refactoring.

Wontfix

public class FakeDatabase extends Database{

    public static final String LAST_FAVOR_TITLE = "Flash needs some help";
    public static FakeDatabase db = null;
    private HashMap<String, DatabaseEntity> database;

Method addExtraToDb has 44 lines of code (exceeds 25 allowed). Consider refactoring.

Wontfix

    public void addExtraToDb(){
        Favor fNew = new Favor("newFavor");
        Favor fNew2 = new Favor("newFavor2");

        fNew.set(Favor.StringFields.ownerID, "U3");

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

Wontfix

    private boolean check(CheckType type, Map.Entry<DatabaseField, Object>  entry, DatabaseEntity entity){
        if(entry == null || entity == null) return false;

        if(entry.getKey() instanceof DatabaseStringField) {
            String temp = entity.get((DatabaseStringField) entry.getKey());

Method getList has 9 arguments (exceeds 4 allowed). Consider refactoring.

Wontfix

    protected  <T extends DatabaseEntity>  void getList(ObservableArrayList<T> list, Class<T> clazz,
                                                        String collection,
                                                        Map<DatabaseField, Object> mapEquals,
                                                        Map<DatabaseField, Object> mapLess,
                                                        Map<DatabaseField, Object> mapMore,

Method getLiveList has 7 arguments (exceeds 4 allowed). Consider refactoring.

Wontfix

    protected  <T extends DatabaseEntity> void getLiveList(ObservableArrayList<T> list, Class<T> clazz,
                                                           String collection,
                                                           DatabaseField key,
                                                           Object value,
                                                           Integer limit,

Method getList has 7 arguments (exceeds 4 allowed). Consider refactoring.

Wontfix

    protected  <T extends DatabaseEntity> void getList(ObservableArrayList<T> list, Class<T> clazz,
                                                       String collection,
                                                       DatabaseField key,
                                                       Object value,
                                                       Integer limit,

Method getElement has 5 arguments (exceeds 4 allowed). Consider refactoring.

Wontfix

    protected  <T extends DatabaseEntity> void getElement(T toUpdate, Class<T> clazz, String collection,
                                                          DatabaseField key, Object value){

Method getAll has 5 arguments (exceeds 4 allowed). Consider refactoring.

Wontfix

    protected <T extends DatabaseEntity> void getAll(ObservableArrayList<T> list, Class<T> clazz, String collection, Integer limit, DatabaseField orderBy) {

Method processMap has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Wontfix

    private boolean processMap(Map<DatabaseField, Object> map, DatabaseEntity entity, Boolean toAdd, CheckType checkType) {
        if (toAdd && map != null) {
            for (Map.Entry<DatabaseField, Object> e : map.entrySet()) {
                if (!check(checkType, e, entity)) {
                    toAdd = false;

• Read up
  Read up

Avoid too many return statements within this method.

Wontfix

                if(type == CheckType.Greater) return out > 0;

Avoid too many return statements within this method.

Wontfix

            if(temp == null || !(entry.getValue() instanceof Boolean)) return false;

Avoid too many return statements within this method.

Wontfix

            if(temp == null || !(entry.getValue() instanceof Long)) return false;

Avoid too many return statements within this method.

Wontfix

            if(type == CheckType.Greater) return temp < (long) entry.getValue();

Avoid too many return statements within this method.

Wontfix

            if(type == CheckType.Less) return temp > (long) entry.getValue();

Avoid too many return statements within this method.

Wontfix

            if(type == CheckType.Equal) return temp == (long) entry.getValue();

Avoid too many return statements within this method.

Wontfix

                if(type == CheckType.Less) return out < 0;

Avoid too many return statements within this method.

Wontfix

            if(type == CheckType.Equal) return temp.equals(entry.getValue());

Avoid too many return statements within this method.

Wontfix

                return ((ArrayList<String>) temp).contains((String) entry.getValue());

Avoid too many return statements within this method.

Wontfix

            if(temp == null || !(entry.getValue() instanceof String)) return false;

Avoid too many return statements within this method.

Wontfix

            else return false; // Comparison of String have no meaning

Avoid too many return statements within this method.

Wontfix

                if(!(entry.getValue() instanceof String)) return false;

Avoid too many return statements within this method.

Wontfix

        return false;

Avoid too many return statements within this method.

Wontfix

            else return false; // Comparison on boolean has no meaning

Avoid too many return statements within this method.

Wontfix

            if(type == CheckType.Equal) return temp == (boolean) entry.getValue();

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

Wontfix

            for(DatabaseEntity entity : database.values()) {
                if (clazz.isInstance(entity) && value instanceof String && entity.get((DatabaseStringField) key).equals(value)) {
                    try {
                        T temp = clazz.newInstance();
                        temp.set(entity.documentID, entity.getEncapsulatedObjectOfMaps());

• Read up
  Read up

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

Wontfix

                    for(DatabaseEntity entity : database.values()) {
                        if (clazz.isInstance(entity) && value instanceof String && entity.get((DatabaseStringField) key).equals(value)) {
                            try {
                                T temp = clazz.newInstance();
                                temp.set(entity.documentID, entity.getEncapsulatedObjectOfMaps());

• Read up
  Read up