conversion problem

[caduel]

while that fix might compile, it is wrong.

try

void checkpos(char cmd[3])
{
  int count=0;
  int move[5]; // why an array? this code never can parse more than one...
  const char *tmp;
  for(int i=0;i<3;++i)
      if (cmd[i]==',')
      {
          tmp=&cmd[i+1];
          move[count++]=atoi(tmp);
 // and now? break? cmd can not contain further moves....
 // what about the (skipped) 'x'?
       }
}

void checkpos(char cmd[3])
{
  int count=0;
  int move[5]; // why an array? this code never can parse more than one...
  const char *tmp;
  for(int i=0;i<3;++i)
      if (cmd[i]==',')
      {
          tmp=&cmd[i+1];
          move[count++]=atoi(tmp);
 // and now? break? cmd can not contain further moves....
 // what about the (skipped) 'x'?
       }
}

To copy to clipboard, switch view to plain text mode 

Note that char cmd[3] is quite a limitation and prevents you from parsing stuff like "15,x"

[pitonyak]

I would be inclined to not worry about such low level processing and simply pass strings around. Something more like this:

void simpleRegEx(const QString& s)
{
    QRegExp regExp("(\\d+)\\s*,\\s*(\\S+)");
    if (regExp.indexIn(s) >= 0)
    {
        qDebug(qPrintable(QString("Number = %1").arg(regExp.cap(1).toLong())));
        qDebug(qPrintable(QString("Command = %1").arg(regExp.cap(2))));
    }
}

void simpleRegEx(const QString& s)
{
    QRegExp regExp("(\\d+)\\s*,\\s*(\\S+)");
    if (regExp.indexIn(s) >= 0)
    {
        qDebug(qPrintable(QString("Number = %1").arg(regExp.cap(1).toLong())));
        qDebug(qPrintable(QString("Command = %1").arg(regExp.cap(2))));
    }
}

To copy to clipboard, switch view to plain text mode 

[Petr_Kropotkin]

Thanks all. The problem has been solved.