Using c++14. I have read many posts regarding the problem.
If I run this code below, it jumps over the getline lines.
#include <iostream>
#include "main_menu.h"
void MainMenu::AddTest()
{
std::string courseName = "";
std::string testName = "";
std::string date = "";
std::cout << "Enter course name: " << std::endl;
std::getline(std::cin, courseName);
std::cout << "Enter test name: " << std::endl;
std::getline(std::cin, testName);
std::cout << "Enter test date: " << std::endl;
std::getline(std::cin, date);
Test test(courseName, testName, date);
tests.Add(test);
std::cout << "Test registered : " << std::endl;
tests.Print(test.id);
}
If I add cin ignore after each getline lines (example below how I implement it), it deletes some characters from the input strings and uses wrong variables to store them. Note that I have strings with whitespaces.
std::getline(std::cin, courseName);
std::cin.ignore();
This is what I get:
Enter course name:
History 2
Enter test name:
History 2 exam
Enter test date:
2017.01.02
Test registered :
test id = 2, course name = , test name = istory 2, date = istory 2 exam
I also tried to flush cout, didn't help.
My Print function works like a charm, if I add courses manually from main, I get the expected output, so the problem is definitely the cin / getline.
Test registered :
test id = 1, course name = History 2, test name = History 2 exam , date = 01.02.2017
I use getline as explained here: http://www.cplusplus.com/reference/string/string/getline/?kw=getline
Any help would be much appreciated, thank you.
By using cin.ignore you are messing with the input itself. If you want to get rid of \n character you don't have to! getline will just do that automatically. So just don't use ignore function and the code will be fine.
This works:
#include<iostream>
using namespace std;
int main()
{
string courseName = "";
string testName = "";
string date = "";
cout << "Enter course name: " << std::endl;
getline(std::cin, courseName);
cout << "Enter test name: " << std::endl;
getline(std::cin, testName);
cout << "Enter test date: " << std::endl;
getline(std::cin, date);
cout << courseName << endl;
cout << testName << endl;
cout << date << endl;
return 0;
}
I'm answering an ancient question, but try clearing the input stream before you use all the getline()'s. It is possible that you have some extra returns in the buffer before you ask for input.
cin.clear();
cin.ignore(INT_MAX);
Related
I am trying to convert a Boost UDP server from using raw pointers to using std::vector which I believe is possible from https://www.boost.org/doc/libs/1_66_0/doc/html/boost_asio/reference/buffer.html
However my vector is always filled with 0 values.
void start_receive()
{
std::vector<unsigned char> recv_buffer_(64);
socket_.async_receive_from(
boost::asio::buffer(recv_buffer_), remote_endpoint_,
boost::bind(&udp_server::handle_receive, this,
boost::asio::placeholders::error,
boost::asio::placeholders::bytes_transferred));
std::cout << "Receive length is :" << recv_buffer_.size() << std::endl;
std::cout << "recv_buffer_ = \n";
for (unsigned char n : recv_buffer_)
std::cout << n << ' ';
std::cout << '\n';
}
Any ideas what I may be doing wrong ? socket_ is a udp::socket socket_; and the above code works with a unsigned char *
I'm wondering if any of you guys have experience in reading/logging the input arguments of an instrumented method?
Technically it's so simple and straightforward when the method has some primitive data type as input parameters. For example, integer or double. However, when it's a more complex data type, like string, I faced some weird behavior from pin framework. In nutshell, I cannot recover the original data input.
Below you see my instrumentation function. It simply tries to print the input arguments passed to the instrumented method.
VOID start_of_execution(uint64_t hash_name, int type,
ADDRINT input_arg1,
ADDRINT input_arg2
){
cout << "int arg:" << input_arg1 << endl; // prints the integer input
cout << "string*:" << hex << input_arg2 << endl; // prints the pointer value
cout << "string arg:" << *(reinterpret_cast<string*>(input_arg2)) << endl; // prints nothing!
}
Here is the instrumented method which it's the only functionality is to print the input arguments.
int sample_method(int i, string* str){
cout << "-----" << endl;
cout << "i: " << i << ", str:" << *str <<", ptr:" << str << endl;
return 0;
}
Below is the output:
$ sudo ../../../pin -t obj-intel64/code/mypin.so -- ./code/a.out
int arg:5
string*:0x1704434923
string arg:
-----
i: 5, str:sample text, ptr:0x1704434923
As you see, the integer argument (number 5) has been retrived properly. However, even though the string pointer is valid, PIN wasn't able to recover the string input argument.
Does anybody have an idea about my problem?
Im not sure how to include a "checker" to prevent incorrect input pls help :(
sorry, just started learning last night forgive me..
How should i do about this, without using if else statement or when should i be using that instead?
#include <iostream>
#include <iomanip>
using namespace std;
int main()
{
int get_number;
char ch;
cout << "Hey! What is your number 1 OR 2?! :";
cin >> get_number;
switch (get_number)
{
case 1: cout << "You pressed 1/nPress anykey to continue!";
break;
case 2: cout << "You pressed 2/nPress anykey to continue!";
break;
default: cout << "Retry! 1 OR 2?! :";
cin >> get_number;
}
cin.ignore(1, '/n');
cout << "Press any key to continue";
cin.get(ch);
return 0;
}
So at my college we haven't covered do while loops yet so I wanted to try one out before we do but I am running into an issue with it, I want the program to carry out different functions based on the users input, A carries out an addition, B a subtraction and C exits the program.
The program I have wrote with a do while in it carries out these functions one after the other regardless of what the user has input and I am unsure how to get this working properly.
Any advice is much appreciated.
Couple things to note. You have user entering a character for a choice, so make userChoice a character. Notice how the do-while is used for input-validation in this case. I changed the extra while loops with semi-colons at end to condition statements. You can substitute the if and else ifs with switch, "menu driven program". Here is a working program that you want. If you want the do-while around the whole condition statement and not use it as input validation then have it around all the condition statements and check while it is not C.
#include <iostream>
#include <string>
using namespace std;
int main()
{
char userChoice;
int vaulue1;
int vaulue2;
int addtionValue;
int subtractionValue;
cout << "Choice A: will preform an addition" << endl;
cout << "Choice B: will preform a subtraction" << endl;
cout << "Choice C: will quit\n" << endl;
do
{
cout << "Enter your choice\n" << endl;
cin >> userChoice;
} while(userChoice != 'A' && userChoice != 'B' && userChoice != 'C');
if(userChoice == 'A')
{
cout << "Addition\n" << endl;
cout << "Please enter the first value you want to add\n" << endl;
cin >> vaulue1;
cout << "Please enter the second value you want to add\n" << endl;
cin >> vaulue2;
addtionValue = vaulue1 + vaulue2;
cout << "The addtion answer is " << addtionValue << endl;
}
else if(userChoice == 'B')
{
cout << "Subtraction\n" << endl;
cout << "Please enter the first value you want to subtract\n" << endl;
cin >> vaulue1;
cout << "Please enter the second value you want to subtract\n" << endl;
cin >> vaulue2;
subtractionValue = vaulue1 - vaulue2;
cout << "The subtract answer is " << subtractionValue << endl;
}
else if(userChoice == 'C')
{
cout << "Exit ";
return -1;
}
return 0;
}
you should use only one "while" - just first while is correct
other "whiles" are empty loop; clean them
it's better to use "switch - case"
while you enter A as input your method wants to give A another one
I'm very new to programming and I seem to have ran into a wall. I'm trying to write a simple code that reads scores from different Bowlers and then finding the average for each Bowler. My code seems to read the first line of scores and then stops. Any advice? Here's what I have
#include <iostream>
#include <fstream>
#include <iomanip>
#include <string>
#include <sstream>
#include <vector>
using namespace std;
int main ()
{
ifstream inFile;
ofstream outFile;
double score1, score2, score3;
double average;
string firstname;
string lastname;
inFile.open ("scores.txt");
outFile.open ("scoreavg.out");
ifstream ("scores.txt");
string content;
while (inFile >> content)
{
cout << content << ' ';
inFile >> firstname >> lastname ;
outFile << "Bowler name: " <<firstname << " " << lastname << endl;
inFile >> score1 >> score2 >> score3;
outFile << "scores: " << setw(4) << score1 << setw(4) << score2 << setw(4)<< score3 << endl;
average = (score1 + score2 + score3)/3;
outFile << "Average score: " << setw(4) << average<< endl;
inFile.close();
outFile.close();
return 0;
}
}
Short answer: the return 0; statement is inside the while loop.
You want your loop to do some reading and averaging for each record in the input. Once that's done, you then cleanup by closing the files. So the loop ends after the record is processed and before the cleanup.
Move the brace from after the return to before the first close.