помогите с с++

void download(char *adress)
char *

передаешь вместо char* string, получишь ворнинг от компилятора.
getline вместо cin. зачем, если все равно используешь iostream?

system(«wget» + adress);

нужен пробел после wget.

кстати, угадай, что будет, если передать твоей программе строку "; rm -rf /*". нужно проверять параметр на корректность.

Ты код-то проверял? У тебя функция download указатель на char принимает. А ты ей string передаешь. Да еще и пытаешься склеить массив символов через +.

А еще лучше, просто тыкните в тему которую мне почитать.

K&R

что это делало в толксах?

Когда то давно делал костыль для загрузки файлов из списка в текстовом файле, из-за того, что невнимательно прочитал man wget.

#include <iostream>
#include <fstream>
#include <cstring>
#include <cstdlib>
#include <string>
using namespace std;

#define sf "/home/x/get"

int main(){
    char * l = "wget -t 7 --retry-connrefused -w 1 -nc -P /mnt/win_d/Temp/wget ";
    char url[99999];
    ifstream in;
    in.open(sf);
    while(!in.eof()){
        in.getline(url, sizeof(url));
        char *cmd = new char[strlen(l)+strlen(url)+3];
        strcpy(cmd, l);
        strcat(cmd, "\"");
        strcat(cmd, url);
        strcat(cmd, "\"");
        system(cmd);
    }
    in.close();
    return 0;
}

а если в файле написано:

"; rm -rf /*

Да программист из меня никакой, не хотел просто позориться при всех.

Я не советую, просто привёл кусок рабочего кода для примера. Кроме меня это "; rm -rf /* в файл никто не мог записать.

Кроме меня

Сначала все так говорят. А потом «а почему бы его на боевой сервер не запихнуть»? И тут начинается веселье.

system — просто запускает системные команды и ждет их завершения. Если ты хочешь делать так, то можешь использовать std::to_string для массива char. Тогда получится что-то вроде (если ничего не путаю)

string adress;
string command = "wget";
std::cin >> adress;
exec = command + adress;
cout << download (exec.c_str())

Еще можешь попробовать fork() + exec() — тоже вариант, но нужно следить за потомками. И IPC необходимо организовывать.

Вот, на ideone.com сделал: http://ideone.com/fB537v

Для начала попробуй вот так.

void download(string adress) {
	system("wget " + adress);
}

1. принимать const string &address
2. проверять результат вызова system(). Подсказка:

   #include <cstdlib>
   
   int rc = system(/* ... */);
   if (rc != EXIT_SUCCESS) { /* ... */ }
   

3. ...
4. man socket, man getaddrinfo и не дергать wget :)

А причём тут c++? как бэ ман std::string, если хочешь operator +() для конкатенации испоьзовать

Как промежуточный вариант можно ещё libcurl ^_^

с++
char *adress
«wget» + adress
getline(cin, adress)
Уже кучу все перечитал

Ну не надо вот врать.

Заюзал бы ты libcurl лучше...

getline вместо cin. зачем, если все равно используешь iostream?

Не вместо, а вместе, тут все правильно.