אורך הסרטה ב #C

[אוריי]

@yossiz , לגבי הקוד הראשון שהבאת
זה מחזיר לי שגיאה בclass

Severity	Code	Description	Project	File	Line	Suppression State
Error	CS1106	Extension method must be defined in a non-generic static class	SearchByTime	C:\Users\uri\source\repos\SearchByTime\SearchByTime\Form1.cs	16	Active

[yossiz]

@אוריי אמר באורך הסרטה ב #C:

לגבי הקוד הראשון שהבאת
זה מחזיר לי שגיאה בclass

אתה מתכוון לקוד השני? בראשון אין בכלל extension method.
הפתרון הוא להכניס את הפונקציה לתוך קלאס סטטי. כזה:

static class Helpers 
{
	public static Dictionary<string, string> GetDetails(this FileInfo fi)
	{
		Dictionary<string, string> ret = new Dictionary<string, string>();
		Shell shl = new Shell();
		Folder folder = shl.NameSpace(fi.DirectoryName);
		FolderItem item = folder.ParseName(fi.Name);

		for (int i = 0; i < 150; i++)
		{
			string dtlDesc = folder.GetDetailsOf(null, i);
			string dtlVal = folder.GetDetailsOf(item, i);

			if (dtlVal == null || dtlVal == "")
				continue;

			ret.Add(dtlDesc, dtlVal);
		}
		return ret;
	}
}

[רחמים]

@אוריי יש פקד של וידאו בדוטנט, תוכל לטעון את הסרטון לפקד ולקבל את אורכו.

[yossiz]

@רחמים אני הייתי פוסל קוד כזה ב-code review (כמובן, ההכרח לא יגונה, אבל אם יש דרך אחר זה לא הייתי עושה את זה)

[רחמים]

@yossiz פוסל? למה? בגלל ביצועים?

[yossiz]

@רחמים לא רק ביצועים, (אבל גם), אלא זה כמו לבנות בנין שלם כדי לבדוק מידות של אחת מהחלונות...

[רחמים]

@yossiz מה חוץ מביצועים? כי המשל שהזכרת מדבר רק על ביצועים, [והוא גם מאוד מוגזם ביחס לנמשל].

[yossiz]

@רחמים זו דרך עקומה. בלי סיבה, למה ללכת בה? כך אני סבור. זכותך לחלוק. (אם בדרך שלך היו ביצועים יותר טובים, ניחא. אבל אין.).
(יש אומנם נקודת זכות לדרך שלך שהיא יותר פשוטה למי שלא מתכנת מנוסה)

[רחמים]

@אוריי יש פה מגוון דרכים ועיצות

[רחמים]

@yossiz אמר באורך הסרטה ב #C:

יש אומנם נקודת זכות לדרך שלך שהיא יותר פשוטה למי שלא מתכנת מנוסה

נכון, בשביל זה הבאתי את העיצה הזו.

[אוריי]

@yossiz ,

כך הפעלתי את הפונקציה

var ee = Helpers.GetDetails(file);

וזה הערך שהפונקציה מחזירה (במקום לקבל זמן)

System.Collections.Generic.Dictionary`2[System.String,System.String]

[אוריי]

@רחמים עד כמה הרעיון הזה מוצלח כאשר אני צריך לסרוק מאות הסרטות... ???

[yossiz]

@אוריי נכון, הפונצקיה מחזרת dictionary של כל המאפיינים של הקובץ. אתה צריך רק מאפיין מס' 27. צריך להתאים את הקוד קצת.

static class Helpers
{
	public static TimeSpan GetMediaLength(this FileInfo fi)
	{
		Shell shl = new Shell();
		Folder folder = shl.NameSpace(fi.DirectoryName);
		FolderItem item = folder.ParseName(fi.Name);
		string timeSpanStr = folder.GetDetailsOf(item, 27);
		return TimeSpan.Parse(timeSpanStr);
	}
}

[אוריי]

@yossiz
תודה רבה אין מילים

[רחמים]

@yossiz המספר 27 לא בטוח יתאים בכל מערכת הפעלה.

[clickone]

@רחמים אתה בהחלט צודק.
אבל חזקה על מייקרוסופט שבד"כ שומרת על תאימות לאחור.