Noob: Code review my first tutorial homework assignment?

B

Brian A.

Hello!
Im reading through tutorials to teach myself Ruby. I'm a business
programmer by day, but I want to learn something new instead of the same
old thing I've been doing for the last 8 years. I don't know any Ruby
programmers local to me, so I'd like to ask you to please code review
the first half of my 'project'. If I am doing something wrong, or could
do it better please remember Im just a ruby-noob so please explain it as
best you can so I can understand why I should do things differently.
Thank you much in advance!

Here is what I was asked to do (note this is only the first half of the
project, but Id like to know Im headed in the right direction before I
start the second half).

==================================================
# Name the project as MyOwnRubyApp.
# Create a new Ruby class called MyOwnRubyClass.
# Define a method called my_own_ruby_method inside of the MyOwnRubyClass
class as following

* The method receives two parameters called my_parameter and
your_parameter.
* The method then prints out current time using Time.now.
* The method then invokes yield(my_parameter, your parameter,
current_month).
o The current_month, is an integer value representing current
month of the year, for example, for the current month of "June", the
current_month should be 6.
o The current_month value needs to computed in your program. For
example, if you are running the same program during July, it should be
set to 7. (In other words, you cannot pass hard-coded value in your
program)

# Create an instance of MyOwnRubyClass. Assign it to a variable called
my_instance.
==================================================

Here is my code. It runs but I get this error also "...My
Documents/NetBeansProjects/HomeWorkLab1/lib/main.rb:23: warning:
multiple values for a block parameter (2 for 1) from .../My
Documents/NetBeansProjects/HomeWorkLab1/lib/main.rb:17"

==================================================
# Create a class called MyOwnRubyClass
class MyOwnRubyClass
# define a method called my_own_ruby_method
# method receives two parameters called my_parameter and your_paramter
def my_own_ruby_method(*parameters)
# method prints our current time using Time.now
yield(Time.now)

# 1) The method then invokes yield(my_parameter, your parameter,
current_month)
# 2) The current_month, is an integer value representing current
month of the year,
# for example, for the current month of "June", the current_month
should be 6.
# 3) The current_month value needs to computed in your program.
# For example, if you are running the same program during July,
it should be set to 7.
# (In other words, you cannot pass hard-coded value in your
program)
current_month = Date.today.month
#yield "#{parameters.join(',')}", current_month{|x| puts x} # doesnt
work
yield(parameters, current_month)
end
end

my_instance = MyOwnRubyClass.new

my_instance.my_own_ruby_method("alpha", "beta"){|s|puts s}
==================================================
 
R

Robert Dober

yield(parameters, current_month)
A: 2
Q: How many paramaters do you pass into the block
my_instance.my_own_ruby_method("alpha", "beta"){|s|puts s}
A: 1
Q: I am sure you know it by now.
Cheers
Robert
--=20
Ne baisse jamais la t=EAte, tu ne verrais plus les =E9toiles.

Robert Dober ;)
 
B

Brian A.

Robert said:
A: 2
Q: How many paramaters do you pass into the block

A: 1
Q: I am sure you know it by now.
Cheers
Robert

Thank you for the help. Does this look better? It runs with no error
and seems to make sense to me... Any input is appreciated:

=================================================================================
# Create a class called MyOwnRubyClass
class MyOwnRubyClass

# define a method called my_own_ruby_method
# method receives two parameters called my_parameter and your_paramter

def my_own_ruby_method(my_parameter, your_parameter, current_month)

# method prints our current time using Time.now
yield(Time.now)

# 1) The method then invokes yield(my_parameter, your parameter,
current_month)
# 2) The current_month, is an integer value representing current
month of the year,
# for example, for the current month of "June", the current_month
should be 6.
# 3) The current_month value needs to computed in your program.
# For example, if you are running the same program during July,
it should be set to 7.
# (In other words, you cannot pass hard-coded value in your
program
)
yield(my_parameter, your_parameter, current_month)

end
end

my_instance = MyOwnRubyClass.new

current_month = Date.today.month

my_instance.my_own_ruby_method("alpha", "beta", Date.today.month){|arg1,
arg2, arg3| puts "#{arg1} #{arg2} #{arg3}"}
 
R

Robert Dober

Robert Dober wrote:
Better, but there is still two yields in your method, one with one
parameter one with three.
If this is really needed I would rather declare the block with
variable arguments look at this
514/15 > irb
irb(main):001:0> def a( *args, &blk)
irb(main):002:1> blk.call( *args )
irb(main):003:1> end
=3D> nil
irb(main):004:0> myblock =3D proc{ |*args| case
irb(main):005:2* when args.empty?
irb(main):006:2> puts "empty"
irb(main):007:2> when args.size=3D=3D1
irb(main):008:2> puts args.first
irb(main):009:2> else
irb(main):010:2* p args
irb(main):011:2> end
irb(main):012:1> }
=3D> #<Proc:0xb7d2f78c@(irb):4>
irb(main):013:0> a &myblock
empty
=3D> nil
irb(main):014:0> a 1, &myblock
1
=3D> nil
irb(main):015:0> a 42, 222, 22, &myblock
[42, 222, 22]
=3D> nil
irb(main):016:0>




--=20
Ne baisse jamais la t=EAte, tu ne verrais plus les =E9toiles.

Robert Dober ;)
 
M

Martin DeMello

Hello!
Im reading through tutorials to teach myself Ruby. I'm a business
programmer by day, but I want to learn something new instead of the same
old thing I've been doing for the last 8 years.

Also, check out Satish Talim's free ruby class:

http://rubylearning.com/

martin
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Members online

Forum statistics

Threads
473,744
Messages
2,569,483
Members
44,901
Latest member
Noble71S45

Latest Threads

Top