Difference between revisions of "OOP344 20131 - Microhard"

From CDOT Wiki
Jump to: navigation, search
(Milestone 0.2)
(Milestone 0.8 and 0.9)
 
(11 intermediate revisions by 2 users not shown)
Line 52: Line 52:
 
'''CValEdit'''
 
'''CValEdit'''
  
Developer:
+
Developer: Hanho Ko
  
Reviewer:
+
Reviewer: Sivathanushan Easwaran
  
  
Line 61: Line 61:
 
Developer: Sivathanushan Easwaran
 
Developer: Sivathanushan Easwaran
  
Reviewer:
+
Reviewer: Hanho Ko
  
  
Line 70: Line 70:
 
Reviewer:
 
Reviewer:
  
 +
==Milestone 0.8 and 0.9==
 +
 +
 +
'''CText'''
 +
 +
Developer: Hanho Ko
 +
 +
Reviewer:
 +
 +
 +
'''CCheckList'''
 +
 +
Developer: Sivathanushan Easwaran
 +
 +
Reviewer:
 +
 +
 +
'''CMenu(optional)'''
 +
 +
Developer:
 +
 +
Reviewer:
  
 
==Git Repository: 10-Microhd==
 
==Git Repository: 10-Microhd==
Line 77: Line 99:
  
 
==Discussion==
 
==Discussion==
 +
 +
For CCheckMark, can anyone tell me how to,
 +
* 'directly initialize _Label with Text, 0, 4, and (Width-4) for Str, Row, Col and Len, arguments of CLabel's Constructor' in function CCheckMark(bool Checked,const char* Format, const char* Text, int Row, int Col, int Width, bool IsRadio = false)
 +
* 'directly initialize the _Label with the _Label of C' in function CCheckMark(const CCheckMark& C)
 +
 +
I tried the following
 +
* CLabel _Label (Text, 0, 4, (Width-4));
 +
* CLabel _Label(C._Label);
 +
 +
But I think I did something wrong, because I get the following error in both functions
 +
 +
: Error: no default constructor exists for class "cio::CLabel"
 +
 +
Edit: If you need to view my full code, you can see it in my branch CCheckMark
 +
 +
Edit2: I think I fixed the error. It would still be great if someone could check it.
 +
 +
Reply by Hanho Ko:
 +
* I've seen your code, and I think everything is good. It's compiled without any errors but 1 warning. I think we don't have to care about this warning...
 +
* and I fixed 2 lines. I push it to your branch, you can check differences.
 +
 +
---S.E.

Latest revision as of 15:31, 30 March 2013


OOP344 | Weekly Schedule | Student List | Teams | Project | Student Resources

Members

First Name Last Name Section Seneca Id wiki id IRC nick GITHUB ID Blog URL
Jungmin Ji A jjungmin Ji Jungmin MistySnake jijungmin90 Hit!
Hanho Ko A hko5 Hko5 hko5 mynameishano I LOVE C & C++
Sivathanushan Easwaran A seaswaran Sivathanushan Easwaran LineCircle sivae OOP344class

Milestone 0.2

CLabel Mock-up Class (issue 2.2)

Developer: Hanho Ko

Reviewer:


CDialog Mock-up Class (issue 2.3)

Developer: Hanho Ko

Reviewer:


CLineEdit Mock-up Class (issue 2.4)

Developer: Sivathanushan Easwaran

Reviewer:


CButton Mock-up Class (issue 2.5)

Developer: Ji Jungmin

Reviewer:

Milestone 0.5

CValEdit

Developer: Hanho Ko

Reviewer: Sivathanushan Easwaran


CCheckMark

Developer: Sivathanushan Easwaran

Reviewer: Hanho Ko


CMenuItem

Developer:

Reviewer:

Milestone 0.8 and 0.9

CText

Developer: Hanho Ko

Reviewer:


CCheckList

Developer: Sivathanushan Easwaran

Reviewer:


CMenu(optional)

Developer:

Reviewer:

Git Repository: 10-Microhd

Master


Discussion

For CCheckMark, can anyone tell me how to,

  • 'directly initialize _Label with Text, 0, 4, and (Width-4) for Str, Row, Col and Len, arguments of CLabel's Constructor' in function CCheckMark(bool Checked,const char* Format, const char* Text, int Row, int Col, int Width, bool IsRadio = false)
  • 'directly initialize the _Label with the _Label of C' in function CCheckMark(const CCheckMark& C)

I tried the following

  • CLabel _Label (Text, 0, 4, (Width-4));
  • CLabel _Label(C._Label);

But I think I did something wrong, because I get the following error in both functions

Error: no default constructor exists for class "cio::CLabel"

Edit: If you need to view my full code, you can see it in my branch CCheckMark

Edit2: I think I fixed the error. It would still be great if someone could check it.

Reply by Hanho Ko:

  • I've seen your code, and I think everything is good. It's compiled without any errors but 1 warning. I think we don't have to care about this warning...
  • and I fixed 2 lines. I push it to your branch, you can check differences.

---S.E.