[C++] Процессы

[ZWYHB]

Необходимо реализовать функцию запуска и параллельного выполнения нескольких программ. Задаётся количество выполняемых программ и набор их имён (пути к исполняемым образам). Исходный процесс завершается сразу после их порождения.
Написать тестовую программу для проверки работоспособности разработанной функции.

вот что получилось:
main.cpp

Код: (C++)

#include <iostream>
#include <vector>
#include <unistd.h>
#include <string>
#include <process.h>

using namespace std;

int main(int argc, char *argv[])
{

    cout << "Vvedite kollichestvo programm:"<<endl;
    unsigned n;
    cin>>n;
    std::vector <string> v;
    v.resize(n);
    if(n<=0) return 0;
    cout << "Vvedite imena programm:"<<endl;
    for (int i =0; i<n; ++i)
    {
        cin>>v[i];
    }

f(n,v);
cout<<"end"<<endl;
    return EXIT_SUCCESS;
}

proc.h

Код: (C++)

#ifndef PROCESS_H
#define PROCESS_H
#include<unistd.h>
#include<iostream>
#include <vector>
#include<string>
class Process
{
public:
    Process();
    ~Process() {}
    operator bool() const;
    pid_t id() const;
    bool run(std::string a);
   
protected:
    int action(std::string a);
private:
    Process(pid_t id);
private:
    pid_t pid;
};
inline Process::operator bool() const
{
    return pid != 0;
}
inline pid_t Process::id() const
{
    return pid;
}
void f(int n, std::vector <std::string> v);
#endif

proc.cpp

Код: (C++)

#include "process.h"
#include <cstdlib>
#include <string>
#include <vector>
#include <signal.h>
#include <sys/wait.h>
using namespace std;

Process::Process()
        : pid(0)
{
}
Process::Process(pid_t id)
        : pid(id)
{
}
bool Process::run(std::string a)
{
    if ( pid )
        return false;
    pid = fork();
    switch ( pid )
    {
    case -1:
        pid = 0;
        return false;
    case 0:
        pid = getpid();
        exit(action(a));
    }
    return true;
}

int Process::action(string a)
{
system(a.c_str());
return 0;
}

void f(int n, vector <string> v)
{
vector <Process> p(n);
for(int i=0; i<n;++i)
{
p[i].run(v[i]);
}
return;
}

Всё работает
Но значительную часть этого  взял из методички - вроде понимаю что здесь происходит - но препод придирается
Не могли бы рассписать - что здесь к чему?
Особенно интересуют это :

Код: (C++)

class Process
{
public:
    Process();
    ~Process() {}
    operator bool() const;
    pid_t id() const;
    bool run(std::string a);
   
protected:
    int action(std::string a);
private:
    Process(pid_t id);
private:
    pid_t pid;
};
inline Process::operator bool() const
{
    return pid != 0;
}
inline pid_t Process::id() const
{
    return pid;
}
void f(int n, std::vector <std::string> v);
#endif

и

Код: (C++)

Process::Process()
        : pid(0)
{
}
Process::Process(pid_t id)
        : pid(id)
{
}
..................
void f(int n, vector <string> v)
{
vector <Process> p(n);
for(int i=0; i<n;++i)
{
p[i].run(v[i]);
}

[Dimka]

А к чему именно препод "придирается", и что тебе нужно объяснить (код тут самый наиочевиднейший)?

[ZWYHB]

ну скажем - для чего вот это :

Код:

Process::Process()
        : pid(0)
{
}
Process::Process(pid_t id)
        : pid(id)
{
}

и
vector <Process> p(n) - это массив векторов
p[i].run(v[i]); -?

[Вад]

это не массив векторов, а массив(вектор) процессов. И вызывается метод run для каждого элемента этого вектора, при этом i-тый процесс получает v[i] в качестве аргумента метода run.

ну скажем - для чего вот это :

Код:

Process::Process()
        : pid(0)
{
}
Process::Process(pid_t id)
        : pid(id)
{
}

а это два конструктора класса Process: конструктор по умолчанию и конструктор с параметром pid - нужны для создания экземпляра Process, выполняют инициализацию полей объекта.

[Dimka]

Интересно, как можно написать лабу на языке программирования "продолжение которого сам не знаешь" (с) х/ф "Кин-дза-дза"...

В общем, проблема не в процессах и не в лабе, а в незнании языка программирования.

[ZWYHB]

В общем, проблема не в процессах и не в лабе, а в незнании языка программирования.

Я только учусь!!

 Вад, ты не мог бы поподробнее написать про вектор процессов?

[Вад]

А что про него подробнее писать? Контейнер из стандартной библиотеки STL, замена обычным динамическим массивам. Построен в виде шаблона - то есть, позволяет адаптировать шаблонную реализацию для хранения элементов любого типа.

Вообще, это настолько естественная часть C++ сейчас, что даже удивительно, почему в методичке он у тебя есть, и при этом требуется объяснять, что это.

[Sla]

Тогда нужно объяснить, что такое конструктор. Что такое вектор и т.д.

[ZWYHB]

Ну вот накинулись!
Знаю я - что такое конструкторы, деструкторы и прочее.
Контейнеры подзабылись.

[Sla]

Накинулись? Пытаемся понять, что ты знаешь.

[lapulya]

Что то я не пойму, в чем собственно вопрос...

[ZWYHB]

Вот ещё маленький вопрос

Код:

 case 0:
        pid = getpid();
        exit(action(a));
    }
    return true;

Вот здесь - я правильно понимаю - в случае если мы в потомке - получаем идентификатор процесса, выполняем action и выходим?
pid = getpid(); - что конкретно означает?

[Вад]

У тебя Process хранит pid для запускаемого процесса. fork возвращает 0 в экземпляр дочернего процесса. И функция getpid вызывается, чтобы узнать pid текущего процесса.

[ZWYHB]

спасибо!

[ZWYHB]

А для чего надо объявлять эти методы константными:

Код:

inline Process::operator bool() const
{
    return pid != 0;
}
inline pid_t Process::id() const
{
    return pid;
}

Код:

int Process::action(string a)
{
system(a.c_str());
return 0;
}

  - здесь (a.c_str()) - преобразование из char в string ?

[Алексей++]

ZWYHB, в константных функциях компилятор не даст явно поменять значение членов класса или вызвать неконстантные методы класса

[lapulya]

Алексей1153++, если только мемберы не были объявлены как мулаблы

[Алексей++]

lapulya, чиво ругаесси ? )) Ну, если вот так, как ты обозвал, объявили, - сами виноваты. Согласись? )

[lapulya]

Алексей1153++, мутаблы* (описался)
Это я так... для полит корректности... (сам ни разу мутабл не использовал, насколько я помню)

ЗЫ
меня как-то по молодости на одном собеседовании спросили, что это, а я не знал. После пришел домой прочитал, нах это надо и понял, что это нах не надо )))) но запомнил так, что уже в жизни не забуду.

[Антон (LogRus)]

lapulya, вообще бывает нужно.
Например, захват мьютекса в константном методе, мьютекс должен быть mutable

[lapulya]

ну не делай функцию мембер константной )))... это если так писать то все функции, которые не setBUBU (т.е. призваны явно менять мембер), можно константными сделать, но это я так... бурчу...

я не говорю, что это абсолютно не нужно, но... странно мне это... короче ни разу не юзал (ну или почти не юзал... может где разочек и затерлось ) )))

[Алексей++]

lapulya, вообще бывает нужно.
Например, захват мьютекса в константном методе, мьютекс должен быть mutable

вах... вещь ) А я всегда приводил указатель к неконстантному. ))) Теперь учту

[lapulya]

LogRus, да и потом, оберни мьютекс во что-нибудь (да так по хорошему и надо сделать) и никаких проблем...
типа так

Код:

class Mutex
{
public:
	void lock() {}
};

class Process
{
	Mutex * theMutex;
public:
	Process()	{theMutex = new Mutex();}
	~Process()	{delete theMutex;}
	void work() const { theMutex->lock(); }
};

void main(void)  
{
	Process process;
	const Process * p = &process;
	process.work();
	p->work();
}

не помню насколько это удовлетворяет стандарту, но компилится если удовлетворяет, то так и надо писать, ну точнее я бы так написал...

[Антон (LogRus)]

Код:

const Process * p = &process;

это запрет делать

Код:

++p;

а не запрет на модификацию памяти по адресу.

[lapulya]

LogRus,
я ж константную функцию вызываю (в ней лочу мьютекс)!!! а это так... для порядку... хотя, для полной верности (я тут чуть налажал, но суть дела не меняется) можно и так написать

const Process * const p = &process;

тут ваще все константное (хотя вызова константной функции было уже достаточно, чтобы продемонстрировать мою мысль).

[Антон (LogRus)]

lapulya, в любом случае в языке есть нормальное средство, для реализации требуемого функционала и строить прослойки я смысла не вижу.

[lapulya]

в принципе согласен, но тут бы я точно обернул (не равен час на линукс переходить надо будет). Кстати, если а можно ли пример где это надо, потому как если юзается мьютекс, так он видимо используется за пределами класса (иначе это была бы критическая секция, хотя и секцию можно передать), причем как часть реализации а не логики... хммм я бы оборачивал подобные куски (речь не о синхронизации потоков, а вообще)... но для наглядности лучше приведи пример или опиши пример... хочется поразмыслить...

[Антон (LogRus)]

lapulya, ничего не понял или запятых не хватает или слов
что касается линукс, то для меня ответ очевиден - возьму чужой велосипед (я так всегда делаю, зачем мне нужны проблемы с отладкой, тестированием и т.д. нового класса)
поэтому мой выбор TBB от Intel или boost
в TBB кстати отличные мьютексы (спиновые, RW, спиновые RW, обычные + всякие другие полезности) и самые лучше имхо это спиновые, а уж спиновые мьютексы я точно сам писать не буду.

[lapulya]

LogRus, Знать на верняка будет ли переход на линукс ты сейчас вряд ли сможешь, как мне кажется...

Вообще сквозное (т.е. повсеместное, где захотел, там и использовал) использование апи платформы по всему коду без оберток, как мне кажется, не лучшее решение. Слишком тяжко будет потом менять апи (ну если придется). Поэтому я сторонник использовать в логике только свои объекты, все чужое имхо, лучше обернуть. Как оборачивать рояля не играет boost, интел или своя обертка разницы нет (лишь бы код был открыт, в противном случае мы просто одно апи на другое меняем).

В предыдущем письме я попросил привести чуть более детальный пример необходимости использование mutable, чтобы подумать настолько ли оно надо

[Антон (LogRus)]

счётчик обращений, например, да мало ли

что касается API - не разу не пользовался