【求助】公司统一规定"{"必须放在行末,这很不习惯,我该怎么

[复制链接]
15421|109
yoyoyun 发表于 2007-9-26 16:13 | 显示全部楼层

7楼的很有意思

呵呵,<br />第二种看起来舒服一些!<br />哈哈<br />行外人说的话<br />
laiqi 发表于 2007-9-26 20:20 | 显示全部楼层

没有最好的,只有合适的

没有最好的,只有合适的,只要在一个项目中风格统一即可。
avrx007 发表于 2007-9-28 18:40 | 显示全部楼层

re: 还有一个无法容忍的规定:不许使用“//”注释

矫枉过正。
drentsi 发表于 2007-9-29 08:34 | 显示全部楼层

死追求形式,没啥前途的,跳吧,微软都用第二种

void&nbsp;CSplitterWnd::StopTracking(BOOL&nbsp;bAccept)<br />{<br />&nbsp;&nbsp;&nbsp;&nbsp;ASSERT_VALID(this);<br /><br />&nbsp;&nbsp;&nbsp;&nbsp;if&nbsp;(!m_bTracking)<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return;<br /><br />&nbsp;&nbsp;&nbsp;&nbsp;ReleaseCapture();<br /><br />&nbsp;&nbsp;&nbsp;&nbsp;//&nbsp;erase&nbsp;tracker&nbsp;rectangle<br />&nbsp;&nbsp;&nbsp;&nbsp;OnInvertTracker(m_rectTracker);<br />&nbsp;&nbsp;&nbsp;&nbsp;if&nbsp;(m_bTracking2)<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;OnInvertTracker(m_rectTracker2);<br />&nbsp;&nbsp;&nbsp;&nbsp;m_bTracking&nbsp;=&nbsp;m_bTracking2&nbsp;=&nbsp;FALSE;<br /><br />&nbsp;&nbsp;&nbsp;&nbsp;//&nbsp;save&nbsp;old&nbsp;active&nbsp;view<br />&nbsp;&nbsp;&nbsp;&nbsp;CWnd*&nbsp;pOldActiveView&nbsp;=&nbsp;GetActivePane();<br /><br />&nbsp;&nbsp;&nbsp;&nbsp;//&nbsp;m_rectTracker&nbsp;is&nbsp;set&nbsp;to&nbsp;the&nbsp;new&nbsp;splitter&nbsp;position&nbsp;(without&nbsp;border)<br />&nbsp;&nbsp;&nbsp;&nbsp;//&nbsp;(so,&nbsp;adjust&nbsp;relative&nbsp;to&nbsp;where&nbsp;the&nbsp;border&nbsp;will&nbsp;be)<br />&nbsp;&nbsp;&nbsp;&nbsp;m_rectTracker.OffsetRect(-CX_BORDER&nbsp;,&nbsp;-CY_BORDER);<br />&nbsp;&nbsp;&nbsp;&nbsp;m_rectTracker2.OffsetRect(-CX_BORDER,&nbsp;-CY_BORDER);<br /><br />&nbsp;&nbsp;&nbsp;&nbsp;if&nbsp;(bAccept)<br />&nbsp;&nbsp;&nbsp;&nbsp;{<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if&nbsp;(m_htTrack&nbsp;==&nbsp;vSplitterBox)<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;SplitRow(m_rectTracker.top);<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;else&nbsp;if&nbsp;(m_htTrack&nbsp;&gt=&nbsp;vSplitterBar1&nbsp;&&&nbsp;m_htTrack&nbsp;&lt=&nbsp;vSplitterBar15)<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;//&nbsp;set&nbsp;row&nbsp;height<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;TrackRowSize(m_rectTracker.top,&nbsp;m_htTrack&nbsp;-&nbsp;vSplitterBar1);<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;RecalcLayout();<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;else&nbsp;if&nbsp;(m_htTrack&nbsp;==&nbsp;hSplitterBox)<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;SplitColumn(m_rectTracker.left);<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;else&nbsp;if&nbsp;(m_htTrack&nbsp;&gt=&nbsp;hSplitterBar1&nbsp;&&&nbsp;m_htTrack&nbsp;&lt=&nbsp;hSplitterBar15)<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;//&nbsp;set&nbsp;column&nbsp;width<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;TrackColumnSize(m_rectTracker.left,&nbsp;m_htTrack&nbsp;-&nbsp;hSplitterBar1);<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;RecalcLayout();<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;else&nbsp;if&nbsp;(m_htTrack&nbsp;&gt=&nbsp;splitterIntersection1&nbsp;&&<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;m_htTrack&nbsp;&lt=&nbsp;splitterIntersection225)<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;//&nbsp;set&nbsp;row&nbsp;height&nbsp;and&nbsp;column&nbsp;width<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;int&nbsp;row&nbsp;=&nbsp;(m_htTrack&nbsp;-&nbsp;splitterIntersection1)&nbsp;/&nbsp;15;<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;int&nbsp;col&nbsp;=&nbsp;(m_htTrack&nbsp;-&nbsp;splitterIntersection1)&nbsp;%&nbsp;15;<br /><br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;TrackRowSize(m_rectTracker.top,&nbsp;row);<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;TrackColumnSize(m_rectTracker2.left,&nbsp;col);<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;RecalcLayout();<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;else&nbsp;if&nbsp;(m_htTrack&nbsp;==&nbsp;bothSplitterBox)<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;//&nbsp;rectTracker&nbsp;is&nbsp;vSplitter&nbsp;(splits&nbsp;rows)<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;//&nbsp;rectTracker2&nbsp;is&nbsp;hSplitter&nbsp;(splits&nbsp;cols)<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;SplitRow(m_rectTracker.top);<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;SplitColumn(m_rectTracker2.left);<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br />&nbsp;&nbsp;&nbsp;&nbsp;}<br /><br />&nbsp;&nbsp;&nbsp;&nbsp;if&nbsp;(pOldActiveView&nbsp;==&nbsp;GetActivePane())<br />&nbsp;&nbsp;&nbsp;&nbsp;{<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;if&nbsp;(pOldActiveView&nbsp;!=&nbsp;NULL)<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;{<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;SetActivePane(-1,&nbsp;-1,&nbsp;pOldActiveView);&nbsp;//&nbsp;re-activate<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;pOldActiveView-&gtSetFocus();&nbsp;//&nbsp;make&nbsp;sure&nbsp;focus&nbsp;is&nbsp;restored<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;}<br />&nbsp;&nbsp;&nbsp;&nbsp;}<br />}<br />
汽车电子 发表于 2007-9-29 10:26 | 显示全部楼层

我喜欢多种风格混用,这就是我的风格

&nbsp;&nbsp;&nbsp;当花括号里的语句很少时,左右括号都放在同一行<br />&nbsp;&nbsp;&nbsp;当花括号里的语句多时,花括号单独占一行<br />&nbsp;&nbsp;&nbsp;坚决使用语句本身就是注释<br /><br />&nbsp;&nbsp;&nbsp;&nbsp;//&nbsp;C程序文件头尾示例<br /><img src="http://www.guoyee.com/GY_Doc/CFileHeadEnd.jpg"><br /><br />&nbsp;&nbsp;&nbsp;&nbsp;//&nbsp;C函数示例<br /><img src="http://www.guoyee.com/GY_Doc/Pro1.jpg"><br /><br />&nbsp;&nbsp;&nbsp;&nbsp;//&nbsp;混合风格示例<br /><img src="http://www.guoyee.com/GY_Doc/Pro2.jpg"><br /><br /><br />&nbsp;&nbsp;&nbsp;
汽车电子 发表于 2007-9-29 10:34 | 显示全部楼层

不同意102楼

&nbsp;&nbsp;&nbsp;第二种看起来舒服一些!&nbsp;&nbsp;不是行外话<br /><br />&nbsp;&nbsp;&nbsp;看起来舒服,就是程序的可读性好。如果一个排版不好,可读性不好的移交给你,恐怕你读起很不舒服吧?可读性非常差的话,或许你会“骂”原来写这程序的工程师。<br />
dragon_hn 发表于 2007-9-29 23:12 | 显示全部楼层

我也喜欢用第二种

我也喜欢用第二种,讨厌第一种,如果我是楼主,早就换公司了。<br />最讨厌大公司的制度化管理、形式主义和管理层的勾心斗角。<br />-------------------------------------------<br />现在在家工作!呵呵,想什么时候起床就什么时候起床,想什么时候睡就什么时候睡!
wj77317 发表于 2007-10-12 14:57 | 显示全部楼层

下面的会更有趣

int&nbsp;Max(int&nbsp;iX,&nbsp;int&nbsp;iY)&nbsp;{<br />&nbsp;&nbsp;&nbsp;&nbsp;if&nbsp;(&nbsp;iX&nbsp;&gt=&nbsp;iY&nbsp;)&nbsp;{<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return&nbsp;iX;<br />&nbsp;&nbsp;&nbsp;&nbsp;}&nbsp;else&nbsp;{<br />&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return&nbsp;iY;<br />&nbsp;&nbsp;&nbsp;&nbsp;}<br />}<br />直接改成:<br />int&nbsp;Max(int&nbsp;iX,&nbsp;int&nbsp;iY)&nbsp;{&nbsp;if&nbsp;(&nbsp;iX&nbsp;&gt=&nbsp;iY&nbsp;)&nbsp;{&nbsp;return&nbsp;iX;}&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;else&nbsp;{&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;return&nbsp;iY;&nbsp;&nbsp;&nbsp;&nbsp;}}<br /><br />
老狼 发表于 2007-10-13 01:21 | 显示全部楼层

我是第二种

我上大学时,老师就这样教的,我的那个老师也很牛,我是老实人!!!
yanfengzhu 发表于 2007-10-13 13:26 | 显示全部楼层

我用第一种,建议楼主用emacs编辑吧,提供了多种风格的转换

还可以自己定制自己的风格。
您需要登录后才可以回帖 登录 | 注册

本版积分规则

快速回复 在线客服 返回列表 返回顶部