how to manage reading a global string from a thread?





.everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty,.everyoneloves__bot-mid-leaderboard:empty{ height:90px;width:728px;box-sizing:border-box;
}







1















I working in a simple program that reads from std input and prints the buffer when a socket message arrives.



std::string data;

void listen_for_messages();

int main(int argc, char *argv)
{
std::thread listenMessages(listen_for_messages);
char c;
while(cin.get(c)) {
data += c;
}
}

void listen_for_messages()
{
while(1) {
//receive a message from the socket
std::cout << "r>" << data << std::endl;
}
}


but data is always empty and I don't know why. Please help me!



I'm using g++ (Ubuntu 7.3.0-16ubuntu3) 7.3.0 compiler and following compile command:



g++ -o myprogram -pthread myprogram.cpp









share|improve this question

























  • Hint: std::mutex

    – NathanOliver
    Nov 16 '18 at 15:10











  • You have no synchronisation on data, so the compiler probably assumes it cannot possibly ever change in listen_for_messages, and optimises the print to an empty string. Just a guess, but you definitely have the synchronisation bug.

    – BoBTFish
    Nov 16 '18 at 15:10











  • I think mutex doesn't really help in this case. I no need synchronization, since one thread write the string and other read it. Is there a way to say the compiler that my string is maybe acces by different threads?

    – ibado
    Nov 16 '18 at 18:19


















1















I working in a simple program that reads from std input and prints the buffer when a socket message arrives.



std::string data;

void listen_for_messages();

int main(int argc, char *argv)
{
std::thread listenMessages(listen_for_messages);
char c;
while(cin.get(c)) {
data += c;
}
}

void listen_for_messages()
{
while(1) {
//receive a message from the socket
std::cout << "r>" << data << std::endl;
}
}


but data is always empty and I don't know why. Please help me!



I'm using g++ (Ubuntu 7.3.0-16ubuntu3) 7.3.0 compiler and following compile command:



g++ -o myprogram -pthread myprogram.cpp









share|improve this question

























  • Hint: std::mutex

    – NathanOliver
    Nov 16 '18 at 15:10











  • You have no synchronisation on data, so the compiler probably assumes it cannot possibly ever change in listen_for_messages, and optimises the print to an empty string. Just a guess, but you definitely have the synchronisation bug.

    – BoBTFish
    Nov 16 '18 at 15:10











  • I think mutex doesn't really help in this case. I no need synchronization, since one thread write the string and other read it. Is there a way to say the compiler that my string is maybe acces by different threads?

    – ibado
    Nov 16 '18 at 18:19














1












1








1








I working in a simple program that reads from std input and prints the buffer when a socket message arrives.



std::string data;

void listen_for_messages();

int main(int argc, char *argv)
{
std::thread listenMessages(listen_for_messages);
char c;
while(cin.get(c)) {
data += c;
}
}

void listen_for_messages()
{
while(1) {
//receive a message from the socket
std::cout << "r>" << data << std::endl;
}
}


but data is always empty and I don't know why. Please help me!



I'm using g++ (Ubuntu 7.3.0-16ubuntu3) 7.3.0 compiler and following compile command:



g++ -o myprogram -pthread myprogram.cpp









share|improve this question
















I working in a simple program that reads from std input and prints the buffer when a socket message arrives.



std::string data;

void listen_for_messages();

int main(int argc, char *argv)
{
std::thread listenMessages(listen_for_messages);
char c;
while(cin.get(c)) {
data += c;
}
}

void listen_for_messages()
{
while(1) {
//receive a message from the socket
std::cout << "r>" << data << std::endl;
}
}


but data is always empty and I don't know why. Please help me!



I'm using g++ (Ubuntu 7.3.0-16ubuntu3) 7.3.0 compiler and following compile command:



g++ -o myprogram -pthread myprogram.cpp






c++






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited Nov 16 '18 at 15:11









NathanOliver

98.5k16138218




98.5k16138218










asked Nov 16 '18 at 15:08









ibadoibado

1144




1144













  • Hint: std::mutex

    – NathanOliver
    Nov 16 '18 at 15:10











  • You have no synchronisation on data, so the compiler probably assumes it cannot possibly ever change in listen_for_messages, and optimises the print to an empty string. Just a guess, but you definitely have the synchronisation bug.

    – BoBTFish
    Nov 16 '18 at 15:10











  • I think mutex doesn't really help in this case. I no need synchronization, since one thread write the string and other read it. Is there a way to say the compiler that my string is maybe acces by different threads?

    – ibado
    Nov 16 '18 at 18:19



















  • Hint: std::mutex

    – NathanOliver
    Nov 16 '18 at 15:10











  • You have no synchronisation on data, so the compiler probably assumes it cannot possibly ever change in listen_for_messages, and optimises the print to an empty string. Just a guess, but you definitely have the synchronisation bug.

    – BoBTFish
    Nov 16 '18 at 15:10











  • I think mutex doesn't really help in this case. I no need synchronization, since one thread write the string and other read it. Is there a way to say the compiler that my string is maybe acces by different threads?

    – ibado
    Nov 16 '18 at 18:19

















Hint: std::mutex

– NathanOliver
Nov 16 '18 at 15:10





Hint: std::mutex

– NathanOliver
Nov 16 '18 at 15:10













You have no synchronisation on data, so the compiler probably assumes it cannot possibly ever change in listen_for_messages, and optimises the print to an empty string. Just a guess, but you definitely have the synchronisation bug.

– BoBTFish
Nov 16 '18 at 15:10





You have no synchronisation on data, so the compiler probably assumes it cannot possibly ever change in listen_for_messages, and optimises the print to an empty string. Just a guess, but you definitely have the synchronisation bug.

– BoBTFish
Nov 16 '18 at 15:10













I think mutex doesn't really help in this case. I no need synchronization, since one thread write the string and other read it. Is there a way to say the compiler that my string is maybe acces by different threads?

– ibado
Nov 16 '18 at 18:19





I think mutex doesn't really help in this case. I no need synchronization, since one thread write the string and other read it. Is there a way to say the compiler that my string is maybe acces by different threads?

– ibado
Nov 16 '18 at 18:19












0






active

oldest

votes












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
});


}
});














draft saved

draft discarded


















StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53340468%2fhow-to-manage-reading-a-global-string-from-a-thread%23new-answer', 'question_page');
}
);

Post as a guest















Required, but never shown

























0






active

oldest

votes








0






active

oldest

votes









active

oldest

votes






active

oldest

votes
















draft saved

draft discarded




















































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.




draft saved


draft discarded














StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53340468%2fhow-to-manage-reading-a-global-string-from-a-thread%23new-answer', 'question_page');
}
);

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







Popular posts from this blog

Bressuire

Vorschmack

Quarantine