No se porque el compilador de c++ no se quejó por tu código.
Problema 1:
El problema es que el compilador toma esta linea de código como prototipos de función(o eso creo):
int pthread_create(pthread_t* proceso1, const pthread_attr_t *_NULL,void *(*suma)(void *),void *&a);// Crea el hilo, pone parámetros por defecto, le muestra su rutina y el valor con el que va a ejecutar la rutina
int pthread_create(pthread_t* proceso2, const pthread_attr_t *_NULL,void *(*resta)(void *),void *&a);
Por lo que la función `pthread_create` nunca se realiza. Y cuando quieres liberar la memoria usada con `pthread_join` el compilador libera un hilo que no existe(recuerda que usar el valor de una variable sin inicializar trae valores impredecibles: `int a;cout
El otro problema que te saldrá después es con los bucles for: `for(int i=0;i=A;i++);`. lo que haces ahí es crear un bucle infinito, que siempre pone a `i` con el valor de `A` con cada iteración: `i=A//cambiar el valor de i a A`, `i==A//Compara el valor de i con A`. Y tampoco es el tipo de comparación que quieres, el que quieres es: `i
Solución 1:
Debes colocar los valores como se debe al llamar la función:
//int pthread_create(pthread_t *th, const pthread_attr_t *attr, void *(*func)(void *), void *arg);
pthread_create(&proceso1, NULL,suma,(void*)&a);
pthread_create(&proceso2, NULL,resta,(void*)&a);
Consejo:
Como dijo @Mateo puedes usar la misma función para los dos hilos, solo debes aplicar esta formula: `+x+=a+=1, +x-=a+=-1;. Solo debes pasar el argumento que cambie el numero a que se va a sumar: a+=num_2;//en el hilo 1 se pasa 1, en el hilo 2 se pasa -1.` . También le debes pasar como argumento A o B para el bucle:
int list[2][2]={
{
A,
1//+1+1: 2
},
{
B,
-1// recuerda que cuando se realiza esta suma: +2-1 es igual a 1. Porque +-==-. Esto nos permite usar la misma funcion para obtener los mismos resultados.
}
};
//No pasamos "a" porque es una variable globa, y se puede acceder sin problema desde la función.
pthread_create(&proceso1, NULL,suma,(void*)list[0]);// Crea el hilo, pone parámetros por defecto, le muestra su rutina y el valor con el que va a ejecutar la rutina
pthread_create(&proceso2, NULL,suma,(void*)list[1]);
Y la función suma quedaría:
void* suma(void* data)
{
const int COUNT=(int)((int*)data)[0];//Recuerda debemos convertir de void* a int* porque le necesitamos acceder a los elemento del array que pasamos(int*).
const int NUM_2=(int)((int*)data)[1];
for(int i=0;i<COUNT;i++)//repare.
{
a+=NUM_2;
sleep(2);// 5 me parece muy lento cuando quiero ver los resultado rápido.
cout<<"El valor de a es:"<< a <<endl;
}
return data;
}
Codigo completo:
#include <stdlib.h>
#include <iostream>
#include<unistd.h>
#include <pthread.h>
using namespace std;
int a=0;
void* suma(void* data)
{
const int COUNT=(int)((int*)data)[0];//Recuerda debemos convertir de void* a int* porque le necesitamos acceder a los elemento del array que pasamos(int*).
const int NUM_2=(int)((int*)data)[1];
for(int i=0;i<COUNT;i++)//repare.
{
a+=NUM_2;
sleep(2);
cout<<"El valor de a es:"<< a <<endl;
}
return data;
}
int main(void)
{
pthread_attr_t atributos;
int A=0,B=0;
cout<<"Valor de A a sumar:"<<endl;
cin>>A;
cout<<"Valor de B a restar:"<<endl;
cin>>B;
pthread_t proceso1; //Identificador de los hilos
pthread_t proceso2;
int list[2][2]={
{
A,
1//+1+1: 2
},
{
B,
-1// recuerda que cuando se realiza esta suma: +2-1 es igual a 1. Porque +-==-. Esto nos permite usar la misma función para obtener los mismos resultados.
}
};
pthread_create(&proceso1, NULL,suma,(void*)list[0]);// Crea el hilo, pone parametros por defecto, le muestra su rutina y el valor con el que va a ejecutar la rutina
pthread_create(&proceso2, NULL,suma,(void*)list[1]);
pthread_join(proceso1,NULL);
pthread_join(proceso2,NULL);
return 0;
}
pthread_create()
? Porque en ambas líneas tienes una declaración y parece que no haces nada con ello. Por otra parte, no es necesario que declares dos funciones iguales. Solo estás cambiando los nombres de algunos parámetros.