EnrollmentMobileSeeder::seedStudies() - Code Metrics - Inspection of "Changing interfaces" - manelgavalda/enrollment_mobile - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 7ec402...2e144b )

by Manel

created 2016-12-13 15:02 UTC

EnrollmentMobileSeeder::seedStudies() A

↳ Parent: EnrollmentMobileSeeder

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	3
ccs	0
cts	3
cp	0
rs	10
c	0
b	0
f	0
cc	1
eloc	1
nc	1
nop	0
crap	2

<?php

namespace Scool\EnrollmentMobile\Database\Seeds;

use Illuminate\Database\Seeder;


/**
 * Class EnrollmentSeeder
 */
class EnrollmentMobileSeeder extends Seeder
{
    /**
     * Run the database Seeds.
     *
     * @return void
     */
    public function run()
    {
        $this->seedStudies();
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}
        $this->seedCourses();
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}
    }

    private function seedStudies()
    {
    }

    private function seedCourses()
    {
    }
}


1			<?php
2
3			namespace Scool\EnrollmentMobile\Database\Seeds;
4
5			use Illuminate\Database\Seeder;
6
7
8			/**
9			* Class EnrollmentSeeder
10			*/
11			class EnrollmentMobileSeeder extends Seeder
12			{
13			/**
14			* Run the database Seeds.
15			*
16			* @return void
17			*/
18			public function run()
19			{
20			$this->seedStudies();
			0 ignored issues – show Unused Code introduced 2016-12-13 15:14 UTC by Report Bug Copy Issue Report The call to the method `Scool\EnrollmentMobile\D...leSeeder::seedStudies()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
21			$this->seedCourses();
			0 ignored issues – show Unused Code introduced 2016-12-13 15:14 UTC by Report Bug Copy Issue Report The call to the method `Scool\EnrollmentMobile\D...leSeeder::seedCourses()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
22			}
23
24			private function seedStudies()
25			{
26			}
27
28			private function seedCourses()
29			{
30			}
31			}
32

manelgavalda / enrollment_mobile

Push — master ( 7ec402...2e144b )

EnrollmentMobileSeeder::seedStudies() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like