Execl will nicht starten.

[SeppJ]

bluepeople12 schrieb:

Und ich hab mir sowohl word.c_str() ausgeben lassen als auch list_files.at(i + 1).c_str()

word ist das gesuchte word und list_files ist die liste der dateien (alle als std::string in einem std::vector) und die dateien gibt es ... so wie auch die im obigen beispiel ...

Ich wette dagegen.

[bluepeople12]

Wieso?

[SeppJ]

bluepeople12 schrieb:

Wieso?

Weil Computer erfahrungsgemäß keine so krassen Fehler machen. Menschen schon.

Um deinem Problem auf die Spur zu kommen, mach mal:
execl( "/bin/ls", "/bin/ls", NULL);

[bluepeople12]

SeppJ schrieb:

bluepeople12 schrieb:

Wieso?

Weil Computer erfahrungsgemäß keine so krassen Fehler machen. Menschen schon.

Um deinem Problem auf die Spur zu kommen, mach mal:
execl( "/bin/ls", "/bin/ls", NULL);

OK, der Befehl funktioniert. Mir wird eine Liste von Dateien ausgegeben ... die in demselben Ordner sind wie ich mich gerade befinde ...

Und nun ?

[rüdiger]

Nun überprüf nochmal, ob die Argumente richtig sind. Hängen da vielleicht noch Leerzeichen oä dran?

[bluepeople12]

Ok, hab nochmal alles durchgecheckt und als ich dann die Längen der strings verglichen hatte, kam raus, dass da doch wohl ein Zeichen zuviel war ...

Aber jetzt habe ich ein ganz anderes Problem. Und zwar wird das Programm gleich nach dem ersten Durchlauf beendet ...
Wie kann ich das verhindern ?

[SeppJ]

bluepeople12 schrieb:

Aber jetzt habe ich ein ganz anderes Problem. Und zwar wird das Programm gleich nach dem ersten Durchlauf beendet ...
Wie kann ich das verhindern ?

Mehr Druchläufe machen!

Ernsthaft: Was soll so eine Frage? Wir kennen von deinem Programm drei Zeilen. Und die drei Zeilen haben allesamt nichts mit der Ablaufsteuerung zu tun. Wie sollen wir da jetzt diagnostizieren, was falsch ist?

[bluepeople12]

Nunja ... wenn ich die Zeilen mit dem Befehl execlp auskommentiere ... zeigt mir mein Programm jeden Durchlauf an.

Wenn ich jedoch da lasse ... dann ist der erste Durchlauf fertig und das wars dann. Und ich denk mir dann als Folge, dass es an dem Befehl liegt.

Hier mal die Schleifen, um die es geht (dabei ist execlp auskommentiert):

Es wird hier geschaut, ob zwei strings gleich sind, und wenn ja, dann soll der Dateiname ausgegeben werden mit ein paar Zeilen aus dem Inhalt (und das geschieht über das Programm grep).

for (size_t i = 0; i < list_files.size() - 1; i++)
    {
      for (size_t j = 0; j < list_fileids.size(); j++)
      {
        // std::cout << "i: " << i << "\tj: " << j << "\n";
        if (list_files.at(i).compare(list_fileids.at(j)) == 0)
        {
          // std::cout << list_files.at(i) << "\n";
          // std::cout << i << "\n";
          list_files.at(i + 1) = list_files.at(i + 1).replace(
            list_files.at(i + 1).size() - 1, 1, "");
          std::cout << list_files.at(i + 1).c_str() << "\n";
/*
          if (execlp("grep", "grep", "-n", "-w",
            word.c_str(), list_files.at(i + 1).c_str(),
            NULL) == -1);
          {
            std::cout << "Error\n";
          }
*/
        }
      }
    }

[SeppJ]

Und was wundert dich da dran jetzt? Weißt du überhaupt, was die exec-Funktionen machen? Mal eine Manpage gelesen oder ein Beispiel angeguckt? Das exec alleine ersetzt den aktuellen Prozess mit einem neuen!

[_matze]

SeppJ schrieb:

bluepeople12 schrieb:

Aber jetzt habe ich ein ganz anderes Problem. Und zwar wird das Programm gleich nach dem ersten Durchlauf beendet ...
Wie kann ich das verhindern ?

Mehr Druchläufe machen!

Der war gut.

[bluepeople12]

SeppJ schrieb:

Und was wundert dich da dran jetzt? Weißt du überhaupt, was die exec-Funktionen machen? Mal eine Manpage gelesen oder ein Beispiel angeguckt? Das exec alleine ersetzt den aktuellen Prozess mit einem neuen!

Und mit welchem Befehl starte ich einfach einen neuen Prozess ohne ihn zu beenden ?

[bluepeople12]

OK, ich habs schon mit system ...

[SeppJ]

bluepeople12 schrieb:

OK, ich habs schon mit system ...

Ähh, was hast du überhaupt vor?

[bluepeople12]

Ich hab hier eine Liste an Dateien (Textdateien). In ihnen soll was gesucht werden und dies soll angezeigt werden.

Dass die Dateien existieren, ist garantiert. Dass da was drin ist, auch.
Und mit dem C++-Befehl system will ich grep starten, sodass mein Prozess nicht beendet wird ... sondern mehrfach, da ich ja jede Datei mit grep starten möchte und nach bestimmten Suchwörtern suche.

Das hab ich vor ...
Ist da was falsch dran?

[Bashar]

bluepeople12 schrieb:

Und mit welchem Befehl starte ich einfach einen neuen Prozess ohne ihn zu beenden ?

fork

edit: "ohne ihn zu beenden" ist natürlich Quark, bei exec wird nichts beendet, sondern ersetzt. Der übliche Weg, ein anderes Programm als Unterprozess zu starten, besteht in einer Kombination aus fork und exec: Erst fork, um einen Kindprozess abzuspalten. Dann im Kindprozess exec.

(oder system)

[bluepeople12]

Ist system schlecht dafür ? Ich mein, bei mir funktionierts ...

[rüdiger]

exec* überschreibt den aktuellen Prozess mit einem neuen Prozess. Wie Bashar bereits gesagt hast, musst du daher vorher einen neuen Prozess anlegen. Das geht mit man: fork bzw. man: vfork.

pid_t pid = vfork();
if(pid == -1) {
  perror("vfork");
  return 1;
}
else if(pid == 0) { // Neuer Prozess
  if(execlp(...) == -1) {
    perror("execlp");
  }
  _exit(1);
}
else {
  int status;
  if(waitpid(pid, &status, 0) == -1) {
    perror("waitpid");
    return 1;
  }
  if(WEXITSTATUS(status) != 0) { // Prozess hat mit einem Fehler beendet.
    // ...
  }
}