Úprava kódu – C / C++ – Fórum – Programujte.com

Ahoj,

předem se omlouvám za triviální dotaz, ale neprogramuju v C++, ale v C# takže je to pro mě nepřekonatelný problém :) 

Měl jsem funkci

#ifndef ABSTRACT_LOGGER_H
#define ABSTRACT_LOGGER_H

#include <string>

namespace Palo {
namespace SpreadsheetFuncs {
class AbstractLogger {
public:
    enum Type {
        LOG_ERROR, LOG_WARNING, LOG_INFO
    };

    virtual void log(Type t, const std::string msg) throw () = 0;
};
}
}
#endif

kterou volala následující funkce 

#ifndef XLL_CELL_LOGGER_H
#define XLL_CELL_LOGGER_H


#include <fstream>

#include <PaloSpreadsheetFuncs/AbstractLogger.h>

namespace Palo {
	namespace XLL {
		class XLLCellLogger : public Palo::SpreadsheetFuncs::AbstractLogger {
		public:
			XLLCellLogger();
			~XLLCellLogger();

			static XLLCellLogger& getInstance() {
				static XLLCellLogger pl;
				return pl;
			}

			void log( Type t, std::string msg, bool force_logging ) throw();
			void log( Type t, std::string msg ) throw() {
				log(t, msg, false);
			};

		private:
			std::ofstream m_log;
		};
	}
}
#endif

bohužel se první funkce změnil string 

#ifndef ABSTRACT_LOGGER_H
#define ABSTRACT_LOGGER_H

#include <string>

namespace Palo {
namespace SpreadsheetFuncs {
class AbstractLogger {
public:
	enum Type {
		LOG_ERROR, LOG_WARNING, LOG_INFO
	};

	virtual void log(Type t, const std::string &msg) throw () = 0;
};
}
}
#endif

zkoušel jsem tedy kód upravit následně 

#ifndef XLL_CELL_LOGGER_H
#define XLL_CELL_LOGGER_H


#include <fstream>

#include <PaloSpreadsheetFuncs/AbstractLogger.h>

namespace Palo {
	namespace XLL {
		class XLLCellLogger : public Palo::SpreadsheetFuncs::AbstractLogger {
		public:
			XLLCellLogger();
			~XLLCellLogger();

			static XLLCellLogger& getInstance() {
				static XLLCellLogger* pl;
				return *pl;
			}

			void log( Type t, std::string msg, bool force_logging ) throw();
			void log( Type t, std::string msg ) throw() {
				log(t, msg, false);
			};

		private:
			std::ofstream m_log;
		};
	}
}
#endif

Což sice projde kompilací, ale v reálu to pak nefunguje. Mohl by mi někdo poradit jak správně kód upravit? Díky.

#1 vesi
A proč měníš getInstance metodu, když jsi upravoval pure virtual funkci třídy AbstractLogger? Přidej si referenci i do funkce log, kterou implementuješ z abstraktní třídy.

void log( Type t, std::string& msg ) throw() {
				log(t, msg, false);
			};

Co se metody getInstance týče, na co si vytváříš statickej ukazatel XLLCellLogger? Nechceš náhodou vytvořit statický objekt? Využij move sémantiky, kterou C++ v nových verzí nabízí. Deklaraci ukazatele a dereferenci pryč.

Každopádně ten ofstream ti tam, myslím si, stejně bude dělat docela bordel, protože by se měl přenést (zkopírovat), což bohužel nejde. Asi bych zvolil jiný typ proměnné pro uchování logu, třeba standardní std::string.

#2 ondrej39
Díky za odpověď. Já bohužel nemůžu měnit kód v AbstractLogger, protože je v jiné knihovně, jehož kód já nemám pod "kontrolou". Takže potřebuji upravit XLLCellLogger, tak aby fungoval se změnou const std::string &msg. Jak jsem psal nahoře, programuju v C#, ale C++ je pro mě španělská vesnice... :(

#3 vesi
Jde o to, že když definuješ virtual / pure virtual metodu z rodiče, tak je třeba, aby ta tvá definice měla stejný přepis (stejnou deklaraci) jako funkce rodičovská.

Původní metoda používala předávání stringu hodnotou, a na to jsi měl tu svou funkci log, oni to změnili na předávání stringu pomocí konstantní reference a kvůli tomu musíš upravit tu svou funkci v XLLCellLogger, aby ten přepis souhlasil...

Takže prostě zkopíruješ deklaraci dané metody, kterou implementuješ. Zmeň si v té své třídě XLLCellLogger metodu na 

void log( Type t, std::string& msg ) throw()

To je jako když v C# děláš 

public override bool Equals(object o)

Ačkoliv toto většinou implementuješ pro konkrétní objekt a chceš proovnávat jen dva objekty stejného typu, tak musíš mít v deklaraci jako parametr object a ne třídu, pro niž danou metodu děláš, a přetypování object si udělat až uvnitř té funkce, protože rodičovská metoda Equals má v parametru právě ten object, a to musí souhlasit.

Divim se, ze slo zkompilovat i tu prvni verzi -  log je tam ciste virtualni a MUSI se definovat v potomku, aby to vubec mohlo kdy jit zkompilovat.

A v potomku nemas ani jednu metodu, ktera by to mela teoreticky predefinovat. Musi tam byt uplne presne to same (akorat bez  = 0; na konci), jako v te AbstractLogger.

#3 vesi

Ak používaš c++11/c++14 tak si môžeš použiť override specifier. Funguje ako v C# len sa dava az za funkciu:

void log( Type t, std::string& msg ) noexcept override{}	//kompilator ti vypise chybu lebo ti tam chyba const

Díky za komentáře.

Upravil jsem kód následovně 

namespace Palo {
	namespace XLL {
		class XLLCellLogger : public Palo::SpreadsheetFuncs::AbstractLogger {
		public:
			XLLCellLogger();
			~XLLCellLogger();

			static XLLCellLogger& getInstance() {
				static XLLCellLogger pl;
				return pl;
			}
			void log( Type t, std::string& msg ) throw();
			void log( Type t, std::string& msg, bool force_logging ) throw();

		private:
			std::ofstream m_log;
		};
	}
}

protože potřebuji i variantu s 3 parametrem, která se používá dál v kódu. 

Ale bohužel mě to vyhází nasledující chyby

Error	1	error C2259: 'Palo::XLL::XLLCellLogger' : cannot instantiate abstract class	d:\develop\xladdin\libraries514\paloxll\XLLCellLogger.h	52
Error	2	error C2664: 'void Palo::XLL::XLLCellLogger::log(Palo::SpreadsheetFuncs::AbstractLogger::Type,std::string &,bool) throw()' : cannot convert parameter 2 from 'const char *' to 'std::string &'	D:\Develop\XlAddin\libraries514\PaloXLL\c2c_plus_plus.cpp	40
Error	3	error C2259: 'Palo::XLL::XLLCellLogger' : cannot instantiate abstract class	d:\develop\xladdin\libraries514\paloxll\XLLCellLogger.h	52
Error	4	error C2259: 'Palo::XLL::XLLCellLogger' : cannot instantiate abstract class	d:\develop\xladdin\libraries514\paloxll\XLLCellLogger.h	52
Error	5	error C2511: 'void Palo::XLL::XLLCellLogger::log(Palo::SpreadsheetFuncs::AbstractLogger::Type,std::string,bool) throw()' : overloaded member function not found in 'Palo::XLL::XLLCellLogger'	D:\Develop\XlAddin\libraries514\PaloXLL\XLLCellLogger.cpp	68
Error	6	error C2259: 'Palo::XLL::XLLCellLogger' : cannot instantiate abstract class	d:\develop\xladdin\libraries514\paloxll\XLLCellLogger.h	52
Error	7	error C2664: 'void Palo::XLL::XLLCellLogger::log(Palo::SpreadsheetFuncs::AbstractLogger::Type,std::string &) throw()' : cannot convert parameter 2 from 'const char [57]' to 'std::string &'	D:\Develop\XlAddin\libraries514\PaloXLL\SpreadsheetFuncsWrapper.cpp	128

to je zase nevyhoda pouziti reference na string - neda se z C style retezce uz automaticky vyrobit std::string

Takže jestli to chápu dobře, tak jediná možnost pro mě je, vrátit zpět v AbstractLogger parametr const std::string &msg na const std::string msg, jak to bylo původně. To sice funguje, ale chtěl jsem se tomu vyhnout, protože AbstractLogger je součástí open source řešení, které je aktualizováno a já to budu muset při každé aktualizaci upravovat ve zdrojovém kódu...

#10 vesi
Pod čím kompiluješ? Ve VS2013 mi to s tímto zápisem C2664 vůbec nehází. (Oddělej referenci u funkce log se třemi parametry.)

#ifndef ABSTRACT_LOGGER_H
#define ABSTRACT_LOGGER_H

#include <string>

namespace Palo {
	namespace SpreadsheetFuncs {
		class AbstractLogger {
		public:
			enum Type {
				LOG_ERROR, LOG_WARNING, LOG_INFO
			};

			virtual void log(Type t, const std::string& msg) throw () = 0;
		};
	}
}
#endif

////////////////////////////////////

#ifndef XLL_CELL_LOGGER_H
#define XLL_CELL_LOGGER_H

#include <fstream>

//includuji to jinak, to si u sebe změň
#include "AbstractLogger.h"

namespace Palo {
	namespace XLL {
		class XLLCellLogger : public Palo::SpreadsheetFuncs::AbstractLogger {
		public:
			XLLCellLogger();
			~XLLCellLogger();

			static XLLCellLogger& getInstance() {
				static XLLCellLogger pl;
				return pl;
			}

			void log(Type t, std::string msg, bool force_logging) throw();
			virtual void log(Type t, const std::string& msg) throw() {
				log(t, msg, false);
			};

		private:
			std::ofstream m_log;
		};
	}
}
#endif

Nicméně to řve kvůli volání smazaného copy konstruktoru třídy std::ofstream (jak jsem již psal ve 2. příspěvku). Použij na logování něco jiného.

Kompiluju to pod VS2010 SP1. Upravil jsem to podle tebe, ale pořád mi to řve na řádku 

static XLLCellLogger pl; 

Error	2	error C2259: 'Palo::XLL::XLLCellLogger' : cannot instantiate abstract class	d:\develop\xladdin\libraries514\paloxll\XLLCellLogger.h	52

a pak na řádku 

XLLCellLogger::getInstance().log( AbstractLogger::LOG_WARNING, "Unexpected exception caught! (This should never happen)." ); 

Error	5	error C2664: 'void Palo::XLL::XLLCellLogger::log(Palo::SpreadsheetFuncs::AbstractLogger::Type,std::string &) throw()' : cannot convert parameter 2 from 'std::string *' to 'std::string &'	D:\Develop\XlAddin\libraries514\PaloXLL\SpreadsheetFuncsWrapper.cpp	129
	6	IntelliSense: no instance of overloaded function "Palo::XLL::XLLCellLogger::log" matches the argument list	d:\develop\xladdin\libraries514\paloxll\spreadsheetfuncswrapper.cpp	128

Tady mě je jasné, že je problém s tím, že se předává string nikoliv reference na string, ale nevím jak to přepsat.

Ještě jednou se omlouvám, je mi jasné, že to co tu řeším jsou asi základy, ale já C++ naposledy viděl asi před 15 lety na vejšce, tak díky za trpělivost :)