I really have no idea what is wrong with my program to control amarok.

G++ gives me a cryptic message...

I need 2 things:

1. To know what I did wrong
2. To be able to debug things better

These may be both stupid questions; I'm an inexperienced C++ programmer.

Here's the code:

***********************************************

#include <cstdio>
#include <cstdlib>
#include <iostream>
using namespace std;

int main(int nNumberofArgs, char* pszArgs[])
{
for(;;)
{
string amacommand;
cout << "Amatrol v0.1/n/nEnter command:";
cin >> amacommand;
if (amacommand = "play")
{
system("dcop amarok player play");
system("festival --tts $HOME/amatrol/play");
}
if (amacommand = "stop")
{
system("dcop amarok player stop");
system("festival --tts $HOME/amatrol/stop");
}
if (amacommand = "vup")
{
system("dcop amarok player volumeUp");
system("festival --tts $HOME/amatrol/vup");
}
if (amacommand = "vdown")
{
system("dcop amarok player volumeDown");
system("festival --tts $HOME/amatrol/vdown");
}
if (amacommand = "shuffle")
{
system("dcop amarok playlist shufflePlaylist");
system("festival --tts $HOME/amatrol/shuffle");
}
if (amacommand = "pause")
{
system("dcop amarok player playPause");
system("festival --tts $HOME/amatrol/pause");
}
if (amacommand = "exit")
{
break;
}
}
return 0;

}
************************************

What did I do wrong? any information would be useful.

( I think I indented correctly, the indents don't like to show up on LQ...

Well, the problem is with the single equal = sign. You are, for example, assigning amacommand = "play", instead of checking if the string matches

This is the correct way: amacommand == "play"

Regards!

You are using wrong escape sequences as well. At the line:

You will want:
But that should still compile. To post code in the forums with indentation, use the <code></code> tags (substitute "<>" for "[]")

Thank you so much! This compiles perfectly! I had a suspicion that I should use == as a comparative operator, but I thought you didn't have to do that in C++!

Thanks for the info on <code> these tags! </code>

I am glad it compiled my friend. Don't feel bad about the comparative operator, we all do the same mistake once in a while between one "if" or another

Cheers!