Oop

🤔 Totally legal kind of comments in the code

So far I can be sure that the only type of comments in code is legal, needed…

Model behaviour

Consider a method:

public function doSomethingWithOrder(Order $order): Order
{
    // ...
}

It takes an Order and returns an Order. In my opinion this method can be moved Order model itself.

Make code reusable

Given:

I’ve got an abstract class, with a needed method, which I want to reuse

Wanted:

Reuse the method in the rest of the system in most appropriate way

Solution:

Getter names for optional properties

Sometimes we have to deal with properties, which can be null

/**
 * @var string|null
 */
private $description;

It’s fine, IDE will generate a getter for us

public function…

Using patterns for reducing complexity

There is a common opinion that OOP patterns reduce complexity of code and project in general.

To me it was sort of obvious, before I spoke to a frontend guy, who hates OOP and does some functional programming. He said that he hates all…

Constants in unit tests

I like to use constant literals in tests instead of constants itself.

Abstraction and encapsulation

Encapsulation picks up where abstraction leaves off. Abstraction says, “You’re allowed to look at an object at a high level of detail.” Encapsulation says, “Furthermore, you aren’t allowed to look at an object at any other level of detail.”
 — Code Complete

Taking ownership

A lot of books/articles say about taking ownership.

The boy-scout rule says about leaving the place cleaner than it used to be.

But what is the scope of ownership? If I am changing a method in a class, am I responsible for all the class? How cam…

Exceptions

1. If it’s a command, do not return a result, just throw an exception in case of fail (also related to CQRS cause you can not say if command succeed by returning value)
2. An exception suppose to be custom and detailed, don’t use basic exceptions, it make it harder to debug