Làm thế nào để tôi im lặng tốt nhất một cảnh báo về các biến không sử dụng?

Tôi có một ứng dụng đa nền tảng và trong một vài chức năng của tôi, không phải tất cả các giá trị được truyền cho các hàm đều được sử dụng. Do đó tôi nhận được cảnh báo từ GCC nói với tôi rằng có các biến không được sử dụng.

Điều gì sẽ là cách tốt nhất của mã hóa xung quanh cảnh báo?

Một #ifdef xung quanh chức năng?

#ifdef _MSC_VER
void ProcessOps::sendToExternalApp(QString sAppName, QString sImagePath, qreal qrLeft, qreal qrTop, qreal qrWidth, qreal qrHeight)
#else
void ProcessOps::sendToExternalApp(QString sAppName, QString sImagePath, qreal /*qrLeft*/, qreal /*qrTop*/, qreal /*qrWidth*/, qreal /*qrHeight*/)
#endif
{

Điều này là rất xấu nhưng có vẻ như cách trình biên dịch sẽ thích.

Hay tôi gán 0 cho biến ở cuối hàm? (điều mà tôi ghét vì nó làm thay đổi thứ gì đó trong luồng chương trình để tắt tiếng cảnh báo trình biên dịch).

Có một cách chính xác?

Bạn có thể đặt nó trong (void)var;biểu thức "" (không làm gì) để trình biên dịch thấy nó được sử dụng. Đây là di động giữa các trình biên dịch.

Ví dụ

void foo(int param1, int param2)
{
    (void)param2;
    bar(param1);
}

Hoặc là,

#define UNUSED(expr) do { (void)(expr); } while (0)
...

void foo(int param1, int param2)
{
    UNUSED(param2);
    bar(param1);
}

Trong GCC và Clang, bạn có thể sử dụng __attribute__((unused))chỉ thị tiền xử lý để đạt được mục tiêu của mình.
Ví dụ:

int foo (__attribute__((unused)) int bar) {
   return 0;
}

C ++ 17 hiện cung cấp [[maybe_unused]]thuộc tính.

http://en.cppreference.com/w/cpp/lingu/attribut

Khá đẹp và chuẩn.

Giải pháp hiện tại của bạn là tốt nhất - nhận xét tên tham số nếu bạn không sử dụng nó. Điều đó áp dụng cho tất cả các trình biên dịch, vì vậy bạn không phải sử dụng bộ xử lý trước để làm điều đó đặc biệt cho GCC.

Cập nhật C ++ 17

Trong C ++ 17, chúng ta có được thuộc tính [[might_unuse]] được bao phủ trong [dcl.attr.unuse]

Mã thông báo thuộc tính có thể_unuse chỉ ra rằng tên hoặc thực thể có thể không được sử dụng có chủ ý. Nó sẽ xuất hiện nhiều nhất một lần trong mỗi danh sách thuộc tính và không có mệnh đề đối số thuộc tính nào xuất hiện. ...

Thí dụ:

 [[maybe_unused]] void f([[maybe_unused]] bool thing1,
                        [[maybe_unused]] bool thing2) {
  [[maybe_unused]] bool b = thing1 && thing2;
    assert(b);
 }

Việc triển khai không nên cảnh báo rằng b không được sử dụng, cho dù NDEBUG có được xác định hay không. Ví dụ

Ví dụ sau:

int foo ( int bar) {
    bool unused_bool ;
    return 0;
}

Cả clang và gcc đều tạo ra chẩn đoán bằng cách sử dụng -Wall -Wextra cho cả thanh và un used_bool ( Xem trực tiếp ).

Trong khi thêm [[may_unuse]] làm im lặng chẩn đoán:

int foo ([[maybe_unused]] int bar) {
    [[maybe_unused]] bool unused_bool ;
    return 0;
}

nhìn thấy nó sống .

Trước C ++ 17

Trong C ++ 11, một dạng thay thế của UNUSEDmacro có thể được hình thành bằng cách sử dụng biểu thức lambda ( thông qua Ben Deane ) với một bản ghi của biến không được sử dụng:

#define UNUSED(x) [&x]{}()

Yêu cầu ngay lập tức của biểu thức lambda nên được tối ưu hóa, đưa ra ví dụ sau:

int foo (int bar) {
    UNUSED(bar) ;
    return 0;
}

chúng ta có thể thấy trong godbolt rằng cuộc gọi được tối ưu hóa đi:

foo(int):
xorl    %eax, %eax
ret

Một cách thậm chí còn sạch hơn là chỉ nhận xét tên biến:

int main(int /* argc */, char const** /* argv */) {
  return 0;
}

Một đồng nghiệp chỉ cho tôi đến macro nhỏ tốt đẹp này ở đây

Để dễ dàng tôi sẽ bao gồm các macro bên dưới.

#ifdef UNUSED
#elif defined(__GNUC__) 
# define UNUSED(x) UNUSED_ ## x __attribute__((unused)) 
#elif defined(__LCLINT__) 
# define UNUSED(x) /*@unused@*/ x 
#else 
# define UNUSED(x) x 
#endif

void dcc_mon_siginfo_handler(int UNUSED(whatsig))

gcckhông gắn cờ các cảnh báo này theo mặc định. Cảnh báo này phải được bật một cách rõ ràng bằng cách chuyển -Wunused-parameterđến trình biên dịch hoặc ngầm định bằng cách chuyển -Wall -Wextra(hoặc có thể một số tổ hợp cờ khác).

Các cảnh báo tham số không được sử dụng đơn giản có thể được loại bỏ bằng cách chuyển -Wno-unused-parameterđến trình biên dịch, nhưng lưu ý rằng cờ vô hiệu hóa này phải xuất hiện sau bất kỳ cờ kích hoạt nào có thể cho cảnh báo này trong dòng lệnh của trình biên dịch, để nó có thể có hiệu lực.

cách ít macro và di động để khai báo một hoặc nhiều tham số là không sử dụng:

template <typename... Args> inline void unused(Args&&...) {}

int main(int argc, char* argv[])
{
    unused(argc, argv);
    return 0;
}

Sử dụng các chỉ thị tiền xử lý được coi là xấu xa hầu hết thời gian. Lý tưởng nhất là bạn muốn tránh chúng như Pest. Hãy nhớ rằng làm cho trình biên dịch hiểu mã của bạn là dễ dàng, cho phép các lập trình viên khác hiểu mã của bạn khó hơn nhiều. Một vài chục trường hợp như thế này ở đây và ở đó làm cho nó rất khó để đọc cho chính bạn sau này hoặc cho người khác ngay bây giờ.

Một cách có thể là đặt các tham số của bạn lại với nhau thành một loại lớp đối số. Sau đó, bạn chỉ có thể sử dụng một tập hợp con của các biến (tương đương với việc gán 0 thực sự của bạn) hoặc có các chuyên môn khác nhau của lớp đối số đó cho mỗi nền tảng. Điều này tuy nhiên có thể không đáng, bạn cần phân tích xem nó có phù hợp không.

Nếu bạn có thể đọc các mẫu không thể, bạn có thể tìm thấy các mẹo nâng cao trong sách "Đặc biệt C ++". Nếu những người đọc mã của bạn có thể có được bộ kỹ năng của họ để bao gồm những thứ điên rồ được dạy trong cuốn sách đó, thì bạn sẽ có mã đẹp mà cũng có thể dễ dàng đọc được. Trình biên dịch cũng sẽ nhận thức rõ về những gì bạn đang làm (thay vì ẩn mọi thứ bằng cách tiền xử lý)

Trước hết cảnh báo được tạo bởi định nghĩa biến trong tệp nguồn chứ không phải tệp tiêu đề. Tiêu đề có thể nguyên sơ và nên, vì bạn có thể đang sử dụng một cái gì đó như doxygen để tạo tài liệu API.

Tôi sẽ giả định rằng bạn có triển khai hoàn toàn khác nhau trong các tệp nguồn. Trong những trường hợp này, bạn có thể nhận xét tham số vi phạm hoặc chỉ viết tham số.

Thí dụ:

func(int a, int b)
{
    b;
    foo(a);
}

Điều này có vẻ khó hiểu, do đó, đã xác định một macro như UNUSED. Cách mà MFC đã làm là:

#ifdef _DEBUG
#define UNUSED(x)
#else
#define UNUSED(x) x
#endif

Như thế này bạn thấy cảnh báo vẫn còn trong các bản dựng gỡ lỗi, có thể hữu ích.

Nó không an toàn để luôn luôn nhận xét tên tham số? Nếu không, bạn có thể làm một cái gì đó như

#ifdef _MSC_VER
# define P_(n) n
#else
# define P_(n)
#endif

void ProcessOps::sendToExternalApp(
    QString sAppName, QString sImagePath,
    qreal P_(qrLeft), qreal P_(qrTop), qreal P_(qrWidth), qreal P_(qrHeight))

Nó hơi xấu xí một chút .

Sử dụng một UNREFERENCED_PARAMETER(p) có thể làm việc. Tôi biết nó được định nghĩa trong WinNT.h cho các hệ thống Windows và cũng có thể dễ dàng được định nghĩa cho gcc (nếu nó chưa có nó).

UNREFERENCED PARAMETER(p) được định nghĩa là

#define UNREFERENCED_PARAMETER(P)          (P)

trong WinNT.h.

Tôi đã thấy điều này thay vì (void)param2cách im lặng cảnh báo:

void foo(int param1, int param2)
{
    std::ignore = param2;
    bar(param1);
}

Có vẻ như điều này đã được thêm vào trong C ++ 11

Sử dụng cờ của trình biên dịch, ví dụ cờ cho GCC: -Wno-unused-variable

Bạn có thể sử dụng __unusedđể báo cho trình biên dịch rằng biến có thể không được sử dụng.

- (void)myMethod:(__unused NSObject *)theObject    
{
    // there will be no warning about `theObject`, because you wrote `__unused`

    __unused int theInt = 0;
    // there will be no warning, but you are still able to use `theInt` in the future
}

Trong C ++ 11, đây là giải pháp tôi đang sử dụng:

template<typename... Ts> inline void Unreferenced(Ts&&...) {}

int Foo(int bar) 
{
    Unreferenced(bar);
    return 0;
}

int Foo2(int bar1, int bar2) 
{
    Unreferenced(bar1, bar2);
    return 0;
}

Được xác minh là có thể mang theo (ít nhất là trên msvc, clang và gcc hiện đại) và không tạo thêm mã khi tối ưu hóa được bật. Không có tối ưu hóa, cuộc gọi chức năng bổ sung được thực hiện và các tham chiếu đến các tham số được sao chép vào ngăn xếp, nhưng không có macro liên quan.

Nếu mã bổ sung là một vấn đề, bạn có thể sử dụng khai báo này thay thế:

(decltype(Unreferenced(bar1, bar2)))0;

nhưng tại thời điểm đó, một macro cung cấp khả năng đọc tốt hơn:

#define UNREFERENCED(...) { (decltype(Unreferenced(__VA_ARGS__)))0; }

Điều này hoạt động tốt nhưng yêu cầu C ++ 11

template <typename ...Args>
void unused(Args&& ...args)
{
  (void)(sizeof...(args));
}

Tôi thấy hầu hết các câu trả lời được trình bày chỉ hoạt động cho biến không sử dụng cục bộ và sẽ gây ra lỗi biên dịch cho biến toàn cục tĩnh không sử dụng.

Một macro khác cần thiết để ngăn chặn cảnh báo của biến toàn cục tĩnh không sử dụng.

template <typename T>
const T* UNUSED_VARIABLE(const T& dummy) { 
    return &dummy;
}
#define UNUSED_GLOBAL_VARIABLE(x) namespace {\
    const auto dummy = UNUSED_VARIABLE(x);\
}

static int a = 0;
UNUSED_GLOBAL_VARIABLE(a);

int main ()
{
    int b = 3;
    UNUSED_VARIABLE(b);
    return 0;
}

Điều này hoạt động vì không có cảnh báo nào được báo cáo cho biến toàn cục không tĩnh trong không gian tên ẩn danh.

C ++ 11 là bắt buộc

 g++  -Wall -O3  -std=c++11 test.cpp

Cười lớn! Tôi không nghĩ rằng có một câu hỏi khác về SO tiết lộ tất cả những kẻ dị giáo bị hỏng bởi Chaos tốt hơn câu hỏi này!

Với tất cả sự tôn trọng đối với C ++ 17, có một hướng dẫn rõ ràng trong Nguyên tắc cốt lõi của C ++ . AFAIR, trở lại vào năm 2009, tùy chọn này đã có sẵn cũng như ngày hôm nay. Và nếu ai đó nói rằng nó được coi là một lỗi trong Doxygen thì có một lỗi trong Doxygen

Tôi không thấy vấn đề của bạn với cảnh báo. Tài liệu này trong tiêu đề phương thức / hàm mà trình biên dịch xy sẽ đưa ra cảnh báo (đúng) ở đây, nhưng các biến đó là cần thiết cho nền tảng z.

Cảnh báo là chính xác, không cần phải tắt nó. Nó không làm mất hiệu lực chương trình - nhưng nó phải được ghi lại, rằng có một lý do.