Fix for C++ Compiler Error - multiple definition

Question

i am trying to make a program that gave you sum of array elements absolute value .

this is the header file :

#include <iostream>
using std::cin,std::cout,std::endl;

#pragma clang diagnostic push
#pragma ide diagnostic ignored "cppcoreguidelines-narrowing-conversions"
void getAbsSum(int arr[10]){
int abs = 0,sum = 0;
int len = *(&arr + 1) - arr;
for (int i = 0; i < len; ++i) {
    if (arr[i] < 0 ) {
        abs++;
        if (abs * *arr == *arr)
            arr[i] = abs;
        else
            continue;
    }
    else
        continue;
    sum+= *arr;
}
cout << sum;
} 
 #pragma clang diagnostic pop

this is the getAbsSum.cpp

#include "getAbsSum.h"
void getAbsSum(int arr[10]);

and this is the main.cpp

#include <iostream>
#include "getAbsSum.h"
using std::cin,std::cout,std::endl;
int main() {
int arr[10];
for (int & i : arr)
  cin >> i;
getAbsSum(arr);
return 0;
}

and this is the error :

    /home/mamog/CLionProjects/Absolute_sum/getAbsSum.h:6: multiple definition of `getAbsSum(int*)'; CMakeFiles/Absolute_sum.dir/main.cpp.o:/home/mamog/CLionProjects/Absolute_sum/getAbsSum.h:6: first defined here
collect2: error: ld returned 1 exit status
make[3]: *** [CMakeFiles/Absolute_sum.dir/build.make:108: Absolute_sum] Error 1
make[2]: *** [CMakeFiles/Makefile2:83: CMakeFiles/Absolute_sum.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:90: CMakeFiles/Absolute_sum.dir/rule] Error 2
make: *** [Makefile:124: Absolute_sum] Error 2

You seem to have switched around which things go in which files. Declarations go in header files, definitions go in source cpp files. Right now both main.cpp and getAbsSum.cpp see a definition for getAbsSum, which is a violation of the One Definition Rule. — Nathan Pierson
– Nathan Pierson, Commented Sep 5, 2021 at 16:15

Iván · Accepted Answer · 2021-09-05 16:16:14Z

2

The .h and the .cpp files are inverted. The code should be in the .cpp, and the declaration, in the .h header.

getAbsSum.h:

void getAbsSum(int arr[10]);

getAbsSum.cpp

#include "getAbsSum.h"
#include <iostream>

using std::cin,std::cout,std::endl;

#pragma clang diagnostic push
#pragma ide diagnostic ignored "cppcoreguidelines-narrowing-conversions"
void getAbsSum(int arr[10]){
int abs = 0,sum = 0;
int len = *(&arr + 1) - arr;
for (int i = 0; i < len; ++i) {
    if (arr[i] < 0 ) {
        abs++;
        if (abs * *arr == *arr)
            arr[i] = abs;
        else
            continue;
    }
    else
        continue;
    sum+= *arr;
}
cout << sum;
} 
 #pragma clang diagnostic pop

answered Sep 5, 2021 at 16:16

Iván

1,0609 silver badges21 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

nanofarad · Accepted Answer · 2021-09-05 16:16:06Z

2

Headers are substituted textually into every compilation unit where they are #include'd; as a result you have definitions of getAbsSum in both getAbsSum.cpp and main.cpp. This violates the one-definition rule.

The correct way to use a header is to include the declaration in the header and the definition in the cpp file; you have this backwards.

Alternatively, for small functions that don't have a lot of dependencies, you may choose to get rid of getAbsSum.cpp, mark the definition in the header as inline, and then anyone including your header will get a definition of the function.

answered Sep 5, 2021 at 16:16

nanofarad

41.6k4 gold badges97 silver badges132 bronze badges

Collectives™ on Stack Overflow

Fix for C++ Compiler Error - multiple definition

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related