Интеграция 2 API в 1 php класс (новичок ООП) - PullRequest
Новая
       10

Интеграция 2 API в 1 php класс (новичок ООП)

0 голосов
/ 18 февраля 2011

Я хочу интегрировать Google Analytics и Marchex (систему отслеживания вызовов) API Call Analytics для получения пользовательских отчетов с коэффициентами конверсии и тому подобными вещами.Я очень, очень новичок в мире объектно-ориентированного программирования, и думаю, что мне нужно какое-то руководство, чтобы начать это.Я хотел бы немного критиковать конструктор, прежде чем двигаться дальше.Я делаю это правильно или как? 

<?php
require_once("gapi.class.php");
require_once("xmlrpc.inc");
class garchex
{
    private $marchex_credentials = array(),
            $ga_credentials = array();
    private $marchex_account, $ga_account;
    public function __construct($marchex_credentials,$ga_credentials) {
        assert(is_array($marchex_credentials));
        assert(is_array($ga_credentials));

        //setup marchex stuff
        $this->marchex_credentials = $marchex_credentials;
        $this->marchex_account = new xmlrpc_client("/api/xmlrpc/1", "api.voicestar.com");
        $this->marchex_account->SetCredentials($marchex_credentials[0],$marchex_credentials[1]);

        //google analytics stuff
        $this->ga_credentials = $ga_credentials;
        $this->ga_account = new gapi($ga_credentials[0],$ga_credentials[1]);
    } // __construct
} // class garchex
?>

1 Ответ

1 голос
/ 18 февраля 2011

Вы присвоили некоторую нестатическую переменную, используя синтаксис -> $, что было не в порядке. Вот фиксированное предложение: 

<?php

// Better use require_once, in case someone loads the file again
require_once( "gapi.class.php" );
require_once( "xmlrpc.inc" );

class garchex
{
    private 
       $marchex_credentials = array(),
       $ga_credentials      = array()
       ;

    public function __construct( $marchex_credentials, $ga_credentials ) {

        // check, if values are fine
        assert( is_array( $marchex_credentials ) );
        assert( is_array( $ga_credentials ) );

        //setup marchex stuff
        $this->marchex_credentials = $marchex_credentials;

        $marchex_account = new xmlrpc_client("/api/xmlrpc/1", "api.voicestar.com");
        $marchex_account->SetCredentials($marchex_credentials[0],$marchex_credentials[1]);

         // google analytics stuff
         // No $ to access non-static ivars
         $this->ga_credentials = $ga_credentials;

        // black hole: local variable assigned.
        // You won't be able to access this from outside this method.
        $ga_account = new gapi($ga_credentials[0],$ga_credentials[1]);

    } // __construct
} // class garchex

// better drop the closing PHP tag
...