PlaceManager::sendNotif() - Code Metrics - Inspection of "Merge pull request #164 from rtfmcorp/dev" - rtfmcorp/asylamba-game - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Push — master ( 710d07...9f3829 )

by Jacky
created 2017-08-09 22:05 UTC
PlaceManager::sendNotif() D

↳ Parent: PlaceManager
Complexity

Conditions	15
Paths	15
Size

Total Lines	302
Code Lines	294
Duplication

Lines	0
Ratio	0 %
Importance

Changes
Metric	Value
cc	15
eloc	294
nc	15
nop	4
dl	0
loc	302
rs	4.9121
c	0
b	0
f	0
How to fix Long Method Complexity

<?php

/**
 * Place Manager
 *
 * @author Jacky Casas
 * @copyright Expansion - le jeu
 *
 * @package Gaia
 * @update 20.05.13
*/
namespace Asylamba\Modules\Gaia\Manager;

use Asylamba\Classes\Entity\EntityManager;
use Asylamba\Classes\Library\Utils;
use Asylamba\Classes\Library\Game;
use Asylamba\Classes\Library\Format;

use Asylamba\Modules\Hermes\Manager\NotificationManager;

use Asylamba\Modules\Gaia\Model\Place;
use Asylamba\Modules\Ares\Model\Commander;
use Asylamba\Modules\Ares\Model\Report;
use Asylamba\Modules\Hermes\Model\Notification;
use Asylamba\Modules\Gaia\Model\System;

use Asylamba\Modules\Gaia\Event\PlaceOwnerChangeEvent;
use Asylamba\Classes\Worker\EventDispatcher;

class PlaceManager {
	/** @var EntityManager **/
	protected $entityManager;
	/** @var NotificationManager **/
	protected $notificationManager;
	/** @var EventDispatcher **/
	protected $eventDispatcher;
	
	/**
	 * @param EntityManager $entityManager
	 * @param NotificationManager $notificationManager
	 * @param EventDispatcher $eventDispatcher
	 */
	public function __construct(EntityManager $entityManager, NotificationManager $notificationManager, EventDispatcher $eventDispatcher)
	{
		$this->entityManager = $entityManager;
		$this->notificationManager = $notificationManager;
		$this->eventDispatcher = $eventDispatcher;
	}
	
	/**
	 * @param int $id
	 * @return Place
	 */
	public function get($id)
	{
		if(($place = $this->entityManager->getRepository(Place::class)->get($id)) !== null) {
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
			return $place;
		}
		return null;
	}
	
	/**
	 * @param int $ids
	 */
	public function getByIds($ids = [])
	{
		return $this->entityManager->getRepository(Place::class)->getByIds($ids);
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
	}
	
	public function getSystemPlaces(System $system)
	{
		return $this->entityManager->getRepository(Place::class)->getSystemPlaces($system->getId());
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
	}

	/**
	 * @param string $search
	 * @return array
	 */
	public function search($search) {
		return $this->entityManager->getRepository(Place::class)->search($search);
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
	}
	
	/**
	 * @return array
	 */
	public function getPlayerPlaces()
	{
		return $this->entityManager->getRepository(Place::class)->getPlayerPlaces();
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
	}
	
	/**
	 * @return array
	 */
	public function getNpcPlaces()
	{
		return $this->entityManager->getRepository(Place::class)->getNpcPlaces();
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
	}

	public function updateNpcPlaces() {
		$places = $this->getNpcPlaces();
		$now   = Utils::now();
		$repository = $this->entityManager->getRepository(Place::class);
		$this->entityManager->beginTransaction();
		
		foreach ($places as $place) {
			if (Utils::interval($place->uPlace, $now, 's') === 0) {
				continue;
			}
			# update time
			$hours = Utils::intervalDates($now, $place->uPlace);
			$place->uPlace = $now;
			$initialResources = $place->resources;
			$initialDanger = $place->danger;
			$maxResources = ceil($place->population / Place::COEFFPOPRESOURCE) * Place::COEFFMAXRESOURCE * ($place->maxDanger + 1);

			foreach ($hours as $hour) {
				$place->danger += Place::REPOPDANGER;
				$place->resources += floor(Place::COEFFRESOURCE * $place->population);
			}
			// The repository method will add the new resources. We have to calculate how many resources have been added
			$place->resources = abs($place->resources - $initialResources);
			// If the max is reached, we have to add just the difference between the max and init value
			if ($place->resources > $maxResources) {
				$place->resources = $maxResources - $initialResources;
			}
			$place->danger = abs($place->danger - $initialDanger);
			// Same thing here
			if ($place->danger > $place->maxDanger) {
				$place->danger = $place->maxDanger - $initialDanger;
			}
			$repository->updatePlace($place, true);

		}
		$repository->npcQuickfix();
		$this->entityManager->commit();
		$this->entityManager->clear(Place::class);
	}

	public function updatePlayerPlaces() {
		$places = $this->getPlayerPlaces();
		$now   = Utils::now();
		$repository = $this->entityManager->getRepository(Place::class);
		$this->entityManager->beginTransaction();
		
		foreach ($places as $place) {
			if (Utils::interval($place->uPlace, $now, 's') === 0) {
				continue;
			}
			# update time
			$hours = Utils::intervalDates($now, $place->uPlace);
			$place->uPlace = $now;
			$initialResources = $place->resources;
			$maxResources = ceil($place->population / Place::COEFFPOPRESOURCE) * Place::COEFFMAXRESOURCE * ($place->maxDanger + 1);
			foreach ($hours as $hour) {
				$place->resources += floor(Place::COEFFRESOURCE * $place->population);
			}
			$place->resources = abs($place->resources - $initialResources);
			if ($place->resources > $maxResources) {
				$place->resources = $maxResources;
			}
			$repository->updatePlace($place);

		}
		$this->entityManager->commit();
		$this->entityManager->clear(Place::class);
	}
    
    /**
     * @param Place $place
     * @return bool
     */
    public function turnAsEmptyPlace(Place $place)
    {
        return $this->entityManager->getRepository(Place::class)->turnAsEmptyPlace($place->getId());
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
    }
	
	public function turnAsSpawnPlace($placeId, $playerId)
	{
		$this->eventDispatcher->dispatch(new PlaceOwnerChangeEvent($this->get($placeId)));
		
		return $this->entityManager->getRepository(Place::class)->turnAsSpawnPlace($placeId, $playerId);
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
	}

	/**
	 * @param Place $place
	 * @param string $case
	 * @param Commander $commander
	 * @param Report $report
	 */
	public function sendNotif(Place $place, $case, Commander $commander, $report = NULL) {
		switch ($case) {
			case Place::CHANGESUCCESS:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Déplacement réussi');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/commander-' . $commander->getId(), $commander->getName())
					->addTxt(' est arrivé sur ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt('.')
					->addEnd();
				$this->notificationManager->add($notif);
				break;

			case Place::CHANGEFAIL:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Déplacement réussi');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/commander-' . $commander->getId(), $commander->getName())
					->addTxt(' s\'est posé sur ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt('. Il est en garnison car il n\'y avait pas assez de place en orbite.')
					->addEnd();
				$this->notificationManager->add($notif);
				break;
			case Place::CHANGELOST:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Déplacement raté');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/commander-' . $commander->getId(), $commander->getName())
					->addTxt(' n\'est pas arrivé sur ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt('. Cette base ne vous appartient pas. Elle a pu être conquise entre temps.')
					->addEnd();
				$this->notificationManager->add($notif);
				break;
			case Place::LOOTEMPTYSSUCCESS:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Pillage réussi');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/commander-' . $commander->getId() . '/sftr-3', $commander->getName())
					->addTxt(' a pillé la planète rebelle située aux coordonnées ')
					->addLnk('map/place-' . $place->id, Game::formatCoord($place->xSystem, $place->ySystem, $place->position, $place->rSector))
					->addTxt('.')
					->addSep()
					->addBoxResource('resource', Format::number($commander->getResources()), 'ressources pillées')
					->addBoxResource('xp', '+ ' . Format::number($commander->earnedExperience), 'expérience de l\'officier')
					->addSep()
					->addLnk('fleet/view-archive/report-' . $report, 'voir le rapport')
					->addEnd();
				$this->notificationManager->add($notif);
				break;
			case Place::LOOTEMPTYFAIL:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Pillage raté');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/view-memorial', $commander->getName())
					->addTxt(' est tombé lors de l\'attaque de la planète rebelle située aux coordonnées ')
					->addLnk('map/place-' . $place->id, Game::formatCoord($place->xSystem, $place->ySystem, $place->position, $place->rSector))
					->addTxt('.')
					->addSep()
					->addTxt('Il a désormais rejoint le Mémorial. Que son âme traverse l\'Univers dans la paix.')
					->addSep()
					->addLnk('fleet/view-archive/report-' . $report, 'voir le rapport')
					->addEnd();
				$this->notificationManager->add($notif);
				break;
			case Place::LOOTPLAYERWHITBATTLESUCCESS:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Pillage réussi');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/commander-' . $commander->getId() . '/sftr-3', $commander->getName())
					->addTxt(' a pillé la planète ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt(' appartenant au joueur ')
					->addLnk('embassy/player-' . $place->rPlayer, $place->playerName)
					->addTxt('.')
					->addSep()
					->addBoxResource('resource', Format::number($commander->getResources()), 'ressources pillées')
					->addBoxResource('xp', '+ ' . Format::number($commander->earnedExperience), 'expérience de l\'officier')
					->addSep()
					->addLnk('fleet/view-archive/report-' . $report, 'voir le rapport')
					->addEnd();
				$this->notificationManager->add($notif);

				$notif = new Notification();
				$notif->setRPlayer($place->rPlayer);
				$notif->setTitle('Rapport de pillage');
				$notif->addBeg()
					->addTxt('L\'officier ')
					->addStg($commander->getName())
					->addTxt(' appartenant au joueur ')
					->addLnk('embassy/player-' . $commander->getRPlayer(), $commander->getPlayerName())
					->addTxt(' a pillé votre planète ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt('.')
					->addSep()
					->addBoxResource('resource', Format::number($commander->getResources()), 'ressources pillées')
					->addSep()
					->addLnk('fleet/view-archive/report-' . $report, 'voir le rapport')
					->addEnd();
				$this->notificationManager->add($notif);
				break;
			case Place::LOOTPLAYERWHITBATTLEFAIL:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Pillage raté');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/view-memorial', $commander->getName())
					->addTxt(' est tombé lors du pillage de la planète ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt(' appartenant au joueur ')
					->addLnk('embassy/player-' . $place->rPlayer, $place->playerName)
					->addTxt('.')
					->addSep()
					->addTxt('Il a désormais rejoint le Mémorial. Que son âme traverse l\'Univers dans la paix.')
					->addSep()
					->addLnk('fleet/view-archive/report-' . $report, 'voir le rapport')
					->addEnd();
				$this->notificationManager->add($notif);

				$notif = new Notification();
				$notif->setRPlayer($place->rPlayer);
				$notif->setTitle('Rapport de combat');
				$notif->addBeg()
					->addTxt('L\'officier ')
					->addStg($commander->getName())
					->addTxt(' appartenant au joueur ')
					->addLnk('embassy/player-' . $commander->getRPlayer(), $commander->getPlayerName())
					->addTxt(' a attaqué votre planète ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt('.')
					->addSep()
					->addTxt('Vous avez repoussé l\'ennemi avec succès.')
					->addSep()
					->addLnk('fleet/view-archive/report-' . $report, 'voir le rapport')
					->addEnd();
				$this->notificationManager->add($notif);
				break;
			case Place::LOOTPLAYERWHITOUTBATTLESUCCESS:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Pillage réussi');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/commander-' . $commander->getId() . '/sftr-3', $commander->getName())
					->addTxt(' a pillé la planète non défendue ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt(' appartenant au joueur ')
					->addLnk('embassy/player-' . $place->rPlayer, $place->playerName)
					->addTxt('.')
					->addSep()
					->addBoxResource('resource', Format::number($commander->getResources()), 'ressources pillées')
					->addBoxResource('xp', '+ ' . Format::number($commander->earnedExperience), 'expérience de l\'officier')
					->addEnd();
				$this->notificationManager->add($notif);

				$notif = new Notification();
				$notif->setRPlayer($place->rPlayer);
				$notif->setTitle('Rapport de pillage');
				$notif->addBeg()
					->addTxt('L\'officier ')
					->addStg($commander->getName())
					->addTxt(' appartenant au joueur ')
					->addLnk('embassy/player-' . $commander->getRPlayer(), $commander->getPlayerName())
					->addTxt(' a pillé votre planète ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt('. Aucune flotte n\'était en position pour la défendre. ')
					->addSep()
					->addBoxResource('resource', Format::number($commander->getResources()), 'ressources pillées')
					->addEnd();
				$this->notificationManager->add($notif);
				break;
			case Place::LOOTLOST:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Erreur de coordonnées');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/commander-' . $commander->getId() . '/sftr-3', $commander->getName())
					->addTxt(' n\'a pas attaqué la planète ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt(' car son joueur est de votre faction, sous la protection débutant ou un allié.')
					->addEnd();
				$this->notificationManager->add($notif);
				break;
			case Place::CONQUEREMPTYSSUCCESS:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Colonisation réussie');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/commander-' . $commander->getId() . '/sftr-3', $commander->getName())
					->addTxt(' a colonisé la planète rebelle située aux coordonnées ')  
					->addLnk('map/place-' . $place->id , Game::formatCoord($place->xSystem, $place->ySystem, $place->position, $place->rSector) . '.')

					->addBoxResource('xp', '+ ' . Format::number($commander->earnedExperience), 'expérience de l\'officier')
					->addTxt('Votre empire s\'étend, administrez votre ')
					->addLnk('bases/base-' . $place->id, 'nouvelle planète')
					->addTxt('.')
					->addSep()
					->addLnk('fleet/view-archive/report-' . $report, 'voir le rapport')
					->addEnd();
				$this->notificationManager->add($notif);
				break;
			case Place::CONQUEREMPTYFAIL:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Colonisation ratée');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/view-memorial', $commander->getName())
					->addTxt(' est tombé lors de l\'attaque de la planète rebelle située aux coordonnées ')
					->addLnk('map/place-' . $place->id, Game::formatCoord($place->xSystem, $place->ySystem, $place->position, $place->rSector))
					->addTxt('.')
					->addSep()
					->addTxt('Il a désormais rejoint le Mémorial. Que son âme traverse l\'Univers dans la paix.')
					->addSep()
					->addLnk('fleet/view-archive/report-' . $report, 'voir le rapport')
					->addEnd();
				$this->notificationManager->add($notif);
				break;
			case Place::CONQUERPLAYERWHITOUTBATTLESUCCESS:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Conquête réussie');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/commander-' . $commander->getId() . '/sftr-3', $commander->getName())
					->addTxt(' a conquis la planète non défendue ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt(' appartenant au joueur ')
					->addLnk('embassy/player-' . $place->rPlayer, $place->playerName)
					->addTxt('.')
					->addSep()
					->addBoxResource('xp', '+ ' . Format::number($commander->earnedExperience), 'expérience de l\'officier')
					->addTxt('Elle est désormais votre, vous pouvez l\'administrer ')
					->addLnk('bases/base-' . $place->id, 'ici')
					->addTxt('.')
					->addEnd();
				$this->notificationManager->add($notif);

				$notif = new Notification();
				$notif->setRPlayer($place->rPlayer);
				$notif->setTitle('Planète conquise');
				$notif->addBeg()
					->addTxt('L\'officier ')
					->addStg($commander->getName())
					->addTxt(' appartenant au joueur ')
					->addLnk('embassy/player-' . $commander->getRPlayer(), $commander->getPlayerName())
					->addTxt(' a conquis votre planète non défendue ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt('.')
					->addSep()
					->addTxt('Impliquez votre faction dans une action punitive envers votre assaillant.')
					->addEnd();
				$this->notificationManager->add($notif);
				break;
			case Place::CONQUERLOST:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Erreur de coordonnées');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/commander-' . $commander->getId() . '/sftr-3', $commander->getName())
					->addTxt(' n\'a pas attaqué la planète ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt(' car le joueur est dans votre faction, sous la protection débutant ou votre allié.')
					->addEnd();
				$this->notificationManager->add($notif);
				break;
			case Place::COMEBACK:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Rapport de retour');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/commander-' . $commander->getId() . '/sftr-3', $commander->getName())
					->addTxt(' est de retour sur votre base ')
					->addLnk('map/place-' . $commander->getRBase(), $commander->getBaseName())
					->addTxt(' et rapporte ')
					->addStg(Format::number($commander->getResources()))
					->addTxt(' ressources à vos entrepôts.')
					->addEnd();
				$this->notificationManager->add($notif);
				break;
			
			default: break;
switch ($expr) {
    default:
        doSomething(); //right
        break;
}


switch ($expr) {
    default:

        doSomething(); //wrong
        break;
}
		}
	}

	/**
	 * @param Place $place
	 * @param string $case
	 * @param Commander $commander
	 * @param array $reports
	 */
	public function sendNotifForConquest(Place $place, $case, $commander, $reports = array()) {
		$nbrBattle = count($reports);
		switch($case) {

			case Place::CONQUERPLAYERWHITBATTLESUCCESS:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Conquête réussie');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/commander-' . $commander->getId() . '/sftr-3', $commander->getName())
					->addTxt(' a conquis la planète ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt(' appartenant au joueur ')
					->addLnk('embassy/player-' . $place->rPlayer, $place->playerName)
					->addTxt('.')
					->addSep()
					->addTxt($nbrBattle . Format::addPlural($nbrBattle, ' combats ont eu lieu.', ' seul combat a eu lieu'))
					->addSep()
					->addBoxResource('xp', '+ ' . Format::number($commander->earnedExperience), 'expérience de l\'officier')
					->addSep()
					->addTxt('Elle est désormais vôtre, vous pouvez l\'administrer ')
					->addLnk('bases/base-' . $place->id, 'ici')
					->addTxt('.');
				for ($i = 0; $i < $nbrBattle; $i++) {
					$notif->addSep();
					$notif->addLnk('fleet/view-archive/report-' . $reports[$i], 'voir le ' . Format::ordinalNumber($i + 1) . ' rapport');
				}
				$notif->addEnd();
				$this->notificationManager->add($notif);

				$notif = new Notification();
				$notif->setRPlayer($place->rPlayer);
				$notif->setTitle('Planète conquise');
				$notif->addBeg()
					->addTxt('L\'officier ')
					->addStg($commander->getName())
					->addTxt(' appartenant au joueur ')
					->addLnk('embassy/player-' . $commander->getRPlayer(), $commander->getPlayerName())
					->addTxt(' a conquis votre planète ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt('.')
					->addSep()
					->addTxt($nbrBattle . Format::addPlural($nbrBattle, ' combats ont eu lieu.', ' seul combat a eu lieu'))
					->addSep()
					->addTxt('Impliquez votre faction dans une action punitive envers votre assaillant.');
				for ($i = 0; $i < $nbrBattle; $i++) {
					$notif->addSep();
					$notif->addLnk('fleet/view-archive/report-' . $reports[$i], 'voir le ' . Format::ordinalNumber($i + 1) . ' rapport');
				}
				$notif->addEnd();
				$this->notificationManager->add($notif);
				break;
			case Place::CONQUERPLAYERWHITBATTLEFAIL:
				$notif = new Notification();
				$notif->setRPlayer($commander->getRPlayer());
				$notif->setTitle('Conquête ratée');
				$notif->addBeg()
					->addTxt('Votre officier ')
					->addLnk('fleet/view-memorial/', $commander->getName())
					->addTxt(' est tombé lors de la tentive de conquête de la planète ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt(' appartenant au joueur ')
					->addLnk('embassy/player-' . $place->rPlayer, $place->playerName)
					->addTxt('.')
					->addSep()
					->addTxt($nbrBattle . Format::addPlural($nbrBattle, ' combats ont eu lieu.', ' seul combat a eu lieu'))
					->addSep()
					->addTxt('Il a désormais rejoint de Mémorial. Que son âme traverse l\'Univers dans la paix.');
				for ($i = 0; $i < $nbrBattle; $i++) {
					$notif->addSep();
					$notif->addLnk('fleet/view-archive/report-' . $reports[$i], 'voir le ' . Format::ordinalNumber($i + 1) . ' rapport');
				}
				$notif->addEnd();
				$this->notificationManager->add($notif);

				$notif = new Notification();
				$notif->setRPlayer($place->rPlayer);
				$notif->setTitle('Rapport de combat');
				$notif->addBeg()
					->addTxt('L\'officier ')
					->addStg($commander->getName())
					->addTxt(' appartenant au joueur ')
					->addLnk('embassy/player-' . $commander->getRPlayer(), $commander->getPlayerName())
					->addTxt(' a tenté de conquérir votre planète ')
					->addLnk('map/place-' . $place->id, $place->baseName)
					->addTxt('.')
					->addSep()
					->addTxt($nbrBattle . Format::addPlural($nbrBattle, ' combats ont eu lieu.', ' seul combat a eu lieu'))
					->addSep()
					->addTxt('Vous avez repoussé l\'ennemi avec succès. Bravo !');
				for ($i = 0; $i < $nbrBattle; $i++) {
					$notif->addSep();
					$notif->addLnk('fleet/view-archive/report-' . $reports[$i], 'voir le ' . Format::ordinalNumber($i + 1) . ' rapport');
				}
				$notif->addEnd();
				$this->notificationManager->add($notif);
				break;

			default: break;
switch ($expr) {
    default:
        doSomething(); //right
        break;
}


switch ($expr) {
    default:

        doSomething(); //wrong
        break;
}
		}
	}
}
GitHub Access Token became invalid

Push — master ( 710d07...9f3829 )

PlaceManager::sendNotif() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

rtfmcorp / asylamba-game

GitHub Access Token became invalid

Push — master ( 710d07...9f3829 )

PlaceManager::sendNotif() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like
