#include <stdio.h>

main()
{
    FILE *fp;
    char buff[256] ;

    fp = fopen("text.txt","r");
    if (fp == NULL) {
        printf("file open error\n");
        exit(1);
    }

    while(fgets(buff,sizeof buff,fp)!=NULL) {
        printf("[%s]\n",buff);
    }

    if(0 != fclose(fp)) {
        printf("file close error\n");
        exit(2);
    }
}

if (ferror(fp)) {
    puts("Read Error !!");
    exit(3);
}
if (feof(fp)) {
    break;
}

#include <stdio.h>
#include <stdlib.h>

int main(void)
{
    FILE *fp;  char buff[256];

    fp = fopen("text.txt","r");
    if (fp == NULL) puts("file open error"), exit(1);

    while (fgets(buff, sizeof buff, fp) != NULL)
        printf("> %s", buff);

    if (ferror(fp)) puts("file read error"), exit(3);
    if (fclose(fp) != 0) puts("file close error"), exit(2);
    return 0;
}

while(fgets(buff,sizeof buff,fp)!=NULL) {
    if (feof(fp)) {
    break;
}

while(!feof(fp)){
    fgets(buff,sizeof buff,fp);
    if(!ferror(fp)) break;
}

> while(!feof(fp)){
>     fgets(buff,sizeof buff,fp);
>     if(!ferror(fp)) break;
> }

読み込んだ buff の処理をどこに書くつもりですか？
feof(fp) は fgets() の実行後でなければ意味がありません。

fgets(buff,sizeof buff,fp);
while(!ferror(fp) && !feof(fp)){
    printf("%s\n",buff);　　//処理
    fgets(buff,sizeof buff,fp);
}

> fgets(buff,sizeof buff,fp);
> while(!ferror(fp) && !feof(fp)){
>     printf("%s\n",buff);　　//処理
>     fgets(buff,sizeof buff,fp);
> }

次のプログラムとどこが違うんでしょうか？

    while (fgets(buff, sizeof buff, fp) != NULL)
        fputs(buff, stdout);


元の質問は、fgets() が NULL を返さなかった場合、EOF 以外に error の
可能性があるのに、なぜそれをチェックしないのか、というものだったはず
です。ferror(fp) と feof(fp) を同じように扱っては意味がありません。

私は、while ループを抜けてから判断すればよい。また、ferror() のチェック
が省略されるのには、それなりの意味があるという回答をしています。

> while(fgets(buff,sizeof buff,fp)!=NULL) {
>     if (feof(fp)) {
>     break;
> }

無意味な if文です。
fgets() が NULL のとき、feof(fp) は常にゼロですから。
それから、{ と } の対応が取れていません。