Thread: Qt5 / C++ - Trying to set up a class

Hello,

I'm tying to convert a function into a class, so I can call it from any cpp module.
I can get this working,
formatSDCard(int type, QString diskNumber, bool &result);
but as soon as I add QProgressBar, I get an error.
Can somebody help, may be I should use another approach.

Regards

Code:

Header file
class formatSDCard
{
public:
    formatSDCard(int type, QString diskNumber, bool &result, QProgressBar &progress);
};

Code:

cpp file
formatSDCard::formatSDCard(int type, QString diskNummber, bool &result, QProgressBar &progress)
{
    progress->setValue(76);
    result = true; //or false;
}

Code:

Calling program
    bool result;
    QString num = "2";
    formatSDCard(1, num, result, ui->progress);
    if (result)
        ...
        ...
    }

Code:

Compile error    
error: no matching function for call to 'formatSDCard::formatSDCard(int, QString&, bool&, QProgressBar*&)'

Hello Salem,

Thanks for your reply.

When I put in those 2 lines of code, compile errors are:-

Code:

class formatSDCard myFooFormatter;  //!! this is an instance

error: no matching function for call to 'formatSDCard::formatSDCard()'
     class formatSDCard myFooFormatter;  //!! this is an instance
                        ^

Code:

myFooFormatter.formatSDCard(1, num, result, ui->progress);

error: invalid use of 'formatSDCard::formatSDCard'
     myFooFormatter.formatSDCard(1, num, result, ui->progress);
                    ^

Regards

Hello Salem,

Thanks for your reply.

I changed "formatSDCard" to "doFormat" as suggested.
Delving more into the compile output I have found these errors as well as the errors previously posted:-

Code:

..\test\noobsform.cpp: In constructor 'noobsForm::noobsForm(const QString&, const QString&, QWidget*)':
..\test\noobsform.cpp:22:20: error: no matching function for call to 'doFormat::doFormat()'
     class doFormat myFooFormatter;  //!! this is an instance
                    ^
In file included from ..\test\noobsform.cpp:12:0:
..\test\doFormat.hpp:11:5: note: candidate: doFormat::doFormat(int, QString, bool&, QProgressBar&)
     doFormat(int type, QString diskNumber, bool &result, QProgressBar &progress);
     ^
..\test\doFormat.hpp:11:5: note:   candidate expects 4 arguments, 0 provided
..\test\doFormat.hpp:7:7: note: candidate: constexpr doFormat::doFormat(const doFormat&)
 class doFormat
       ^
..\test\doFormat.hpp:7:7: note:   candidate expects 1 argument, 0 provided
..\test\noobsform.cpp:23:20: error: invalid use of 'doFormat::doFormat'
     myFooFormatter.doFormat(1, num, result, ui->progress);
                    ^

Regards

Hello Salem,

Sorry, my mistake.

I didn't say change ALL the names!

Code:

#ifndef FORMATSD_HPP
#define FORMATSD_HPP

class formatSDCard
{
public:
    //formatSDCard();
    doFormat(int type, QString diskNumber, bool &result); //, QProgressBar &progress);
};

#endif // FORMATSD_HPP

Code:

#include <QString>
#include "formatSD.hpp"

formatSDCard::doFormat(int type, QString diskNummber, bool &result) //, QProgressBar &progress)
{
    //progress->setValue(76);
    result = true; //or false;
}

Code:

    bool result;

    formatSDCard myFooFormatter;  //!! this is an instance
    myFooFormatter.doFormat(1, num, result); //, ui->progress);
    qDebug() << "result" << result;
    
output:-
result true

Code:

Compiler warning:-
..\test\formatSD.cpp: In member function 'int formatSDCard::doFormat(int, QString, bool&)':
..\test\formatSD.cpp:14:1: warning: no return statement in function returning non-void [-Wreturn-type]
 }
 ^

This works without QProgressBar, apart from a warning.
I will post later the output when including the QProgressBar.

Regards