bykovme/nswallet

Class BL has 40 methods (exceeds 20 allowed). Consider refactoring.

Open

    public static partial class BL
    {
        static string lang;
        static string lastCurrentID;
        public static string CurrentFieldID { get; private set; }

File BusinessLayer.cs has 381 lines of code (exceeds 250 allowed). Consider refactoring.

Open

using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
using NSWallet.Shared.Helpers.Logs.AppLog;

Method GetFieldTypeNameByShortName has 42 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        public static string GetFieldTypeNameByShortName(string fieldTypeShortName)
        {
            switch (fieldTypeShortName) {
                case GConsts.FLDTYPE_ACNT:
                    return TR.Tr("ACNT");

Method GetCurrentPath has 32 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        public static string GetCurrentPath()
        {
            if (CurrentItemID == GConsts.EXPIRING_SOON_ID)
            {
                return "/" + TR.Tr("expiringsoon_folder") + "/";

Method CreateSampleItems has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        public static void CreateSampleItems()
        {
            var dal = DataAccessLayer.GetInstance();

            // Banking

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

Open

        public static bool UpdateIcon(string iconID, string name, byte[] blob = null, int groupID = -1, int isCircle = -1)

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

Open

        public static void ResetData(bool resetItems, bool resetFields, bool resetLabels, bool icons = false, bool groups = false)

Avoid too many return statements within this method.

Open

                return bufferedPath[CurrentItemID];

Avoid too many return statements within this method.

Open

            return bufferedPath[CurrentItemID];

Avoid too many return statements within this method.

Open

                return "/" + TR.Tr("mostlyviewed_folder") + "/";

Avoid too many return statements within this method.

Open

                return "/" + TR.Tr("mostlyviewed_folder") + "/" + GetCurrentItem().Name + "/";

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

Open

        public static bool UpdateGroup(int groupID, string name)
        {
            var dal = DataAccessLayer.GetInstance();
            var res = dal.UpdateGroup(groupID, name);
            ResetData(false, false, false, false, true);

• Read up
  Read up

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

Open

        public static bool InsertGroup(int groupID, string name)
        {
            var dal = DataAccessLayer.GetInstance();
            var result = dal.InsertGroup(groupID, name);
            ResetData(false, false, false, false, true);

• Read up
  Read up