Question

The following code produces a “lvalue required as left operand of assignment”

if( c >= 'A' && c <= 'Z'  || c = " " || c = ",") {

I assume I’m writing this wrong, what is wrong? and how would I write it correctly?

Answer 1

You should use single quotes for chars and do double equals for equality (otherwise it changes the value of c)

if( c >= 'A' && c <= 'Z'  || c == ' ' || c == ',') {

Furthermore, you might consider something like this to make your boolean logic more clear:

if( (c >= 'A' && c <= 'Z')  || c == ' ' || c == ',') {

Although your boolean logic structure works equivalently (&& takes precedence over ||), things like this might trip you up in the future.