C / C++ / MFC

Nandu_77b wrote:

if ( (' ' != *from ) || ('\t' != *from))

SandipG wrote:

Nandu_77b wrote:

if ( (' ' != *from ) || ('\t' != *from))

Don't you think it should be &&

Can anyone confirm??

Nandu_77b wrote:

string removeSpaces( string & str )
{
std::string::iterator to = str.begin();
std::string::iterator from;

for( from = str.begin(); '\0' != *from; ++ from )
{
if ( (' ' != *from ) || ('\t' != *from))
*(to ++) = *from;
}
if( *to != '\0' )
str.erase(to, from);

*to = '\0';
return str;
}

string removeSpaces( string & str )
{
  std::string::iterator to = str.begin();
  std::string::iterator from;

  for( from = str.begin(); str.end() != from; ++from )
  {
    if ( (' ' != *from ) && ('\t' != *from))
      *(to++) = *from;
  }
  if( to != str.end() )
    str.erase(to, from);
}

std::string& removeSpaces( std::string& str )
{
	std::string::iterator to = str.begin();
	std::string::iterator from = to;

	// Loop until we reach the end of the string
	while( from != str.end() )
	{
		// Check if the current char is a space OR a tab
		if( (*from == ' ') || (*from == '\t') )
		{
			// The current char is a space OR a tab, skip it!
			++from;
		}
		else
		{
			// The current char is NOT a space or a tab, copy it!
			*to++ = *from++;
		}
	}
	
	// If the iterator has not reached the end of the string, delete the rest
	if( to != str.end() )
	{
		str.erase( to, from );
	}

	// Return a REFERENCE to the same string we got as formal parameter
	// instead of making a copy of it on the stack
	// There's no point in returning a copy since the function operates
	// directly on the provided string buffer
	return str;
}

VCProgrammer wrote:

how can i do it...