Název metody vracející exception handler

besir

Ahoj,
v práci s kolegou řešíme menší problém v konvenci. Máme metodu co vrací exception, její název je sice vypovídající, ale chtěli bychom se vyhnout mít v metodě název handle viz. kod níže..

Napadá Vás nějaký rozumnější název?

	...
	catch (\Exception $exception)
		{
			$this->handleException($exception);
		}
	}

	/**
	 * Exception handler
	 * @author
	 * @param \Exception $exception
	 * @return void
	 */
	private function handleException(\Exception $exception)
	{
		if (get_class($exception) === 'Nette\\Application\\AbortException')
		{
			throw $exception;/** @internal Used by Nette for different response type */
		}

		$this->sendResponse(new \Nette\Application\Responses\JsonResponse(array(
					'status' => 'ERROR',
					'code' => $exception->getCode(),
					'message' => $exception->getMessage(),
					'type' => get_class($exception),
				)));
	}

Vojtěch Dobeš

$this->fail($e);

besir

Když on to vlastně fail také neni :-/ Tak jsme se shodli na nazvu exceptionHandler :D a spokojeny jsme oba :D

Editoval besir (21. 9. 2012 16:01)

paranoiq

jméno metody by vždy mělo začínat slovesem

besir

njn, ale nic jiného nás nenapadlo :D

paranoiq

processException? manageException?

Filip Procházka

handle je taky sloveso :P byť nevhodné

duke

Co třeba resolveException?

studna

A co třebaa.. reportError(či Exception) nebo sendError. Pak je jasnější, co se uvnitř děje..no ne? :)

Filip Procházka

Abych se vrátil k původnímu tématu..

use Nette\Application\Responses\JsonResponse;
use Nette\Diagnostics\Debugger;

class MyPresenter extends BasePresenter
{
	protected function startup()
	{
		parent::startup();
		$httpResponse = $this->getHttpResponse();
		$this->application->onError[] = function ($app, \Exception $e) use ($httpResponse) {
			Debugger::log($e);
			$httpResponse->setCode(500);
			$httpResponse->setContentType('application/json');
			echo Nette\Utils\Json::encode(array(
                                'status' => 'ERROR',
                                'code' => $e->getCode(),
                                'message' => $e->getMessage(),
                                'type' => get_class($e),
                        ));
			exit;
		};
	}
}

Popřemýšlel bych, jestli je vracení obsahu výjimky dobrý nápad.

Editoval HosipLan (22. 9. 2012 16:56)