StackOverflow2013

Note that there are some explanatory texts on larger screens.

plurals

PO
text
Body
copied!<p>There's already an extension method which will do this. Just stick this at the end of your existing query:</p> <pre><code>.ToDictionary(x => x.FileHash, x => x.Files); </code></pre> <p>However: using <code>Encoding.UTF8.GetString</code> to convert <em>arbitrary binary data</em> into a string is a really bad idea. Use <code>Convert.ToBase64String</code> instead. The hash is <em>not</em> a UTF-8 encoded string, so don't treat it as one.</p> <p>You're also grouping by hash twice, which I suspect isn't really what you want to do.</p> <p>Alternatively, remove the previous <code>GroupBy</code> calls and use a <code>Lookup</code> instead:</p> <pre><code>var query = Directory.GetFiles(searchFolder, "*.*") .Select(f => new { FileName = f, FileHash = Convert.ToBase64String( new SHA1Managed().ComputeHash(...)) }) .ToLookup(x => x.FileHash, x => x.FileName); </code></pre> <p>That will give you a <code>Lookup<string, string></code>, which is basically the files grouped by hash.</p> <p>One further thing to note: I suspect you'll be leaving file streams open with this method. I suggest you write a small separate method to compute the hash of a file based on its name, but making sure you close the stream (with a <code>using</code> statement in the normal way). This will also end up making your query simpler - something along the lines of:</p> <pre><code>var query = Directory.GetFiles(searchFolder) .ToLookup(x => ComputeHash(x)); </code></pre> <p>It's hard to simplify it much further than that :)</p>

Querying!

Guidance

An individual column

Larger individual text columns get their own page to allow for proper reading.

SQuiL has stopped working due to an internal error.

If you are curious you may find further information in the browser console, which is accessible through the devtools (F12).

Reload