qcubed/framework

Method __set has 45 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        public function __set($strName, $mixValue) {
            try {
                switch ($strName) {
                    case 'KeyName':
                        return $this->strKeyName = QType::Cast($mixValue, QType::String);

Method __get has 45 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        public function __get($strName) {
            switch ($strName) {
                case 'KeyName':
                    return $this->strKeyName;
                case 'Table':

The class QManyToManyReference has 18 fields. Consider redesigning QManyToManyReference to keep the number of fields under 15.

Open

    class QManyToManyReference extends QBaseClass {

        /////////////////////////////
        // Protected Member Variables
        /////////////////////////////

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Person {
   protected $one;
   private $two;
   private $three;
   [... many more fields ...]
}

Avoid too many return statements within this method.

Open

                    return $this->strOppositeColumn;

Avoid too many return statements within this method.

Open

                    return $this->strOppositeDbType;

Avoid too many return statements within this method.

Open

                    return $this->strVariableName;

Avoid too many return statements within this method.

Open

                    return $this->strObjectDescriptionPlural;

Avoid too many return statements within this method.

Open

                        return $this->strOppositeColumn = QType::Cast($mixValue, QType::String);

Avoid too many return statements within this method.

Open

                        return $this->strOppositeVariableName = QType::Cast($mixValue, QType::String);

Avoid too many return statements within this method.

Open

                    return $this->strOppositeObjectDescription;

Avoid too many return statements within this method.

Open

                    return $this->strVariableType;

Avoid too many return statements within this method.

Open

                    return $this->strObjectDescription;

Avoid too many return statements within this method.

Open

                    return $this->blnIsTypeAssociation;

Avoid too many return statements within this method.

Open

                    return $this->options;

Avoid too many return statements within this method.

Open

                    return $this->strOppositeVariableType;

Avoid too many return statements within this method.

Open

                    return $this->strOppositePropertyName;

Avoid too many return statements within this method.

Open

                        return $this->strOppositeVariableType = QType::Cast($mixValue, QType::String);

Avoid too many return statements within this method.

Open

                    return $this->strOppositeVariableName;

Avoid too many return statements within this method.

Open

                    return $this->objColumnArray;

Avoid too many return statements within this method.

Open

                    return $this->strAssociatedTable;

Avoid too many return statements within this method.

Open

                        return parent::__get($strName);

Avoid too many return statements within this method.

Open

                        return $this->strOppositeDbType = QType::Cast($mixValue, QType::String);

Avoid too many return statements within this method.

Open

                        return $this->strOppositePropertyName = QType::Cast($mixValue, QType::String);

Avoid too many return statements within this method.

Open

                        return $this->blnIsTypeAssociation = QType::Cast($mixValue, QType::Boolean);

Avoid too many return statements within this method.

Open

                        return $this->strAssociatedTable = QType::Cast($mixValue, QType::String);

Avoid too many return statements within this method.

Open

                        return parent::__set($strName, $mixValue);

Avoid too many return statements within this method.

Open

                        return $this->strObjectDescription = QType::Cast($mixValue, QType::String);

Avoid too many return statements within this method.

Open

                        return $this->strVariableName = QType::Cast($mixValue, QType::String);

Avoid too many return statements within this method.

Open

                        return $this->strVariableType = QType::Cast($mixValue, QType::String);

Avoid too many return statements within this method.

Open

                        return $this->strObjectDescriptionPlural = QType::Cast($mixValue, QType::String);

Avoid too many return statements within this method.

Open

                        return $this->strOppositeObjectDescription = QType::Cast($mixValue, QType::String);

Avoid too many return statements within this method.

Open

                        return $this->options = QType::Cast($mixValue, QType::ArrayType);

Avoid too many return statements within this method.

Open

                        return $this->objColumnArray = QType::Cast($mixValue, QType::ArrayType);                        

The method __get() has a Cyclomatic Complexity of 20. The configured cyclomatic complexity threshold is 10.

Open

        public function __get($strName) {
            switch ($strName) {
                case 'KeyName':
                    return $this->strKeyName;
                case 'Table':

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

The method __set() has a Cyclomatic Complexity of 20. The configured cyclomatic complexity threshold is 10.

Open

        public function __set($strName, $mixValue) {
            try {
                switch ($strName) {
                    case 'KeyName':
                        return $this->strKeyName = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strObjectDescriptionPlural = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->options = QType::Cast($mixValue, QType::ArrayType);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strAssociatedTable = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strPropertyName = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strOppositeVariableType = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strOppositeObjectDescription = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strOppositeVariableName = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->blnIsTypeAssociation = QType::Cast($mixValue, QType::Boolean);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->objColumnArray = QType::Cast($mixValue, QType::ArrayType);                        

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strColumn = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strVariableName = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strKeyName = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strOppositePropertyName = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strOppositeColumn = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strTable = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strOppositeDbType = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strVariableType = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'QType' in method '__set'.

Open

                        return $this->strObjectDescription = QType::Cast($mixValue, QType::String);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid excessively long variable names like $strOppositePropertyName. Keep variable name length under 20.

Open

        protected $strOppositePropertyName;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $strObjectDescriptionPlural. Keep variable name length under 20.

Open

        protected $strObjectDescriptionPlural;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $strOppositeVariableName. Keep variable name length under 20.

Open

        protected $strOppositeVariableName;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $strOppositeObjectDescription. Keep variable name length under 20.

Open

        protected $strOppositeObjectDescription;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}

Avoid excessively long variable names like $strOppositeVariableType. Keep variable name length under 20.

Open

        protected $strOppositeVariableType;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    protected $reallyLongIntName = -3; // VIOLATION - Field
    public static function main( array $interestingArgumentsList[] ) { // VIOLATION - Formal
        $otherReallyLongName = -5; // VIOLATION - Local
        for ($interestingIntIndex = 0; // VIOLATION - For
             $interestingIntIndex < 10;
             $interestingIntIndex++ ) {
        }
    }
}