Разбить программу на потоки

@Vexel · Регистрация: 18.10.2009

Author24 — интернет-сервис помощи студентам

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

C++

#include <iostream>
#include <windows.h>
#include <pthread>
void * func_MyThread(void * args)
{
int S=0; //
int i;
int j;
int k; 
for (i=1; i <=8; i++)//
{
S +=i+1; //
}
int main (void)
{
pthread_t MyThread;
pthread_create(&MyThread,NULL,func_MyThread,NULL);
 
for (j=4; j <=12; j++)//
{
S+=j; //
}
for (k=5; k<=20; k++)
{
S += k*(2*k-1); 
}
pthread_join(MyThread,NULL);
std::cout <<"S= \t" <<S; //
return 0;// 
}

@niXman · 09.09.2010, 02:22

я так понял, программа под вендаус? а почему используешь pthread?

Добавлено через 33 секунды
отформатируй код. глаза сломать можно.

@Vexel · 09.09.2010, 12:17 **[ТС]**

Сообщение от niXman

я так понял, программа под вендаус? а почему используешь pthread?

Добавлено через 33 секунды
отформатируй код. глаза сломать можно.

C++

#include <iostream>
 
#include <windows.h>
 
#include <pthread>
 
void * func_MyThread(void * args)
 
{
 
    int S=0; //
 
int i;
  
     int j;
 
   int k; 
 
for (i=1; i <=8; i++)//
 
    {
 
   S +=i+1; //
 
}
 
   int main (void)
 
{
 
       pthread_t MyThread;
 
pthread_create(&MyThread,NULL,func_MyThread,NULL);
 
    for (j=4; j <=12; j++)//
 
{
 
S+=j; //
 
}
 
    for (k=5; k<=20; k++)
 
{
 
       S += k*(2*k-1); 
 
}
 
       pthread_join(MyThread,NULL);
 
 
std::cout <<"S= \t" <<S; //
 
       return 0;// 
 
}

Программа под Windows! Pthread использую поскольку преподаватель сказал, что в данной программе для создания потоков нужно использовать именно её!

@Andrew_Lvov · 09.09.2010, 13:01

Сообщение от niXman

я так понял, программа под вендаус? а почему используешь pthread?

Реализация pthread есть и под windows. А собсно почему бы и нет ?

Добавлено через 6 минут
Vexel, Вы не закрыли ф-цию func_myThread. Обьявления функции (main) внутри другой запрещены.

@Vexel · 09.09.2010, 18:09 **[ТС]**

Сообщение от Andrew_Lvov

Добавлено через 6 минут
Vexel, Вы не закрыли ф-цию func_myThread. Обьявления функции (main) внутри другой запрещены.

Т.е строки pthread_t MyThread; pthread_create(&MyThread,NULL,func_MyThread,NULL); и pthread_join(MyThread,NULL); должны до мэина стоять ?

@fasked · 09.09.2010, 18:17

Сообщение от Vexel

должны до мэина стоять ?

Просто Вы пропустили одну фигурную скобку, закрывающую функцию.

@Vexel · 09.09.2010, 18:28 **[ТС]**

Сообщение от fasked

Просто Вы пропустили одну фигурную скобку, закрывающую функцию.

А не подскажете в какой строчке ?

@niXman · 09.09.2010, 18:32

в 26ой

Добавлено через 41 секунду

Сообщение от Andrew_Lvov

А собсно почему бы и нет ?

нелепо выглядит)

Добавлено через 1 минуту

Сообщение от Vexel

for (j=4; j <=12; j++)//

декларации j и k, в другой функции. как вы собрались их использовать?

@Vexel · 09.09.2010, 18:41 **[ТС]**

Сообщение от niXman

декларации j и k, в другой функции. как вы собрались их использовать?

Ну можно их объявить например так:

C++

pthread_t MyThread;
pthread_create(&MyThread,NULL,func_MyThread,NULL);
int j;
int k; 
int S=0; 
for (j=4; j <=12; j++)//

Когда компилировать начинаю выписывает:
[Linker error] undefined reference to `_imp__pthread_create'
[Linker error] undefined reference to `_imp__pthread_join'
C:\Program Files\Dev-Cpp\Makefile.win[Build Error] [Project1.exe] Error 1

@niXman · 09.09.2010, 18:43

либу прилинковать нужно

@Vexel · 09.09.2010, 18:58 **[ТС]**

Можете посмотреть, правильно ли я сделал ? Создал ли я три потока, которые общий результат выдают ? Программа дает верный результат!

C++

#include <iostream>
#include <windows.h>
#include <pthread.h>
void * func_MyThread(void * args)
{
int S=0; //
int i;
int j;
int k; 
}
int main (void)
{
pthread_t MyThread;
pthread_create(&MyThread,NULL,func_MyThread,NULL);
int i;
int j;
int k; 
int S=0;
for (i=1; i <=8; i++)//
{
S +=i+1; //
} 
for (j=4; j <=12; j++)//
{
S+=j; //
}
pthread_create(&MyThread,NULL,func_MyThread,NULL);
for (k=5; k<=20; k++)
{
S += k*(2*k-1); 
}
pthread_join(MyThread,NULL);
std::cout <<"S= \t" <<S; //
return 0;// 
}

@niXman · 09.09.2010, 19:02

все неправильно.
потоки пустые. они ничего не выполняют. и их не три.
удаляйте все, и начинайте заново.

_И_ОТФОРМАТИРУЙТЕ_КОД_

@Vexel · 09.09.2010, 19:19 **[ТС]**

Может так:

C++

#include <iostream>
#include <windows.h>
#include <pthread.h>
 
void * func_MyThread(void * args)
{
 
int S=0; //
 
   int i;
 
   int j;
 
int k;
 
pthread_t MyThread;
 
pthread_create(&MyThread,NULL,func_MyThread,NULL); 
 
for (i=1; i <=8; i++)//
 
pthread_join(MyThread,NULL);
 
{
 
    S +=i+1; //
}
 
   pthread_create(&MyThread,NULL,func_MyThread,NULL); 
 
 for (j=4; j <=12; j++)//
 
pthread_join(MyThread,NULL);
 
{
 
S+=j; //
 
}
 
pthread_create(&MyThread,NULL,func_MyThread,NULL);
 
for (k=5; k<=20; k++)
 
pthread_join(MyThread,NULL);
 
{
 
    S += k*(2*k-1); 
}
 
{
 
std::cout <<"S= \t" <<S; //
 
return 0;// 
}

@niXman · 09.09.2010, 19:21

не нужно в потоке, создавать поток. это называется рекурсия.
дальше вообще полный бред... вы вообще понимаете что пишете?

@Vexel · 09.09.2010, 19:24 **[ТС]**

Сообщение от niXman

не нужно в потоке, создавать поток. это называется рекурсия.
дальше вообще полный бред... вы вообще понимаете что пишете?

Тогда я не знаю как реализовать данную программу с помощью потоков. Идей вообще нету никаких.

@niXman · 09.09.2010, 19:26

Сообщение от Vexel

Тогда я не знаю как реализовать данную программу с помощью потоков. Идей вообще нету никаких.

ну это нормально...

а теперь, не спеша, внятно, объясни - что программа должна делать?

@MikeSoft · 09.09.2010, 19:30

Vexel, возможно, вам поможет мой простенький пример:

C++

//---------------------------------------------------------------------------
#include <iostream>
#include <windows.h>
//---------------------------------------------------------------------------
using namespace std;
//---------------------------------------------------------------------------
struct Param {
  int Sum;
  int HowMuch;
  bool Ready;
} *SumParam;
//---------------------------------------------------------------------------
DWORD WINAPI SumThread(LPVOID lParam)
{
  Param *SumParamThr = (Param *)lParam;
 
  while (SumParamThr->Sum < SumParamThr->HowMuch) {
    SumParamThr->Sum += 10;
  }
 
  std::cout << SumParamThr->Sum << std::endl;
  SumParamThr->Ready = true;
 
  return 0;
}
//---------------------------------------------------------------------------
int main()
{
  int Count = 3;
  SumParam = new Param[Count];
  for (int i = 0; i < Count; i++) {
    SumParam[i].Sum = 0;
    SumParam[i].HowMuch = 50*(i+1);
    SumParam[i].Ready = false;
 
    DWORD dwThreadId;
    HANDLE hThread = CreateThread(0,0,SumThread,&SumParam[i],0,&dwThreadId);
  }
 
  while (true) {
    bool Ready = true;
    for (int i = 0; i < Count; i++) {
      if (SumParam[i].Ready == false) {
        Ready = false;
        break;
      }
    }
 
    if (Ready) {
      delete []SumParam;
      system("pause");
      break;
    }
  }
 
  return 0;
}
//---------------------------------------------------------------------------

В качестве параметра можете передавать что угодно

@Vexel · 09.09.2010, 19:32 **[ТС]**

В общем программа должна считать сумму. Всего в программе три суммы, для каждой суммы должен быть свой поток! Т.е. сумма считаться не последовательно а параллельно должна и в конце все результаты трех сумм должны быть объеденены в один! В итоге должно получиться число 5596.

@niXman · 09.09.2010, 19:34

MikeSoft, полагаю, вы его только больше запутаете.

@MikeSoft · 09.09.2010, 19:35

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

@niXman 3211 / 1459 / 74 Регистрация: 09.08.2009 Сообщений: 3,441 Записей в блоге: 2
	09.09.2010, 19:21	14
	не нужно в потоке, создавать поток. это называется рекурсия. дальше вообще полный бред... вы вообще понимаете что пишете? 1

@Vexel 12 / 12 / 2 Регистрация: 18.10.2009 Сообщений: 158
	09.09.2010, 19:24 [ТС]	15
	Сообщение от niXman не нужно в потоке, создавать поток. это называется рекурсия. дальше вообще полный бред... вы вообще понимаете что пишете? Тогда я не знаю как реализовать данную программу с помощью потоков. Идей вообще нету никаких. 0

@niXman 3211 / 1459 / 74 Регистрация: 09.08.2009 Сообщений: 3,441 Записей в блоге: 2
	09.09.2010, 19:26	16
	Сообщение от Vexel Тогда я не знаю как реализовать данную программу с помощью потоков. Идей вообще нету никаких. ну это нормально... а теперь, не спеша, внятно, объясни - что программа должна делать? 1

@niXman 3211 / 1459 / 74 Регистрация: 09.08.2009 Сообщений: 3,441 Записей в блоге: 2
	09.09.2010, 02:22	2
	я так понял, программа под вендаус? а почему используешь pthread? Добавлено через 33 секунды отформатируй код. глаза сломать можно. 1

@Andrew_Lvov 261 / 191 / 10 Регистрация: 19.08.2010 Сообщений: 760 Записей в блоге: 1
	09.09.2010, 13:01	4
	Сообщение от niXman я так понял, программа под вендаус? а почему используешь pthread? Реализация pthread есть и под windows. А собсно почему бы и нет ? Добавлено через 6 минут Vexel, Вы не закрыли ф-цию func_myThread. Обьявления функции (main) внутри другой запрещены. 0

@Vexel 12 / 12 / 2 Регистрация: 18.10.2009 Сообщений: 158
	09.09.2010, 18:09 [ТС]	5
	Сообщение от Andrew_Lvov Добавлено через 6 минут Vexel, Вы не закрыли ф-цию func_myThread. Обьявления функции (main) внутри другой запрещены. Т.е строки pthread_t MyThread; pthread_create(&MyThread,NULL,func_MyThread,NULL); и pthread_join(MyThread,NULL); должны до мэина стоять ? 0

@fasked 5043 / 2622 / 241 Регистрация: 07.10.2009 Сообщений: 4,310 Записей в блоге: 1
	09.09.2010, 18:17	6
	Сообщение от Vexel должны до мэина стоять ? Просто Вы пропустили одну фигурную скобку, закрывающую функцию. 1

@Vexel 12 / 12 / 2 Регистрация: 18.10.2009 Сообщений: 158
	09.09.2010, 18:28 [ТС]	7
	Сообщение от fasked Просто Вы пропустили одну фигурную скобку, закрывающую функцию. А не подскажете в какой строчке ? 0

@niXman 3211 / 1459 / 74 Регистрация: 09.08.2009 Сообщений: 3,441 Записей в блоге: 2
	09.09.2010, 18:32	8
	в 26ой Добавлено через 41 секунду Сообщение от Andrew_Lvov А собсно почему бы и нет ? нелепо выглядит) Добавлено через 1 минуту Сообщение от Vexel for (j=4; j <=12; j++)// декларации j и k, в другой функции. как вы собрались их использовать? 1

@niXman 3211 / 1459 / 74 Регистрация: 09.08.2009 Сообщений: 3,441 Записей в блоге: 2
	09.09.2010, 18:43	10
	либу прилинковать нужно 1

@niXman 3211 / 1459 / 74 Регистрация: 09.08.2009 Сообщений: 3,441 Записей в блоге: 2
	09.09.2010, 19:02	12
	все неправильно. потоки пустые. они ничего не выполняют. и их не три. удаляйте все, и начинайте заново. _И_ОТФОРМАТИРУЙТЕ_КОД_ 1

@Vexel 12 / 12 / 2 Регистрация: 18.10.2009 Сообщений: 158
	09.09.2010, 19:32 [ТС]	18
	В общем программа должна считать сумму. Всего в программе три суммы, для каждой суммы должен быть свой поток! Т.е. сумма считаться не последовательно а параллельно должна и в конце все результаты трех сумм должны быть объеденены в один! В итоге должно получиться число 5596. 0

@niXman 3211 / 1459 / 74 Регистрация: 09.08.2009 Сообщений: 3,441 Записей в блоге: 2
	09.09.2010, 19:34	19
	MikeSoft, полагаю, вы его только больше запутаете. 1

@MikeSoft 3953 / 1808 / 184 Регистрация: 21.11.2009 Сообщений: 2,540
	09.09.2010, 19:35	20
	niXman, полагаю, что там нет ничего сверхсложного (заметьте, я даже делал проверку на завершение вычислений не через события, а обычными булевыми переменными) ... к тому же, код ему прийдётся модифицировать. Думаю, всё сразу станет понятно. 0