warum wird mir beim Beenden des Prozesses eine illegale PID angezeigt?

Question 1

Hier ist die Ursache Ihres Problems:

PID=`ps -ef|grep -w ${APP_NAME}|grep -v grep|cut -c 9-15`

Setzen Sie Ihre Variablen in doppelte Anführungszeichen, wenn Sie sie verwenden, z. B. „$APP_NAME“ oder „${APP_NAME}“.

Übrigens werden die geschweiften Klammern nur benötigt, wenn Sie eine Variable von anderem Text in einem String unterscheiden müssen. Wenn die Variable z. B. tatsächlich wäre, $APPSie sie aber in einem String mit verwenden müssten _NAME, würden Sie "${APP}_NAME"- verwenden. Dadurch würde verhindert, dass _NAMEdie Shell sie als Teil des Variablennamens interpretiert.
Sie verwenden Backticks anstelle von $(). Sie sind seit Jahren veraltet und das aus gutem Grund. Sie sind hier nicht die Ursache des Problems, sondern nur eine schlechte Angewohnheit, die Sie aufgeben müssen.
cutist kein gutes Tool zum Extrahieren von Feldern, die durch 1 oder mehr Zeichen getrennt werden können. Es ist nur dann gut, wenn es ein (und genau ein) Trennzeichen zwischen den Feldern gibt. Viele Textdateien verwenden 1 oder mehr Leerzeichen (und/oder Tabulatoren und/oder andere Leerzeichen) als Feldtrennzeichen und können nicht einfach mit verarbeitet werden cut. Verwenden Sie stattdessen awkoder perl.

Ihre Verwendung von cut -c 9-15führt dazu, dass mindestens ein zusätzliches Leerzeichen in $PID erfasst wird. Um eine PID aus zu extrahieren ps -ef, verwenden Sie awk '{print $2}'anstelle von cut -c 9-15.
Verwenden Sie diese Option pgrep "$APP_NAME", um die PID eines Prozesses nach Namen abzurufen. Oder pgrep -f "$APP_NAME"wenn die gesuchte Zeichenfolge ein Argument ist (z. B. wenn der Name des Skripts als Argument an einen Interpreter übergeben wird).

Cargo-Kult ps | grep ... | grep -v grepist seit Jahrzehnten überholt – und war ohnehin nie eine gute Methode dafür. ps -ef | awk '/[p]rocess_name/ {print $2}'war immer eine bessere Methode (das Setzen eines []um das erste Zeichen des Musters verhindert, dass awk, grep oder was auch immer sich selbst in der ps-Ausgabe zuordnet), aber selbst das ist mittlerweile mit überholt pgrep.

psselbst verfügt über eine -COption zum Abgleichen von Prozessnamen und eine hOption zum Unterdrücken von Headern sowie -ozum Angeben der gewünschten Ausgabe ps. zBps h -o pid -C "$APP_NAME"

Um das alles zusammenzufassen, verwenden Sie:

    PID=$(pgrep "$APP_NAME")
or
    PID=$(pgrep -f "$APP_NAME")
or
    PID=$(ps h -o pid -C "$APP_NAME")

Übrigens, wenn esbeliebigpgrepDie Wahrscheinlichkeit, dass mehrere PIDs von oder zurückgegeben werden ps, sollten Sie die Ausgabe in einem Array erfassen, nicht in einer Skalarvariable. Im Folgenden werden beispielsweise die PIDs aller sichtbaren Bash-Prozesse in dem Array erfasst $BASHPIDS.

$ BASHPIDS=( $(ps h -o pid -C bash) )

$ typeset -p BASHPIDS
declare -a BASHPIDS=([0]="68910" [1]="71059" [2]="71634" [3]="71641" [4]="71643"
[5]="71680" [6]="71683" [7]="71684" [8]="71687" [9]="71693" [10]="71712" [11]="72394"
[12]="72568" [13]="72589" [14]="970222" [15]="974740" [16]="1078757" [17]="1278073"
[18]="1365082" [19]="1405642" [20]="1458889" [21]="2278763" [22]="2466442" [23]="2876831"
[24]="2955565" [25]="3260896" [26]="3261235" [27]="3269020" [28]="3281961" [29]="3702104" 
[30]="4038149")

Wenn Sie sie aus irgendeinem Grund alle töten möchten, können Sie dies folgendermaßen tun:

kill "${BASHPIDS[@]}"

Answer

Hier ist die Ursache Ihres Problems:

PID=`ps -ef|grep -w ${APP_NAME}|grep -v grep|cut -c 9-15`

Setzen Sie Ihre Variablen in doppelte Anführungszeichen, wenn Sie sie verwenden, z. B. „$APP_NAME“ oder „${APP_NAME}“.

Übrigens werden die geschweiften Klammern nur benötigt, wenn Sie eine Variable von anderem Text in einem String unterscheiden müssen. Wenn die Variable z. B. tatsächlich wäre, $APPSie sie aber in einem String mit verwenden müssten _NAME, würden Sie "${APP}_NAME"- verwenden. Dadurch würde verhindert, dass _NAMEdie Shell sie als Teil des Variablennamens interpretiert.
Sie verwenden Backticks anstelle von $(). Sie sind seit Jahren veraltet und das aus gutem Grund. Sie sind hier nicht die Ursache des Problems, sondern nur eine schlechte Angewohnheit, die Sie aufgeben müssen.
cutist kein gutes Tool zum Extrahieren von Feldern, die durch 1 oder mehr Zeichen getrennt werden können. Es ist nur dann gut, wenn es ein (und genau ein) Trennzeichen zwischen den Feldern gibt. Viele Textdateien verwenden 1 oder mehr Leerzeichen (und/oder Tabulatoren und/oder andere Leerzeichen) als Feldtrennzeichen und können nicht einfach mit verarbeitet werden cut. Verwenden Sie stattdessen awkoder perl.

Ihre Verwendung von cut -c 9-15führt dazu, dass mindestens ein zusätzliches Leerzeichen in $PID erfasst wird. Um eine PID aus zu extrahieren ps -ef, verwenden Sie awk '{print $2}'anstelle von cut -c 9-15.
Verwenden Sie diese Option pgrep "$APP_NAME", um die PID eines Prozesses nach Namen abzurufen. Oder pgrep -f "$APP_NAME"wenn die gesuchte Zeichenfolge ein Argument ist (z. B. wenn der Name des Skripts als Argument an einen Interpreter übergeben wird).

Cargo-Kult ps | grep ... | grep -v grepist seit Jahrzehnten überholt – und war ohnehin nie eine gute Methode dafür. ps -ef | awk '/[p]rocess_name/ {print $2}'war immer eine bessere Methode (das Setzen eines []um das erste Zeichen des Musters verhindert, dass awk, grep oder was auch immer sich selbst in der ps-Ausgabe zuordnet), aber selbst das ist mittlerweile mit überholt pgrep.

psselbst verfügt über eine -COption zum Abgleichen von Prozessnamen und eine hOption zum Unterdrücken von Headern sowie -ozum Angeben der gewünschten Ausgabe ps. zBps h -o pid -C "$APP_NAME"

Um das alles zusammenzufassen, verwenden Sie:

    PID=$(pgrep "$APP_NAME")
or
    PID=$(pgrep -f "$APP_NAME")
or
    PID=$(ps h -o pid -C "$APP_NAME")

Übrigens, wenn esbeliebigpgrepDie Wahrscheinlichkeit, dass mehrere PIDs von oder zurückgegeben werden ps, sollten Sie die Ausgabe in einem Array erfassen, nicht in einer Skalarvariable. Im Folgenden werden beispielsweise die PIDs aller sichtbaren Bash-Prozesse in dem Array erfasst $BASHPIDS.

$ BASHPIDS=( $(ps h -o pid -C bash) )

$ typeset -p BASHPIDS
declare -a BASHPIDS=([0]="68910" [1]="71059" [2]="71634" [3]="71641" [4]="71643"
[5]="71680" [6]="71683" [7]="71684" [8]="71687" [9]="71693" [10]="71712" [11]="72394"
[12]="72568" [13]="72589" [14]="970222" [15]="974740" [16]="1078757" [17]="1278073"
[18]="1365082" [19]="1405642" [20]="1458889" [21]="2278763" [22]="2466442" [23]="2876831"
[24]="2955565" [25]="3260896" [26]="3261235" [27]="3269020" [28]="3281961" [29]="3702104" 
[30]="4038149")

Wenn Sie sie aus irgendeinem Grund alle töten möchten, können Sie dies folgendermaßen tun:

kill "${BASHPIDS[@]}"

Question 2

dieser Befehl behebt das Problem:

 echo "${PID}" | xargs kill -9

Answer

dieser Befehl behebt das Problem:

 echo "${PID}" | xargs kill -9

warum wird mir beim Beenden des Prozesses eine illegale PID angezeigt?

Antwort1

Antwort2

verwandte Informationen