有关位操作的C语言编程问题

Question

题目是对16种不同项目投票，分别用16位中的1个位表示，编写函数，头部如下：
short majority(short a,short b,short c)
{
... ...
这个函数接受3个人的输入，分别存于a,b,c中，函数返回的整数的各个位表示投票结果，如果某个项目3个人中有2个人以上同意，对应位置为1，否则为0
下面是我编写的程序：

当运行程序时输入 15 11 4 正确的结果应该为15， 我的运行结果是-2，不知道哪出问题了，求大侠解答...

Answer 1

#include <limits.h>
short majority(short,short,short);
int main(void)
{
short a,b,c,d;
scanf("%hd",&a);//*** short int格式化输入输出 d 加前缀 h 否则，32位以上系统中有越界行为
scanf("%hd",&b);
scanf("%hd",&c);
d=majority(a,b,c);
printf("%hd\n",d);
return 0;
}
short majority(short a,short b,short c)
{
int i;
short p,q,v,r = 0;
short n = sizeof(short)*CHAR_BIT;
short mask = 1<<(n-1); //*** 减号的优先级低于移位操作符，要用括号，才得到得掩码0x8000
for(i=1;i<=n;i++)
{
r<<=1; //***** <<<< 放到这里 关键所在
p=((a&mask)==0 ? 0 : 1 ); //*** 不要引号 要0和1 不是'0'和'1'
a<<=1;
q=((b&mask)==0 ? 0 : 1 );
b<<=1;
v=((c&mask)==0 ? 0 : 1 );
c<<=1;
r|=((p+q+v)>=2 ? 1 : 0 );
//r<<=1; //***** <<<放在这里是算法错误 放到循环开始处，最初的0多移一回没关系，当前结果不应当位移，只对上一循环结果移位!!
}
return r;
}