Skip to main content
CS50 Beta

You are not logged in. Your edit will be placed in a queue until it is peer reviewed.

We welcome edits that make the post easier to understand and more valuable for readers. Because community members review edits, please try to make the post substantially better than how you found it, for example, by fixing grammar or adding additional resources and hyperlinks.

Required fields*

3
  • Hey, so I've taken upon the considerations you've given me & it's made the code much more sleek and got rid of the time taken in the size function! I'm not sure with what to do with the EOF loop though, i've gotten rid of the feof and the code compiles fine & has the same output as before (644 words misspelled etc.). I thought that this would work, but i'm still getting the same error as before i.e. :( handles most basic words properly expected "MISSPELLED WORD...", not "" Any more that could be done? Thank you heaps for the help so far! Commented Aug 8, 2017 at 10:57
  • lmao, so it's cos when i was opening the file i was opening 'DICTIONARY' which i defined to be "dictionaries/large", changing it to 'dictionary' fixed it up & now it passes all the checks - thanks for the help ! Commented Aug 8, 2017 at 11:28
  • The use of fscanf seems correct, I must have confused that with another function. I compared the output of your programme, and it seems the same as mine, as long as I use the large dict. There's no use for DICTIONARYSMALL and DICTIONARY macros, remove them, and use dictionary in load (if you count the words on loading, there's no need for the file name in size). valgrind seems happy with my changes. Commented Aug 8, 2017 at 11:41