PHP - Loop Class Objects

I'm looking for a more efficient way of declaring variables within a class. What I normally use is a class which is actually a database object. Every time I make a new table in my database I make a new class and new attributes to match the fields in the db table. Looks something like this:

    Code: [Select]
protected static $table_name="myTable";
protected static $db_fields=array('id', 'name', 'address', 'tel_number', 'fax_number');

public $name;
public $address;
public $tel_number;
public $fax_number;

//etc. etc.
At the moment I am manually adding the attribute, everytime I add a field to the db table. I'm guessing there should be a way to loop through the $db_fields array and add an attribute in the loop? Can anyone help me with the syntax for this. Is there also a way that I don't have to list the fields everytime in $db_fields....can $db_fields not just look at the relevant db table and pull out the field names in some kind of clever method?

Hello all!   I have this array of objects:

Array
(
    [0] => stdClass Object
        (
            [first_name] => test
            [last_name] => test
            [title] => test
            [id] => 34
            [type] => 4
            [manager] => 4
            [email] => p@yahoo.com
            [date] => 2014-09-21 07:23:12
            [status] => 2
            [approval_date] => 2014-09-21 07:31:10
            [assessment_id] => 1
            [supervisor_approval] => 1
            [manager_approval_date] => 2014-09-21 07:31:27
            [mainmanger] => 3
        )

    [1] => stdClass Object
        (
            [first_name] => test
            [last_name] => test
            [title] => test
            [id] => 34
            [type] => 4
            [manager] => 4
            [email] => p@yahoo.com
            [date] => 2014-09-20 07:39:55
            [status] => 2
            [approval_date] => 2014-09-20 07:40:41
            [assessment_id] => 3
            [supervisor_approval] => 1
            [manager_approval_date] => 2014-09-20 07:41:07
            [mainmanger] => 3
        )

    [2] => stdClass Object
        (
            [first_name] => jimmy john john
            [last_name] => john
            [title] => Lorad and Master
            [id] => 32
            [type] => 4
            [manager] => 3
            [email] => j@j.com
            [date] => 2014-09-21 07:38:50
            [status] => 2
            [approval_date] => 2014-09-19 07:39:25
            [assessment_id] => 2
            [supervisor_approval] => 1
            [manager_approval_date] => 2014-09-19 07:39:25
            [mainmanger] => 0
        )

)

Where "id" = the user ID.    If there are two or more entries for a user, I want to remove all but the most recent by the "approval_date",... So for the above example it would return:

Array
(
    [0] => stdClass Object
        (
            [first_name] => test
            [last_name] => test
            [title] => test
            [id] => 34
            [type] => 4
            [manager] => 4
            [email] => piznac@yahoo.com
            [date] => 2014-09-21 07:23:12
            [status] => 2
            [approval_date] => 2014-09-21 07:31:10
            [assessment_id] => 1
            [supervisor_approval] => 1
            [manager_approval_date] => 2014-09-21 07:31:27
            [mainmanger] => 3
        )

    [1] => stdClass Object
        (
            [first_name] => jimmy john john
            [last_name] => john
            [title] => Lorad and Master
            [id] => 32
            [type] => 4
            [manager] => 3
            [email] => j@j.com
            [date] => 2014-09-21 07:38:50
            [status] => 2
            [approval_date] => 2014-09-19 07:39:25
            [assessment_id] => 2
            [supervisor_approval] => 1
            [manager_approval_date] => 2014-09-19 07:39:25
            [mainmanger] => 0
        )

)

I'm having a hard time wrapping my head around this concept, so I don't have any example code. And I don't really need or expect someone to code this for me,. just a push in the right direction would be awesome.

I have the following three tables (and also delta_point and hist_point which are purposely left out and just mentioned for context).  Before going down the Doctrine ORM long curvy road, aggrType was just used to restrict possible values so that I didn't need to use MySQL's ENUM, but now I am thinking there may be value to making it an entity and include some methods (and a second reason because I still can't figure out how to make Doctrine create a FK constraint yet not populate the parent entity with a AggrType object).

point
-id (int PK)
-data
-dType /* descriminator.  Options are AggregatePoint, DeltaPoint, and HistoricPoint */

aggr_point
-id (int PK, FK to point)
-aggrType (char FK to aggrType)
-data

aggrType
-type (char PK) /* there are about five types */

I have a service to create a new point based on the provided desired type and user data.  I don't like the switch statement to determine which type of point to create, but don't know a better way.  Also my entity is responsible to provide the appropriate validation rules for the given point type which might not be considered proper but it seems to work for me.  Any constructive criticism is welcomed.

<?php
namespace NotionCommotion\PointMapper\Api\Point;
use NotionCommotion\PointMapper\Domain\Entity\Point;
class PointService
{
    public function create(string $type, array $params):integer {
        $class=[
            'aggr'=>'AggregatePoint',
            'delta'=>'DeltaPoint',
            'hist'=>'HistoricPoint',
        ][$type];
        //$class='Point\\'.$class;  //This doesn't seem to be possible and I needed to use the fully qualifed name without the short cut used namespace.
        $class='\\Greenbean\Datalogger\Domain\Entity\Point\\'.$class;
        $point=new $class();
        $validator=$this->getValidator($point);
        $params=$validator->sanitize($params);
        $validator->validate($params);
        $point->setAccount($this->account);
        $point->populate($params, $this->em); //Will be addressed later
        $this->em->persist($point);
        $this->em->flush();
        $point->setIdPublic($this->em->getRepository(Point\Point::class)->getPublicId($point->getId()));
        return $point->getIdPublic();
    }
}

Now, this is the part I am struggling with the most.  I only have one service to create any type of point, so I do wish to put a bunch of if/thens in it and instead move logic to the individual point type being created.  The AggrPoint in question requires a sub-point and I don't think it is appropriate to inject the entity manager into a point entity and as an alternative solution find an existing point which is associated with the account which resides in the new AggrPoint.  Maybe not appropriate, but it works.  This type of point also has a sub-object AggrType as discussed at the very beginning of this post, so I first attempt to create a new one from scratch, but that doesn't work and I need to set $aggrType with an existing one.  As an alternative, I passed the entity the entity manager, but as stated I think this is a bad idea.

Any recommendations where to go from here?  Thank you

namespace NotionCommotion\PointMapper\Domain\Entity\Point;
class AggregatePoint extends Point
{
    public function populate(array $params, \Doctrine\ORM\EntityManager $em):\NotionCommotion\PointMapper\Domain\Entity\Entity{
        $criteria = \Doctrine\Common\Collections\Criteria::create()
        ->where(\Doctrine\Common\Collections\Criteria::expr()->eq("id_public", $params['pointId']));
        if(!$subpoint = $this->getAccount()->getPoints()->matching($criteria)->current()){
            throw new \Exception("Invalid subpoint ID $params[pointId]");
        }
        $this->setSubpoint($subpoint);

        //This won't work since I must use an existing AggregateType and not create a new one
        $aggrType=new AggregateType();
        $aggrType->setType($params['aggrType']);

        //This just seems wrong and I shouldn't be passing the entity manager to an entity
        $aggrType=$em->getRepository(AggregateType::class)->findOneBy(['type'=>$params['aggrType']]);

        $this->setAggregateType($aggrType);

        unset($params['pointId'], $params['aggrType']);
        // parent::populate($params) not shown, but it basically calles appropriate setters based on the property name
        return parent::populate($params);
    }

    public function getValidatorRules():array
    {
        return $this->filesToArr(['/point/base.json', '/point/aggr.json']);
    }

}

 

I have mysqli object in Database class base:

[color=]database class:[/color]

class Database {
  private $dbLink = null;
  
  public function __construct() {
    if (is_null($this->dbLink)) {
      // load db information  to connect
      $init_array = parse_ini_file("../init.ini.inc", true);
      $this->dbLink = new mysqli($init_array['database']['host'],
                           $init_array['database']['usr'],
                           $init_array['database']['pwd'],
                           $init_array['database']['db']);
  
      if (mysqli_connect_errno()) {
        $this->dbLink = null;
      }
    }
  }

  public function __destruct() {
    $this->dbLink->close();
  }
}


Class derived is Articles where I use object dBLink in base (or parent) class and I can't access to mysqli methods (dbLink member of base class):

Articles class:

require_once ('./includes/db.inc');

class Articles extends Database{
  private $id,
              ....
              ....
             $visible = null;
  
  public function __construct() {
    // Set date as 2009-07-08 07:35:00
    $this->lastUpdDate = date('Y-m-d H:i:s');
    $this->creationDate = date('Y-m-d H:i:s');
  }
  
  // Setter
  ....
  ....

  // Getter
  ....
  ....

  public function getArticlesByPosition($numArticles) {
    if ($result = $this->dbLink->query('SELECT * FROM articles ORDER BY position LIMIT '.$numArticles)) {
      $i = 0;
      while ($ret = $result->fetch_array(MYSQLI_ASSOC)) {
        $arts[$i] = $ret;
      }
      $result->close();
      return $arts;
    }
  }
}


In my front page php I use article class:


include_once('./includes/articles.inc');
$articlesObj = new articles();
$articles = $articlesObj->getArticlesByPosition(1);
var_dump($articles);


[color=]Error that go out is follow[/color]

Notice: Undefined property: Articles::$dbLink in articles.inc  on line 89

Fatal error: Call to a member function query() on a non-object in articles.inc on line 89

If I remove constructor on derived class Articles result don't change 
Please help me

If a class has a constructor but also has a static method, if I call the static method does the constructor run so that I can use an output from the constructor in my static method?

--Kenoli

Hi

Can you call Class A's methods or properties from Class B's methods?

Thanks.

Ok. I know you can pass the object of a class as an argument. Example:


class A {
    function test() {
        echo "This is TEST from class A";
    }
}


class B {
    function __construct( $obj ) {
        $this->a = $obj;
    }

    function test() {
       $this->a->test();
    }

}


Then you could do:

$a = new A();
$b = new B($a);


Ok so that's one way i know of.

I also thought that you could make a method static, and do this: (assuming class A's test is 'static')

class B {
    function test() {
       A::test(); 
   }
}


But that is not working.


I'd like to know all possible ways of accomplishing this. Any hints are appreciated. thanks

I have an existing instance of my class Database, now I want to call that instance in my Session class, how would I go about doing this?

Hey.   So the issue I'm having is consecutive loops on semi-large arrays, over and over. Consider this array:

$firstArray = array(
'row1' => array(
'dates' => array(
'2014-01-01' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-01-02' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-01-03' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-01-04' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-01-05' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-01-06' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-01-07' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
)
),
'row2' => array(
'dates' => array(
'2014-02-01' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-02-02' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-02-03' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-02-04' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-02-05' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-02-06' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-02-07' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-02-08' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
'2014-02-09' => array('key1' => 'value1', 'key2' => 'value2', 'key3' => 'value3', 'key4' => 'value4', 'key5' => 'value5', 'key6' => 'value6', 'key7' => 'value7', 'key8' => 'value8', 'key9' => 'value9', 'key10' => 'value10'),
)
)
);

Originally the data comes from ~2-3 database tables, of course. But to ilustrate the point, this is how the main array looks like. This array usually contains anywhere between 10-50 rows, each row containing at least 10 dates, with 10 key/values each.   And after setting up all the data, it needs to be processed. Currently this is how a friend of mine did it..

$placeDataHere = array();
foreach($firstArray as $key => $dates) {
foreach($dates as $date => $values) {
foreach($values as $key => $value) {
$placeDataHere['DV_' . $date]['SM_' . $key] = 'KS_' . $value;
//Followed by another ~50-70 lines of processing the 3 loop's data..
...
...
....
....
....
....
....
....
}
}
}

Obviously this isn't good practise, but we can't seem to figure out a better way of doing it, since both the data and the loops are horribly nested.   This loop and setup of $firstArray is run anywhere between 10-20 times/request, due to amount of users we wish to process. So, the result is that this code can take up to over 2-3 minutes to complete, which isn't really optimal performance.   In short my question is, are there any better methods of handling this with the data setup we currently have?

Hi,

I need to be able to call a class based on variables.  E.G.

I would normally do:
Code: [Select]
$action = new pattern1()
but i would like to be able to do it dynamicaly:
Code: [Select]
$patNum = 1;
$action = new pattern.$patNum.()
Im wondering if that's possible?  If so what would the correct syntax be?

Many Thanks.

Hi people!

class FirstOne{

public function FunctionOne($FirstInput){
//do stuff and output value
return $value1;
}

}


Then:-


class SecondOne{

public function FunctionTwo($AnotherInput){
//do stuff and output value
return $value2;
}

}


What I want to know is this, if I want to use FunctionOne() in Class SecondOne do I do it like this:-

(Assume as I have instantiated the first class using $Test = new FirstOne(); )


class SecondOne{

function SecondedFunction(){
global $Test;
return $Test->FunctionOne();
}

public function FunctionTwo($AnotherInput){
//do stuff and output value
return $value2;
}

public function FunctionThree(){

//some code here
$this->Test->SecondedFunction();<--I think as I can omit the $this-> reference
}
}


My point is: Do I have to do it this way or is there way of having this done through __construct() that would negate the need for a third party function?

I have a version working, I just think that it is a little convoluted in the way as I have done it, so I thought I would ask you guys.

Any help/advice is appreciated.

Cheers
Rw

I have two classes:

## Admin.php

<?php
class Admin
{
public function __construct()
{
include("Config.php");
}

/**
* deletes a client
* @returns true or false
*/
function deleteClient($id)
{
return mysql_query("DELETE FROM usernames WHERE id = '$id'");
}
}
?>


## Projects.php

<?php
class Projects
{
public function __construct()
{
include("Config.php");
$this->admin = $admin;
$this->dataFolder = $dataFolder;
}

/**
* Deletes a project
* @returns true or false
*/
function deleteProject($id)
{
$root  = $_SERVER['DOCUMENT_ROOT'];
$theDir  = $root . $this->dataFolder;
$sql  = mysql_query("SELECT * FROM projectData WHERE proj_id = '$id'");
while ($row = mysql_fetch_array($sql))
{
$mainFile = $row['path'];
$thumb = $row['thumbnail'];
if ($thumb != 'null')
{
unlink($theDir . "/" . substr($thumb,13));
}
unlink($theDir . "/" . substr($mainFile,13));
}
$delete = mysql_query("DELETE FROM projectData WHERE proj_id = '$id'");
$getDir = mysql_query("SELECT proj_path FROM projects WHERE id = '$id'");
$res = mysql_fetch_array($getDir);
rmdir($theDir . "/" . $res['proj_path']);
return mysql_query("DELETE FROM projects WHERE id = '$id'");
}
}
?>


How can I call deleteProject() from within Admin.php?

Below is my output on the browser:

    Student: Kevin Smith (u0867587)
    Course: INFO101 - Bsc Information Communication Technology Course Mark 70 Grade
    Year: 3
   
   
    Module: CHI2550 - Modern Database Applications Module Mark: 41 Mark Percentage: 68 Grade: B
   
    Session: AAB Session Mark: 72 Session Weight Contribution 20%
   
    Session: AAE Session Mark: 67 Session Weight Contribution 40%


    Module: CHI2513 - Systems Strategy Module Mark: 31 Mark Percentage: 62 Grade: B
   
    Session: AAD Session Mark: 61 Session Weight Contribution 50%

Now where it says course mark above it says 70. This is incorrect as it should be 65 (The average between the module marks percentage should be 65 in the example above) but for some stange reason I can get the answer 65. I have a variable called $courseMark and that does the calculation. Now if the $courseMark is echo outside the where loop, then it will equal 65 but if it is put in while loop where I want the variable to be displayed, then it adds up to 70. Why does it do this.
   
    Below is the code:

Code: [Select]
    $sessionMark = 0;
    $sessionWeight = 0;
    $courseMark = 0;
   
    $output = "";
   
    $studentId = false;
    $courseId  = false;
    $moduleId  = false;
   
    while ($row = mysql_fetch_array($result))
    {
   
   
    $sessionMark += round($row['Mark'] / 100 * $row['SessionWeight']);
    $sessionWeight  += ($row['SessionWeight']);
        $courseMark = ($sessionMark /  $sessionWeight * 100);
   
          if($studentId != $row['StudentUsername'])
        {
   
            //Student has changed
            $studentId = $row['StudentUsername'];
   
            $output .= "<p><strong>Student:</strong> {$row['StudentForename']} {$row['StudentSurname']} ({$row['StudentUsername']})\n";
   
        }
             
        if($courseId != $row['CourseId'])
        {
   
            //Course has changed
            $courseId = $row['CourseId'];
   
            $output .= "<br><strong>Course:</strong> {$row['CourseId']} - {$row['CourseName']} <strong>Course Mark</strong>" round($courseMark) "<strong>Grade</strong>  <br><strong>Year:</strong> {$row['Year']}</p>\n";
   
        }
   
        if($moduleId != $row['ModuleId'])
        {
   
            //Module has changed
            if(isset($sessionsAry)) //Don't run function for first record
            {
   
                //Get output for last module and sessions
                $output .= outputModule($moduleId, $moduleName, $sessionsAry);
            }
   
            //Reset sessions data array and Set values for new module
   
            $sessionsAry = array();
            $moduleId    = $row['ModuleId'];
            $moduleName  = $row['ModuleName'];
        }
             
        //Add session data to array for current module
             
        $sessionsAry[] = array('SessionId'=>$row['SessionId'], 'Mark'=>$row['Mark'], 'SessionWeight'=>$row['SessionWeight']);
             
    }    //Get output for last module
   
    $output .= outputModule($moduleId, $moduleName, $sessionsAry);
   
    //Display the output
    echo $output;
I think the problem is that it is outputting the answer of the calculation only for the first session mark. How in the while loop can I do it so it doesn't display it for the first mark only but for all the session marks so that it ends up showing the correct answer 65 and not 72?

Hi I am new to PHP and I need to create an array and add both books to the array and than loop through the array this is where I am having trouble if the price of a book is less than $20.00 output the book’s name and price if the book’s title comes after the letter R in the alphabet output the book’s title.

I am getting these error:

Notice: Trying to get property 'price' of non-object in C:\xampp\htdocs\PHP objects\books.php on line 45

Notice: Trying to get property 'title' of non-object in C:\xampp\htdocs\PHP objects\books.php on line 49

Notice: Trying to get property 'price' of non-object in C:\xampp\htdocs\PHP objects\books.php on line 50

Fatal error: Uncaught Error: Call to a member function get_first_char() on string in C:\xampp\htdocs\PHP objects\books.php:52 Stack trace: #0 {main} thrown in C:\xampp\htdocs\PHP objects\books.php on line 52

My code is:

<?php
class Book {
  // Properties
  public $price;
  public $title;
  public function get_first_char() {
    $titleArray = str_split($this->title);
    return reset($titleArray);
}

  // Methods
  function set_price($price) {
    $this->price = $price;
  }
  function get_price() {
    return $this->price;
  }
  function set_title($title) {
    $this->title = $title;
  }
  function get_title() {
    return $this->title;
  }
  
}

$first_book = new Book();
$first_book->set_price('$13.95');
$first_book->set_title('Moby Dick');
echo "Price: " . $first_book->get_price();
echo "<br>";
echo "Title: " .  $first_book->get_title();
echo "<br>";
$second_book = new Book();
$second_book->set_price('$23.99');
$second_book->set_title('Wuthering Heights');
echo "Price: " . $second_book->get_price();
echo "<br>";
echo "Title: " .  $second_book->get_title();
echo "<br>";
$book = array("Moby Dick", "Wuthering Heights");
echo "Book Titles: " . $book[0] . ", "  . " and " . $book[1] . ".";

foreach ($book as $bookItem) {
    $price = $bookItem->price;
    // Right now $price is a string ('$xx.xx'), we need it to be numeric
    $price = (float) substr($price, 1);
    if ($price < 20) {
        echo $bookItem->title;
        echo $bookItem->price;
    }
    if (ord(strtoupper($bookItem->get_first_char())) > ord('R')) {
        echo $bookItem->title;
    }
}
 ?>

Any help will be very much appreciated

Thank you

I took an object, and saved it into a database serialized.

Code: [Select]
$object_here = serialize($theobject);
Then later on, I go and I get that data from a database, and try to unserialize it..

Code: [Select]
$objecthere = unserialize($row->theobject)

And it throws the following error...any advice?
Quote

Warning: unserialize() [function.unserialize]: Node no longer exists in /path/to/file on line 20