Предисловие
Современные компиляторы обладают огромным количеством диагностик. И удивительно, что очень малая их часть включена по умолчанию.
Огромное количество претензий, которые предъявляют к языку C++ в этих ваших интернетах, — про сложность, небезопасность, стрельбу по ногам и т.п., — относятся как раз к тем случаям, когда люди просто не знают о том, что можно решить эти проблемы лёгким движением пальцев по клавиатуре.
Давайте же исправим эту вопиющую несправедливость, и прольём свет истины на возможности компилятора по предотвращению ошибок.
Содержание
- Ода компилятору
- Игнорировать нельзя исправить
- -Wall
- -Wextra
- -Wpedantic
- Нужно больше предупреждений
- -Werror
- Заключение
- Ссылки
Ода компилятору
Компилятор – лучший друг плюсовика. Компилятор — это не просто транслятор формального человекочитаемого языка в машинные коды. Компилятор — лучший помощник в написании программ.
Важная (и не единственная) помощь, которую оказывает компилятор — поиск ошибок. И я говорю не об опечатках, несовпадении типов и прочих синтаксических ошибках. Я говорю об огромном наборе ошибок, которые можно выловить с помощью механизма предупреждений.
Часто встречаю мнение о том, что предупреждений слишком много, они дают ложноположительные результаты, мешают работать, замыливают глаз, отвлекают от «настоящих» ошибок и т.п. Такое действительно бывает, но это большая редкость.
Игнорировать нельзя исправить
Большинство предупреждений — это не «бзик» компилятора, который можно просто проигнорировать. Предупреждение — это потенциальная ошибка. Предупреждение — это сигнал от компилятора о том, что написано одно, а требуется, возможно, что-то совершенно иное.
Поэтому программист должен помочь компилятору понять, как трактовать спорную ситуацию. То есть либо поправить свою ошибку, либо сообщить компилятору явно о том, что нужно верить программисту и делать именно то, что написано. Причём это поможет не только компилятору, но и человеку, который будет читать код. Лишний static_cast
или пара скобок будут явно свидетельствовать о том, что имелось в виду именно то, что написано.
Далее я расскажу о наиболее важных на мой взгляд предупреждениях и покажу, какие ошибки можно отловить с их помощью.
Надеюсь, что данное не слишком занимательное чтиво поможет правильно поставить запятую в заголовке этого раздела.
Сразу хочу оговориться, что далее речь пойдёт исключительно о языке C++ и компиляторе GCC (впрочем, подавляющая часть информации актуальна и для компилятора Clang). Информацию о других компиляторах и языках придётся искать в соответствующих справочниках.
-Wall
-Wall
— это «агрегатор» базовых предупреждений. В языке C++ он включает в себя длинный перечень предупреждений, каждое из которых будет рассмотрено отдельно (на самом деле, рассмотрены будут не все, а только те, которые непосредственно помогают выявлять ошибки).
Важно:
Несмотря на название, этот флаг включает далеко не все предупреждения, которые умеет обнаруживать компилятор.
В этом разделе:
- -Waddress
- -Warray-bounds=1
- -Wbool-compare
- -Wbool-operation
- -Wcatch-value
- -Wchar-subscripts
- -Wcomment
- -Wint-in-bool-context
- -Winit-self
- -Wlogical-not-parentheses
- -Wmaybe-uninitialized
- -Wmemset-elt-size
- -Wmemset-transposed-args
- -Wmisleading-indentation
- -Wmissing-attributes
- -Wmultistatement-macros
- -Wnonnull
- -Wnonnull-compare
- -Wparentheses
- -Wpessimizing-move
- -Wreorder
- -Wreturn-type
- -Wsequence-point
- -Wsign-compare
- -Wsizeof-pointer-div
- -Wsizeof-pointer-memaccess
- -Wstrict-aliasing
- -Wswitch
- -Wtautological-compare
- -Wtrigraphs
- -Wuninitialized
- -Wunused-function
- -Wunused-variable
-Waddress
Предупреждает о странной работе с адресами. Например, об использовании адреса функции в условном выражении. Такое может произойти, если забыть поставить скобки после имени функции:
bool func () {return false;}
int main ()
{
if (func)
{
return 1;
}
}
prog.cc: In function 'int main()':
prog.cc:5:9: warning: the address of 'bool func()' will never be NULL [-Waddress]
5 | if (func)
| ^~~~
Также этот флаг может спасти от типичной ошибки новичка — сравнения строкового литерала с адресом. Очевидно, программист хотел сравнить строки, но в результате сравнил два указателя:
int main ()
{
const char * a = "abc";
if (a == "abc")
{
return 0;
}
}
Компилятор бдит:
prog.cc: In function 'int main()':
prog.cc:4:11: warning: comparison with string literal results in unspecified behavior [-Waddress]
4 | if (a == "abc")
| ~~^~~~~~~~
-Warray-bounds=1
Предупреждает о выходе за пределы массивов. Используется только вместе с -O2
.
int main()
{
int a[5] = {1, 2, 3, 4, 5};
return a[5];
}
prog.cc: In function 'int main()':
prog.cc:4:15: warning: array subscript 5 is above array bounds of 'int [5]' [-Warray-bounds]
4 | return a[5];
| ~~~^
prog.cc:3:9: note: while referencing 'a'
3 | int a[5] = {1, 2, 3, 4, 5};
| ^
-Wbool-compare
Предупреждает о сравнении булева выражения с целым числом, которое нельзя трактовать как булево:
int main ()
{
int n = 5;
if ((n > 1) == 2)
{
return 0;
}
}
prog.cc: In function 'int main()':
prog.cc:5:17: warning: comparison of constant '2' with boolean expression is always false [-Wbool-compare]
5 | if ((n > 1) == 2)
| ~~~~~~~~^~~~
-Wbool-operation
Предупреждает о подозрительных операциях с булевыми выражениями. Например, о побитовом отрицании:
int main ()
{
bool b = true;
auto c = ~b;
}
prog.cc: In function 'int main()':
prog.cc:4:15: warning: '~' on an expression of type 'bool' [-Wbool-operation]
4 | auto c = ~b;
| ^
prog.cc:4:15: note: did you mean to use logical not ('!')?
Что касается инкрементов и декрементов булевых переменных, то в C++17 это просто ошибки, безо всяких предупреждений.
-Wcatch-value
Предупреждает о обработчиках исключений, которые принимают полиморфные объекты по значению:
struct A
{
virtual ~A () {};
};
struct B: A{};
int main ()
{
try {}
catch (A) {}
}
prog.cc: In function 'int main()':
prog.cc:11:12: warning: catching polymorphic type 'struct A' by value [-Wcatch-value=]
11 | catch (A) {}
| ^
Есть и более сильные версии предупреждения: -Wcatch-value=n
(см. справку к компилятору).
-Wchar-subscripts
Предупреждает об обращении к массиву по индексу, тип которого char
. А ведь char
является знаковым на многих машинах:
int main ()
{
int a[] = {1, 2, 3, 4};
char index = 'a' - 'b';
return a[index];
}
prog.cc: In function 'int main()':
prog.cc:5:14: warning: array subscript has type 'char' [-Wchar-subscripts]
5 | return a[index];
| ^~~~~
Предупреждает о наличии последовательности, начинающей новый комментарий (/*
), внутри многострочного комментария, либо о разрыве строки при помощи обратного слеша внутри однострочного комментария.
int main ()
{
/* asd /* fgh */
//qwe
rty
}
prog.cc:3:8: warning: "/*" within comment [-Wcomment]
3 | /* /* */
|
prog.cc:4:5: warning: multi-line comment [-Wcomment]
4 | //ssd
| ^
-Wint-in-bool-context
Предупреждает о подозрительном использовании целых чисел там, где ожидаются булевы выражения, например, в условных выражениях:
int main ()
{
int a = 5;
if (a <= 4 ? 2 : 3)
{
return 0;
}
}
prog.cc: In function 'int main()':
prog.cc:4:16: warning: ?: using integer constants in boolean context, the expression will always evaluate to 'true' [-Wint-in-bool-context]
4 | if (a <= 4 ? 2 : 3)
| ~~~~~~~^~~~~~~
Другой пример — операции побитового сдвига в булевом контексте. Вполне вероятно, что здесь произошла опечатка, и имелся в виду не сдвиг, а сравнение:
int main ()
{
for (auto a = 0; 1 << a; a++);
}
prog.cc: In function 'int main()':
prog.cc:3:24: warning: '<<' in boolean context, did you mean '<' ? [-Wint-in-bool-context]
3 | for (auto a = 0; 1 << a; a++);
| ~~^~~~
А также сообщает о любых видах умножения в булевом контексте.
int main ()
{
int a = 5;
if (a * 5);
}
prog.cc: In function 'int main()':
prog.cc:4:11: warning: '*' in boolean context, suggest '&&' instead [-Wint-in-bool-context]
4 | if (a * 5);
| ~~^~~
-Winit-self
Предупреждает об инициализации переменных самими сабями. Используется только вместе с флагом -Wuninitialized
.
int main ()
{
int i = i;
return i;
}
prog.cc: In function 'int main()':
prog.cc:3:9: warning: 'i' is used uninitialized in this function [-Wuninitialized]
3 | int i = i;
| ^
-Wlogical-not-parentheses
Предупреждает об использовании логического отрицания в левой части сравнения. При этом если правая часть сравнения является сама по себе булевым выражением, то предупреждения не будет.
Используется для того, чтобы отлавливать подозрительные конструкции вроде следующей:
int main ()
{
int a = 9;
if (!a > 1);
}
prog.cc: In function 'int main()':
prog.cc:5:12: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses]
5 | if (!a > 1);
| ^
prog.cc:5:9: note: add parentheses around left hand side expression to silence this warning
5 | if (!a > 1);
| ^~
| ( )
Традиционный способ сообщить компилятору, что так и было задумано — поставить скобки, о чём и сообщает компилятор.
-Wmaybe-uninitialized
Предупреждает о том, что существует возможность использования непроинициализированной переменной.
int main (int argc, const char * argv[])
{
int x;
switch (argc)
{
case 1: x = 1;
break;
case 2: x = 4;
break;
case 3: x = 5;
}
return x;
}
prog.cc: In function 'int main(int, const char**)':
prog.cc:3:9: warning: 'x' may be used uninitialized in this function [-Wmaybe-uninitialized]
3 | int x;
| ^
В данном конкретном случае решается с помощью конструкции default
:
int main (int argc, const char * argv[])
{
int x;
switch (argc)
{
case 1: x = 1;
break;
case 2: x = 4;
break;
case 3: x = 5;
break;
default:
x = 17;
}
return x;
}
-Wmemset-elt-size
Предупреждает о подозрительных вызовах функции memset
, когда первый аргумент — это массив, а третий аргумент — количество элементов в массиве, но не количество байт, занимаемой этим массивом в памяти.
#include <cstring>
int main ()
{
constexpr auto size = 20ul;
int a[size];
std::memset(a, 0, size);
}
prog.cc: In function 'int main()':
prog.cc:7:27: warning: 'memset' used with length equal to number of elements without multiplication by element size [-Wmemset-elt-size]
7 | std::memset(a, 0, size);
| ^
-Wmemset-transposed-args
Предупреждает о том, что пользователь, вероятно, перепутал порядок аргументов в функции memset
:
#include <cstring>
int main ()
{
constexpr auto size = 20ul;
int a[size];
std::memset(a, size, 0);
}
prog.cc: In function 'int main()':
prog.cc:7:27: warning: 'memset' used with constant zero length parameter; this could be due to transposed parameters [-Wmemset-transposed-args]
7 | std::memset(a, size, 0);
| ^
-Wmisleading-indentation
Предупреждает о том, что отступы в коде не отражают структуру этого кода. Особенно это актуально для конструкций if
, else
, while
и for
. Пример:
int main ()
{
int x;
if (true)
x = 3;
return x;
}
prog.cc: In function 'int main()':
prog.cc:4:5: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
4 | if (true)
| ^~
prog.cc:6:9: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if'
6 | return x;
| ^~~~~~
См. также -Wempty-body
.
-Wmissing-attributes
Предупреждает о ситуации, когда специализация шаблона объявлена не с тем же списком атрибутов, что и оригинальный шаблон.
template <class T>
T* __attribute__ ((malloc, alloc_size (1)))
foo (long);
template <>
void* __attribute__ ((malloc))
foo<void> (long);
int main ()
{
}
prog.cc:7:1: warning: explicit specialization 'T* foo(long int) [with T = void]' may be missing attributes [-Wmissing-attributes]
7 | foo<void> (long);
| ^~~~~~~~~
prog.cc:3:1: note: missing primary template attribute 'alloc_size'
3 | foo (long);
| ^~~
-Wmultistatement-macros
Предупреждает о макросах, состоящих из нескольких инструкций, и используемых в выражениях if
, else
, while
и for
. В такой ситуации под действие выражений попадает только первая инструкция макроса, и это, вероятно, ошибка:
#define INCREMENT x++; y++
int main ()
{
int x = 0;
int y = 0;
if (true)
INCREMENT;
}
prog.cc: In function 'int main()':
prog.cc:1:19: warning: macro expands to multiple statements [-Wmultistatement-macros]
1 | #define INCREMENT x++; y++
| ^
prog.cc:8:9: note: in expansion of macro 'INCREMENT'
8 | INCREMENT;
| ^~~~~~~~~
prog.cc:7:5: note: some parts of macro expansion are not guarded by this 'if' clause
7 | if (true)
| ^~
См. также -Wmisleading-indentation
.
-Wnonnull
Предупреждает о передаче нулевого указателя в функцию, аргументы которой помечены атрибутом nonnull
.
void f (int * ptr) __attribute__((nonnull));
int main ()
{
f(nullptr);
}
void f (int *) {}
prog.cc: In function 'int main()':
prog.cc:5:14: warning: null argument where non-null required (argument 1) [-Wnonnull]
5 | f(nullptr);
| ^
-Wnonnull-compare
Предупреждает о сравнении с нулём аргумента функции, помеченного атрибутом nonnull
.
bool f (int * ptr) __attribute__((nonnull));
int main ()
{
f(nullptr);
}
bool f (int * p)
{
return p == nullptr;
}
prog.cc: In function 'bool f(int*)':
prog.cc:10:17: warning: nonnull argument 'p' compared to NULL [-Wnonnull-compare]
10 | return p == nullptr;
| ^~~~~~~
-Wparentheses
Типичный случай — опечатались, и вместо равенства написали присвоение:
int main ()
{
int x = 5;
if (x = 4)
{
x = 3;
}
}
Компилятор, естественно, сомневается:
prog.cc: In function 'int main()':
prog.cc:4:11: warning: suggest parentheses around assignment used as truth value [-Wparentheses]
4 | if (x = 4)
| ~~^~~
Либо исправляем код, либо убеждаем компилятор в том, что мы хотели именно присвоение:
int main ()
{
int x = 5;
if ((x = 4))
{
x = 3;
}
}
-Wpessimizing-move
Иногда явная попытка переноса может ухудшить производительность. Пример:
#include <utility>
struct A {};
A f ()
{
A a;
return std::move(a);
}
int main ()
{
f();
}
prog.cc: In function 'A f()':
prog.cc:8:21: warning: moving a local object in a return statement prevents copy elision [-Wpessimizing-move]
8 | return std::move(a);
| ~~~~~~~~~^~~
prog.cc:8:21: note: remove 'std::move' call
-Wreorder
Предупреждает о том, что порядок инициализации членов класса не соответствует порядку их объявления. Поскольку компилятор может переупорядочить инициализацию этих членов, результат может быть неочевидным.
struct A
{
int i;
int j;
A(): j(0), i(1)
{
}
};
int main () {}
prog.cc: In constructor 'A::A()':
prog.cc:4:9: warning: 'A::j' will be initialized after [-Wreorder]
4 | int j;
| ^
prog.cc:3:9: warning: 'int A::i' [-Wreorder]
3 | int i;
| ^
prog.cc:6:5: warning: when initialized here [-Wreorder]
6 | A(): j(0), i(1)
| ^
-Wreturn-type
Предупреждает о том, что из функции не вернули заявленный результат:
int f ()
{
}
int main () {}
prog.cc: In function 'int f()':
prog.cc:3:1: warning: no return statement in function returning non-void [-Wreturn-type]
3 | }
| ^
-Wsequence-point
Сообщает о подозрительных операциях относительно точек следования. Любимый пример (никогда так не делайте):
int main ()
{
int a = 6;
++a = a++;
}
prog.cc: In function 'int main()':
prog.cc:4:12: warning: operation on 'a' may be undefined [-Wsequence-point]
4 | ++a = a++;
| ~^~
-Wsign-compare
Одно из важнейших предупреждений. Сообщает о сравнении знаковых и беззнаковых чисел, которое может произвести некорректный результат из-за неявных преобразований. К примеру, отрицательное знаковое число неявно приводится к беззнаковому и внезапно становится положительным:
int main ()
{
short a = -5;
unsigned b = 4;
if (a < b)
{
return 0;
}
return 1;
}
prog.cc: In function 'int main()':
prog.cc:5:11: warning: comparison of integer expressions of different signedness: 'short int' and 'unsigned int' [-Wsign-compare]
5 | if (a < b)
| ~~^~~
-Wsizeof-pointer-div
Предупреждает о подозрительном делении друг на друга двух результатов выражения sizeof
, когда размер указателя делится на размер объекта. Обычно это бывает, когда пытаются вычислить размер массива, но вместо массива по ошибке берут указатель:
int main ()
{
const char * a = "12345";
auto size = sizeof (a) / sizeof (a[0]);
}
prog.cc: In function 'int main()':
prog.cc:4:28: warning: division 'sizeof (const char*) / sizeof (const char)' does not compute the number of array elements [-Wsizeof-pointer-div]
4 | auto size = sizeof (a) / sizeof (a[0]);
| ~~~~~~~~~~~^~~~~~~~~~~~~~~
prog.cc:3:18: note: first 'sizeof' operand was declared here
3 | const char * a = "12345";
| ^
-Wsizeof-pointer-memaccess
Предупреждает о подозрительных параметрах, передаваемых в строковые функции и функции для работы с памятью (str...
, mem...
и т.п.), и использующих оператор sizeof
. Например:
#include <cstring>
int main ()
{
char a[10];
const char * s = "qwerty";
std::memcpy (a, s, sizeof(s));
}
prog.cc: In function 'int main()':
prog.cc:7:24: warning: argument to 'sizeof' in 'void* memcpy(void*, const void*, size_t)' call is the same expression as the source; did you mean to provide an explicit length? [-Wsizeof-pointer-memaccess]
7 | std::memcpy (a, s, sizeof(s));
| ^~~~~~~~~
-Wstrict-aliasing
Каламбур типизации (strict aliasing) — это отдельная большая тема для разговора. Предлагаю читателю найти литературу по этой теме самостоятельно.
В общем, это тоже крайне полезное предупреждение.
-Wswitch
Предупреждает о том, что не все элементы перечисления задействованы в конструкции switch
:
enum struct enum_t
{
a, b, c
};
int main ()
{
enum_t e = enum_t::a;
switch (e)
{
case enum_t::a:
case enum_t::b:
return 0;
}
}
prog.cc: In function 'int main()':
prog.cc:9:12: warning: enumeration value 'c' not handled in switch [-Wswitch]
9 | switch (e)
| ^
-Wtautological-compare
Предупреждает о бессмысленном сравнении переменной с самой собой:
int main ()
{
int i = 1;
if (i > i);
}
prog.cc: In function 'int main()':
prog.cc:4:11: warning: self-comparison always evaluates to false [-Wtautological-compare]
4 | if (i > i);
| ~ ^ ~
Кроме того, сообщает о сравнениях при участии битовых операций, которые имеют всегда один и тот же результат (всегда истинно или всегда ложно):
int main ()
{
int i = 1;
if ((i & 16) == 10);
}
prog.cc: In function 'int main()':
prog.cc:4:18: warning: bitwise comparison always evaluates to false [-Wtautological-compare]
4 | if ((i & 16) == 10);
| ~~~~~~~~ ^~ ~~
-Wtrigraphs
Предупреждает о наличии триграфов, которые могут изменить смысл программы. Не сообщается о триграфах в теле комментария, за исключением случаев, когда триграф трактуется как перевод строки.
См. также -Wcomment
.
-Wuninitialized
Предупреждает об использовании переменных и членов класса, которые не были проинициализированы:
int main ()
{
int x;
return x;
}
prog.cc: In function 'int main()':
prog.cc:4:12: warning: 'x' is used uninitialized in this function [-Wuninitialized]
4 | return x;
| ^
-Wunused-function
Предупреждает о том, что статическая функция объявлена, но не определена, либо о том, что статическая функция, не помеченная как inline
, не используется.
-Wunused-variable
Предупреждает о том, что переменная не используется.
int main ()
{
int x = 0;
}
prog.cc: In function 'int main()':
prog.cc:3:9: warning: unused variable 'x' [-Wunused-variable]
3 | int x = 0;
| ^
Для того, чтобы помочь компилятору понять, что так и задумывалось, можно использовать конструкцию static_cast<void>(...)
:
int main ()
{
int x = 0;
static_cast<void>(x);
}
-Wextra
«Агрегатор» дополнительных предупреждений. Включает много интересного, чего нет в -Wall
(как и в случае с -Wall
, рассмотрены будут не все возможности).
В этом разделе:
- -Wempty-body
- -Wimplicit-fallthrough
- -Wmissing-field-initializers
- -Wredundant-move
- -Wtype-limits
- -Wshift-negative-value
- -Wunused-parameter
- -Wunused-but-set-parameter
-Wempty-body
Предупреждает о пустом теле условных выражений или цикла do-while
. Чаще всего это говорит об опечатке, меняющей логику программы:
int main ()
{
if (true);
{
return 1;
}
}
prog.cc: In function 'int main()':
prog.cc:3:14: warning: suggest braces around empty body in an 'if' statement [-Wempty-body]
3 | if (true);
| ^
См. также -Wmisleading-indentation
.
Предупреждает о «проваливании» в операторе switch
:
int main ()
{
int x = 7;
int a;
switch (x)
{
case 1:
a = 1;
break;
case 2:
a = 2;
case 3:
a = 3;
break;
default:
a = 0;
}
return a;
}
Компилятор предполагает, что программист забыл break
, и case 2
не должен проваливаться:
prog.cc: In function 'int main()':
prog.cc:11:15: warning: this statement may fall through [-Wimplicit-fallthrough=]
11 | a = 2;
| ~~^~~
prog.cc:13:9: note: here
13 | case 3:
| ^~~~
В C++17 для обозначения явного намерения появился специальный атрибут — fallthrough
:
int main ()
{
int x = 7;
int a;
switch (x)
{
case 1:
a = 1;
break;
case 2:
a = 2;
[[fallthrough]];
case 3:
a = 3;
break;
default:
a = 0;
}
return a;
}
Предупреждает о том, что отдельные члены структуры не были проинициализированы. Скорее всего это просто забыли сделать:
struct S
{
int f;
int g;
int h;
};
int main ()
{
S s{3, 4};
}
prog.cc: In function 'int main()':
prog.cc:10:13: warning: missing initializer for member 'S::h' [-Wmissing-field-initializers]
10 | S s{3, 4};
| ^
Предупреждает о ненужном вызове std::move
в случаях, когда компилятор сам сделает всё, что нужно:
#include <utility>
struct S {};
S f (S s)
{
return std::move(s);
}
int main ()
{
auto s = f(S{});
}
prog.cc: In function 'S f(S)':
prog.cc:7:21: warning: redundant move in return statement [-Wredundant-move]
7 | return std::move(s);
| ~~~~~~~~~^~~
prog.cc:7:21: note: remove 'std::move' call
Предупреждает о сравнениях, которые всегда имеют один и тот же результат. Например, когда беззнаковое число проверяется на неотрицательность. Если программист делает такую проверку, то, видимо, предполагает, что число в теории может быть отрицательным, однако, это не так. Видимо, он где-то ошибся:
int main ()
{
unsigned x = 17;
if (x >= 0)
{
return 1;
}
}
prog.cc: In function 'int main()':
prog.cc:4:11: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
4 | if (x >= 0)
| ~~^~~~
Предупреждает об операциях сдвига для отрицательных значений. Отрицательными могут быть только знаковые числа, а для них это некорректно:
int main ()
{
const int x = -7;
return x << 4;
}
prog.cc: In function 'int main()':
prog.cc:4:17: warning: left shift of negative value [-Wshift-negative-value]
4 | return x << 4;
| ^
Предупреждает о неиспользуемом параметре функции. Возможно, про него просто забыли, и в этом случае функция может работать некорректно.
void f (int x) {}
int main ()
{
}
prog.cc: In function 'void f(int)':
prog.cc:1:13: warning: unused parameter 'x' [-Wunused-parameter]
1 | void f (int x) {}
| ~~~~^
В C++17 для явного выражения намерения существует атрибут maybe_unused
:
void f ([[maybe_unused]] int x) {}
int main ()
{
}
Предупреждает о том, что в параметр функции было записано значение, но после этого он ни разу не использовался. Возможно, про него снова забыли:
void f (int x)
{
x = 7;
}
int main ()
{
}
prog.cc: In function 'void f(int)':
prog.cc:1:13: warning: parameter 'x' set but not used [-Wunused-but-set-parameter]
1 | void f (int x)
| ~~~~^
-Wpedantic
-Wall
и -Wextra
— это не всё, на что способен компилятор.
В дополнение к ним существует флаг -Wpedantic
(он же -pedantic
), который проверяет соответствие кода стандарту ISO C++, сообщает об использовании запрещённых расширений, о наличии лишних точек с запятой, нехватке переноса строки в конце файла и прочих полезных штуках.
Нужно больше предупреждений
Но и это ещё не всё. Есть несколько флагов, которые почему-то не входят ни в один из «аргегаторов», но крайне важны и полезны.
В этом разделе:
- -Wctor-dtor-privacy
- -Wnon-virtual-dtor
- -Wold-style-cast
- -Woverloaded-virtual
- -Wsign-promo
- -Wduplicated-branches
- -Wduplicated-cond
- -Wfloat-equal
- -Wshadow=compatible-local
- -Wcast-qual
- -Wconversion
- -Wzero-as-null-pointer-constant
- -Wextra-semi
- -Wsign-conversion
- -Wlogical-op
-Wctor-dtor-privacy
Предупреждает о том, что класс выглядит неиспользуемым, потому что конструкторы и деструкторы закрыты, а друзей и открытых статических функций-членов у него нет.
class base
{
base () {};
~base() {};
};
int main ()
{
}
prog.cc:1:7: warning: 'class base' only defines a private destructor and has no friends [-Wctor-dtor-privacy]
1 | class base
| ^~~~
Аналогично, сообщает, что у класса есть закрытые функции-члены, а открытых нет ни одной.
-Wnon-virtual-dtor
Предупреждает о том, что у класса есть виртуальные функции-члены, но деструктор при этом не виртуальный. Очень сложно представить себе такой класс. Вероятнее всего, это ошибка.
struct base
{
virtual void f (int) {}
~base() {};
};
int main ()
{
}
prog.cc:1:8: warning: 'struct base' has virtual functions and accessible non-virtual destructor [-Wnon-virtual-dtor]
1 | struct base
| ^~~~
-Wold-style-cast
Предупреждает о приведении типов в стиле языка C. В плюсах есть прекрасные и ужасные static_cast
, dynamic_cast
, reinterpret_cast
и const_cast
, которые более локальны и более описательны. Сишный способ слишком сильный и — о, ужас, — небезопасный. Лучше его не использовать вообще.
-Woverloaded-virtual
Предупреждает о попытке в классе-наследнике перегрузить виртуальную функцию базового класса:
struct base
{
virtual void f (int) {}
};
struct derived: base
{
void f () {};
};
int main ()
{
}
prog.cc:3:18: warning: 'virtual void base::f(int)' was hidden [-Woverloaded-virtual]
3 | virtual void f (int) {}
| ^
prog.cc:8:10: warning: by 'void derived::f()' [-Woverloaded-virtual]
8 | void f () {};
| ^
-Wsign-promo
Крайне полезный флаг. Предупреждает о неочевидном выборе перегруженной функции:
void f (int) {}
void f (unsigned) {}
int main ()
{
unsigned short x = 7;
f(x);
}
prog.cc: In function 'int main()':
prog.cc:7:8: warning: passing 'short unsigned int' chooses 'int' over 'unsigned int' [-Wsign-promo]
7 | f(x);
| ^
prog.cc:7:8: warning: in call to 'void f(int)' [-Wsign-promo]
Вероятнее всего, хотели-таки позвать вторую перегрузку, а не первую. А если всё-таки первую, то будьте любезны сказать об этом явно.
-Wduplicated-branches
Предупреждает о том, что ветви if
и else
одинаковы:
int main ()
{
if (true)
return 0;
else
return 0;
}
prog.cc: In function 'int main()':
prog.cc:3:5: warning: this condition has identical branches [-Wduplicated-branches]
3 | if (true)
| ^~
Условный оператор ?:
также под прицелом:
int main ()
{
auto x = true ? 4 : 4;
}
prog.cc: In function 'int main()':
prog.cc:3:19: warning: this condition has identical branches [-Wduplicated-branches]
3 | auto x = true ? 4 : 4;
| ~~~~~^~~~~~~
Для меня абсолютная загадка, почему этот флаг не включён не то, что в -Wall
, а вообще по умолчанию.
См. также -Wduplicated-cond
.
-Wduplicated-cond
Предупреждает об одинаковых условиях в цепочках if-else-if
:
int main ()
{
auto x = 6;
if (x > 7) {return 0;}
else if (x > 7) {return 1;}
}
prog.cc: In function 'int main()':
prog.cc:5:10: warning: duplicated 'if' condition [-Wduplicated-cond]
5 | else if (x > 7) {return 1;}
| ^~
prog.cc:4:5: note: previously used here
4 | if (x > 7) {return 0;}
| ^~
См. также -Wduplicated-branches
.
-Wfloat-equal
Предупреждает о проверке на равенство между двумя числами с плавающей точкой. Скорее всего, это ошибка, и сравнение нужно проводить с заданной точностью.
Если же требуется именно сравнить на равенство (такое редко, но бывает), то можно использовать std::equal_to
, который под предупреждение не попадает.
-Wshadow=compatible-local
Полезная опция, которая не даёт перекрыть локальную переменную другой локальной переменной при условии, что они имеют совместимые типы.
-Wcast-qual
Предупреждает о преобразовании указателя, при котором сбрасываются квалификаторы. Например, чтобы случайно не потерять const
.
-Wconversion
Очень, очень, очень важный флаг. Он предупреждает об огромном количестве неявных сужающих (то есть потенциально приводящих к потере информации) преобразований, которые могут быть следствием ошибки программиста. Например:
int main ()
{
double x = 4.5;
int y = x;
}
prog.cc: In function 'int main()':
prog.cc:12:13: warning: conversion from 'double' to 'int' may change value [-Wfloat-conversion]
12 | int y = x;
| ^
Если вы раньше никогда не включали этот флаг, то будет интересно.
-Wzero-as-null-pointer-constant
Предупреждает об использовании целочисленного нуля вместо nullptr
.
Флаг для педантов. Сообщает о лишней точке с запятой после определения функции-члена.
-Wsign-conversion
Как и -Wconversion
помогает предотвратить большое количество неявных преобразований, которые запросто могут быть ошибками:
int main ()
{
signed si = -8;
unsigned ui;
ui = si;
}
prog.cc: In function 'int main()':
prog.cc:5:10: warning: conversion to 'unsigned int' from 'int' may change the sign of the result [-Wsign-conversion]
5 | ui = si;
| ^~
-Wlogical-op
Предупреждает о подозрительных логических выражениях. Например, когда вместо побитового «И» поставили логическое «И», или логическое выражение имеет одинаковые операнды:
int main ()
{
int a = 8;
if (a < 0 && a < 0)
{
return 1;
}
}
prog.cc: In function 'int main()':
prog.cc:4:15: warning: logical 'and' of equal expressions [-Wlogical-op]
if (a < 0 && a < 0)
~~~~~~^~~~~~~~
-Werror
С этого, вообще говоря, стоило бы начать. Данный флаг делает все предупреждения ошибками. Код не скомпилируется при наличии хотя бы одного предупреждения.
Без этого флага всё остальное имеет мало смысла. Но если понять и принять мысль о том, что предупреждение — это что-то подозрительное, и их быть не должно, то именно этот флаг и позволит поддерживать код в чистоте.
В дополнение к
-Werror
существует флаг-pedantic-errors
, который не эквивалентен комбинации-Wpedantic -Werror
.Да, всё непросто.
Заключение
Резюмируя, для компилятора GCC (Clang кое-что из этого не умеет, к сожалению) я рекомендую включать следующий минимальный набор флагов, по необходимости дополняя его более сложными диагностиками.
-Werror
-pedantic-errors
-Wall
-Wextra
-Wpedantic
-Wcast-align
-Wcast-qual
-Wconversion
-Wctor-dtor-privacy
-Wduplicated-branches
-Wduplicated-cond
-Wextra-semi
-Wfloat-equal
-Wlogical-op
-Wnon-virtual-dtor
-Wold-style-cast
-Woverloaded-virtual
-Wredundant-decls
-Wsign-conversion
-Wsign-promo
Да, такой список флагов может породить большое количество ошибок, которые поначалу могут показаться излишними. Но явное лучше неявного. Если знаешь, что делаешь — делай. Но делай это так, чтобы всем было понятно, что именно так ты и хотел. Поработав таким образом хотя бы неделю, вы поймёте, насколько это прекрасно, и уже не сможете вернуться обратно.
Любите ваш компилятор и помогайте ему помогать вам писать программы.
Ссылки
- Документация к компилятору GCC
- Документация к компилятору Clang
- Forum
- Beginners
- template nested classes and compilation
template nested classes and compilation error: no match for operator!=
Hi everybody,
I’m trying to create a matrix class. This class has a nested class called linear_t (which represents both columns and rows) and linear_t has another nested class called iterator_t.
Here is the code:
|
|
I get a compilation error when I try to use the operator!= from the main function. It seems the compiler cannot find the implementation, but I don’t understand the reason.
Here is the error:
main.cpp: In function `int main(int, char**)’:
main.cpp:10: error: no match for ‘operator!=’ in ‘it_1 != it_2’
And here is the fragment of code where the operator is invoked:
|
|
I don’t understand what is the problem. Any help or suggestion will be more than welcomed.
Thanks
What is more, there is no compilation errors if I invoke the operator!= method in the following way:
|
|
I cannot understand why in the first post the invokation is failing.
That is very strange. I have the same problem over here.
I can get around it by making the operator overload part of the iterator class rather than a global function (which is my preferred style). So I guess this is a solution:
|
|
As for why this error is happening, I don’t have the first idea. What’s even more baffling… if you make a friend declaration:
|
|
Now it turns into a linker error instead! So it finds the friend declaration in the class, but still doesn’t find the global function.
Very very strange. I don’t know why it’s doing that.
What compiler are you using?
According to
http://www.devx.com/cplus/10MinuteSolution/30302/1954
the correct declaration is
|
|
but neither GCC 4.x nor Comeau online compiler compile this successfully, however
GCC 3.x does like it.
Hi,
First of all, thanks to all of you for your replies.
jsmith, I’m using g++ 3.4.5. But the initial problem is not the friend declaration. The original problem is that the compiler cannot find the implementation of the operator!= () if I declare it as a global function (not as a member function) as I commented in the first post.
This is actually a headache-inducing problem. I discussed this with a colleague over lunch.
The problem is that the compiler is not able to deduce T from the invocation site.
Why? Let’s reduce the problem size a bit:
|
|
The above code still has the same problem.
Let’s say you had a specialization:
|
|
Now, what is T in the following code?
|
|
(Since matrix_t<int>::iterator is simply a typedef for int at that point,
it can’t deduce T.)
jsmith, thanks very much for your latest post. Your example is very clear. Does anybody know how to solve it?
Thanks in advance
I solved it in my first post.
Just make the function a member of iterator:
|
|
Yes, but I also want to define some algorithms (apart of the != or == operators) like for example:
|
|
And in that case I don’t think it is a good idea to define it as a member function (adding the Cmp template parameter to the class)
There isn’t a solution AFAIK except for Disch’s — you have to make them members that are
declared and implemented in the class.
Is not std::max_element() good enough? Do you really need to specialize?
Sorry for my insistence but I think there is something else. For example, when using the STL, we usually do things like this:
|
|
I think inheritance could be a possible solution (I don know if it is used in the STL) but we could do something like this (please, read the first post to understand the following fragment of code):
|
|
This solution presents two disadvantages:
1. inheritance implies less performance
2. each function requires two versions (see the following example for the operator!= method)
|
|
Ok, this is an easier way (in fact, this is a simplification of what I’ve seen for std::vector):
|
|
In regard to the original problem:
Here is an extract from the book «C++ templates the complete guide» concerning template argument deduction:
Qualified type names. A type name like Q<T>::X will never be used to deduce a template parameter T, for example.
Topic archived. No new replies allowed.
I have tried both the CLI and the Eclipse development environments, but each give me the same problems with the errors of:
In function ‘int main(int, char**)’: error: ‘loadImage’ was not declared in this scope; error: ‘showImage’ was not declared in this scope; error: ‘releaseImage’ was not declared in this scope;
I have the following include directives:
‘#include «cv.h»‘
‘#include «highgui.h»‘
and I have the exported environmental $LD_LIBRARY_PATH=»/usr/local/lib» which is correct as it contains opencv & opencv2 directories.
A> What include files are I missing, but surely the cv.h (aka cv.hpp) should have taken care of the declarations themselves?
B> Do I have to do a «nm <library_name>» to get the includes from each library or is there a easier way to get what header file prototypes are within which library ?
C> Could you advise what I need to do to fix the problems with the 2.5.4 release examples.
Actually to assist newbies to CV, it would be a good idea to have all the tutorial examples tested under each new release so it doesn’t turn newbies off, as they consider it all too hard just trying to break into OpenCV.
Thanks.
2 answers
A> Since the includes are below the opencv2 folder you need to specify those, too. Also please use the newer include-modules, i.e.:
#include <opencv2/core/core.hpp>
#include <opencv2/highgui/highgui.hpp>
Also note that it is not necessary to set the LD_LIBRARY_PATH to /usr/local/lib! However don’t forget to link against the libraries (if you search for Makefile in this Q&A forum you will also find several examples for working Makefiles).
B> You can see at the top of each documentation page (http://docs.opencv.org/) to which module each method belongs to.
C> The most examples should run out-of-the box and can be build via cmake (or ccmake). But you are right, it can happen that an example doesn’t work. I am not quite sure if they are tested via buildbot (http://build.opencv.org/) or not, seems like only the tests were built and run and not the example-module. Thus, please report if you find an example which doesn’t build by submitting an issue ticket (http://code.opencv.org/projects/opencv/issues/new) so that it can be fixed for the next release.
The following answers my predictament exactly.
IplImage* img = cvLoadImage( argv[1] );
cv::namedWindow( "Test3-Example1", CV_WINDOW_AUTOSIZE );
cvShowImage( "Test3-Example1", img );
cv::waitKey(0);
cvReleaseImage( &img );
cv::destroyWindow( "Test3-Example1" );
return 0;
LoadImage, ShowImage and ReleaseImage needed to have pre-appended «cv» with the third character capitalised.
I found this out by just trying different combinations of function calls from the CV library headers.
Question Tools
Related questions
rotciv 0 / 0 / 0 Регистрация: 04.03.2015 Сообщений: 55 |
||||||||||||
1 |
||||||||||||
Какая может быть причина ошибки компилятора?18.05.2015, 17:46. Показов 3528. Ответов 13 Метки нет (Все метки)
Добрый день! Не понимаю, что у меня неправильно. In file included from /usr/lib/gcc/x86_64-redhat-linux/4.4.7/../../../../include/c++/4.4.7/ios: Если добавить прототип копи конструктора в объявление, то ошибка пропадает. Но мне непонятно, какой код должен содержать этот копи конструктор и для чего он вообще нужен. Весь мой код довольно громоздкий, поэтому я постарался его минимизировать, при этом ошибка не воспроизвелась.
Файл pfr.cpp
Добавлено через 33 минуты
убирает и ошибку компилятора, и ошибку линковки. Код-то будет при этом работать правильно? Это можно как-то объяснить?
0 |
3257 / 2059 / 351 Регистрация: 24.11.2012 Сообщений: 4,909 |
|
18.05.2015, 17:55 |
2 |
В приведенном коде ошибки нет, а вот там, куда объект класса передается по значению — ожидается конструктор копирования, о чем нам и сообщают:
main.cpp:71: note: synthesized method ?PsfFileReader::PsfFileReader(const PsfFileReader&)? firs
0 |
18490 / 9642 / 2361 Регистрация: 30.01.2014 Сообщений: 16,899 |
|
18.05.2015, 17:55 |
3 |
Код-то будет при этом работать правильно? Вероятно, нет.
Весь мой код довольно громоздкий, поэтому я постарался его минимизировать, при этом ошибка не воспроизвелась. Что поделать, но без полного кода сложно диагностировать проблему.
1 |
3257 / 2059 / 351 Регистрация: 24.11.2012 Сообщений: 4,909 |
|
18.05.2015, 17:57 |
4 |
Но вот почему компилятор не устроил автоматически сгенерированный конструктор У класса один из мемберов — типа ifstream, а потоки не копируются по дефолту.
1 |
18490 / 9642 / 2361 Регистрация: 30.01.2014 Сообщений: 16,899 |
|
18.05.2015, 19:16 |
5 |
У класса один из мемберов — типа ifstream, а потоки не копируются по дефолту. Телепатия в ход пошла? Я тоже мог предположить (например насчет ссылки в качестве поля класса). Но лучше когда автор сам скажет.
0 |
236 / 196 / 21 Регистрация: 04.06.2014 Сообщений: 1,309 |
|
18.05.2015, 19:23 |
6 |
Не-а. Не скажет. Давно завис.
0 |
3257 / 2059 / 351 Регистрация: 24.11.2012 Сообщений: 4,909 |
|
18.05.2015, 19:25 |
7 |
Телепатия в ход пошла? Да почему. Первый фрагмент кода, 28 строка: ifstream File;
2 |
18490 / 9642 / 2361 Регистрация: 30.01.2014 Сообщений: 16,899 |
|
18.05.2015, 22:11 |
8 |
Да почему. Первый фрагмент кода, 28 строка: ifstream File; А ты глазастый
0 |
Alex5 1130 / 789 / 232 Регистрация: 12.04.2010 Сообщений: 2,012 |
||||||||
18.05.2015, 22:25 |
9 |
|||||||
Сообщение было отмечено rotciv как решение Решение
/usr/lib/gcc/x86_64-redhat-linux/4.4.7/../../../../include/c++/4.4.7/bits/ios_base.h:790: error Получается попытка скопировать частные данные.
main.cpp:71: error: initializing argument 1 of ?int Analyzer::StepThroughData(PsfFileReader, Может быть, попробовать использовать ссылки?
или
1 |
3257 / 2059 / 351 Регистрация: 24.11.2012 Сообщений: 4,909 |
|
19.05.2015, 03:41 |
10 |
Получается попытка скопировать частные данные. Попытка вызвать приватный конструктор копирования.
1 |
Alex5 1130 / 789 / 232 Регистрация: 12.04.2010 Сообщений: 2,012 |
||||
19.05.2015, 10:40 |
11 |
|||
/usr/lib/gcc/x86_64-redhat-linux/4.4.7/../../../../include/c++/4.4.7/bits/ios_base.h:790: error
Попытка вызвать приватный конструктор копирования. Верно, здесь конструктор копирования. Добавлено через 12 минут error C2248: std::basic_ios<_Elem,_Traits>::basic_ios: невозможно обратиться к private член, объявленному в классе «std::basic_ios<_Elem,_Traits>» Сообщение диагностики возникло в созданной компилятором функции «std::basic_ifstream<_Elem,_Traits>::basic_ifstream(const std::basic_ifstream<_Elem,_Traits> &)»
0 |
rotciv 0 / 0 / 0 Регистрация: 04.03.2015 Сообщений: 55 |
||||||||||||
19.05.2015, 12:26 [ТС] |
12 |
|||||||||||
Воспроизвелась ошибка! Я не догадался сам, что это с передачей в функцию связано.
Добавлено через 4 минуты In file included from /usr/lib/gcc/x86_64-redhat-linux/4.4.7/../../../../include/c++/4.4.7/ios:39, Добавлено через 10 минут
Может быть, попробовать использовать ссылки?
Это — помогло.
Всем огромное спасибо!
0 |
2021 / 1348 / 385 Регистрация: 16.05.2013 Сообщений: 3,482 Записей в блоге: 6 |
|
19.05.2015, 12:33 |
13 |
Смею предположить, что у вас в классе PsfFileReader есть поле File которое является потоком. При прередаче объекта данного класса в функцию int StepThroughData(PsfFileReader PFR, PsfFileReader PFR2); вызывается конструктор копирования. Однако вы не удосужились его написать и он генерируется по умолчанию, что в данном случае приводит к генерации кода приводящего к копированию полей класса. Однако поле File является потоком и для него конструктор копий закрыт. Пишите свой конструктор копий для класса PsfFileReader или передавайте параметр по ссылке.
1 |
0 / 0 / 0 Регистрация: 04.03.2015 Сообщений: 55 |
|
19.05.2015, 13:06 [ТС] |
14 |
И Вы правы!
0 |
- Печать
Страницы: [1] 2 3 Все Вниз
Тема: простая программа на с++ не компилируется (Прочитано 4401 раз)
0 Пользователей и 1 Гость просматривают эту тему.
Abbattar
помогите разобраться, что не так вот программа из учебника
kdevelop в проблемах пишет, что nNcelsius — declaration not found, терминал выдаёт то-же:
~$ g++ -c -o conversation /home/deny/projects/kdevelop/conversion.cpp
/home/deny/projects/kdevelop/conversion.cpp: In function ‘int main(int, char**)’:
/home/deny/projects/kdevelop/conversion.cpp:18: error: ‘nNcelsius’ was not declared in this scope
как правильно работать с функцией cin?
Новое — не значит лучшее.
Not eXist
Вам же компилятор всё и так говорит.
Переменная nNcelsius не объявлена. И если посмотреть, то так и есть. Вы объявили переменную nNCelsius, а пытаетесь использовать nNcelsius
[DarkNet]Alpha
1. Зачем Вам stdio?
2. За using namespace std надо бить. Так нельзя. надо std::cin и std::cout.
И да, что это за ужасные аргументы main? Какой редактор их создал?
int main(int argc, char *argv[]) же
« Последнее редактирование: 20 Февраля 2011, 09:33:38 от [DarkNet]Alpha »
Orfer
как правильно работать с функцией cin?
Я бы не стал называть cin функцией. Стандартным потоком, объектом, но не функцией.
[DarkNet]Alpha
Чо умный да?
Orfer
2. За using namespace std надо бить. Так нельзя. надо std::cin и std::cout.
За using namespace std бить НЕ надо. Так пишут повсеместно в небольших программах
[DarkNet]Alpha
Даже в маленькой программе есть вероятность сделать переменную с таким же именем, как у какой-нибудь из std. И тогда будет плохо.
kstati
помогите разобраться, что не так вот программа из учебника
int nNCelsius;
cout << «Введите температуру по Цельсию: » ;
cin >> nNcelsius;/home/deny/projects/kdevelop/conversion.cpp:18: error: ‘nNcelsius’ was not declared in this scope
Риторический вопрос. Что такое регистрозависимость?
Sova777
Даже в маленькой программе есть вероятность сделать переменную с таким же именем, как у какой-нибудь из std. И тогда будет плохо.
И более глобальная переменная будет скрыта. Нет? В этом нет ужаса. В крайнем случаи получишь ошибку компилятора. Что-то типа:
welcome.cc:39:5: error: reference to 'a' is ambiguous
«using namespace» не нужно использовать заголовочных файлах (*.hpp). Но это другая история.
b.h:12:9: error: candidates are: int bbb::a
a.h:12:9: error: int aaa::a
Пользователь OpenSolaris 2008.11, Ubuntu 8.10, Windows XP. Mac OS X не нравится, стараюсь не использовать.
Orfer
помогите разобраться, что не так вот программа из учебника
int nNCelsius;
cout << «Введите температуру по Цельсию: » ;
cin >> nNcelsius;/home/deny/projects/kdevelop/conversion.cpp:18: error: ‘nNcelsius’ was not declared in this scope
Риторический вопрос. Что такое регистрозависимость?
Для компилятора имя nNCelsius будет отличаться от имени nNcelsius, поскольку буквы, набранные в разных регистрах будут иметь разный ASCII-код. Нужно объявлять и использовать переменные, пользуясь строго одним и тем же именем (с точки зрения регистра символов). Иначе — ошибка!
Думаю, что это приучает к аккуратности и внимательности!
Abbattar
Вам же компилятор всё и так говорит.
Переменная nNcelsius не объявлена. И если посмотреть, то так и есть. Вы объявили переменную nNCelsius, а пытаетесь использовать nNcelsius
Это задачка из учебника, который только начал изучать и как правильно объявить эту переменную я не в курсе, по-сути скопировал учебник (2001 года, старьё конечно. но другого пока нет), вот я и пытаюсь разобраться как всё сделать.
Пользователь решил продолжить мысль 20 Февраля 2011, 18:24:10:
помогите разобраться, что не так вот программа из учебника
int nNCelsius;
cout << «Введите температуру по Цельсию: » ;
cin >> nNcelsius;/home/deny/projects/kdevelop/conversion.cpp:18: error: ‘nNcelsius’ was not declared in this scope
Риторический вопрос. Что такое регистрозависимость?
О да-да, всё ошибка пропала фух! , только как теперь проверить результат, тип скомпилированного файла >> application/x-object ?
« Последнее редактирование: 20 Февраля 2011, 18:46:11 от Abbattar »
Новое — не значит лучшее.
Orfer
Abbattar
В терминале перейти в каталог, где скомпилирована программа, затем её запустить
./имя_программыну это я и попробовал — не вышло потому и спрашиваю
Новое — не значит лучшее.
Orfer
А почему не вышло? Что в терминале появилось?
kstati
Если не указывать gcc куда переносить скомпилированный код, то он будет располагаться в a.out
То есть запускать надо ./a.out
А за одно изучить ключики gcc
- Печать
Страницы: [1] 2 3 Все Вверх
От: |
wayfaring |
||
Дата: | 17.12.13 06:27 | ||
Оценка: |
Привет.
Скажите, почему последний mingw не может собрать такой код:
#include <stdio.h>
int main( int argc, char ** argv )
{
FILE * f = _popen( argv[ 1 ], "r" );
}
g++ -c -pipe -fno-keep-inline-dllexport -O2 -std=c++0x -fno-exceptions -frtti -Wall -Wextra main.cpp
main.cpp: In function ‘int main(int, char**)’:
main.cpp:6:39: error: ‘_popen’ was not declared in this scope
FILE * f = _popen( argv[ 1 ], «r» );
От: |
MTD |
https://github.com/mtrempoltsev | |
Дата: | 17.12.13 06:37 | ||
Оценка: |
Здравствуйте, wayfaring, Вы писали:
W>Скажите, почему последний mingw не может собрать такой код:
W>main.cpp:6:39: error: ‘_popen’ was not declared in this scope
Он не знает, что такое _popen, так как это специфично для VS.
От: |
niXman
|
https://github.com/niXman | |
Дата: | 17.12.13 06:37 | ||
Оценка: |
собирается и работает. проверил на последней сборке.
$ g++ -pipe -fno-keep-inline-dllexport -O2 -std=c++0x -fno-exceptions -frtti -Wall -Wextra popen.cpp -opopen
popen.cpp: In function ‘int main(int, char**)’:
popen.cpp:5:12: warning: unused variable ‘f’ [-Wunused-variable]
FILE * f = _popen( argv[ 1 ], «r» );
^
popen.cpp: At global scope:
popen.cpp:3:5: warning: unused parameter ‘argc’ [-Wunused-parameter]
int main( int argc, char ** argv )
^
пачка бумаги А4 стОит 2000 р, в ней 500 листов. получается, лист обычной бумаги стОит дороже имперского рубля =)
От: |
niXman
|
https://github.com/niXman | |
Дата: | 17.12.13 06:38 | ||
Оценка: |
6 (1) |
Здравствуйте, MTD, Вы писали:
MTD>Он не знает, что такое _popen, так как это специфично для VS.
mingw пока еще использует msvcrt.dll
пачка бумаги А4 стОит 2000 р, в ней 500 листов. получается, лист обычной бумаги стОит дороже имперского рубля =)
От: |
visual_wind |
||
Дата: | 17.12.13 12:39 | ||
Оценка: |
4 (1) |
Здравствуйте, wayfaring, Вы писали:
W>Привет.
W>Скажите, почему последний mingw не может собрать такой код:
W>
W>#include <stdio.h>
W>int main( int argc, char ** argv )
W>{
W> FILE * f = _popen( argv[ 1 ], "r" );
W>}
W>
W>g++ -c -pipe -fno-keep-inline-dllexport -O2 -std=c++0x -fno-exceptions -frtti -Wall -Wextra main.cpp
W>main.cpp: In function ‘int main(int, char**)’:
W>main.cpp:6:39: error: ‘_popen’ was not declared in this scope
W> FILE * f = _popen( argv[ 1 ], «r» );
Похоже, что вы комилируете с включенной поддержкой С++11, что автоматически добавляет __STRICT_ANSI__. Подробнее можно посмотреть здесь
От: |
niXman
|
https://github.com/niXman | |
Дата: | 17.12.13 13:13 | ||
Оценка: |
Здравствуйте, visual_wind, Вы писали:
_>Похоже, что вы комилируете с включенной поддержкой С++11, что автоматически добавляет __STRICT_ANSI__. Подробнее можно посмотреть здесь
я тоже компилировал с включенной поддержкой С++11, и ничего
пачка бумаги А4 стОит 2000 р, в ней 500 листов. получается, лист обычной бумаги стОит дороже имперского рубля =)
От: |
visual_wind |
||
Дата: | 17.12.13 14:18 | ||
Оценка: |
Здравствуйте, niXman, Вы писали:
X>я тоже компилировал с включенной поддержкой С++11, и ничего
Ну, тогда еще шанс — неизвестно, что там за версия компилятора у топикстартера.
- Переместить
- Удалить
- Выделить ветку
Пока на собственное сообщение не было ответов, его можно удалить.