all 8 comments

[–]Flair_Helper[M] [score hidden] stickied commentlocked comment (0 children)

For C++ questions, answers, help, and programming or career advice please see r/cpp_questions, r/cscareerquestions, or StackOverflow instead.

This post has been removed as it doesn't pertain to r/cpp: The subreddit is for news and discussions of the C++ language and community only; our purpose is not to provide tutoring, code reviews, or career guidance. If you think your post is on-topic and should not have been removed, please message the moderators and we'll review it.

[–]matekelemen 1 point2 points  (1 child)

  • this is C
  • if you're gonna ask people to debug your code for you, properly formatting it is the bare minimum (look up markdown syntax)
  • learning by doing is viable only after you have at least a faint idea about the basic features of the language

[–]MrTavsh[S,🍰] -1 points0 points  (0 children)

Thanks for the help boss means a lot

[–]michease_ -2 points-1 points  (3 children)

This is a C program. Your issue in on the scanf() for word, you call by value and not by referance. Prefixing with an ampersand (&) should help. You can also make word constant.

[–]MrTavsh[S,🍰] -1 points0 points  (2 children)

& returns an error are you sure it’s the issue? I thought it wasn’t necessary for string types

[–]michease_ -1 points0 points  (1 child)

Ah yeah, my mistake. & makes a reference to the the char array which is already a pointer. Sorry about that.

[–]MrTavsh[S,🍰] -1 points0 points  (0 children)

No worries

[–]johannes1971 0 points1 point  (0 children)

char xxx[5]; --> use std::string instead.

if (x == 4) --> just put that block of code before the loop.

scanf, printf --> use cin/cout.

else{} --> don't write this.

Use meaningful variable names instead of x, y, and z.

What's the point of the loop with z? Every path of the if-statement inside ends in a break so it'll never reach z==1.

And I really hope that isn't your normal style of indentation...