Mixing
Cannot write objects containing Strings to File
I am working on a phone book project in C++. I made a class and have string objects inside to store some strings.
My Program is unable to write the string data into the file and is troubling me. It only shows me the last data inserted.
Here is the sample program giving the basic idea of what I want to do and where does the problem reside.
#include <iostream>
#include <fstream>
using namespace std;
class student {
private:
string name;
int age;
string mail;
public:
void getData(student &s) {
ofstream file;
file.open("file.txt", ios::app);
if(!file) {
cout << "Error in creating file.." << endl;
return;
}
cout << "nFile created successfully." << endl;
cout << "Enter name: ";
cin.ignore();
getline(cin, name);
cout << "Enter age: ";
cin >> age;
cout<<"Enter mail: ";
cin.ignore();
getline(cin, mail);
file.write((char*)&s, sizeof(s));
file.close();
cout << "nFile saved and closed succesfully." << endl;
}
void showData(student &s) {
ifstream file1;
file1.open("file.txt", ios::in);
if(!file1){
cout<<"Error in opening file..";
return;
}
while(!file1.eof()){
file1.read((char*) &s, sizeof(s));
cout << "Name: " <<name << "nAge : " << age << "nMail: "<< mail << endl;
}
file1.close();
}
};
int main() {
student s;
s.getData(s);
s.getData(s);
s.getData(s);
s.showData(s);
return 0;
}
I need help in this case.
c++ c++11
edited Nov 20 '18 at 16:37
kiner_shah
1,23221423
asked Nov 20 '18 at 15:04
Vimal KumawatVimal Kumawat
13
add a comment |
I am working on a phone book project in C++. I made a class and have string objects inside to store some strings.
My Program is unable to write the string data into the file and is troubling me. It only shows me the last data inserted.
Here is the sample program giving the basic idea of what I want to do and where does the problem reside.
#include <iostream>
#include <fstream>
using namespace std;
class student {
private:
string name;
int age;
string mail;
public:
void getData(student &s) {
ofstream file;
file.open("file.txt", ios::app);
if(!file) {
cout << "Error in creating file.." << endl;
return;
}
cout << "nFile created successfully." << endl;
cout << "Enter name: ";
cin.ignore();
getline(cin, name);
cout << "Enter age: ";
cin >> age;
cout<<"Enter mail: ";
cin.ignore();
getline(cin, mail);
file.write((char*)&s, sizeof(s));
file.close();
cout << "nFile saved and closed succesfully." << endl;
}
void showData(student &s) {
ifstream file1;
file1.open("file.txt", ios::in);
if(!file1){
cout<<"Error in opening file..";
return;
}
while(!file1.eof()){
file1.read((char*) &s, sizeof(s));
cout << "Name: " <<name << "nAge : " << age << "nMail: "<< mail << endl;
}
file1.close();
}
};
int main() {
student s;
s.getData(s);
s.getData(s);
s.getData(s);
s.showData(s);
return 0;
}
I need help in this case.
c++ c++11
edited Nov 20 '18 at 16:37
kiner_shah
1,23221423
asked Nov 20 '18 at 15:04
Vimal KumawatVimal Kumawat
13
Unrelated to your problem, but please read Why is iostream::eof inside a loop condition considered wrong?
– Some programmer dude
Nov 20 '18 at 15:08
The concept you're looking for is called (de-)serialization. Dumping a binary blob containingstd::stringobjects isn't going to work for that.
– Useless
Nov 20 '18 at 15:17
You are passingsas reference, but not filling any values in its members. Do you intend to fill age, name, mail ins?
– kiner_shah
Nov 20 '18 at 15:35
add a comment |
I am working on a phone book project in C++. I made a class and have string objects inside to store some strings.
My Program is unable to write the string data into the file and is troubling me. It only shows me the last data inserted.
Here is the sample program giving the basic idea of what I want to do and where does the problem reside.
#include <iostream>
#include <fstream>
using namespace std;
class student {
private:
string name;
int age;
string mail;
public:
void getData(student &s) {
ofstream file;
file.open("file.txt", ios::app);
if(!file) {
cout << "Error in creating file.." << endl;
return;
}
cout << "nFile created successfully." << endl;
cout << "Enter name: ";
cin.ignore();
getline(cin, name);
cout << "Enter age: ";
cin >> age;
cout<<"Enter mail: ";
cin.ignore();
getline(cin, mail);
file.write((char*)&s, sizeof(s));
file.close();
cout << "nFile saved and closed succesfully." << endl;
}
void showData(student &s) {
ifstream file1;
file1.open("file.txt", ios::in);
if(!file1){
cout<<"Error in opening file..";
return;
}
while(!file1.eof()){
file1.read((char*) &s, sizeof(s));
cout << "Name: " <<name << "nAge : " << age << "nMail: "<< mail << endl;
}
file1.close();
}
};
int main() {
student s;
s.getData(s);
s.getData(s);
s.getData(s);
s.showData(s);
return 0;
}
I need help in this case.
c++ c++11
edited Nov 20 '18 at 16:37
kiner_shah
1,23221423
asked Nov 20 '18 at 15:04
Vimal KumawatVimal Kumawat
13
I am working on a phone book project in C++. I made a class and have string objects inside to store some strings.
My Program is unable to write the string data into the file and is troubling me. It only shows me the last data inserted.
Here is the sample program giving the basic idea of what I want to do and where does the problem reside.
#include <iostream>
#include <fstream>
using namespace std;
class student {
private:
string name;
int age;
string mail;
public:
void getData(student &s) {
ofstream file;
file.open("file.txt", ios::app);
if(!file) {
cout << "Error in creating file.." << endl;
return;
}
cout << "nFile created successfully." << endl;
cout << "Enter name: ";
cin.ignore();
getline(cin, name);
cout << "Enter age: ";
cin >> age;
cout<<"Enter mail: ";
cin.ignore();
getline(cin, mail);
file.write((char*)&s, sizeof(s));
file.close();
cout << "nFile saved and closed succesfully." << endl;
}
void showData(student &s) {
ifstream file1;
file1.open("file.txt", ios::in);
if(!file1){
cout<<"Error in opening file..";
return;
}
while(!file1.eof()){
file1.read((char*) &s, sizeof(s));
cout << "Name: " <<name << "nAge : " << age << "nMail: "<< mail << endl;
}
file1.close();
}
};
int main() {
student s;
s.getData(s);
s.getData(s);
s.getData(s);
s.showData(s);
return 0;
}
I need help in this case.
c++ c++11
c++ c++11
edited Nov 20 '18 at 16:37
kiner_shah
1,23221423
asked Nov 20 '18 at 15:04
Vimal KumawatVimal Kumawat
13
edited Nov 20 '18 at 16:37
kiner_shah
1,23221423
asked Nov 20 '18 at 15:04
Vimal KumawatVimal Kumawat
13
edited Nov 20 '18 at 16:37
kiner_shah
1,23221423
edited Nov 20 '18 at 16:37
kiner_shah
1,23221423
edited Nov 20 '18 at 16:37
kiner_shah
1,23221423
1,23221423
asked Nov 20 '18 at 15:04
Vimal KumawatVimal Kumawat
13
asked Nov 20 '18 at 15:04
Vimal KumawatVimal Kumawat
13
asked Nov 20 '18 at 15:04
Vimal KumawatVimal Kumawat
13
13
Unrelated to your problem, but please read Why is iostream::eof inside a loop condition considered wrong?
– Some programmer dude
Nov 20 '18 at 15:08
The concept you're looking for is called (de-)serialization. Dumping a binary blob containingstd::stringobjects isn't going to work for that.
– Useless
Nov 20 '18 at 15:17
You are passingsas reference, but not filling any values in its members. Do you intend to fill age, name, mail ins?
– kiner_shah
Nov 20 '18 at 15:35
add a comment |
Unrelated to your problem, but please read Why is iostream::eof inside a loop condition considered wrong?
– Some programmer dude
Nov 20 '18 at 15:08
The concept you're looking for is called (de-)serialization. Dumping a binary blob containingstd::stringobjects isn't going to work for that.
– Useless
Nov 20 '18 at 15:17
You are passingsas reference, but not filling any values in its members. Do you intend to fill age, name, mail ins?
– kiner_shah
Nov 20 '18 at 15:35
Unrelated to your problem, but please read Why is iostream::eof inside a loop condition considered wrong?
– Some programmer dude
Nov 20 '18 at 15:08
Unrelated to your problem, but please read Why is iostream::eof inside a loop condition considered wrong?
– Some programmer dude
Nov 20 '18 at 15:08
The concept you're looking for is called (de-)serialization. Dumping a binary blob containing
std::string objects isn't going to work for that.– Useless
Nov 20 '18 at 15:17
The concept you're looking for is called (de-)serialization. Dumping a binary blob containing
std::string objects isn't going to work for that.– Useless
Nov 20 '18 at 15:17
You are passing
s as reference, but not filling any values in its members. Do you intend to fill age, name, mail in s?– kiner_shah
Nov 20 '18 at 15:35
You are passing
s as reference, but not filling any values in its members. Do you intend to fill age, name, mail in s?– kiner_shah
Nov 20 '18 at 15:35
add a comment |
2 Answers
2
active
oldest
votes
A std::string object is basically nothing more than a wrapper around a pointer (to the actual string contents). A pointer is unique for each process and can not readily be shared between processes even on the same system, not even processes running the very same program.
For you to be able to read or write write a raw object like you do, the object needs to be trivial, which your object isn't since it contains the non-trivial std::string members.
answered Nov 20 '18 at 15:09
Some programmer dudeSome programmer dude
297k24252414
add a comment |
A std::ifstream provides the same interface as std::cin and a std::ofstream provides the same interface as std::cout. Rather than copying the bytes that denote your object, you can use a function to read and write to the appropriate stream.
class student {
private:
string name;
int age;
string mail;
public:
void getData();
void showData();
};
std::istream& operator >> (std::istream& is, const student & s)
{
is.ignore(); std::getline(is, s.name);
is >> s.age;
is.ignore(); std::getline(is, s.mail);
return is;
}
std::ostream& operator << (std::ostream& os, const student & s)
{
return os << s.name << s.age << s.mail;
}
You also don't need to pass an object to it's own methods. You use the members, but then read or write the passed student.
void student::getData() {
std::ofstream file("file.txt",ios::app);
if(!file) {
std::cout << "Error in creating file.." << std::endl;
return;
}
std::cout << "nFile created successfully." << std::endl;
std::cout << "Enter name: ";
std::cin.ignore();
std::getline(std::cin, name);
std::cout << "Enter age: ";
std::cin >> age;
std::cout << "Enter mail: ";
std::cin.ignore();
std::getline(std::cin, mail);
file << *this;
std::cout << "nFile saved and closed succesfully." << std::endl;
}
void student::showData() {
ifstream file1("file.txt", ios::in);
if(!file1){
std::cout << "Error in opening file..";
return;
}
while(file1 >> *this){
std::cout << "Name: " << name << "nAge : " << age << "nMail: " << mail << std::endl;
}
}
int main() {
student s;
s.getData();
s.getData();
s.getData();
s.showData();
return 0;
}
But getData and showData don't really belong to student. I would change it to void getStudent(std::istream & src, std::ostream & dest) and void showStudents(std::istream & src, std::ostream & dest), passing in cin, cout and the file stream from main.
answered Nov 20 '18 at 16:29
CalethCaleth
17.3k22139
add a comment |
Your Answer
StackExchange.ifUsing("editor", function () {
StackExchange.using("externalEditor", function () {
StackExchange.using("snippets", function () {
StackExchange.snippets.init();
});
});
}, "code-snippets");
StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "1"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);
StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});
function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: true,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});
}
});
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53395878%2fcannot-write-objects-containing-strings-to-file%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
2 Answers
2
active
oldest
votes
2 Answers
2
active
oldest
votes
active
oldest
votes
active
oldest
votes
A std::string object is basically nothing more than a wrapper around a pointer (to the actual string contents). A pointer is unique for each process and can not readily be shared between processes even on the same system, not even processes running the very same program.
For you to be able to read or write write a raw object like you do, the object needs to be trivial, which your object isn't since it contains the non-trivial std::string members.
answered Nov 20 '18 at 15:09
Some programmer dudeSome programmer dude
297k24252414
add a comment |
A std::string object is basically nothing more than a wrapper around a pointer (to the actual string contents). A pointer is unique for each process and can not readily be shared between processes even on the same system, not even processes running the very same program.
For you to be able to read or write write a raw object like you do, the object needs to be trivial, which your object isn't since it contains the non-trivial std::string members.
answered Nov 20 '18 at 15:09
Some programmer dudeSome programmer dude
297k24252414
add a comment |
A std::string object is basically nothing more than a wrapper around a pointer (to the actual string contents). A pointer is unique for each process and can not readily be shared between processes even on the same system, not even processes running the very same program.
For you to be able to read or write write a raw object like you do, the object needs to be trivial, which your object isn't since it contains the non-trivial std::string members.
answered Nov 20 '18 at 15:09
Some programmer dudeSome programmer dude
297k24252414
A std::string object is basically nothing more than a wrapper around a pointer (to the actual string contents). A pointer is unique for each process and can not readily be shared between processes even on the same system, not even processes running the very same program.
For you to be able to read or write write a raw object like you do, the object needs to be trivial, which your object isn't since it contains the non-trivial std::string members.
answered Nov 20 '18 at 15:09
Some programmer dudeSome programmer dude
297k24252414
answered Nov 20 '18 at 15:09
Some programmer dudeSome programmer dude
297k24252414
answered Nov 20 '18 at 15:09
Some programmer dudeSome programmer dude
297k24252414
answered Nov 20 '18 at 15:09
Some programmer dudeSome programmer dude
297k24252414
297k24252414
add a comment |
add a comment |
A std::ifstream provides the same interface as std::cin and a std::ofstream provides the same interface as std::cout. Rather than copying the bytes that denote your object, you can use a function to read and write to the appropriate stream.
class student {
private:
string name;
int age;
string mail;
public:
void getData();
void showData();
};
std::istream& operator >> (std::istream& is, const student & s)
{
is.ignore(); std::getline(is, s.name);
is >> s.age;
is.ignore(); std::getline(is, s.mail);
return is;
}
std::ostream& operator << (std::ostream& os, const student & s)
{
return os << s.name << s.age << s.mail;
}
You also don't need to pass an object to it's own methods. You use the members, but then read or write the passed student.
void student::getData() {
std::ofstream file("file.txt",ios::app);
if(!file) {
std::cout << "Error in creating file.." << std::endl;
return;
}
std::cout << "nFile created successfully." << std::endl;
std::cout << "Enter name: ";
std::cin.ignore();
std::getline(std::cin, name);
std::cout << "Enter age: ";
std::cin >> age;
std::cout << "Enter mail: ";
std::cin.ignore();
std::getline(std::cin, mail);
file << *this;
std::cout << "nFile saved and closed succesfully." << std::endl;
}
void student::showData() {
ifstream file1("file.txt", ios::in);
if(!file1){
std::cout << "Error in opening file..";
return;
}
while(file1 >> *this){
std::cout << "Name: " << name << "nAge : " << age << "nMail: " << mail << std::endl;
}
}
int main() {
student s;
s.getData();
s.getData();
s.getData();
s.showData();
return 0;
}
But getData and showData don't really belong to student. I would change it to void getStudent(std::istream & src, std::ostream & dest) and void showStudents(std::istream & src, std::ostream & dest), passing in cin, cout and the file stream from main.
answered Nov 20 '18 at 16:29
CalethCaleth
17.3k22139
add a comment |
A std::ifstream provides the same interface as std::cin and a std::ofstream provides the same interface as std::cout. Rather than copying the bytes that denote your object, you can use a function to read and write to the appropriate stream.
class student {
private:
string name;
int age;
string mail;
public:
void getData();
void showData();
};
std::istream& operator >> (std::istream& is, const student & s)
{
is.ignore(); std::getline(is, s.name);
is >> s.age;
is.ignore(); std::getline(is, s.mail);
return is;
}
std::ostream& operator << (std::ostream& os, const student & s)
{
return os << s.name << s.age << s.mail;
}
You also don't need to pass an object to it's own methods. You use the members, but then read or write the passed student.
void student::getData() {
std::ofstream file("file.txt",ios::app);
if(!file) {
std::cout << "Error in creating file.." << std::endl;
return;
}
std::cout << "nFile created successfully." << std::endl;
std::cout << "Enter name: ";
std::cin.ignore();
std::getline(std::cin, name);
std::cout << "Enter age: ";
std::cin >> age;
std::cout << "Enter mail: ";
std::cin.ignore();
std::getline(std::cin, mail);
file << *this;
std::cout << "nFile saved and closed succesfully." << std::endl;
}
void student::showData() {
ifstream file1("file.txt", ios::in);
if(!file1){
std::cout << "Error in opening file..";
return;
}
while(file1 >> *this){
std::cout << "Name: " << name << "nAge : " << age << "nMail: " << mail << std::endl;
}
}
int main() {
student s;
s.getData();
s.getData();
s.getData();
s.showData();
return 0;
}
But getData and showData don't really belong to student. I would change it to void getStudent(std::istream & src, std::ostream & dest) and void showStudents(std::istream & src, std::ostream & dest), passing in cin, cout and the file stream from main.
answered Nov 20 '18 at 16:29
CalethCaleth
17.3k22139
add a comment |
A std::ifstream provides the same interface as std::cin and a std::ofstream provides the same interface as std::cout. Rather than copying the bytes that denote your object, you can use a function to read and write to the appropriate stream.
class student {
private:
string name;
int age;
string mail;
public:
void getData();
void showData();
};
std::istream& operator >> (std::istream& is, const student & s)
{
is.ignore(); std::getline(is, s.name);
is >> s.age;
is.ignore(); std::getline(is, s.mail);
return is;
}
std::ostream& operator << (std::ostream& os, const student & s)
{
return os << s.name << s.age << s.mail;
}
You also don't need to pass an object to it's own methods. You use the members, but then read or write the passed student.
void student::getData() {
std::ofstream file("file.txt",ios::app);
if(!file) {
std::cout << "Error in creating file.." << std::endl;
return;
}
std::cout << "nFile created successfully." << std::endl;
std::cout << "Enter name: ";
std::cin.ignore();
std::getline(std::cin, name);
std::cout << "Enter age: ";
std::cin >> age;
std::cout << "Enter mail: ";
std::cin.ignore();
std::getline(std::cin, mail);
file << *this;
std::cout << "nFile saved and closed succesfully." << std::endl;
}
void student::showData() {
ifstream file1("file.txt", ios::in);
if(!file1){
std::cout << "Error in opening file..";
return;
}
while(file1 >> *this){
std::cout << "Name: " << name << "nAge : " << age << "nMail: " << mail << std::endl;
}
}
int main() {
student s;
s.getData();
s.getData();
s.getData();
s.showData();
return 0;
}
But getData and showData don't really belong to student. I would change it to void getStudent(std::istream & src, std::ostream & dest) and void showStudents(std::istream & src, std::ostream & dest), passing in cin, cout and the file stream from main.
answered Nov 20 '18 at 16:29
CalethCaleth
17.3k22139
A std::ifstream provides the same interface as std::cin and a std::ofstream provides the same interface as std::cout. Rather than copying the bytes that denote your object, you can use a function to read and write to the appropriate stream.
class student {
private:
string name;
int age;
string mail;
public:
void getData();
void showData();
};
std::istream& operator >> (std::istream& is, const student & s)
{
is.ignore(); std::getline(is, s.name);
is >> s.age;
is.ignore(); std::getline(is, s.mail);
return is;
}
std::ostream& operator << (std::ostream& os, const student & s)
{
return os << s.name << s.age << s.mail;
}
You also don't need to pass an object to it's own methods. You use the members, but then read or write the passed student.
void student::getData() {
std::ofstream file("file.txt",ios::app);
if(!file) {
std::cout << "Error in creating file.." << std::endl;
return;
}
std::cout << "nFile created successfully." << std::endl;
std::cout << "Enter name: ";
std::cin.ignore();
std::getline(std::cin, name);
std::cout << "Enter age: ";
std::cin >> age;
std::cout << "Enter mail: ";
std::cin.ignore();
std::getline(std::cin, mail);
file << *this;
std::cout << "nFile saved and closed succesfully." << std::endl;
}
void student::showData() {
ifstream file1("file.txt", ios::in);
if(!file1){
std::cout << "Error in opening file..";
return;
}
while(file1 >> *this){
std::cout << "Name: " << name << "nAge : " << age << "nMail: " << mail << std::endl;
}
}
int main() {
student s;
s.getData();
s.getData();
s.getData();
s.showData();
return 0;
}
But getData and showData don't really belong to student. I would change it to void getStudent(std::istream & src, std::ostream & dest) and void showStudents(std::istream & src, std::ostream & dest), passing in cin, cout and the file stream from main.
answered Nov 20 '18 at 16:29
CalethCaleth
17.3k22139
answered Nov 20 '18 at 16:29
CalethCaleth
17.3k22139
answered Nov 20 '18 at 16:29
CalethCaleth
17.3k22139
answered Nov 20 '18 at 16:29
CalethCaleth
17.3k22139
17.3k22139
add a comment |
add a comment |
Thanks for contributing an answer to Stack Overflow!
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53395878%2fcannot-write-objects-containing-strings-to-file%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Unrelated to your problem, but please read Why is iostream::eof inside a loop condition considered wrong?
– Some programmer dude
Nov 20 '18 at 15:08
The concept you're looking for is called (de-)serialization. Dumping a binary blob containing
std::stringobjects isn't going to work for that.– Useless
Nov 20 '18 at 15:17
You are passing
sas reference, but not filling any values in its members. Do you intend to fill age, name, mail ins?– kiner_shah
Nov 20 '18 at 15:35