<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
</head>
<body dir="ltr" bgcolor="#ffffff" text="#000000">
Diego Iastrubni wrote:
<blockquote cite="mid:200904261719.38384.elcuco@kde.org" type="cite">
<pre wrap="">On Sunday 26 April 2009 15:38:53 Erez D wrote:
</pre>
<blockquote type="cite">
<pre wrap="">files:
== a.h ===
</pre>
</blockquote>
<pre wrap=""><!---->MISSING:
#ifndef __FILE_H__
</pre>
</blockquote>
There is no syntactical requirement to place those. If you do place
those, it is customary to call them after the file's name. Also, "__"
is only prepended to system includes, so it should have been:<br>
#ifndef A_H<br>
<br>
Also, you forgot to add:<br>
#define A_H<br>
<br>
which is, after all, the reason we go through the trouble of putting
the #ifndef to begin with.<br>
<blockquote cite="mid:200904261719.38384.elcuco@kde.org" type="cite">
<blockquote type="cite">
<pre wrap="">class a {
int m_a;
a (int val);
void do_somthing()
}
</pre>
</blockquote>
<pre wrap=""><!---->MISSING: semicolon, last line should be
};
MISSING:
#endif // __FILE_H__
</pre>
<blockquote type="cite">
<pre wrap="">== a.cpp ===
#include "a.h"
a::do_somthing {a.m_a++;};
a::a(int val):m_a(val) {};
============
</pre>
</blockquote>
</blockquote>
Missed lots and lots and lots of errors in this file.<br>
<blockquote cite="mid:200904261719.38384.elcuco@kde.org" type="cite">
<pre wrap=""><!---->
GRADE: C- ~ 70.
Go back to school, and as punishment re-implement quick_sort in VB.NET.
</pre>
</blockquote>
Now that's going the "cruel and unusual" route.<br>
<br>
I do think every programmer should take the time to implement binary
search and quick sort at least once from scratch. The number of corner
cases there is outstanding, and it is a great practice of thinking of
the fine details.<br>
<br>
Shachar<br>
<br>
<pre class="moz-signature" cols="72">--
Shachar Shemesh
Lingnu Open Source Consulting Ltd.
<a class="moz-txt-link-freetext" href="http://www.lingnu.com">http://www.lingnu.com</a>
</pre>
</body>
</html>