[2 days left] What’s wrong with your cloud strategy? Learn why multicloud solutions matter with Nimble Storage.Register Now

x
?
Solved

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

Posted on 2007-12-06
8
Medium Priority
?
324 Views
Last Modified: 2007-12-09
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

0
Comment
Question by:jschmuff
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 4
  • 3
8 Comments
 
LVL 55

Accepted Solution

by:
Jaime Olivares earned 1200 total points
ID: 20425122
>>      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
 

Author Comment

by:jschmuff
ID: 20425168
what does that have to do with taking the input only a through z?
0
 
LVL 55

Expert Comment

by:Jaime Olivares
ID: 20425181
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
VIDEO: THE CONCERTO CLOUD FOR HEALTHCARE

Modern healthcare requires a modern cloud. View this brief video to understand how the Concerto Cloud for Healthcare can help your organization.

 
LVL 55

Expert Comment

by:Jaime Olivares
ID: 20425186
sorry, in this case, exception will always occur.
0
 

Author Comment

by:jschmuff
ID: 20425202
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
 

Author Comment

by:jschmuff
ID: 20425211
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
 
LVL 55

Expert Comment

by:Jaime Olivares
ID: 20425236
>>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
 
LVL 40

Assisted Solution

by:evilrix
evilrix earned 800 total points
ID: 20434994
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

Featured Post

New feature and membership benefit!

New feature! Upgrade and increase expert visibility of your issues with Priority Questions.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

This article shows you how to optimize memory allocations in C++ using placement new. Applicable especially to usecases dealing with creation of large number of objects. A brief on problem: Lets take example problem for simplicity: - I have a G…
Whether you’re a college noob or a soon-to-be pro, these tips are sure to help you in your journey to becoming a programming ninja and stand out from the crowd.
The viewer will learn how to implement Singleton Design Pattern in Java.
This video will show you how to get GIT to work in Eclipse.   It will walk you through how to install the EGit plugin in eclipse and how to checkout an existing repository.
Suggested Courses

649 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question