[Klasa][PHP7] Prosta klasa do kilku operacji bitowych., "Dwuwartościowe przechowywanie" Opcje

[KsaR]

class.Bits.php

[PHP] pobierz, plaintext 
<?php
abstract class Bits
{
    protected $bits = 0;
 
    public function __construct($bits = NULL)
    {
        if ($bits !== NULL) $this->bits = $bits;
    }
 
    public function has(int $bit): bool
    {
       return ($this->bits & $bit) === $bit;
    }
 
    public function set(int $bit, bool $bool)
    {
       if ($bool)
          $this->bits |= $bit;
       else
          $this->bits &=~ $bit;
 
       return $this;
    }
 
    public function get(): int
    {
       return $this->bits;
    }
}
[PHP] pobierz, plaintext 

class.Flags.php

[PHP] pobierz, plaintext 
class Flags extends Bits
{
    const _BAN_ = 1;
    const _ONLINE_ = 2;
    const _GLOBALS_ = 4;
    const _ECHO_ = 8;
    const _SPY_ = 16;
}
[PHP] pobierz, plaintext 

Klasa jak widać prosta, zainspirowałem się do zrobienia jej po artykule nospor-a.
(Dwuwartościowe przechowywanie)

Oceniać można wszystko (nazewnictwo, co można by lepiej) itp.

Przykłady użycia:

Rejestracja Użytkownika

[PHP] pobierz, plaintext 
$flags = new Flags;
$flags->set(Flags::_GLOBALS_, true);
# ... Potem przy insercie do bazy użyjemy dla pola `options`
$flags->get();
[PHP] pobierz, plaintext 

Gdy już mamy zarejestrowanego a chcemy np. Zaktualizować status online/spy:

[PHP] pobierz, plaintext 
$flags = new Flags($user['options']);
$flags
  ->set(Flags::_ONLINE_, true)
  ->set(Flags::_SPY_, true);
# przy update do bazy....
$flags->get();
[PHP] pobierz, plaintext 

Aby sprawdzić czy uzytkownik jest zalogowany:

[PHP] pobierz, plaintext 
$flags = new Flags($user['options']);
#...
if ($flags->has(Flags::_ONLINE_))
{
   # online
}
else
{
   # offline
}
[PHP] pobierz, plaintext 

Oczywiście klasę Flags trzeba dopasować pod swoje wymagania.
@edit. Zmiana nazw klas na duże.

Ten post edytował KsaR 21.02.2016, 21:05:37

[com]

_BAN_ czemu taka konwencja?

I raczej już nazewnictwa plików tak klas się nie stosuje, a klasa z dużej litery.

[KsaR]

_BAN_ czemu taka konwencja?

I raczej już nazewnictwa plików tak klas się nie stosuje, a klasa z dużej litery.

Przy echo problem był (w edytorze lub w php już nie pamiętam) dlatego na szybko przedrostki dodałem na ewentualność) - co do nazw, dzięki, zaraz pozmieniam

[Pyton_000]

const: https://github.com/php-fig/fig-standards/bl...md#41-constants
- Klasę Bits dałbym abstract, bo nie ma sensu robienie z niej instancji
- if (NULL !== $bits) - Yoda Conditions? Nie używa się
- metoda isset zmienił bym na has

Warto dodać że min. wersja PHP 7.x

Poza tym ok, nie ma za bardzo więcej co oceniać bo i nie wiele jest kodu

[com]

problem z czym?

[KsaR]

const: https://github.com/php-fig/fig-standards/bl...md#41-constants
- Klasę Bits dałbym abstract, bo nie ma sensu robienie z niej instancji
- if (NULL !== $bits) - Yoda Conditions? Nie używa się
- metoda isset zmienił bym na has

Warto dodać że min. wersja PHP 7.x

Poza tym ok, nie ma za bardzo więcej co oceniać bo i nie wiele jest kodu

Co do constant - nie rozumiem, co trzeba zmienić?

Bits było na początku abstract ale na potrzeby forum skasowałem bo myślałem że źle ;p

Yoda Conditions - ... Mikrooptymalizacja, zawsze tak robię, problemów z tym nigdy nie miałem..
Jak doczytałem problemy mogą byc tylko w np. ''=$x; jednak tak nigdy nie robiłem.
Ps. PHP pierw sprawdza typ dlatego pierw typ potem ===.
Nie wiem dlaczego to złe w moim przypadku, jakieś wyjaśnienie?

Co do php7, jest w tytule [PHP7] od samego początku
A klasa mała bo tylko do "wartości dwustanowych"

Zaraz dodam abstract i zmienie isset na has, dzięki
--
@com, chyba składnię mi źle podswietlało.
A innej nazwy odchodzącej od echo nie mogłem zastosować bo tak mam w bazie danych,
I w moim czacie (na gg) oznacza to czy ma odsylac wlasna wiadomosc wyslana na czacie czy nie

Ten post edytował KsaR 21.02.2016, 21:04:16

[com]

no formę jego zapisu trzeba zmienić.

Mikrooptymalizacja

Faktycznie to dużo zmienia

[KsaR]

no formę jego zapisu trzeba zmienić.


Faktycznie to dużo zmienia

Ok, zaraz poprawie mi się zawsze w oczy poprostu rzuca jak jest na odwrót
WordPress też tak ma;;;

[com]

no to czas na zmiany, nie stosuje się nazw zastrzeżonych, ECHO nie zdziała bo to to samo co echo

Nie ucz sie złych praktyk

bo Yoda było kiedyś popularne bo ludzie nie rozróżniali = i ==

[KsaR]

no to czas na zmiany, nie stosuje się nazw zastrzeżonych, ECHO nie zdziała bo to to samo co echo

Nie ucz sie złych praktyk

A mógłbyś jeszcze objaśnić czemu to złe? Bo nadal nic nie wiem, a do-googlować nic nie mogę.

Yoda Conditions #

[PHP] pobierz, plaintext 
if ( true == $the_force ) {
    $victorious = you_will( $be );
}
[PHP] pobierz, plaintext 

When doing logical comparisons, always put the variable on the right side, constants or literals on the left.

In the above example, if you omit an equals sign (admit it, it happens even to the most seasoned of us), you?ll get a parse error, because you can?t assign to a constant like true. If the statement were the other way around ( $the_force = true ), the assignment would be perfectly valid, returning 1, causing the if statement to evaluate to true, and you could be chasing that bug for a while.

A little bizarre, it is, to read. Get used to it, you will.

This applies to ==, !=, ===, and !==. Yoda conditions for <, >, <= or >= are significantly more difficult to read and are best avoided.

Edit. No to nie rozróżniali, a coś wiekszego przeciw temu? Tu z wordpressa jak widac tylko do operatorów wiekszosci/mniejszosci się doczepili

Ten post edytował KsaR 21.02.2016, 21:14:39

[com]

https://www.ibuildings.nl/blog/2016/01/prog...cing-complexity

Napisałem Ci poco się Yoda stosuje, to jest ominiecie problemów słabych programistów którzy popełniają trywialne błędy, zaciemniając czytelność kodu

A wp to nie jest dobrym przykładem niczego

A tak wgl to nie do tego był mój komentarz a do nazw stałych w tym co zacytowałeś

[KsaR]

https://www.ibuildings.nl/blog/2016/01/prog...cing-complexity

Napisałem Ci poco się Yoda stosuje, to jest ominiecie problemów słabych programistów którzy popełniają trywialne błędy, zaciemniając czytelność kodu

A wp to nie jest dobrym przykładem niczego

A tak wgl to nie do tego był mój komentarz a do nazw stałych w tym co zacytowałeś

No juz poprawione, ja nigdy nie mialem problemow z = ale znam osoby co zaczynaja z php (aktualnie) i mają ja tak jak pisalem mikro/makro-optymalizacja... Dla mnie tak czytelniej
Gdzieś o tym było na potwierdzenie ale nie chce mi się googlowac - ale bylo ze php pierw sprawdza typ zmiennej (tak jak w ===) i jesli typ jest pierwszy to by jeszcze lepiej, ale skoro zle to zle, więc nie bede uzywal juz

Co do komentarza to dodalem w odpowiedzi na "problem z czym?"

Jednak oznaczylem "@com" - i tak 2 postow sie nie da pod rzad bo doda do 1

[com]

to złego, że trzeba się zastanawiać nad takim kodem, czytamy od lewej do prawej a ludzie, którzy pracują z tym strasznym kodem wp, często popełniali błędy bo go się źle czyta, no to ktoś wpadł na pomysł użyjmy yoda to nam się kody wysypie to poprawimy błąd

[KsaR]

To cza by więcej Arabów zatrudnić, hehe

[com]

Tobie może się tak lepiej czytać ale ktoś kto odziedziczy kod może już nie być z tego zadowolony, na tym poziomie ta optymalizacja nie ma znaczenia a typ sprawdzany jest tylko przy ===, po to się stosuje 3 a nie 2 =

Ten post edytował com 21.02.2016, 21:39:41

[lukaskolista]

Co do yoda conditions - jest ok, używa się, przykład z najnowszego symfony:
https://github.com/symfony/symfony/blob/mas.../Constraint.php

[Pyton_000]

@lukaskolista zobacz z którego roku są te commity 2014, 2010

[com]

lukaskolista

Yoda pojawia się w rożnych miejscach, ale nie znaczy, że to jest zalecana praktyka.

https://github.com/zendframework/zend-diact...tTrait.php#L110 tu tez ja znajdziesz, ale to jest kwestia przyzwyczajenia konkretnego programisty który to stworzył, nie dobrych praktyk

[lukaskolista]

https://github.com/symfony/symfony/blob/mas...Yaml/Inline.php - commit sprzed 5 godzin.

Dodatkowo w żadnym PSR nic o tym nie ma, jakoś też w google nic nie mogę znaleźć na temat tego, żeby była to dobra czy zła praktyka.

[com]

co z tego ze commit 5 h temu, kod się utrzymuje latami przecież, dałem linka do strony gdzie jest napisane na bazie książek uncle boba