Cannot jump to label 'fin', error: from here, and crosses initialization - c++

Question

My biggest problems are all stated above, the inability to jump to label fin (error on line 27), the error: from here (error on lines 12 and 14) and the crosses initialization error (error on line 20) Please help!
#include <iostream>
#include <string>
int main()
{
std::string name;
std::cout << "Please comply. y/n: ";
std::string answer;
std::cin >> answer;
if (answer == "y"){std::cout << "You were spared." << std::endl; goto fin;}
if (answer == "Miche"){std::cout << "The killers understood that you understood the prophecy, so they took you to their master" << std::endl; goto secret;}
if (answer == "n"){std::cout << "You were brutally killed." << std::endl; goto fin;}
else {std::cout << "You randomly babled " << answer << ", getting yourself killed."; goto fin;}
secret:
std::cout << "In order to fully find out if you are the legendary Miche, they took you to their leader."
<< " The master looked you over, and asked you one final question. The master asks you, fish?" << std::endl;
std::string fish; fish = "none";
std::cin >> fish;
if (fish == "fish."){std::cout << "You were put in the throne of the king, where you ruled your near killers and their species for eternity."
<< std::endl; goto fin;}
else {std::cout << "You failed and were immediately killed." << std::endl; goto fin;}
goto fin;
fin:
return 0;
}

Answer 1

The problem is, essentially this:
int main() {
if (whatever)
goto fin;
std::string fish;
fin:
return 0;
}
If whatever is true, the goto jumps past the construction of fish. That's not allowed, because the compiler can't generate sensible code to destroy fish or not, depending on whether the goto was executed.
Solution: don't use goto.
Possibilities:
int main() {
if (whatever)
goto fin;
{
std::string fish;
}
fin:
return 0;
Here, fish gets destroyed at the end of the block, so the goto doesn't cause a problem (aside from its inherent unstructured nature).
Better:
int main() {
if (!whatever) {
std::string fish;
}
return 0;
}

Answer 2

You can duplicate the problem using a much simpler function.
void foo()
{
goto fin;
std::string fish = "none";
std::cin >> fish;
fin:
}
Why is that a problem? When the execution jumps to fin:, the code to initialize fish is not executed. Its destructor will be called when the function returns. Since the destructor will be called on an uninitialized object, the program will exhibit undefined behavior.
The simplest fix for your problem is to put almost everything in main in another function, and then use return instead of goto.
void do_stuff()
{
std::string name;
std::cout << "Please comply. y/n: ";
std::string answer;
std::cin >> answer;
if (answer == "y")
{
std::cout << "You were spared." << std::endl;
return;
}
if (answer == "n")
{
std::cout << "You were brutally killed." << std::endl;
return;
}
if (answer == "Miche")
{
std::cout << "The killers understood that you understood the prophecy, so they took you to their master" << std::endl;
}
else
{
std::cout << "You randomly babled " << answer << ", getting yourself killed.";
return;
}
std::cout << "In order to fully find out if you are the legendary Miche, they took you to their leader."
<< " The master looked you over, and asked you one final question. The master asks you, fish?" << std::endl;
std::string fish = "none";
std::cin >> fish;
if (fish == "fish")
{
std::cout << "You were put in the throne of the king, where you ruled your near killers and their species for eternity." << std::endl;
}
else
{
std::cout << "You failed and were immediately killed." << std::endl;
}
}
int main()
{
do_stuff();
}