string/concatenation.rb tests are misleading

オープン

#64 2015年8月10日に作成コメント 3 件リアクション 0 件担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度: 2/5
見積もり時間: 1〜3時間
初心者へのやさしさ: 45/100

issue の種類: リファクタリング
明瞭さ: 明確に書かれている
活発さ: 停滞
技術スタック: ruby
領域: performance, testing

調査の方向性

string/concatenation.rb から始めて、現在のベンチマークが高速なメソッドをどのように定義し、連結方法を比較しているかを調べます。比較には実行時変数を使い、issue の benchmark-ips の例を基準にします。ベンチマークがパーサー時の連結を実行時の操作として扱わず、示されている代替手段を公平に比較するようになれば完了です。

索引モデルが issue の本文から書いたものです。

説明

Hi, the tests in string/concatenation.rb are quite misleading.

The fast method consists of this

def fast
  'foo' 'bar'
end

That's not concatenating during calling of fast but on parsing the code. If you write another method just returning foobar, it is as fast as this method.

So I think this is not fair comparison and what you usually want is to concatenate two variables during runtime.

For this use case concat and << are calling the same code, so they have the same performance and both are fine if you want to change the string on the left and not just get two strings concatenated. If you want a new string you can use +.

Some better test could be to compare + and String interpolation

Benchmark.ips do |x|
  foo = 'foo'
  bar = 'bar'

  x.report('String#+') do
    foo + bar
  end

  x.report('String interpolation') do
    "#{foo}#{bar}"
    end

  x.compare!
end

This still has the difference that interpolation can handle nil values, while + cannot.

主要言語: Ruby
スター: 5.7k
フォーク: 370
PR マージ指標: 30日以内にマージされた PR はありません

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
リポジトリをフォークし、ブランチを切って変更します。
issue 番号を参照したプルリクエストを送ります。

fastruby/fast-ruby のほかの issue

`dup`ing an Identity Hash/Set is faster than creating a new one オープン

難易度 2/5 1〜3時間初心者へのやさしさ 52/100

fastruby/fast-ruby#230 · コメント 2 件 · 2024年7月15日に作成
[Documentation] Could more explanations be added? オープン

難易度 4/5 3〜5日初心者へのやさしさ 32/100

fastruby/fast-ruby#229 · 2024年4月1日に作成
Comparison logic vs `between?` vs `include?` オープン

難易度 3/5 1〜2日初心者へのやさしさ 35/100

fastruby/fast-ruby#225 · 2023年11月5日に作成
`str.count("\n")` is 1.3-170 times faster than `str.lines.count` or `str.each_line.count` depending on the string size オープン

難易度 2/5 1〜3時間初心者へのやさしさ 45/100

fastruby/fast-ruby#220 · コメント 6 件 · 2023年9月11日に作成
Parallel versus sequential assignment is microbenchmark (not representative) and scales with number of inputs オープン

難易度 4/5 3〜5日初心者へのやさしさ 42/100

fastruby/fast-ruby#208 · コメント 7 件 · リアクション 1 件 · 2022年8月31日に作成

fastruby/fast-ruby の issue をすべて見る

似ている issue

P4 FR: Add one-click copy-to-clipboard button to code snippets in static workshop HTML オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

palladius/rails8-app-on-gcp#145 · 2026年9月23日に作成
Restore `preview_sensitive_media` to the admin UI オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

mastodon/mastodon#40662 · 2026年9月23日に作成
Cop idea: prefer `be_an_instance_of` over `be_instance_of` オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

rubocop/rubocop-rspec#2236 · 2026年9月23日に作成
condition solver and JSON Schema are out of sync? オープン

bug

難易度 2/5 1〜3時間初心者へのやさしさ 70/100

riscv/riscv-unified-db#2624 · リアクション 1 件 · 2026年9月23日に作成
Bounces#match crashes with NameError for already-bounced recipients オープン

難易度 1/5 1時間未満初心者へのやさしさ 88/100

yegor256/mailanes#322 · 2026年9月22日に作成

Ruby の issue をもっと見る