Wie verwenden von Variablen im system() in c

Bitte helfen Sie mir mit meinem code. Ich bin mit dem system () - Funktion in c#. Lassen Sie uns sagen ich will ein neues Verzeichnis mit C-code.

char name[];

printf("Enter the name of directory: ");
scanf("%s", &name);

Dann mit system() ich wollen, verwenden Sie die variable name[]
anstelle der Verwendung oder setzen Sie einen festen Wert wie system("mkdir ryan"); macht ein neues Verzeichnis ryan; was ich will, geschehen wird, dass zu jedem Namen von Verzeichnis und speichern Sie es auf variable name[] und umzusetzen, dass Wert statt ryan. Ihre Antworten sind sehr geschätzt.

  • system ist nicht der richtige Weg, um call mkdir. Es ist eine ganz gute mkdir - Funktion aufrufen. Mit system führt zu allen Arten von Sicherheit, Stabilität und performance-Probleme, die sollten nie eingeführt worden, um mit zu beginnen. Im Grunde vergessen Sie jemals gehört, der system. Es sollte absolut nie verwendet werden.
  • Außer, wenn Sie es brauchen 🙂
InformationsquelleAutor ryan nuada | 2012-01-16
  1. 6

    Sollten Sie mit so etwas wie:

    char name[100];
printf("Enter the name of the directory: ");
if (scanf("%99s", name) == 1)  //Not &name
{
    char command[120];
    sprintf("%s %s", "mkdir", name);
    if (system(command) != 0)
        ...oops...
}
    • Oder verwenden Sie snprintf um sicherzustellen, dass Sie nicht überlauf-Puffer, unabhängig davon, wie groß Sie denken, ist "groß genug".
    • Da ich begrenzt die Eingabe auf weniger als 100 Zeichen, und strlen("mkdir ") ist weniger als 20, das ist sicher so wie es da steht. In anderen Umständen, dann snprintf() wäre besser.
    • Sehr gute Lösung.
    InformationsquelleAutor Jonathan Leffler
  2. 1

    Zuerst aus (und dies wahrscheinlich spielt keine Rolle, ob Ihr code ist nur ein Beispiel), verwenden Sie nicht immer ein unbegrenztes %s scanf - das öffnet Sie auf Puffer-überläufe.

    Wenn Sie eine Zeichenfolge wie:

    char name[] = "paxdiablo";

    können Sie nur verwenden, um erstellen Sie Ihre eigene Zeichenfolge für die Ausführung.

    char cmd[1000];
strcpy (cmd, "mkdir ");
strcat (cmd, name);
system (cmd);

    Und stellen Sie sicher, dass Sie wissen (oder überprüfen Sie mit strlen oder dynamisch zuordnen des Puffers, so dass es groß genug ist) die Größe der name so dass Sie nicht am Ende mit einem Puffer-überlauf, der es als gut.

    Ein Beispiel für die dynamische Zuteilung einer:

    void tryMkdir (char *dir) {
    static char prefix[] = "mkdir ";
    //Use sizeof to allow for null char at end.
    char *cmd = malloc (sizeof (prefix) + strlen (dir));
    if (cmd != NULL) {
        strcpy (cmd, prefix);
        strcat (cmd, dir);
        system (cmd);
        free (cmd);
    }
}

    (obwohl würden Sie wahrscheinlich wollen einige Fehler-Check-in gibt es im Fall der mkdir oder malloc fehlgeschlagen).

    • Was ist Los mit snprintf?
    • Nichts. Es gibt viele Möglichkeiten, um die Arbeit zu tun. Für einfache string-Verkettung, ziehe ich strcat anstatt die volle Kraft der snprintf (es gibt keine Notwendigkeit für die Länge prüfen, hier seit wir sorgen für die Puffer ist groß genug). Wenn ich den Bau einer Zeichenfolge aus mehr als zwei Dinge oder sich nicht sicher von der Länge, würde ich wahrscheinlich wechseln, aber ich finde es nicht notwendig in diesem Fall.
    • Es klingt alles kompliziert mit der malloc, mehrere Anrufe etc - wo man snprintf tun würde, mit ein array auf dem stack von MAX_PATH Größe (nicht 100% sicher über die übertragbarkeit von der makro -) Und wirklich, der OP sollte mit einem system-Aufruf, um das Verzeichnis zu erstellen - weil es so begrenzt, wie mit einem system-Aufruf, aber ohne einige seltsame Seite beeinflussen die Ausführung einer nicht unterstützten Befehl gegen die OS
    • es gibt keine max_path/path_max in C99 oder C11. Es gibt eine filename_max, aber das kann auch einfach der empfohlen Größe eher als das maximium. Und du hast wahrscheinlich Recht, über die Verwendung von Plattform-spezifischen Aufrufe eher als system aber die Frage war speziell über system mit Variablen.
    • Einverstanden - konnte ihn nicht finden entweder vorausgesetzt - dann war es ein POSIX-Erweiterung Sache, obwohl ich glaube, es ist verfügbar in VS als gut.
    • vielen Dank. es hat mir wirklich geholfen. dies ist die Antwort auf mein problem:)

    InformationsquelleAutor paxdiablo
  3. 1

    hier ist ein Beispiel, wie es umgesetzt werden könnte:

    #include <stdio.h>
#include <stdlib.h>
#include <sys/wait.h>

int main() {
    char *dirname = NULL;
    char *cmdline = NULL;
    size_t len;
    size_t dirlen = 0;
    int rv = 0;

    printf("Enter directory: ");
    if ( (len = getline(&dirname, &dirlen, stdin)) < 0) {
        perror("getline");
        exit(-1);
    }

    dirname[len-1] = 0;
    cmdline = malloc(len+8);
    snprintf(cmdline, dirlen+8, "mkdir %s", dirname);
    rv = system(cmdline);
    free(cmdline);
    free(dirname);
    rv = WEXITSTATUS(rv);
    return rv;
}
    • Vergessen Sie nicht, free dirname zu. Es mag auch erwähnenswert sein, dass getline() ist in POSIX 2008 und, während unter Linux und *BSD (einschließlich MacOS X), es ist nicht überall verfügbar.
    InformationsquelleAutor Ahmed Masud
  4. 0

    Müssen Sie erstellen Sie ein string w/Ihre full-Befehl zu senden, um system() in der it, was bedeutet, zuweisen von Speicherplatz für Sie (die Sie nicht für Namen in Ihrer ursprünglichen code, so dass Ihr Aufruf von scanf würde wahrscheinlich scheitern). Für sowas wie dein mkdir Beispiel könnten Sie eine Zeichenfolge, die beginnt mit mkdir & hat genug Raum nach, die für Ihr Verzeichnis-Namen, und wenn Sie den Aufruf scanf, zeigen Sie es an die Position innerhalb dieser Zeichenfolge, die Sie wollen, dass der name zu gehen; spart Sie zu tun eine strcat oder somesuch danach, und wenn dies die einzige Sache, die Sie müssen den Namen des Verzeichnisses, warum speichern Sie es zweimal?

    InformationsquelleAutor Scott Hunter
  5. -3

    Versuchen:

    char command[80];
strcpy(command, "mkdir ");
strcat(command, name);
system(command);
    • Sie können NICHT verwenden strcat mit einem const char * " als ersten parameter.
    • Hoppla, danke für die Warnung! Ist es jetzt besser?
    • Warum machen Sie es so kompliziert snprintf verwenden
    • Sorry, dass ich noch nie davon gehört, dass Funktion vor, habe ich hauptsächlich C++ so weit, und die string-Verkettung war die erste Sache, gewölbt, in den Sinn
    • vielen Dank 🙂
    • kein problem, obwohl meine Antwort war nicht so hilfreich 😛
    • Wenn Sie vor allem mit C++, strcpy/strcat sollte der letzten, was auf Ihre Meinung!

    InformationsquelleAutor wrongusername
Schreibe einen Kommentar