Improved set function

gistfile1.php

/**
 * Original
 * Set name of the the item 
 *
 * @param string $name
 */
public function setName($name = "")
{
	if (empty($name)) {
		throw new \Exception("Item name can't be empty");
	} else {
		$this->name = $name;
	}
}

/**
 * Improved
 * The problem with the previous approach was that if $name is a boolean or soehting like " " the exception wouldn't be fired
 * We need to add some extra checks on the varialbe $name
 *
 * @param string $name
 */
 
public function setName($name = "")
{
	$name = trim($name);
	if (empty($name) || !is_string($name)) {
		throw new \Exception("Item must be a string and can't empty");
	}
	$this->name = $name;
}