-
Notifications
You must be signed in to change notification settings - Fork 163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Grep実行結果に対してGrepを実行すると検索対象に存在しない行が検索結果に出力される #1936
Comments
「編集中のテキストから検索」にチェックを入れると文書中のテキスト全体が対象になるのでそういう挙動になりますね。 Grep実行後に追加される検索条件と検索対象の行を自動的に検索対象から外すのは難しい気がします。Grep実行結果の文書から「編集中のテキストから検索」にチェックを入れてGrepを実行すると同じ文書に検索結果の文章が追記されます。その場合に各行が検索対象かどうかを適切に選別するのは難しいと思います。 1.「選択中のテキストから検索」みたいなチェックを新規に用意、ユーザーは検索対象のテキストを選択してからGrepする 上記のようないくつかの対処方法が考えられます。 |
「2. Grep実行後に追加される検索条件と検索対象の行をユーザーが自分で削除してからGrep」を試してみましたが、再現手順3と同じ結果になりました(検索条件と検索対象の行だけでなくGrep実行結果をすべて削除しても同様)。 再現手順2の結果にはhugaという文字列が存在しないのですが、その手順2の結果の文書に直接Grepを実行すると検索結果にhugaが含まれる挙動となるのであれば、Grep検索の結果に対して直接Grepを実行するのはあきらめたほうがよさそうですね。 とりあえず「Grep結果を別の文書にコピーしてから再度Grepする」方法で回避しようと思います。 |
@musagi さん どう改善したらよいかの提案は可能ですか? 再現手順2で、検索結果が2行となっている点に違和感ありますが、「前回の検索条件」が検索対象になる挙動は妥当に思えます。「開いているウインドウ」を対象にGrepする場合、ウインドウに表示されている文字列の全文が検索対象になる仕様だからです。 使い勝手の問題として、「編集中のウインドウ」が「Grep検索結果」である場合、Grepの検索条件と思われる部分を除外して検索する、の仕様を追加したほうがよいの主張を否定するつもりはありません。 その場合、「なんで?なんでそうすると便利になる?」をプレゼンする感じになります。 |
「前回の検索条件」ではなく、これから実行しようとしている、ウインドウに表示されていない「今回の検索条件」が検索対象となっているようです。hogeとhugaが紛らわしいようですいません。 再現手順3の結果の検索対象が「 Window:[ 【Grep】"huga"]」となっていますので、おそらくGrep結果のウインドウにこれから検索しようとしている検索条件hugaについての文章を追加してから、その内容に対して検索を実行しているのではないでしょうか。 改善するとすれば、再現手順2のGrep結果のウインドウを対象としてGrepをする場合は、「今回の検索条件」を検索対象から外すか、再現手順2のGrep結果のウインドウに対してGrep結果を追記するのではなく、新しくウインドウを作成してGrep結果を表示するほうが直感的なのではないかと思います。 例えば、再現手順2のGrep結果のウインドウ(hogeの検索結果)に対してhugaでGrepする場合は、結果を表示するウインドウには以下が表示されることになります。
|
理解しました。仕様バグですね。 検索対象が「現在のウインドウ」であり、かつ、「現在のウインドウ」が検索結果ウインドウである場合に 簡単な直し方は、「その場合は新しい検索結果ウインドウを開く」だと思いますが、ユーザビリティどうなんでしょう。 |
問題内容
Grep実行結果に対してGrepを実行すると、Grep実行後に追加される検索条件と検索対象の行が検索結果に出力される。
Grep実行後に追加される検索条件と検索対象の行は検索対象とならないのが正しいと思われる。
再現手順
再現頻度
100%
問題のカテゴリ
環境情報
OS バージョン
Windows 10 Enterprise 64bit
サクラエディタバージョン
PC情報
スクリーンショット
なし
The text was updated successfully, but these errors were encountered: