Comments on: Code Review – ActiveMQ-CPP http://fsfoundry.org/codefreak/2009/03/08/code-review-activemq-cpp/ Weblog of a lively geek. Tue, 27 Jul 2010 06:54:46 -0700 http://wordpress.org/?v=2.8.6 hourly 1 By: fr3@K http://fsfoundry.org/codefreak/2009/03/08/code-review-activemq-cpp/comment-page-1/#comment-4953 fr3@K Tue, 10 Mar 2009 01:53:07 +0000 http://fsfoundry.org/codefreak/2009/03/08/code-review-activemq-cpp/#comment-4953 好樣的 Keiko. 你列的第一個 item 算是 platform-dependent behavior, 我沒把它算在文中提到的四個 exception 裏面. 事實上這也反應了 ActiveMQ-CPP 的 debugging code 的不足, 沒有在被呼叫 function 裏面以 <a href="http://fsfoundry.org/codefreak/2006/04/02/c-cpp-tips-the-marco-assert/" rel="nofollow"><code>assert()</code> 檢查 parameter 是否符合 expectation</a>. 另外, 關於 item 3, <a href="http://www.dinkumware.com/manuals/?manual=compleat&page=utility.html#make_pair" rel="nofollow"><code>make_pair</code></a> 呼叫的 <a href="http://www.dinkumware.com/manuals/?manual=compleat&page=utility.html#pair" rel="nofollow"><code>pair</code></a> ctor 應為: <ul><pre>template <class Ty1, class Ty2> pair<Ty1, Ty2>::pair(const Ty1& val1, const Ty2& val2);</pre></ul> 而非 copy ctor. Well done. :) 好樣的 Keiko.

你列的第一個 item 算是 platform-dependent behavior, 我沒把它算在文中提到的四個 exception 裏面. 事實上這也反應了 ActiveMQ-CPP 的 debugging code 的不足, 沒有在被呼叫 function 裏面以 assert() 檢查 parameter 是否符合 expectation.

另外, 關於 item 3, make_pair 呼叫的 pair ctor 應為:

    template <class Ty1, class Ty2>
pair<Ty1, Ty2>::pair(const Ty1& val1, const Ty2& val2);

而非 copy ctor.

Well done. :)

]]> By: Keiko http://fsfoundry.org/codefreak/2009/03/08/code-review-activemq-cpp/comment-page-1/#comment-4950 Keiko Mon, 09 Mar 2009 16:13:13 +0000 http://fsfoundry.org/codefreak/2009/03/08/code-review-activemq-cpp/#comment-4950 讓我猜猜至少是哪四個 exceptions： 1. file 是 NULL 時，會讓 std::string ctor 產生 access violation。 2. string::string( const char* ) 可能會產生記憶體配置不足或是類似的記憶體 exception。 3. pair<string,int>的 copy constructor 。 4. stackTrace 利用 allocator 產生一塊記憶體給要 push_back 進來的 object。 5. stackTrace 在 4. 上利用 placement new 呼叫 pair<string,int> 的copy ctor。(placement new 不會產生 exception) PS. 第一種情況會直接丟出 access violation 讓程式結束，不知道算不算在你提的 exception 裡頭？ 讓我猜猜至少是哪四個 exceptions：
1. file 是 NULL 時，會讓 std::string ctor 產生 access violation。
2. string::string( const char* ) 可能會產生記憶體配置不足或是類似的記憶體 exception。
3. pair<string,int>的 copy constructor 。
4. stackTrace 利用 allocator 產生一塊記憶體給要 push_back 進來的 object。
5. stackTrace 在 4. 上利用 placement new 呼叫 pair<string,int> 的copy ctor。(placement new 不會產生 exception)

PS. 第一種情況會直接丟出 access violation 讓程式結束，不知道算不算在你提的 exception 裡頭？

]]>