Nicht genutzen Laufwerksbuchstaben herausfinden

GPC

Na ja, also die "n00bige"-Variante sähe dann wohl so aus:

using System.Linq;

//Aus LinqPAD kopiert
void Main()
{
	foreach (var unassignedLetter in getUnassignedNetworkLetter()) {
		Console.WriteLine(unassignedLetter);
	}
}

// Define other methods and classes here
private IEnumerable<string> getUnassignedNetworkLetter() {
	string[] assignedLetters = Directory.GetLogicalDrives();
    List<string> freeLetters = new List<string>();

	char c = 'A';
	while (c <= 'Z') {
		string letter = c + @":\";
		if (assignedLetters.Contains(letter) == false)
			freeLetters.Add(letter);;
		++c;
	}

    return freeLetters;
}

Das ist jetzt nicht mehr ganz so cool, aber immer noch cool genug
Ein HashSet (Oder nur "Set") ist eine Datenstruktur, die du dir mal genauer anschauen solltest. Die Wikipedia hat sicher einen Artikel.

private static IEnumerable<Char> getUnassignedNetworkLetter()
{
  List<Char> unassigned = "ABCDEFGHIJKLMNOPQRSTUVWXYZ".ToList<Char>();

  foreach (String drive in Environment.GetLogicalDrives())
    unassigned.Remove(drive[0]);

  return unassigned;
}

EXDW

fkaskd schrieb:

private static IEnumerable<Char> getUnassignedNetworkLetter()
{
  List<Char> unassigned = "ABCDEFGHIJKLMNOPQRSTUVWXYZ".ToList<Char>();

  foreach (String drive in Environment.GetLogicalDrives())
    unassigned.Remove(drive[0]);

  return unassigned;
}

Wird nicht funktionieren, da GetLogicalDrives es mit + $:\$ zurück gibt.
Siehe: http://msdn.microsoft.com/en-us/library/system.environment.getlogicaldrives.aspx

D.h. diese Methode gibt alle Laufwerke als unbenutzt zurück

EXDW

Wie wäre es damit?:

static void Main(string[] args)
{
    foreach (var drive in GetFreeDriveLetters())
        Console.WriteLine(drive);
}

private static IEnumerable GetFreeDriveLetters()
{
    var drives = "ABCDEFGHIJKLMNOPQRSTUVWXYZ".ToList<char>().Select(d => d + ":\\").ToList();
    var usedDrives = Environment.GetLogicalDrives();
    drives.RemoveAll(d => usedDrives.Contains(d));
    return drives;
}

Blacky666

Ich habe jetzt auch nochmal ein wenig rumprobiert. Morgen werde ich mich mal in IEnumeral und HashSet einlesen um eure Codebeispiele richtig zu verstehen.

private string getUnassignedNetworkLetter()
        {
            string[] assignedNetworkLetters = Directory.GetLogicalDrives();
            char letter = 'A';

            while (letter <= 'Z')
            {
                foreach (string currentCheckDrive in assignedNetworkLetters)
                {
                    if (currentCheckDrive.Contains(letter) == false)
                        return letter.ToString() + ":";
                    letter++;
                }
            }
            return null;
        }

Th69

Hallo David,

der Code funktioniert aber, denn es wird dort ja "drive[0]" (also nur das erste Zeichen) benutzt...

GPC

Fear teh power of my linq skillz!

Enumerable.Range((int)'A', (int)'Z' - (int)'A' + 1).Select(i => (char)i + @":\").Except(Directory.GetLogicalDrives())

:p

David W schrieb:

fkaskd schrieb:
private static IEnumerable<Char> getUnassignedNetworkLetter()
{
  List<Char> unassigned = "ABCDEFGHIJKLMNOPQRSTUVWXYZ".ToList<Char>();

  foreach (String drive in Environment.GetLogicalDrives())
    unassigned.Remove(drive[0]);

  return unassigned;
}
Wird nicht funktionieren, da GetLogicalDrives es mit + $:\$ zurück gibt.
Siehe: http://msdn.microsoft.com/en-us/library/system.environment.getlogicaldrives.aspx

D.h. diese Methode gibt alle Laufwerke als unbenutzt zurück

Nein, wie Th69 schon gesagt hat, funktioniert der Code korrekt.

EXDW

Stimmt, hab das [0] übersehen - Sorry

Die ersten beiden Lösungen von GPC finde ich am schönsten, klar geht es kürzer, aber im Sinne der Wartbarkeit und vor allem Übersicht sind diese beiden zu bevorzugen

Ich find die Lösung von fkaskd sehr schön. Durchaus elegant und tut genau das, was man erwartet. Einen Buchstaben (Letter) zurückgeben.

Insbesondere die while-Schleife in den anderen Codes ist nicht so toll. Hier wäre eine For-Schleife um einiges schöner:

for (char c = 'A'; c <= 'Z'; ++c)
{
 // ...
}

GPC

Flurry schrieb:

Ich find die Lösung von fkaskd sehr schön. Durchaus elegant und tut genau das, was man erwartet. Einen Buchstaben (Letter) zurückgeben.

Die Lösung finde ich auch gut, aber sie gibt dennoch eine Sequenz von Buchstaben und nicht nur einen einzelnen zurück

Insbesondere die while-Schleife in den anderen Codes ist nicht so toll. Hier wäre eine For-Schleife um einiges schöner:

Das halte ich für eine Geschmacksfrage.. auf mich wirkt die for-Schleife irgendwie "unpassend"

und ich dachte immer wenn A-Z belegt ist geht es bei AA, AB, AC, ... weiter.

inflames2k

brzzz schrieb:

und ich dachte immer wenn A-Z belegt ist geht es bei AA, AB, AC, ... weiter.

Nicht so viel denken. Windows kennt nur 26 Laufwerkbezeichner