Вопрос. Можно как то код сделать по проще php?

Question

[Malias]

Можно вод этот код сделать, как-то по проще?

$MysqlConnect = array(
"link" => "192.168.0.222",
"port" => "29000"
);
if(fsockopen($MysqlConnect['link'], $MysqlConnect['port'], $errno, $errstr, 30)) {
$IsOnline = "<font color='green' size='5'>ONLINE</font>";
} else $IsOnline = "<font color='red' size='5'>OFFLINE</font>";

Сделать всё через функцию?

Answer 1

[Валентин]

function checkStatus($link, $port){
	if (fsockopen($link, $port, $errno, $errstr, 30)){
		return array('color' => 'green', 'text' => 'ONLINE');
	}
	
	else {
		return array('color' => 'red', 'text' => 'OFFLINE');
	}
}

$res = checkStatus('192.168.0.222', '29000');
echo "<font color='{$res['color']}' size='5'>{$res['text']}</font>";

Comments

[Станислав Шендаков]

+1. Только исходя из 6-го принципа структурного программирования ( Все перечисленные конструкции должны иметь один вход и один выход ) нагляднее и правильнее будет так:

function checkStatus($link, $port){
  $arr = ['color' => 'green', 'text' => 'ONLINE'];
  if ( !fsockopen($link, $port, $errno, $errstr, 30) )
  	$arr = ['color' => 'red', 'text' => 'OFFLINE', 'errno' => $errno, 'errstr' => $errstr ];

  return $arr;
}
$res = checkStatus('192.168.0.222', '29000');
echo "<font color='{$res['color']}' size='5'>{$res['text']}</font>";

[di]

Станислав Шендаков, Я бы сделал так. Правилом хорошего тона считаю постановку фигурных скобок в любом случае. Лучше читается вроде как

function checkStatus($link, $port){
  if ( !fsockopen($link, $port, $errno, $errstr, 30) ) {
      return ['color' => 'red', 'text' => 'OFFLINE', 'errno' => $errno, 'errstr' => $errstr ];
  }
  return  ['color' => 'green', 'text' => 'ONLINE'];
}
$res = checkStatus('192.168.0.222', '29000');
echo "<font color='{$res['color']}' size='5'>{$res['text']}</font>";

[Станислав Шендаков]

Я в курсе про фигурные скобки, но их использование при выполнении одного оператора в for, while, if, do..while, не является обязательным. Обязательным является использование единого стиля т.е.:

if( ... ){
$x = true;
}
else{
$x = false;
}

или

if( ... )
$x = true;
else
$x = false;

В моем коде и использован частный случай последнего блока кода без ветки else.