По фэншую ли синтаксис?

Question

[0ldn0mad]

Добрый день.
Изучаю ООП (Интерфейсы, Классы). Попутно решаю задачки.
В задаче есть условие - собрать объекты классов в массив.
Для компактного, а тем более для читабельного вида написал код таким способом:

$arr = [];

$arr[] = $objQuadrate = new Quadrate(6);
$arr[] = $objCube = new Cube(5);
$arr[] = $objRectangle = new Rectangle (2);
$arr[] = $objQuadrate = new Quadrate(3);
$arr[] = $objRectangle = new Rectangle (4);
$arr[] = $objCube = new Cube(10);
$arr[] = $objRectangle = new Rectangle (6);
$arr[] = $objCube = new Cube(15);
$arr[] = $objQuadrate = new Quadrate(9);

Так писать по фэншую код или есть запись объектов в массив ровнее?

Comments

[0ldn0mad]

Евгений Ромашкан, Я понимаю что стандартов нет, я спрашиваю у профи читабельно ли так писать? Меня смущает два знака равно в одной строке....

[Konata Izumi]

0ldn0mad, сами объекты помимо записи в массив как-то еще используются?

[0ldn0mad]

Да, далее из массива объекты проверяются на принадлежность интерфейсам и в зависимости от интерфейса выполняется то или иное действие (вызывается необходимый метод).

Answer 1

[Евгений Вольф]

Так писать по фэншую код или есть запись объектов в массив ровнее?

У Вас тут не столько запись чего-то в массив, сколько предварительное присвоение значения переменной, с уже последующем добавлением оной в массив. Оно работать конечно будет, но выглядит несколько странно, на мой взгляд. Подобные принципы обычно используются в условных операторах или циклах (if/while), для последующей обработки переменной (результата).

Если Вам нужно просто собрать массив, обычно пишут так:

$arr = [
    new Quadrate(6),
    new Cube(5),
    new Rectangle (2),
    ... ,
];

P.S. Так же этот код можно написать в строчку, если он не выходит за границу (80 или 120 символов, в IDE и редакторах эта линия почти всегда есть по умолчанию) или если просто если Вам очень хочется написать этот код в одну строку.

Comments

[0ldn0mad]

Во! Спасибо большое за такой комментарий!
Мне тоже как-то режет глаза два знака равно, а Ваше предложение очень даже правильное! Обязательно возьму на заметку!

[Евгений Вольф]

0ldn0mad, не за что, рад был Вам чем-то помочь :)

[Вася]

Евгений Вольф, проверять принадлежность к интерфейсами не по фен-шую будет при вызовах $arr[2] instanceof IFigure

[0ldn0mad]

Вася, спасибо за комментарий. Я вижу вы понимаете откуда ноги растут у задачи. Чтобы было понятно до конца в каком виде лучше формировать массив, я расскажу всю задачу от начала и до конца.
В начале нам дается два интерфейса - для трехмерных фигур и двумерных с методами в них для реализации оных в классах:

<?php

interface iFigure3D
{
	public function getVolume();
	public function getSurfaceSquare();
}

<?php

interface iFigure
{
	public function getPerimeter();
	public function getSquare();
}

Далее реализуем методы в классах:
Для КУБА

<?php

class Cube implements iFigure3D
{
	private $a;

	public function __construct($a)
	{
		$this->a = $a;
	}

	public function getVolume()
	{
		// TODO: Implement getVolume() method.
		return pow($this->a,3);
	}

	public function getSurfaceSquare()
	{
		// TODO: Implement getSurfaceSquare() method.
		return 6 * pow($this->a, 2);
	}
}

Для КВАДРАТА

<?php

class Quadrate implements iFigure
{
	private $a;

	public function __construct($a)
	{
		$this->a = $a;
	}

	public function getSquare()
	{
		return pow($this->a,2);
	}

	public function getPerimeter()
	{
		return 4 * $this->a;
	}
}

И ПРЯМОУГОЛЬНИКА
P.S.Так как в условии присутствует только одна сторона A, то мы считаем что прямоугольник = квадрат и наследуем все методы у квадрата

<?php

class Rectangle extends Quadrate
{

}

Далее собираем массив фигур в случайном порядке и далее, пропустив через цикл массив вывести информацию о принадлежности к тому или иному интерфейсу объекта.
Чтобы контролировать содержание массива, я включил дебаг для него.

<?php
error_reporting(E_ALL);
ini_set('display_errors', 'on');
setlocale(LC_ALL, 'ru_RU.UTF-8');
#-------------------------------------------------
require_once 'interface/iFigure3d.php';
require_once 'interface/iFigure.php';
require_once 'classes/Cube.php';
require_once 'classes/Quadrate.php';
require_once 'classes/Rectangle.php';

function debug($data){
	echo '<pre>' . print_r($data, 1) . '</pre>';
}

$arr = [
	new Quadrate(6),
	new Cube(5),
	new Rectangle (2),
	new Quadrate(3),
	new Rectangle (4),
	new Cube(10),
	new Rectangle (6),
	new Cube(15),
	new Quadrate(9),
];

debug($arr);

foreach($arr as $elem){
	if($elem instanceof iFigure){
		echo $elem->getSquare() . '<br>';
	}
}
echo "<hr color='3C3F41'  size=1px width=500px align=\"left\"/>";

foreach ($arr as $elem){
	if ($elem instanceof iFigure){
		echo "Это фигура 2D, ее площадь равна {$elem->getSquare()}" . '<br>';
	}

	if ($elem instanceof iFigure3D){
		echo "Это фигура 3D, площадь ее поверхности равна {$elem->getSurfaceSquare()}" . '<br>';
	}
}

В итоге, получается ожидаемый результат. Поэтому предложенный Евгением Вольфом вариант сбора массива очень даже симпатичный.

Вывод всего этого скрипта:

Array
(
[0] => Quadrate Object
(
[a:Quadrate:private] => 6
)

[1] => Cube Object
(
[a:Cube:private] => 5
)

[2] => Rectangle Object
(
[a:Quadrate:private] => 2
)

[3] => Quadrate Object
(
[a:Quadrate:private] => 3
)

[4] => Rectangle Object
(
[a:Quadrate:private] => 4
)

[5] => Cube Object
(
[a:Cube:private] => 10
)

[6] => Rectangle Object
(
[a:Quadrate:private] => 6
)

[7] => Cube Object
(
[a:Cube:private] => 15
)

[8] => Quadrate Object
(
[a:Quadrate:private] => 9
)

)

36
4
9
16
36
81
Это фигура 2D, ее площадь равна 36
Это фигура 3D, площадь ее поверхности равна 150
Это фигура 2D, ее площадь равна 4
Это фигура 2D, ее площадь равна 9
Это фигура 2D, ее площадь равна 16
Это фигура 3D, площадь ее поверхности равна 600
Это фигура 2D, ее площадь равна 36
Это фигура 3D, площадь ее поверхности равна 1350
Это фигура 2D, ее площадь равна 81

[Вася]

0ldn0mad, понятно, вы молодец

[Евгений Вольф]

Вася, спасибо на добром слове :)

Answer 2

[Gip]

не ну можно и по рефлексии циклом пройти:)

Comments

[Вася]

Я бы генератор добавил ещё