Strange, non-repetitious error

Ruby
1

Hello

I have that exception sometimes: “can’t convert false into Integer”
This error is non-repetitious.
This exception occur once in a while when I run progam on the same data.

sample code:

for z in 1…lines.length
line = lines[z-1] # this is the line where error occur
end

Is it a ruby implementation problem?

Greetings

2

2008/4/7, Pan K. [email protected]:

end

Is it a ruby implementation problem?

Most likely not. We can’t say without seeing more code. What type of
object is “lines”?

Why don’t you just do

lines.each do |line|
end

?

Kind regards

robert

3

I don’t use “lines.each do |line|” because I need number of line.

This is the code:

File.open(procsw[j-1],‘r+’) do |file|
lines = file.readlines
for z in 1…lines.length
line = lines[z-1] #can’t convert false into integer
if(line!=nil and line!="") then
line.downcase.gsub(“external name”) do
if(line.index(/’.’/)!=nil) then
line.gsub(/’.’/) {|match|
k = match.rindex("/")
line = “’” + bladePath + match.slice(k…match.length)

  • “\n”
    }
    else
    lines[z].gsub(/’.*’/) {|match|
    k = match.rindex("/")
    lines[z] = “’” + bladePath +
    match.slice(k…match.length) + “\n”
    }
    end
    end
    end
    end
    file.pos = 0
    file.print lines
    file.truncate(file.pos)
    file.close
    end
4

On Mon, Apr 7, 2008 at 7:33 AM, Pan K. [email protected] wrote:

I don’t use “lines.each do |line|” because I need number of line.

lines.each_with_index do | line, z|
…

–
Rick DeNatale

My blog on Ruby
http://talklikeaduck.denhaven2.com/

5

On Monday 07 April 2008, Pan K. wrote:

I don’t use “lines.each do |line|” because I need number of line.

Do you know about Array#each_index and Array#each_with_index?:

[‘a’, ‘b’, ‘c’].each_index{|i| puts i}

Output:
0
1
2

[‘a’, ‘b’, ‘c’].each_with_index{|obj, i| puts “i: #{i}, obj: #{obj}”}

Output:
i: 0, obj: a
i: 1, obj: b
i: 2, obj: c

Stefano

6

I will try with
lines.each_with_index do | line, z|

Thank you all for replies.
Greetings

7

2008/4/7, Pan K. [email protected]:

I will try with

lines.each_with_index do | line, z|

You should change a lot more. For example, you have nested
invocations of gsub on the same line (albeit sometimes with a
#downcase so it’s probably not the same instance).

You also do not need the “file.close” because your block ensures the
file will be closed properly.

Also, I do not see how z can ever be a non Fixnum in your code. Are
you sure this is the code that produces the error? Do you happen to
have multiple threads all of them using a z which is defined outside
the code you show?

In your case I’d rather use #map instead of normal iteration since you
seem to be replacing line contents with something else.

Or wait, there seems to be a off by one error lurking: you use
1…lines.length for iteration, read lines[z-1] but later lines[z].
lines[z] does not exist for z==lines.length. You either need to
change logic or use the three dots range for iteration.

Kind regards

robert

8

On Mon, Apr 7, 2008 at 2:25 PM, Robert K.
[email protected] wrote:

2008/4/7, Pan K. [email protected]:
You should change a lot more. For example, you have nested
…

I agree with Robert. Maybe if you describe what you are trying to
achieve, somebody would help with a cleaner solution.

J.