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?

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.

$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

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:)

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.

Jak ci ta jedna linijka jest potrzebna do regexpa to

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

I masz jedna linijke

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ł?


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".

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:


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



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

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

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?

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.

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 

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

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 ?

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

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

[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

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:

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

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

[PHP] pobierz, plaintext 
$values[1] ?? ''
[PHP] pobierz, plaintext 

Tak nie całkiem offtopic. Czy po dwóch linijkach kodu które napisałeś mam podstawy wnioskować, że nie będziesz rozumiał co napisał @trueblue, czy może się mylę? To tak a propos kopiuj/wklej do swojego kodu.

Widziałem już gdzieś taki przykład @gino. To uproszczenie tego ifa które z powodzeniem udało mi się zastosować:

[PHP] pobierz, plaintext 
if(strpos($values[1] ?? '',$node['ip']) !== false) {
[PHP] pobierz, plaintext