dataslang / dataslang-php

DataslangApi.php

<?php

use GuzzleHttp\Client;

class DataslangApi {

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

	
	const BASE_URI = 'http://api.dataslang.com';
	
	private $client = null;
	private $settings = array();
	private $timeout = 2.0;
  
	public function __construct(){
		$this->initSettings();
		$this->client = new Client($this->settings);
	}

	public function validate($xml, $xsd){

The parameter $xml is not used and could be removed.

The parameter $xsd is not used and could be removed.

		$result = null;
		
		try {
		

			//     	$response = $this->client->request('POST', '/validate', [

56% of this comment could be valid code. Did you maybe forget this after debugging?

			//         	'form_params' => [

50% of this comment could be valid code. Did you maybe forget this after debugging?

			//             	'field_name' => 'abc',

58% of this comment could be valid code. Did you maybe forget this after debugging?

			//             	'other_field' => '123',

58% of this comment could be valid code. Did you maybe forget this after debugging?

			//             	'nested_field' => [

50% of this comment could be valid code. Did you maybe forget this after debugging?

			//                 	'nested' => 'hello'

50% of this comment could be valid code. Did you maybe forget this after debugging?

			//             	]
			//         	]
			//     	]);
			
			$response = $this->client->request('POST', '/validate', [

$response is not used, you could remove the assignment.

				'form_params' => [
					'parametro' => 'prova'
				]
			]);
			
			$result = $res->getBody();

The variable $res does not exist. Did you forget to declare it?

			
		} catch (\Exception $e) {

Consider adding a comment why this CATCH block is empty.

			
		}
		
		return $result;
  	}
  
  	private function initSettings(){
  		$this->settings['base_uri'] = $this->getBaseUri();
  		$this->settings['timeout'] = $this->getTimeout();
  	}
  
	public function setTimeout($timeout){
		$this->timeout = $timeout;
	}
	
	public function getTimeout(){
		return $this->timeout;
	}
	
	public function getBaseUri(){
		return self::BASE_URI;
	}

}