Неожиданные результаты использования istringstream для двойного преобразования строки в C++

StackOverflow https://stackoverflow.com/questions/1698265

Вопрос

В настоящее время я пытаюсь взять строку («0.1») и преобразовать ее в двойное число, используя C++ в Xcode в версии 10.6 с помощью gcc4.2.

Я использую функцию, которую я взял из Другой вопрос, но когда я пытаюсь использовать эту функцию, мой ввод в соответствии с GDB равен (строка) «0,1», но мой вывод — (двойной) 2.1220023981051542e-314.

Вот мой фрагмент, скопированный прямо из кода:

double strToDouble(std::string const &numberString)
{
    istringstream inStream(numberString);
    double doubleValue;
    inStream >> doubleValue;
    if(!(inStream && (inStream >> std::ws).eof()))
    {
        return 0.0;  
    }

    return doubleValue;
};

Я использую C++, а не Obj-C, так как в конечном итоге его, вероятно, придется скомпилировать на машине *nix или Windows.

Я, естественно, PHP-программист, но мне нужно ускорить некоторые вычисления, поэтому я делаю это на компилируемом языке.Прошло много времени с тех пор, как университет занимался языком более низкого уровня =P.Так что, если у кого-то есть идеи, буду очень признателен...

Дрю Дж.Зонне.

Полный код:

#include <iostream>
#include <vector>
#include <string>
#include <sstream>
using namespace std;

/*
 * @var delimiters the string to define a break by
 * @var str the string to break
 * @var tokens where to store the broken string parts.
 */
void explode(const string& delimiters, const string& str, vector<string>& tokens)
{
    // Skip delimiters at beginning.
    string::size_type lastPos = str.find_first_not_of(delimiters, 0);
    // Find first "non-delimiter".
    string::size_type pos     = str.find_first_of(delimiters, lastPos);

    while (string::npos != pos || string::npos != lastPos)
    {
        // Found a token, add it to the vector.
        tokens.push_back(str.substr(lastPos, pos - lastPos));
        // Skip delimiters.  Note the "not_of"
        lastPos = str.find_first_not_of(delimiters, pos);
        // Find next "non-delimiter"
        pos = str.find_first_of(delimiters, lastPos);
    }
};

/*
 * @var numberString double to be converted as a string.
 */
double strToDouble(std::string const &numberString)
{
    istringstream inStream(numberString);
    double doubleValue;
    inStream >> doubleValue;
    if(!(inStream && (inStream >> std::ws).eof()))
    {
        return 0.0;  
    }

    return doubleValue;
};

class Cluster {
private:
    vector<double> latStore;
    vector<double> lngStore;
public:
    Cluster(int,string,string);
};

/*
 * @var latString a comma seperated list of doubles for the latitudes.
 * @var lngString a comma separated list of doubles for the longitudes.
 */
Cluster::Cluster(int newLocationLength, string latString, string lngString)
{
    // mark our vectors
    vector<string> latStoreString;
    vector<string> lngStoreString;

    // Explode our input strings.
    explode(",", latString, latStoreString);
    explode(",", lngString, lngStoreString);

    for( int i = 0; i < latStoreString.size(); i++)
    {
        // Grab the latitude and store it.
        string tempLat = latStoreString[i];
        double latDouble = strToDouble(tempLat);
        latStore.push_back( strToDouble(tempLat) );

        // Grab the longitude and store it.
        string tempLng = lngStoreString[i];
        lngStore.push_back( strToDouble(tempLng) );
    }
}

int main (int argc, char * const argv[]) {

    Cluster *rect = new Cluster("0.1,0.4","0.1,0.4");

    return 0;
}
Это было полезно?

Решение

Это может быть вызвано режимом отладки STL.Удалите макросы _GLIBCXX_DEBUG в настройках сборки макросов препроцессора вашей цели.

Сборки отладки C++ сломались в X-Code Snow Leopard

Другие советы

Почему бы вместо этого не использовать atof()? связь

Я не вижу ничего плохого в вашем фрагменте кода.Можете ли вы показать код, который вы используете для выход результат

Как предположил Майлс, atof() может быть проще.Вы можете преобразовать строковый объект в строку c, вызвав функцию-член c_str():

double result = atof(inputString.c_str()); // 0.0 if a double couldn't be read

Вот еще информация: http://www.cplusplus.com/reference/clibrary/cstdlib/atof/

#include <iostream>
#include <sstream>

double strToDouble(std::string const &numberString)
{
    std::istringstream inStream(numberString);
    double doubleValue;
    std::cout << doubleValue << '\n';
    inStream >> doubleValue;
    if(!(inStream && (inStream >> std::ws).eof()))
    {
        return 0.0;  
    }

    return doubleValue;
};

int main()
{
    std::cout << strToDouble("0.1") << '\n';

    return 0;
}

Приведенный выше код дает мне следующий вывод:
-2.36907e-39
0.1

Вы случайно не проверяете значение doubleValue перед возвратом?


Cluster::Cluster(string latString, string lngString)
{
    // mark our vectors
    vector<string> latStoreString;
    vector<string> lngStoreString;

    // Explode our input strings.
    explode(",", latString, latStoreString);
    explode(",", lngString, lngStoreString);

    for( int i = 0; i < latStoreString.size(); i++)
    {
        // Grab the latitude and store it.
        string tempLat = latStoreString[i];
        std::cout << "tempLat=" << tempLat << '\n';
        double latDouble = strToDouble(tempLat);
        std::cout << "latDouble=" << latDouble << '\n';
        latStore.push_back( strToDouble(tempLat) );

        // Grab the longitude and store it.
        string tempLng = lngStoreString[i];
        lngStore.push_back( strToDouble(tempLng) );
    }
}

int main (int argc, char * const argv[]) {

    Cluster *rect = new Cluster("0.1,0.4","0.1,0.4");

    return 0;
}

Выход:
темпЛат=0,1
latDouble=0,1
tempLat=0,4
latDouble=0,4

Можете ли вы запустить код с этими изменениями (вы не использовали параметр int в ctor, поэтому я его просто удалил).Я также запустил ваш код с помощью valgrind без ошибок (только утечки памяти).

Похоже, вы не получили ответа на свой вопрос с реализацией.Другие предложили использовать atof().Если вы хотите решить проблему в своей реализации, посмотрите приведенный ниже код.Вместо метода векторного разнесения используйте вектор, чтобы избежать избыточного кода.Я также изменил код в strToDouble.

void explode(const string& delimiters, const string& str, vector<double>& tokens)
{    
     // Skip delimiters at beginning.   
     string::size_type lastPos = str.find_first_not_of(delimiters, 0);   
     // Find first "non-delimiter".    
     string::size_type pos = str.find_first_of(delimiters, lastPos);
     while (string::npos != pos || string::npos != lastPos)   
     {        
              // Found a token, add it to the vector.       
             string temp = str.substr(lastPos, pos - lastPos);
             tokens.push_back(strToDouble(temp));    
             // Skip delimiters.  Note the "not_of"        
            lastPos = str.find_first_not_of(delimiters, pos);        
            // Find next "non-delimiter"        
            pos = str.find_first_of(delimiters, lastPos);    
     }
}

double strToDouble(std::string const &numberString)
{  
   istringstream inStream(numberString);    
   int pos = numberString.find_first_not_of("1234567890.", 0);

   if (string::npos != pos)
   {
       return 0.0;
   }
   else
   {
      double doubleValue;   
      inStream >> doubleValue;        
      return doubleValue;
  }
}
Лицензировано под: CC-BY-SA с атрибуция
Не связан с StackOverflow
scroll top