2011-12-13 10 views
4

私は現在C + +を使ってC++をDummies All-In-Oneに使っています。第2版​​。このプログラムを作成するにはQtを使用しています。ヘッダーファイル内のオブジェクトとクラスを整理し、main.cppに加えて.cppファイルでメンバー関数をプロスペクティブに構成することをお勧めします。この点に関して私はこの本の演習をそのように実行しようとしますが、最近次のエラーが発生しました。エラー: '。'の前にprimary-expressionが必要です。トークン

このエラーは、31,32、および37行目で発生します。このエラーは、クラスメンバー関数との関連性が高いようです。

私のmain.cppに

#include "controlinginput.h" 
#include <QtCore/QCoreApplication> 
#include <iostream> 
#include <sstream> 


using namespace std; 

int main(int argc, char *argv[]) 
{ 
QCoreApplication a(argc, argv); 


// just a basic name-entering 
string name; 
cout << "What is your name?"; 
cin >> name; 
cout << "Hello " << name << endl; 

/* now you are asked for a number 
    but the computer will allow you to enter anything*/ 
int x; 
cout << endl << "Enter a number! Any Number!" << endl; 
cin >> x; 
cout << "You choose " << x << endl; 

/* now youll be asked for a number again 
    but the computer will only allow numbers */ 
cout << endl<< "This time you will ONLY be able to enter a number! " << endl; 
cout << "SO, Pick a number! any number!" << endl; 
string entered = ControlingInput.enterOnlyNumbers(); // ###Error###   
int num = ControlingInput.stringToANumber(entered); // ###Error### 
cout << endl << "You entered " << num << endl; // value is displayed 
//Now finally we enter the password 
cout << endl; 
cout << "Please enter a password" << endl; 
string password = ControlingInput.EnterPassword(); // ###Error### 
cout << "shh... your password is " << password << endl; 
return a.exec(); 
} 

私は、このエラーは、構文の誤用のかなり幅広い範囲を示していることを見つけるために、いくつかの研究をしました。残念ながら、私は鉱山に似たインスタンスを見つけることができませんでした。私は経験豊富なプログラマーの一部からいくつかの洞察を得ることを望んでいました。これが私の最後の過失のための単純な問題であれば、私は事前に謝罪し、フィードバックを感謝します。それは少しとは対照的に、トラブルのアロット私を与えた場合、私はこれらが私のメンバ関数が含まれているので、私はまた、私は私のヘッダが含まれている私のヘッダファイルと.CPP

controlingInput.cppを(含まれている

..よりよい学びます編集者が私にここに問題を与えていたいくつかの理由でファイルとiostreamsstreamここではなく)

using namespace std; 

ControlingInput::ControlingInput() 
{ 

} 
int ControlingInput::stringToANumber(string MyString) 
{ 
istringstream converter(MyString); //Holds the string that was passed to this function 
int result;      //Holds the integer result 

//perform the conversion 
converter >> result; 
return result; //function completes and returns converted string 

} 

string ControlingInput::enterOnlyNumbers() 
{ 
string numbAsString = ""; // this holds our numeric string 
     char ch = getch(); // This gets a single character from our user 
//Says to keep gettting characters from our user untill user presses enter 
     while (ch != '\r') // \r is the enter key 
     { 
      //This says to add characters only if they are numbers 
      if (ch >= '0' && ch <='9') 
      { 
       cout << ch; // show 
       numbAsString += ch; // add character to the string 
      } 

      ch = getch(); // get the next character from the user 

     } 
     return numbAsString; 

} 

string ControlingInput::EnterPassword() 
{ 
string numbAsString = ""; //this will hold our password string 
char ch = getch(); // this gets a single char from our users just like before 
//keep gettting characters from the user until enter/return is pressed 
while (ch != '\r'); // \r is the enter or return key 
{ 
    //for security passwords are displayed as asterisks instead of characters 
    cout << '*'; 

    //add character input into the password string 
    numbAsString += ch; 

    //Get the next character from the user 
    ch = getch(); 
} 
return numbAsString; // return the user input from this function 

そして、ここでは私のcontrolingInput.hは

#ifndef CONTROLINGINPUT_H 
#define CONTROLINGINPUT_H 
#include <iostream> 

using namespace std; 

class ControlingInput 
{ 
public: 
int stringToANumber(string MyString); 
string EnterPassword(); 
string enterOnlyNumbers(); 

}; 

#endif // CONTROLINGINPUT_H 
です

フィードバックをいただきありがとうございます。

+0

あなたは 'static'関数のように関数を扱っていますが、' static'ではありません。あなたは 'ControlingInput'のインスタンスを作成する必要があります。 – birryree

答えて

6

インスタンス変数を静的であるかのようにクラス自体で呼び出そうとしています(これは無効な構文です)。これが正しく動作するためには、ControlingInputのインスタンスが必要です。

int main(int argc, char *argv[]) 
{ 

    QCoreApplication a(argc, argv); 

    ControlingInput ctrlInput; //Create instance 
    ... 

    string entered = ctrlInput.enterOnlyNumbers();   
    int num = ctrlInput.stringToANumber(entered); 
    cout << endl << "You entered " << num << endl; // value is displayed 
    ... 

    string password = ctrlInput.EnterPassword(); 
    cout << "shh... your password is " << password << endl; 
    return a.exec(); 

} 
+0

Ahh;分かりました!オブジェクトのインスタンスを作成するという考え方が間違っていたので、それも良いことでした。私はオブジェクトが含まれていたのでそれが作成されたと信じていました。それは魅力のように働いた。あなたの時間をありがとう。 –

関連する問題