Thread: Problem with writing

And what is price value? Because right now, your data runs contrary to your expectation. Look:

3 ASD 77 15000
4 BSAD 77 3000

Item ASD is bigger than 3600, not BSAD.

My own tinkering confirms this result for the goods file you gave.

Code:

#include <stdio.h>

int main()
{
    char itemDesc[32] = "";
    long price = 0;
    long enteredPrice = 0;
    int c;
    FILE *fp = NULL;
    
    if ((fp = fopen("goods.txt", "r")) == NULL)
    {
        fprintf(stderr, "Couldn't open goods.txt file!\n");
        return 1;
    }
    
    printf("Enter price: ");
    while (scanf("%ld", &enteredPrice) != 1) 
    {
        printf("Price not understood. Try entering a number again.\n");
        while ((c = getchar()) != '\n' && c != EOF) 
            /* nothing */;
    }
    
    /* Note that for scanf functions, a * means that the item is not assigned.
       http://linux.die.net/man/3/scanf 
    */
    while (fscanf(fp, "%*d %31s %*d %ld", itemDesc, &price) == 2)
    {
        if (price > enteredPrice) 
        {
            puts(itemDesc);
        }
    }
    fclose(fp);
    return 0;
}

/* my output

Enter price: 3600                                                                                                                                                       
ASD 
*/

That is about as simple as this task gets, though. You might need to do some edits to get it to behave exactly how you want.

Code:

 while(fscanf(fp, "%*d %31s %*d %ld", itemDesc, &price)== 2)

I don't understand what ==2 mean can you explain that part ?

EDIT!!!
J try this but didn't working and give me one warning:comparison between pointer and integer.

Code:

#include <stdio.h>int main()
{
    char st[3],nam[15],nmb[4],pr[5];
    int asd,i;
    char ch;
    int l=0;
    int c=0;
    FILE *fp,*ft;
    fp=fopen("items.txt","r");
    if(fp==NULL)
    {
        printf("Error!");
    }
    fscanf(fp,"%s %s %s %s\n",st,nam,nmb,pr);
    printf("Enter number:");
    scanf("%d",&asd);
    ft=fopen("enter.txt","w");
    while(fscanf(fp,"%s %s %s %s\n",st,nam,nmb,pr) == 2)
    {
        if(pr>asd)
        {
            fputs(nam, ft);
        }
    }
    fclose(fp);
    fclose(ft);
}

I don't understand what ==2 mean can you explain that part ?

In at least two places, I've linked the manual page. It explains everything about what I was doing. I hoped you would read it by now. It's big though... so I understand why you didn't.

Anyway, there are two parts of that document which really matter to the code.

Code:

    /* Note that for scanf functions, a * means that the item is not assigned.
       http://linux.die.net/man/3/scanf 
    */
    while (fscanf(fp, "%*d %31s %*d %ld", itemDesc, &price) == 2)

To understand that comment, there is a section that reads as follows (emphasis mine):

Each conversion specification in format begins with either the character '%' [...] followed by:
• An optional '*' assignment-suppression character: scanf() reads input as directed by the conversion specification, but discards the input. No corresponding pointer argument is required, and this specification is not included in the count of successful assignments returned by scanf().

Hopefully, that is mostly clear. Since there are %*d parts to the format string, those inputs are read, and then discarded. I then added two pointers to the argument list. This is where the == 2 comparison came from.

To explain that in detail, you need to read another part of the manual:

Return Value

These functions return the number of input items successfully matched and assigned, which can be fewer than provided for, or even zero in the event of an early matching failure.

You know what fscanf() is going to return by counting the arguments, excluding the FILE pointer and the format string. When you write a different fscanf() function call from mine, the return value you should expect to compare with will also be different.

The big idea is that fscanf() is going to scan the file, grabbing 2 specific columns from every line, until there are no more lines.

J try this but didn't working and give me one warning:comparison between pointer and integer.

I think you mean to use I to refer to yourself, but type J instead.

Anyway, it doesn't matter because your programs are still small, but when you post about errors, at least include the line number. It helps everyone know where to look for the problem.

Code:

main.c:22:14: warning: comparison between pointer and integer                                                                                                           

         if(pr>asd)                                                                                                                                                     

              ^

The error may even include code to explicitly point out the error. pr is a string, and asd is an integer. These two types cannot be compared in C.
You would need to, at the very least, call atoi() on pr again for the sake of that comparison. This goes back to my advice about using fscanf() earlier in the thread. You're using it to read strings when you want to compare integers. You can ask fscanf() to do the conversions on the input for you, and that makes writing the program a lot easier.