Предупреждения о конверсиях для std :: copy и std :: vector :: assign - программирование
Подтвердить что ты не робот

Предупреждения о конверсиях для std :: copy и std :: vector :: assign

Когда число с плавающей запятой вставляется в std::vector<int>, число должно быть преобразовано путем какого-либо округления. Обычно это изменяет число, 1.5 изменяется на 1 или 2, и я ожидал бы, что компилятор должен предупредить об этом преобразовании. Таким образом, я использую -Wconversion флаг g++ или клана g++. Это позволяет использовать предупреждения для std::vector::push_back или прямого назначения, но не для std::copy или std::vector::assign(iterator first, iterator end).

Теперь мой вопрос: как мне получить предупреждения о конверсиях для std::copy и std::vector::assign?

Вот моя примерная программа:

#include <iostream>
#include <vector>
#include <algorithm>

using source_type = std::vector<double>;
using target_type = std::vector<int>;

int main() {
    source_type vsource;
    target_type vtarget1;
    target_type vtarget2;
    target_type vtarget3;
    target_type vtarget4;

    // Fill source with a number
    vsource.push_back(1.5);

    // This will give a compiler warning as expected
    vtarget1.push_back(vsource.at(0));

    // This does not give a warning, why not?
    vtarget2.assign(vsource.begin(), vsource.end());

    // Also this does not give a warning, why not?
    vtarget3.assign(vsource.size(), 0);
    std::copy(vsource.begin(), vsource.end(), vtarget3.begin());


    // The following should be equivalent to std::copy according to
    // http://www.cplusplus.com/reference/algorithm/copy/?kw=copy
    // Here we get a warning as expected (in contrast to std::copy).
    vtarget4.assign(vsource.size(), 0);
    auto source = vsource.begin();
    auto target = vtarget4.begin();
    while (source != vsource.end()) {
       *target = *source;
       ++target; ++source;
    }

    std::cout << vsource.at(0)  << " "
              << vtarget1.at(0) << " "
              << vtarget2.at(0) << " "
              << vtarget3.at(0) << " "
              << vtarget4.at(0) << std::endl;

    return 0;
}

Я компилирую с:

g++ -Wall -Wextra -Wconversion -std=c++11 -pedantic 
clang++ -Wall -Wextra -Wconversion -std=c++11 -pedantic

Я получаю только два предупреждения, я хочу иметь еще несколько:

question.cpp:22:24: warning: implicit conversion turns floating-point number into integer: 'value_type' (aka 'double')
      to 'value_type' (aka 'int') [-Wfloat-conversion]
    vtarget1.push_back(vsource.at(0));

question.cpp:40:18: warning: implicit conversion turns floating-point number into integer: 'double' to 'int'
      [-Wfloat-conversion]
       *target = *source;
4b9b3361

Ответ 1

Вы можете использовать следующие флаги -std=c++11 -Wfloat-conversion -Wsystem-headers чтобы GCC дополнительно по крайней мере предупреждал об этом LOC

std::copy(vsource.begin(), vsource.end(), vtarget3.begin());

а также вы можете увидеть здесь.

Затем вывод будет

<source>: In function 'int main()':

<source>:21:34: warning: conversion from '__gnu_cxx::__alloc_traits<std::allocator<double>, double>::value_type' {aka 'double'} to 'std::vector<int>::value_type' {aka 'int'} may change value [-Wfloat-conversion]

21 |     vtarget1.push_back(vsource.at(0));

   |                        ~~~~~~~~~~^~~

<source>:38:18: warning: conversion from 'double' to 'int' may change value [-Wfloat-conversion]

38 |        *target = *source;

   |                  ^~~~~~~

In file included from /opt/compiler-explorer/gcc-trunk-20180917/include/c++/9.0.0/bits/char_traits.h:39,

                 from /opt/compiler-explorer/gcc-trunk-20180917/include/c++/9.0.0/ios:40,

                 from /opt/compiler-explorer/gcc-trunk-20180917/include/c++/9.0.0/ostream:38,

                 from /opt/compiler-explorer/gcc-trunk-20180917/include/c++/9.0.0/iostream:39,

                 from <source>:1:

/opt/compiler-explorer/gcc-trunk-20180917/include/c++/9.0.0/bits/stl_algobase.h: In instantiation of 'static _OI std::__copy_move<false, false, std::random_access_iterator_tag>::__copy_m(_II, _II, _OI) [with _II = double*; _OI = int*]':

/opt/compiler-explorer/gcc-trunk-20180917/include/c++/9.0.0/bits/stl_algobase.h:400:30:   required from '_OI std::__copy_move_a(_II, _II, _OI) [with bool _IsMove = false; _II = double*; _OI = int*]'

/opt/compiler-explorer/gcc-trunk-20180917/include/c++/9.0.0/bits/stl_algobase.h:437:30:   required from '_OI std::__copy_move_a2(_II, _II, _OI) [with bool _IsMove = false; _II = __gnu_cxx::__normal_iterator<double*, std::vector<double> >; _OI = __gnu_cxx::__normal_iterator<int*, std::vector<int> >]'

/opt/compiler-explorer/gcc-trunk-20180917/include/c++/9.0.0/bits/stl_algobase.h:470:7:   required from '_OI std::copy(_II, _II, _OI) [with _II = __gnu_cxx::__normal_iterator<double*, std::vector<double> >; _OI = __gnu_cxx::__normal_iterator<int*, std::vector<int> >]'

<source>:28:63:   required from here

/opt/compiler-explorer/gcc-trunk-20180917/include/c++/9.0.0/bits/stl_algobase.h:338:18: warning: conversion from 'double' to 'int' may change value [-Wfloat-conversion]

338 |        *__result = *__first;

    |                  ^

Compiler returned: 0

Что касается clang: этот набор флагов довольно многословен и не дает больше понимания.