C++ Boost: Was ist die Ursache für diese Warnung?

Question

Ich habe ein einfaches C++ mit Boost wie dieses:

#include <boost/algorithm/string.hpp>

int main()
{
  std::string latlonStr = "hello,ergr()()rg(rg)";
  boost::find_format_all(latlonStr,boost::token_finder(boost::is_any_of("(,)")),boost::const_formatter(" "));

Dies funktioniert gut; jedes Vorkommen von ( ) wird durch ein " " ersetzt.

Beim Kompilieren erhalte ich jedoch diese Warnung:

Ich verwende MSVC 2008, Boost 1.37.0.

1>Compiling...
1>mainTest.cpp
1>c:\work\minescout-feat-000\extlib\boost\algorithm\string\detail\classification.hpp(102) : warning C4996: 'std::copy': Function call with parameters that may be unsafe - this call relies on the caller to check that the passed values are correct. To disable this warning, use -D_SCL_SECURE_NO_WARNINGS. See documentation on how to use Visual C++ 'Checked Iterators'
1>        c:\program files (x86)\microsoft visual studio 9.0\vc\include\xutility(2576) : see declaration of 'std::copy'
1>        c:\work\minescout-feat-000\extlib\boost\algorithm\string\classification.hpp(206) : see reference to function template instantiation 'boost::algorithm::detail::is_any_ofF<CharT>::is_any_ofF<boost::iterator_range<IteratorT>>(const RangeT &)' being compiled
1>        with
1>        [
1>            CharT=char,
1>            IteratorT=const char *,
1>            RangeT=boost::iterator_range<const char *>
1>        ]
1>        c:\work\minescout-feat-000\minescouttest\maintest.cpp(257) : see reference to function template instantiation 'boost::algorithm::detail::is_any_ofF<CharT> boost::algorithm::is_any_of<const char[4]>(RangeT (&))' being compiled
1>        with
1>        [
1>            CharT=char,
1>            RangeT=const char [4]
1>        ]

Ich könnte die Warnung natürlich deaktivieren, indem ich

-D_SCL_SECURE_NO_WARNINGS

aber ich zögere ein bisschen, das zu tun, bevor ich herausgefunden habe, was falsch ist, oder noch wichtiger, ob mein Code falsch ist.

Answer 1

Das ist kein Grund zur Sorge. In den letzten Versionen von MSVC sind sie in den vollen Sicherheits-Paranoia-Modus gegangen. std::copy gibt diese Warnung aus, wenn es mit rohen Zeigern verwendet wird, weil bei unsachgemäßer Verwendung kann es zu Pufferüberläufen kommen.

Ihre Iterator-Implementierung führt eine Überprüfung der Grenzen durch, um sicherzustellen, dass dies nicht geschieht, was jedoch erhebliche Leistungseinbußen zur Folge hat.

Sie können die Warnung also gerne ignorieren. Sie bedeutet nicht, dass mit Ihrem Code etwas nicht stimmt. Sie besagt nur, dass wenn etwas mit Ihrem Code nicht stimmt, dann werden schlimme Dinge passieren. Das ist eine merkwürdige Sache, vor der man warnen sollte ;)

Answer 2

Sie können diese Warnung auch in bestimmten Kopfzeilen deaktivieren:

#if defined(_MSC_VER) && _MSC_VER >= 1400 
#pragma warning(push) 
#pragma warning(disable:4996) 
#endif 

/* your code */ 

#if defined(_MSC_VER) && _MSC_VER >= 1400 
#pragma warning(pop) 
#endif

Answer 3

Wenn Sie sich sicher fühlen, diesen Fehler zu deaktivieren:

• Gehen Sie zu den Eigenschaften Ihres C++-Projekts
• Erweitern Sie die "C/C++"
• Markieren Sie "Befehlszeile".
• Fügen Sie unter "Additional Options" (Zusätzliche Optionen) den folgenden Text an den Text an, der sich in diesem Feld befinden könnte

" -D_SCL_SECURE_NO_WARNINGS"

Answer 4

Die Warnung stammt von den nicht standardmäßigen "sicheren" Bibliotheksprüfungen von Visual Studio, die mit MSVC 8.0 eingeführt wurden. Microsoft hat "potenziell gefährliche" APIs identifiziert und Warnungen eingefügt, die von deren Verwendung abraten. Es ist zwar technisch möglich, std::copy auf unsichere Weise aufzurufen, aber 1) diese Warnung zu erhalten bedeutet nicht, dass Sie dies tun, und 2) die Verwendung von std::copy, wie Sie es normalerweise tun sollten, ist nicht gefährlich.

Answer 5

Wenn Sie C++11 verwenden und die Warnungen nicht abschalten wollen, haben Sie alternativ die schmerzhafte Möglichkeit, die

boost::is_any_of(L"(,)")

mit dem folgenden Lambda-Ausdruck

[](wchar_t &c) { for (auto candidate : { L'(', L',', L')' }) { if (c == candidate) return true; }; return false; }

Möglicherweise können Sie das auch in ein Makro packen