Hello Thomas,Alvaro,Fernando and all,<br>I have made the changes to my first commit as per the reviews. <br><a href="goog_1583234049" target="_blank">https://svn.kde.org/home/kde/trunk/playground/office/alkimia/alkdbus/AlkRec</a>ord<br>
<br>The changes made are as follows:<br><br><br>1)The database connection is now established in the constructor itself  and disconnected in destructor .Before it was created at each function call.<br>2)Binding variables are used in place of normal sql queries. SqlTableModel is also replaced with the QSqlQuery with binding variables.<br>
3)All function calls are made using call by reference.Earlier it was call by value.Const correctness also implemented.<br>4)Enumerated data types are used for status and transaction variables.<br>5)Getter and Setter functions are used for private members.<br>
6)Database connection object db is made static .<br>7)Code indented using astyle and kmymoney guidelines<br>8)Documentation prepared using Doxygen.<br>9)Licence provided in all the files.<br>10)Database name made dynamic for the class methods.<br>
<br>Thomas,Alvaro, Fernando please check if  i missed out something ? <br>Please point out anything you feel is worth changing.<br><br>Alvaro , sorry i made so many changes all at once.I got your mail after i had already made these changes. Will keep that in mind next time.<br>
<br>Regards,<br>Mukesh<br>