Uninitialized constant Connector(Name Error)

I have seen a thread that has nearly the same problems but I guess mine
should be some other problems.

I had my Connector.rb and test.rb classes next to each other so it
should be able to interact.

Do bear with me as I am very new to ruby. I just started to experiment
abit with ruby lately.

Attachments:
http://www.ruby-forum.com/attachment/5042/problem.zip

···

--
Posted via http://www.ruby-forum.com/.

Is this problem really that uncommon ?

Tay Thotheolh wrote:

···

I have seen a thread that has nearly the same problems but I guess mine
should be some other problems.

I had my Connector.rb and test.rb classes next to each other so it
should be able to interact.

Do bear with me as I am very new to ruby. I just started to experiment
abit with ruby lately.

--
Posted via http://www.ruby-forum.com/\.

Hi Tay,

For future questions, please describe the results you are trying to achieve and how what you see differs from what you want. Include error messages if they exist. If your code is not too long (and in this case it really isn't) it is easier for people to help if it is directly included in your email.

It appears that you are expecting Ruby to "know" that the Connector class (really the Connector constant) is defined in the Connector.rb file (in a Java-ish) way. That's not how Ruby works. You need to explicitly "load" or "require" a file. For example, add

   require "connector" #if you are using Ruby 1.8

or

   require_relative "connector" #if you are using 1.9.2

to the top of your test.rb file.

However, you will quickly find that you get a new error. Something like:

   test.rb:13:in `dotest': undefined method `new' for Connector:Module (NoMethodError)
       from test.rb:28:in `<main>'

I think you want Connector to be a class, not a module. Modules are not instantiated as instances, only classes are.

Okay. Now your code will likely work, but there are still some issues. In Connector.rb (by the way, the convention would be to have the file name lower-cased), you have this:

   #Globals
   @hostname;
   @port;
   @soc;

First of all, conventionally semicolons are only used when you want multiple expressions sharing the same line. They are completely redundant here.

Secondly, these are not really globals. Globals begin with a dollar sign ($). But why even bother? You do not use these as globals. You can get rid of these lines completely.

Here's some more code:

   def send(message) # send tcp messages
     @soc.puts(message);
     return @soc.gets;
   end

Methods in Ruby return the last value by default, so "return" here is not needed.

And some more:

   def close()

     if(!(@soc.closed?)==true)
       @soc.close
     end

   end

A more idiomatic way to write this would be:

   def close

     @soc.close unless @soc.closed?

   end

And for this:

   def is_closed()
     return @soc.closed?
   end

The Rubyish way would be:

   def closed?
     @soc.closed?
   end

Hope that helps.

-Justin

···

On 09/12/2010 06:20 PM, Tay Thotheolh wrote:

Is this problem really that uncommon ?

Tay Thotheolh wrote:
   

I have seen a thread that has nearly the same problems but I guess mine
should be some other problems.

I had my Connector.rb and test.rb classes next to each other so it
should be able to interact.

Do bear with me as I am very new to ruby. I just started to experiment
abit with ruby lately.

Hi Collins. Thanks for the reply. Would look into it.

file (in a Java-ish) way

I came from a Java background where my idea was you just call Connector
like in Java and it should be loaded. This is my first attempt of really
sitting down and creating a ruby program by trial and error and learning
along the way.

···

--
Posted via http://www.ruby-forum.com/\.