[问题]一个线程初级问题

[Roots]

在C程序中第一次使两个线程对同一个变量操作，看下这样用是否有问题，会不会产生混乱：

int g_num = 0;

线程1:
{
while(1)
{
if(g_num <= 15)
{
g_num++;
}
sleep(1);
}
}

主程序:
{
……
……
if(g_num >= 15)
{
g_num = 0;
}
}

[BigSnake.NET]

会的

[Roots]

详细点撒，
g_num起一个标志的作用，线程1计数，主进程来判断处理

[kofshower]

怎么能这么写多线程.线程1和主线程的操作是atom操作么?

[Roots]

不是很懂俄，
线程有线程1其实主要是敢干的事，定时每秒一次，因为主程序要用到一个时间判断，为方便所以把这个加线程里了

[BigSnake.NET]

不是很懂俄，
线程有线程1其实主要是敢干的事，定时每秒一次，因为主程序要用到一个时间判断，为方便所以把这个加线程里了

多线程操作共享对象很易出问题的
例如

代码： 全选

if(g_num <= 15)
{
# 另一个线程在这时 g_num++ ...
g_num++;
}

另外 g_num++ 是不是原子性也是个问题..

[kofshower]

下面给你贴的是Netscape的代码

代码： 全选

165 PRInt32
166 _PR_MD_ATOMIC_INCREMENT(PRInt32 *val)
167 {
168     PRInt32 rv;
169     PRInt32 idx = _PR_HASH_FOR_LOCK(val);
170 
171     pthread_mutex_lock(&atomic_locks[idx]);
172     rv = ++(*val);
173 #ifdef DEBUG
174     hash_lock_counts[idx]++;
175 #endif
176     pthread_mutex_unlock(&atomic_locks[idx]);
177     return rv;
178 }

为什么人家不和你一样直接
xxx++了事.
你贴的代码问题很多.
------------------------------------------------------------------
当然人人家的代码是为了跨平台,这样的实现不是很高效

[Roots]

看来还是要锁一下，我就怕出现缩死现象

我是怕内存出错，其实那个时间标志的准确性不是很重要，只是根据情况估测了一个值，主程序的清0操作在前一次还是后一次，不是很重要

[kofshower]

小心循环锁

[poet]

你这个地方，因为只有一个线程进行写操作，不锁是没有问题的。虽然++不是原子操作但对此没有影响。

不过因为你一边用 >= 15， 另一边用 <= 15，导致了实际运行结果的不确定（15还是16次，都有可能）。也许你可以无所谓一个tick的sleep，但是这种散漫的编程习惯似乎很有问题。

[shellex]

锁.....

[kofshower]

你这个地方，因为只有一个线程进行写操作，不锁是没有问题的。虽然++不是原子操作但对此没有影响。

不过因为你一边用 >= 15， 另一边用 <= 15，导致了实际运行结果的不确定（15还是16次，都有可能）。也许你可以无所谓一个tick的sleep，但是这种散漫的编程习惯似乎很有问题。

我也认为这个编码风格不行

[Roots]

“一边用 >= 15， 另一边用 <= 15”，这个其实是有意为之，保险一下。

只有一个线程进行写操作?
g_num++;
g_num = 0;
哪个不算？