Как перенести длинную строку в коде?

Question

[Александр Назаров]

Есть вот такая строка, но она длинная и неудобная.
Буду благодарен, если кто-нибудь объяснит как правильно перенести строки не нарушая работу скрипта.

$send .= "sudo screen -dmS screen_name docker run " . $user_container . " " . $memory_container . " -d -it --cpus '1' --memory=" . $mem . "m " . $v1 . " " . $v2 . " " . $v3 . " " . $v4 . " " . $v5 . " -e LANG=C.UTF-8 -e LC_ALL=C.UTF-8 " . $v6 . " " . $v7 . " " . $v8 . " " . $v9 . " " . $v10 . " " . $v11 . " " . $v12 . " " . $v13 . ";";

Пробовал вот так:

$send .= "sudo screen -dmS screen_name docker run  \
 " . $user_container . " " . $memory_container . " -d -it --cpus '1' --memory=" . $mem . "m \
" . $v1 . " " . $v2 . " " . $v3 . " " . $v4 . " " . $v5 . " -e LANG=C.UTF-8 -e LC_ALL=C.UTF-8  \
" . $v6 . " " . $v7 . " " . $v8 . " " . $v9 . " " . $v10 . " " . $v11 . " " . $v12 . " " . $v13 . ";";

Answer 1

[Ипатьев]

$send .= "sudo screen -dmS screen_name docker run $user_container $memory_container -d -it --cpus '1' --memory=" . $mem."m $v1 $v2 $v3 $v4 $v5 -e LANG=C.UTF-8 -e LC_ALL=C.UTF-8 $v6 $v7 $v8 $v9 $v10 $v11 $v12 $v13;";

но как же я задолбался все эти кавычечки выковыривать

Comments

[Dmitry Bay]

Вковыряй обратно!))

[Dmitry Bay]

Кстати как вариант, я бы подготовил строку шаблон. А затем просто заменил бы данные. Скорее всего v1->v5 можно поместить в массив, а дальше join(). тоже самое с v6-v13. "$user_container $memory_container " можно в сам шаблон поместить.

$prepaiderCommand = "sudo screen -dmS screen_name docker run $user_container $memory_container -d -it --cpus '1' --memory=" . $mem."m $v1 $v2 $v3 $v4 $v5 -e LANG=C.UTF-8 -e LC_ALL=C.UTF-8 $v6 $v7 $v8 $v9 $v10 $v11 $v12 $v13;";

Answer 2

[Newto]

Практика писать
$text = "text " . "$param" . "...";
Весьма распространенная, к сожалению.
Гораздо читабельней написать
$text = "text {$param}...";
Исходя из этого и измените вашу строку. Вот как выглядело бы начало:

$send .= "sudo screen -dmS screen_name docker run {$user_container } {$memory_container}  -d -it --cpus '1' --memory={$mem}m"

А там уже переносы строк ставьте где хотите.

P.S. Фигурные скобки, обрамляющие переменные, не обязательны. Но они выделяют эти переменные зрительно, а так же выделяются в разных IDE(средах разработки, проще говоря редакторах кода).

Comments

[Ипатьев]

вообще-то у него шелловая команда, и "где хотите" переносы поставить не получится

Answer 3

[Adamos]

$send .= <<<LONGSTRING
sudo screen -dmS screen_name docker run $user_container $memory_container -d -it \
--cpus '1' --memory=${mem}m $v1 $v2 $v3 $v4 $v5 -e LANG=C.UTF-8 -e LC_ALL=C.UTF-8 \
$v6 $v7 $v8 $v9 $v10 $v11 $v12 $v13;
LONGSTRING;

$send .= implode(' ', [
  'sudo screen -dmS screen_name docker run',
  $user_container,
  $memory_container,
  "-d -it --cpus '1'",
  '--memory=' . $mem . 'm',
  $v1, $v2, $v3, $v4, $v5,
  '-e LANG=C.UTF-8 -e LC_ALL=C.UTF-8',
  $v6, $v7, $v8, $v9, $v10, $v11, $v12, $v13
]);

Answer 4

[Денис Радченко]

Можете использовать sprintf() или

$send .= "sudo screen -dmS screen_name docker run "
. $user_container . " " . $memory_container .
" -d -it --cpus '1' --memory=" . $mem . "m " . $v1 . " " . $v2 . " " . $v3 . " " . $v4 . " " . $v5
. " -e LANG=C.UTF-8 -e LC_ALL=C.UTF-8 " . $v6 . " " . $v7 . " " . $v8 . " " . $v9 . " " . $v10 . " " . $v11 . " " . $v12 . " " . $v13 . ";";

Comments

[Ипатьев]

а почему точки вразнобой, то слева, то справа?

[Денис Радченко]

Ипатьев, моя ошибка, точки лучше написать вначале или в конец строки.