aws sdk for ruby アップロードについて

Question

AWS と rubyについて勉強中です。
AWS-sdk-for-rubyを使いデータをs3にアップロードしようとしています。
その際、php側から送られてきたデータをもとにrubyを使ってs3にアップロードしようと思っています。

$results = shell_exec("ruby s3uplod.rb devtouch2 51 scdci filename.txt Horseracing.mp4");
    echo "<pre>".$results . "</pre>";

require "aws-sdk-core"
require 'aws-sdk'
#require 'exec.php'

def test_bucket

bucket = Aws::S3::Resource.new(
    :region => 'ap-northeast-1',
    :access_key_id => '***********',
    :secret_access_key => '********************************',
    ).bucket('test')
end


def upload_file(object_name, target_file)
    # 正常終了すれば true が返る
    ARGV.each_with_index do |arg, i|
            puts "ARGV[#{i}]：#{arg}"
    end
    
    $siteid = ARGV[0]
    $contentsid = ARGV[1]
    $contentspasskey = ARGV[2]
    $filename = ARGV[3]
    $uploadfile = ARGV[4]
    
    contentsfolder = (($contentsid.to_i / 100).floor + 1)*100
    
    $contentsfolder1 = contentsfolder.to_s

    puts  $contentsfolder1
    
    str = $contentsid + (Digest::MD5.digest($contentsid + $contentspasskey + $siteid ))


    contentsfolder2 = str[0...10].to_s

    puts = contentsfolder2
    
    apfiles = apfiles
    $siteid_ture = $siteid
    test_bucket.object(apfiles/$siteid_ture/$contents/$contentsfolder1/$contentsfolder2/$uploadfile).upload_file($uploadfile)
rescue Aws::S3::MultipartUploadError => _e
    # (e.errors にStandardError オブジェクトの配列が格納されている)

    # 失敗しても(リトライ処理など)何もしない
    false
    
end

if upload_file($uploadfile, $uploadfile) # => true/false
    puts 'アップロードに成功しました'
else
    puts 'アップロードに失敗しました'
end

しかし
test_bucket.object(apfiles/$siteid_ture/$contents/$contentsfolder1/$contentsfolder2/$uploadfile).upload_file($uploadfile)

のところがうまくいかずに困っています。
アップロードするファイルをバケットの中のこの階層に入れるよって指示したつもりですが
undefined method `/' for "":String (NoMethodError)
このようなエラーげ出てしまったりうまくいきません。どのように解決すればよろしいでしょうか

Answer 1

Rubyの文字列連結がうまくいっていないですね。

apfiles/$siteid_ture/$contents/$contentsfolder1/$contentsfolder2/$uploadfileはおそらく変数に入っている文字列を/で連結したものを取得したいのだとおもうのですが、これだと変数同士で割り算をしているようなコードになってしまっています。たとえば以下のようなコードと同じです。

i = 10
j = 5
k = 2

puts(i / j / k)

エラーメッセージは「文字列は割り算ができないよ」という内容です。

一番簡単な対応方法は "#{apifiles}/#{$siteid_true}/... という感じで、Rubyの式展開を使うのが良いかと思います。

ただ、この場合は文字列中にスラッシュなどパスの区切り文字が入ると意図しない挙動になることが多いので、 class Pathname (Ruby 2.4.0) とjoinメソッドを使うと、文字列から安全にパスに変換できます。

また、本筋とはずれますが、Rubyの場合はPHPと違って変数の先頭に$を付ける必要や習慣はないので、それに慣れるとより良いと思います！