If statement accepting a through z only... need help!

I need help with this code I need my if statement to accept a through z only and if a number or something else is entered it would fall into the else statement. When I entered a word if goes into the else I do not know what it is doing this.

Here is what I have for this file:
#include <iostream>
#include <string>
#include <fstream>
#include "game.h"
 
#define MaxWordSize 10
#define MaxWords 50
typedef char String[MaxWordSize];
String Words[MaxWords - 1];
int Count;
char Word[MaxWordSize];
int Size;
int Loop;
Game myGame;
 
using namespace std;
 
void Game::addWord()
{
    myGame.loadFile();
    ofstream Datfile("words.txt");
try
{
    cout << "\tEnter a new word, keep it below " << MaxWordSize << " characters: ";
    cin >> Word;
	if(Word == "ABCDEFGHIJKLMNOPQRSTUVWXYZ")
	{
    Size = strlen(Word);
 
    strcpy_s(Words[Count++],MaxWordSize,Word);
 
    for (int i = 0; i < Count; ++i) Datfile << Words[i] << endl;
	}
	else
	{
		throw 1;
	}
}
	catch(int e)
	{
		cout << "Error...\n";
	}
}
void Game::deleteWord()
{
	cout << "you have chosen to delete a word...\n";
}
 
void Game::loadFile()
{
	char C;
	ifstream Datfile;
 
	Count = 0;
 
	Datfile.open("words.txt");
 
	while((C=Datfile.peek()) !=EOF)
	{
		Datfile >> Words[Count++];
 
		if(Count > MaxWords - 1)
		{
			cout << "\nToo many words in the file, stopping with " << MaxWords << " words." << endl;
			Count = MaxWords;
			break;
		}
	}
 
	Count--;
 
	Datfile.close();
}

Open in new window

jschmuffAsked:
Who is Participating?

Improve company productivity with a Business Account.Sign Up

x
 
Jaime OlivaresConnect With a Mentor Software ArchitectCommented:
>>      if(Word == "ABCDEFGHIJKLMNOPQRSTUVWXYZ")

this won't work at all, you can design your own function, so you can use  like:

      if(validateWord(Word))

Here is the validating funcion:


// this will require: #include <stdio.h>
 
bool validateWord(char *word)
{
       for (int i=0; word[i]; i++)
            if (!isletter(word[i])
                  return false;
       return true;
}

Open in new window

0
 
jschmuffAuthor Commented:
what does that have to do with taking the input only a through z?
0
 
Jaime OlivaresSoftware ArchitectCommented:
it validates if all characters in Word are A though Z and returns you a boolean value.
your code:
if(Word == "ABCDEFGHIJKLMNOPQRSTUVWXYZ")
doesn't work as you expected and always return false, so exception will never occur.
0
What Kind of Coding Program is Right for You?

There are many ways to learn to code these days. From coding bootcamps like Flatiron School to online courses to totally free beginner resources. The best way to learn to code depends on many factors, but the most important one is you. See what course is best for you.

 
Jaime OlivaresSoftware ArchitectCommented:
sorry, in this case, exception will always occur.
0
 
jschmuffAuthor Commented:
so that is something that would be done before the if statement since if(validateWord(Word)) is that correct.
so to actually do a - z I have to use a for loop? What is the isletter in this example you have given me? cause it is something that will need to be declared.
0
 
jschmuffAuthor Commented:
Is there another way this can be done like if(number is input) { throw 1; } else then the code to add and save word to file.. catch blah blah blah?
0
 
Jaime OlivaresSoftware ArchitectCommented:
>>so that is something that would be done before the if statement since if(validateWord(Word)) is that correct
validateWord is a separate function.

>>so to actually do a - z I have to use a for loop?
yes, you have to use a loop

>>What is the isletter in this example you have given me
isletter is a function declared in the stdio.h header, it test if a character is a letter or not. But you can use isalpha instead, it is more compatible.
If you don't want to use it, you can do this instead:


bool validateWord(char *word)
{
       char c; 
       for (int i=0; c = word[i]; i++)
            if (!(c>='A' && c<='Z') && !(c>='a' && c<='z'))
                  return false;
       return true;
}

Open in new window

0
 
evilrixConnect With a Mentor Senior Software Engineer (Avast)Commented:
use std::string::find_first_not_of: -

http://www.cppreference.com/cppstring/find_first_not_of.html
char const * Word = "qwerty"; // Your word!
 
if(std::string::npos != std::string(Word).find_first_not_of("abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ"))
{
	throw std::runtime_error("Invalid data");
}

Open in new window

0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

All Courses

From novice to tech pro — start learning today.