[PHP] 1 linijka zamiast 3 Opcje

[KR2615]

Hej
Zmieniłem bibliotekę do komunikacji mojego skryptu PHP z Mikrotikiem z https://github.com/EvilFreelancer/routeros-api-php na https://github.com/BenMenking/routeros-api
Wcześniej posługiwałem się takim kodem do odczytu:

[PHP] pobierz, plaintext 
$identity = $client->query('/system/identity/print')->read();
[PHP] pobierz, plaintext 

Teraz w przykładzie znalazłem:

[PHP] pobierz, plaintext 
$client -> write('/system/identity/print');
$read = $client -> read(false);
$identity = $client -> parseResponse($read);
[PHP] pobierz, plaintext 

Jak ten drugi kod przerobić żeby też mieścił się w jednej linijce?

[Salvation]

Nie przerobisz tego na jednolinijkowca, bo metoda `write` nie zwraca obieku, a jedynie typ prosty: bool.
Z resztą, ja osobiście nie widzę potrzeby. Tak jak jest, jest czytelniej, a dla interpretera i tak ta różnica jest niezauważalna.

Max co można zrobić, to tak:

[PHP] pobierz, plaintext 
$client->write('/system/identity/print');
 
$parsedResponse = $client->parseResponse($client->read(false));
[PHP] pobierz, plaintext 

Z kodu klasy wynika jeszcze, że możesz po prostu zrobić takie coś:

[PHP] pobierz, plaintext 
$client->write('/system/identity/print');
 
$parsedResponse = $client->read();
[PHP] pobierz, plaintext 

I też będzie działać jak powyżej.

Ten post edytował Salvation 18.10.2022, 09:56:16

[gino]

$client jest u Ciebie instancją klasy, która odwołuje się do różnych metod: $client->write($patch), $client->read itd. Po co chcesz to łączyć w jedna linijkę?
Nie da się podmienić obiektu $client na write($patch) bo to już nie będzie instancja i dostaniesz masę błędów.
Ewentualnie jak napisał @Salvation

Ten post edytował gino 18.10.2022, 09:52:43

[KR2615]

Z resztą, ja osobiście nie widzę potrzeby.

A ja mam to w 26 miejscach w kodzie i wolałbym poprawić to jednym regexpem zamiast ręcznie. Ale skoro nie ma możliwości to ok:)

[Salvation]

A ja mam to w 26 miejscach w kodzie (...)

To zamknij ten kod w metodę i utwórz Traita. Wtedy tylko raz zmienisz to w 26 miejsach, a później już tylko w jednym.

Ten post edytował Salvation 18.10.2022, 10:00:23

[nospor]

Jak ci ta jedna linijka jest potrzebna do regexpa to

$client->write('/system/identity/print'); $parsedResponse = $client->read();

I masz jedna linijke

[KR2615]

Tylko, że ja tak za bardzo nie umiem w programwanie obiektowe Czytałem poradniki, robiłem kursy ale nic. Może jakiś gotowiec na tym przykładzie by mnie oświecił?

Jak ci ta jedna linijka jest potrzebna do regexpa to

$client->write('/system/identity/print'); $parsedResponse = $client->read();

I masz jedna linijke

Zamieniłem ale wyszło coś takiego:

[PHP] pobierz, plaintext 
$users = $client->write('/user/print'); $parsedResponse = $client->read();
$dns = $client->write('/ip/dns/print'); $parsedResponse = $client->read();
$ntp_serv = $client->write('/system/ntp/client/print'); $parsedResponse = $client->read();
[PHP] pobierz, plaintext 

a chodziło by mi o to, żeby pod zmiennymi $users, $dns i $ntp_serv pojawiła się już sparsowana odpowiedź. Dlatego tak zależy mi na "jednej linijce".

Ten post edytował KR2615 18.10.2022, 10:15:36

[gino]

Najprościej dopisz w klasie RouterosAPI metodę np. clientConnect(string $patch) i wrzuć w nią kod, który masz w pierwszym poście, coś takiego:

Kod

public function connectClient(string $patch)
{
  $this->write($patch);
  return $this->parseResponse($this->read(false));
}

i będziesz miał jedną linijkę w stylu:

Kod

$identity = $client->connectClient('/system/identity/print');

zmieniając sobie ścieżkę jak Ci się podoba.

[KR2615]

Fajnie, dzięki, o to właśnie mi chodziło!

[Salvation]

Nie nalezy edytować plików źródłowych. Teraz wyobraź sobie, że robisz update biblioteki i wszystkie Twoje zmiany idą się paść na zieloną łączkę...
Dlatego napisałem, byś obudował bibliotekę własną abstrakcją.

Korzystasz z composera może?

[gino]

dlatego napisałem najprościej a nie najlepiej i to chyba jest logiczne. Ucięło mi też z jakiegoś powodu resztę posta, bo pisałem, żeby zrobił jak radzisz, albo napisał własne rozszerzenie klasy i podałem tam też przykład.


Ten post edytował gino 18.10.2022, 13:14:19

[KR2615]

Korzystasz z composera może?

W tym rozwiązaniu nie. Poprzednią bibliotekę  ładowałem composerem:

[PHP] pobierz, plaintext 
//include("vendor/autoload.php");
//use \RouterOS\Client;
//use \RouterOS\Query;
[PHP] pobierz, plaintext 

[Salvation]

To pamiętaj o zmianach jakie wykonałeś przy update'cie biblioteki.

[KR2615]

Nie dotykałem biblioteki bo już kiedyś brutalnie się przekonałem, co to znaczy updatować biblioteki, zmieniać strukturę danych itp na przykładzie LMSa (Lan Management System), że wyszła mi z niego kompletnie osobna gałąź i nie mogłem w łatwy sposób korzystać z feature'ów wprowadzanych przez Twórców. Wszystko zrobiłem w "mojej" części kodu.
Działa dokładnie tak, jakbym chciał.
Dzięki wszystkim za pomoc!

----------------


Jeszcze jedno pytanie w tym samym temacie. Dostaję od urządzenia taką odpowiedź:

[PHP] pobierz, plaintext 
array(2) {
  [0]=>
  array(8) {
    [".id"]=>
    string(2) "*1"
    ["address"]=>
    string(13) "10.9.4.109/24"
    ["network"]=>
    string(8) "10.9.4.0"
    ["interface"]=>
    string(7) "ether1"
    ["actual-interface"]=>
    string(7) "ether1"
    ["invalid"]=>
    string(5) "false"
    ["dynamic"]=>
    string(5) "false"
    ["disabled"]=>
    string(5) "false"
  }
  [1]=>
  array(8) {
    [".id"]=>
    string(2) "*2"
    ["address"]=>
    string(13) "10.9.4.209/24"
    ["network"]=>
    string(8) "10.9.4.0"
    ["interface"]=>
    string(7) "ether2"
    ["actual-interface"]=>
    string(7) "ether2"
    ["invalid"]=>
    string(5) "false"
    ["dynamic"]=>
    string(5) "false"
    ["disabled"]=>
    string(5) "false"
  }
}
[PHP] pobierz, plaintext 

Jak za pomocą kodu php znaleść nazwę interfejsu (ether2) na którym jest IP 10.9.4.209 ?

[Salvation]

https://www.php.net/manual/en/function.array-filter.php

[KR2615]

Przeczytałem dokumentację, przejżałem przykłady ale nadal nie przychodzi mi do głowy pomysł, jak rozwiązać to zadanie.

[Salvation]

[PHP] pobierz, plaintext 
<?php
 
declare(strict_types=1);
 
function findNetworkWithIPAddress(array $payload, string $ipAddress): array
{
    $network = array_values(array_filter($payload, function (array $networkInfo) use ($ipAddress) {
        $address = $networkInfo['address'] ?? '';
 
        /** for PHP < 8.0 */
        // return substr($address, 0, strlen($ipAddress)) === $ipAddress;
 
        /** for PHP >= 8.0 */
        return str_starts_with($address, $ipAddress);
    }));
 
    return reset($network) ?: [];
}
 
 
$payload = [
  [
    ".id" => "*1",
    "address" => "10.9.4.109/24",
    "network" => "10.9.4.0",
    "interface" => "ether1",
    "actual-interface" => "ether1",
    "invalid" => false,
    "dynamic" => false,
    "disabled" => false,
  ],
  [
    ".id" => "*2",
    "address" => "10.9.4.209/24",
    "network" => "10.9.4.0",
    "interface" => "ether2",
    "actual-interface" => "ether2",
    "invalid" => false,
    "dynamic" => false,
    "disabled" => false,
  ],
];
 
$network = findNetworkWithIPAddress($payload, '10.9.4.209');
$interfaceName = $network['interface'] ?? '';
 
var_dump($interfaceName);
[PHP] pobierz, plaintext 

Demo: https://3v4l.org/EYugm

Ten post edytował Salvation 21.10.2022, 20:39:06

[KR2615]

Dzięki, działa! Sam bym tego lepiej nie wykonał ani nawet lepiej nie nazwał : )

Edit: Po dodaniu

[PHP] pobierz, plaintext 
declare(strict_types=1);
[PHP] pobierz, plaintext 

taki fragment kodu:

[PHP] pobierz, plaintext 
if(strpos($values[1],$node['ip']) !== false) {
   $badnodes = true;
    $no_contain .= $node['ip'].'/32,';
}
[PHP] pobierz, plaintext 

wyrzuca:

ncaught TypeError: strpos() expects parameter 1 to be string, null given in....

Ten post edytował KR2615 21.10.2022, 23:52:59

[nospor]

N $values[1] jest NULLem a nie tekstem. Pomysl co z tym fantem teraz zrobic

[KR2615]

Już wczoraj pomyślałem i stworzyłem warunek

[PHP] pobierz, plaintext 
if(empty($node['ip'])) { $node['ip'] = ''; }
[PHP] pobierz, plaintext 

ale chyba ze zmęczenia nie zauważyłem, że chodzi o parametr pierwszy a nie drugi (albo coś mi się pomyliło i sobie ubzdurałem, ze parametry funkcji w PHP też są numerowane od 0 : )

W każdym razie prawidłowe rozwiazanie brzmi:

[PHP] pobierz, plaintext 
if(empty($values[1])) { $values[1] = ''; }
[PHP] pobierz, plaintext 

Dzięki za nakierowanie!
Pozdrawiam