Merge pull request #8 from scento/exceptions

Exceptions

src/Isbn/Check.php

@@ -53,9 +53,14 @@ public function identify($isbn)
  *
  * @param string $isbn
  * @return boolean
+ * @throws Exception
  */
  public function is10($isbn)
  {
+ if(is_string($isbn) === false) {
+ throw new Exception('Invalid parameter type.');
+ }
+
  $isbn = $this->hyphens->removeHyphens($isbn);
  return (strlen($isbn) === 10);
  }
@@ -65,9 +70,14 @@ public function is10($isbn)
  *
  * @param string $isbn
  * @return boolean
+ * @throws Exception
  */
  public function is13($isbn)
  {
+ if(is_string($isbn) === false) {
+ throw new Exception('Invalid parameter type.');
+ }
+
  $isbn = $this->hyphens->removeHyphens($isbn);
  return (strlen($isbn) === 13);
  }

src/Isbn/CheckDigit.php

@@ -39,6 +39,10 @@ public function __construct(Hyphens $hyphens)
  */
  public function make($isbn)
  {
+ if(is_string($isbn) === false) {
+ throw new Exception('Invalid parameter type.');
+ }
+
  $isbn = $this->hyphens->removeHyphens($isbn);
  if (strlen($isbn) === 12 or strlen($isbn) === 13) {
  return $this->make13($isbn);
@@ -53,20 +57,25 @@ public function make($isbn)
  * Calculate the check digit of the ISBN-10 $isbn.
  *
  * @param string $isbn
- * @return boolean|string|int
+ * @return string|int
+ * @throws Exception
  */
  public function make10($isbn)
  {
+ if(is_string($isbn) === false) {
+ throw new Exception('Invalid parameter type.');
+ }
+
  //Verify length
  $isbnLength = strlen($isbn);
  if ($isbnLength < 9 or $isbnLength > 10) {
- return false;
+ throw new Exception('Invalid ISBN-10 format.');
  }
 
  //Calculate check digit
  $check = 0;
  for ($i = 0; $i < 9; $i++) {
- if ($isbn[$i] === "X") {
+ if ($isbn[$i] === 'X') {
  $check += 10 * intval(10 - $i);
  } else {
  $check += intval($isbn[$i]) * intval(10 - $i);
@@ -87,14 +96,19 @@ public function make10($isbn)
  * Calculate the check digit of the ISBN-13 $isbn
  *
  * @param string $isbn
- * @return boolean|int
+ * @return int
+ * @throws Exception
  */
  public function make13($isbn)
  {
+ if(is_string($isbn) === false) {
+ throw new Exception('Invalid parameter type.');
+ }
+
  //Verify length
  $isbnLength = strlen($isbn);
  if ($isbnLength < 12 or $isbnLength > 13) {
- return false;
+ throw new Exception('Invalid ISBN-13 format.');
  }
 
  //Calculate check digit

src/Isbn/Exception.php

@@ -0,0 +1,17 @@
+<?php
+/**
+ * ISBN Exception
+ *
+ * @author Wenzel Pünter <[email protected]>
+ * @version 2.0.0
+ * @package ISBN
+*/
+namespace Isbn;
+
+/**
+ * ISBN Exception
+*/
+class Exception extends \Exception
+{
+
+}

src/Isbn/Hyphens.php

@@ -34,11 +34,16 @@ class Hyphens
  *
  * @param string $isbn
  * @return string
+ * @throws Exception
  */
  public function removeHyphens($isbn)
  {
- $isbn = str_replace(" ", "", $isbn);
- $isbn = str_replace("-", "", $isbn);
+ if(is_string($isbn) === false) {
+ throw new Exception('Invalid parameter type.');
+ }
+
+ $isbn = str_replace(' ', '', $isbn);
+ $isbn = str_replace('-', '', $isbn);
  return $isbn;
  }
 
@@ -49,7 +54,7 @@ public function removeHyphens($isbn)
  * @param string $char
  * @return string
  */
- public function fixHyphens($isbn, $char = "-")
+ public function fixHyphens($isbn, $char = '-')
  {
  $isbn = $this->removeHyphens($isbn);
  return $this->addHyphens($isbn, $char);
@@ -60,9 +65,15 @@ public function fixHyphens($isbn, $char = "-")
  *
  * @param string $isbn
  * @param string $char
+ * @throws Exception
  */
- public function addHyphens($isbn, $char = "-")
+ public function addHyphens($isbn, $char = '-')
  {
+ if(is_string($isbn) === false ||
+ is_string($char) === false) {
+ throw new Exception('Invalid parameter type.');
+ }
+
  $this->isbn = $isbn;
  $this->isbnSplit = array();
 
@@ -157,7 +168,7 @@ private function getRegistrantElement()
  if (isset($this->isbnSplit[0]) === true) {
  $soFar = implode('-', $this->isbnSplit);
  } else {
- $soFar = "978-".$this->isbnSplit[1];
+ $soFar = '978-'.$this->isbnSplit[1];
  }
 
  switch ($soFar) {

src/Isbn/Translate.php

@@ -38,9 +38,14 @@ public function __construct(CheckDigit $checkDigit)
  *
  * @param string $isbn
  * @return string
+ * @throws Exception
  */
  public function to10($isbn)
  {
+ if(is_string($isbn) === false) {
+ throw new Exception('Invalid parameter type.');
+ }
+
  if (strlen($isbn) > 13) {
  $isbn = substr($isbn, 4, -1);
  } else {
@@ -55,15 +60,20 @@ public function to10($isbn)
  *
  * @param string $isbn
  * @return string
+ * @throws Exception
  */
  public function to13($isbn)
  {
+ if(is_string($isbn) === false) {
+ throw new Exception('Invalid parameter type.');
+ }
+
  $isbn = substr($isbn, 0, -1);
 
  if (strlen($isbn) > 9) {
- $isbn = "978-".$isbn;
+ $isbn = '978-'.$isbn;
  } else {
- $isbn = "978".$isbn;
+ $isbn = '978'.$isbn;
  }
 
  return $isbn.$this->checkDigit->make($isbn);

src/Isbn/Validation.php

@@ -61,22 +61,27 @@ public function isbn($isbn)
  *
  * @param string $isbn
  * @return boolean
+ * @throws Exception
  */
  public function isbn10($isbn)
  {
+ if(is_string($isbn) === false) {
+ throw new Exception('Invalid parameter type.');
+ }
+
  //Verify ISBN-10 scheme
  $isbn = $this->hyphens->removeHyphens($isbn);
  if (strlen($isbn) != 10) {
  return false;
  }
- if (preg_match("/\d{9}[0-9xX]/i",$isbn) == false) {
+ if (preg_match('/\d{9}[0-9xX]/i',$isbn) == false) {
  return false;
  }
 
  //Verify checksum
  $check = 0;
  for ($i = 0; $i < 10; $i++) {
- if ($isbn[$i] === "X") {
+ if ($isbn[$i] === 'X') {
  $check += 10 * intval(10 - $i);
  } else {
  $check += intval($isbn[$i]) * intval(10 - $i);
@@ -90,15 +95,20 @@ public function isbn10($isbn)
  *
  * @param string $isbn
  * @return boolean
+ * @throws Exception
  */
  public function isbn13($isbn)
  {
+ if(is_string($isbn) === false) {
+ throw new Exception('Invalid parameter type.');
+ }
+
  //Verify ISBN-13 scheme
  $isbn = $this->hyphens->removeHyphens($isbn);
  if (strlen($isbn) != 13) {
  return false;
  }
- if (preg_match("/\d{13}/i",$isbn) == false) {
+ if (preg_match('/\d{13}/i',$isbn) == false) {
  return false;
  }