#include <stdio.h>
int main(void)
{
      char ch;
      int n = 0;
      printf ("Input some characters ending with %: ");
      scanf_s ("%c", &ch);
      while (ch != '%')
      {
          if (ch >= 'a' && ch <= 'z' || ch >= 'A' && ch <= 'Z')
              n++;
          scanf_s ("%c", &ch);
      }
      printf ("n = %d\n", n);
      return 0;
}

是程序问题还是编译器有问题？？？

简单说说：
程序初步看起来没有什么故障

f (ch >= 'a' && ch <= 'z' || ch >= 'A' && ch <= 'Z')
这句优先级有问题

没啥问题呀  你的字符串结束标志被人为的设置成了 %

问题可能是出在你用scanf_s函数上，这个函数是vc里加的，你是用的方式对吗，应该还有一个表示读入字符个数的参数，是不是应该这样写scanf_s("%c",&ch,1),此处用getchar（）不挺好的吗，(ch >= 'a' && ch <= 'z' || ch >= 'A' && ch <= 'Z')这个建议你加上括号(（ch >= 'a' && ch <= 'z'） || （ch >= 'A' && ch <= 'Z')），这样容易阅读（编程风格）

那个%是我改的，我在试试我哪错了，谢了。。。

我试出来了，不是这个优先级的问题...谢了

恩，把scanf_S改成getchar()就能计算出结果了。难道是VC++2008输出字符只能用getchar()。而不能用scanf_s,谢了，好建议

建议少用vc里自己添加的函数，用编译器自带的东西，那你的代码对编译器就有依赖性，换成别的编译器就不能用了，基本的输入输出标准库里都有。