#include<stdio.h>
void main()
{
    char j;
    scanf("%d",&j);
    while(j!='\n')
    {
        if('a'<=j&&j<='w'||'A'<=j&&j<='W')
        j=j+3;
        else if('x'<=j&&j<='z'||'X'<=j&&j<='Z')
        j=(j+3)-26;
        printf("%c\n",j);
    }

}

scanf("%d",&j);改成scanf("%c",&j);
while(j!='\n')去掉  j 一直都不等于‘\n'

你想得到什么啊

你的代码想做出什么效果？？

真的，错了不少地方。
scanf("%d",&j);
%d是对整数型变量使用的，你定义的是char j；是一个字符，要用%c
还有怎么有输入就最好使用printf函数在屏幕上显示提示下
if('a'<=j&&j<='w'||'A'<=j&&j<='W')
应该写为if（('a'<=j&&j<='w'）||（'A'<=j&&j<='W')），类推都改了
还有在你的嵌套中
while(j!='\n')
    {
j!='\n'是要满足条件再进入循环的，不是否。去掉“！”
在你的这句else if('x'<=j&&j<='z'||'X'<=j&&j<='Z')中去掉else，直接用if，不然又else if就得使用else。可以n个if并列使用的，不必非要用if……else if
最后你的printf输出杉树语句应该提出while语句循环中，不然不当输入回车键后是不进入循环了，可也没得结果显示啊。

还有一个建议：定义char ch比char j更有利于读写。j很让人混淆的