# Good code?

**URL:** <https://www.ruby-forum.com/t/good-code/103110>\
**Category:** Rails\
**Created:** [May 1, 2007, 11:08pm UTC](https://www.ruby-forum.com/t/good-code/103110 "2007-05-01T23:08:58Z")\
**Posts on this page:** 2\
**Page:** 1

<div class="post-metadata">

**Author:** ![leonb](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/l/22d042/32.png) [@leonb](https://www.ruby-forum.com/u/leonb)\
**Post date:** [May 1, 2007, 11:08pm UTC](https://www.ruby-forum.com/t/good-code/103110/1 "2007-05-01T23:08:58Z")

</div>

Hi,

Is this ok as helper code?  
[http://pastie.caboo.se/58054](http://pastie.caboo.se/58054)

I especially was wondering if it is ok to use the “current\_user”  
method in a helper method?

---

<div class="post-metadata">

**Author:** ![leonb](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/l/22d042/32.png) [@leonb](https://www.ruby-forum.com/u/leonb)\
**Post date:** [May 6, 2007, 4:45pm UTC](https://www.ruby-forum.com/t/good-code/103110/2 "2007-05-06T16:45:45Z")

</div>

Anyone has got any thoughts about it?