Förenkla och förminska?

användarkontots rättigheter borde jag väl med fördel kunna använda mig av enum istället och ha en metod för getUserType istället och byta ut alla access mot UserType eller MemberType?

Jag hade gjort såhär.

Du har t.ex. flera IM fält bryt ut dom mot en lista med IM objekt.
Du har flera Access bryt ut dom till ett eget objekt.

Om du hade haft fler parametrar till Adress och namn hade jag brutit ut dom också.

Nickemannen skrev:

Jag hade gjort såhär.

Du har t.ex. flera IM fält bryt ut dom mot en lista med IM objekt.
Du har flera Access bryt ut dom till ett eget objekt.

Om du hade haft fler parametrar till Adress och namn hade jag brutit ut dom också.

Hmm jag funderade på en lista men innebär inte det att jag måste stoppa just IM i en egen klass?

Ne det kan visserligen vara en lista med strängar.
Borde vara det eftersom det bara är en sträng som skall sparas.

Nån speciell anledning till att du har en konstruktor med alla dessa variablar?
Själv är jag lite allergisk mot sånt! :)
För samtidigt har du ju alla fält som publika egenskaper, känns som om det räcker!
Men vad vet jag, är lite full just nu! Skål! :bire
Annars är nog klassen så rolig som den kan bli! :)

Ja ibland känner jag att det rentav kan räcka men en liten struct istället (less is more som vi säger när vi spelar i bandet)

När det gäller rättigheter så har jag lite dåliga erfarenheter när man sätter dem direkt på användaren. Ju mer man bygger ut och ju fler användare det blir så vet man till slut inte vad användarna får göra och inte får göra. Bättre då att flytta all behörighet till grupper istället och så gör du en List<> med användarens grupper i. :)

doggelito skrev:

När det gäller rättigheter så har jag lite dåliga erfarenheter när man sätter dem direkt på användaren. Ju mer man bygger ut och ju fler användare det blir så vet man till slut inte vad användarna får göra och inte får göra. Bättre då att flytta all behörighet till grupper istället och så gör du en List<> med användarens grupper i. :)

Ja det var ungefär vad nicke förslog ;)

Jag är lite vilsen om jag ska vara helt ärlig!

enum IM { Skype, MsnMessenger, Icq, YahooMessenger }

skulle väl gå att använda till nåt. Annars kanske en IList skulle funka men jag begriper inte hur jag skulle binda värden till min IList...

		public IList IM
		{
			get { return im; }
			set { im = value; }
		}

lär ju inte vara rätt metod. Man kan ju lägga till i en ILIst men hur då i det här fallet? Jag har läst för mycket om DDD och Patterns det sista och nu är jag helt vilsen i programmeringen.... allt har blivit kaos :)

Varför inte skapa en im klass som du sedan fyller i din user med en List<im> som innehåller alla messangers som användaren har tillgång till?

Jag hade tänkt det från början, sedan hörde jag att varje klass ska ha en ege fil och det känns väldigt lite för en sådan fil när det ändå bara kommer tillhöra user...

Antar att jag får lämna det bakom mig och ha dubbla klasser tillsvidare

Har möblerat om lite... tror det kommer bli lite snyggare såhär.

[Flags()]
public enum Permissions
{ 
	None = 0,
	Administer = 1, 
	PostResults = 2, 
	PostNews = 4,
	User = 8,
	All = Administer | PostResults | PostNews | User
}

public class User
{
	#region Private Member Variables
	private int userid;
	private string username;
	private string email;
	private string password;
	private string fullname;
	private string address;
	private int mpnid;
	private int phone;
	private string signature;
	private IList<IM> im;
	private DateTime lastlogindate;
	private Permissions _permission = Permissions.None;
	private bool active;
	#endregion

	#region Constructors
	public User() { }
	#endregion

	#region Public Properties
	public int UserID
	{
		get { return userid; }
		set { userid = value; }
	}
	public string UserName
	{
		get { return username; }
		set { username = value; }
	}
	public string Email
	{
		get { return email; }
		set { email = value; }
	}
	public string Password
	{
		get { return password; }
		set { password = value; }
	}
	public string FullName
	{
		get { return fullname; }
		set { fullname = value; }
	}
	public string Address
	{
		get { return address; }
		set { address = value; }
	}
	public int MpnID
	{
		get { return mpnid; }
		set { mpnid = value; }
	}
	public int Phone
	{
		get { return phone; }
		set { phone = value; }
	}
	public string Signature
	{
		get { return signature; }
		set { signature = value; }
	}
	public IList<IM> IM;

	public DateTime LastLoginDate
	{
		get { return lastlogindate; }
		set { lastlogindate = value; }
	}
	public Permissions Permission
	{
		get { return _permission; }
		set { _permission = value; }
	}
	public bool Active
	{
		get { return active; }
		set { active = value; }
	}
	#endregion
}

public class IM
{
	private string _skype;
	private string _msnmessenger;
	private string _yahoomessenger;
	private int _icq;

	public string Skype
	{
		get { return _skype; }
		set { _skype = value; }
	}
	public string MsnMessenger
	{
		get { return _msnmessenger; }
		set { _msnmessenger = value; }
	}
	public int ICQ
	{
		get { return _icq; }
		set { _icq = value; }
	}
	public string YahooMessenger
	{
		get { return _yahoomessenger; }
		set { _yahoomessenger = value; }
	}
}

Fiffig enum där med All, visste jag inte:-)

Och inte vet jag vad Flags() attributet gör för något?

Det är för att du ska kunna jämföra har jag för mig

CatZ skrev:

Det är för att du ska kunna jämföra har jag för mig

Det stämmer. Flags-attributet säger att följande enum är ett bitfält som logiska uttryck som t.ex. AND och OR ska kunna operera på.

Ah vad smutt, det är ju verkligen användbart!

Tror att detta ska fungera om du sedan vill kolla om en användare har tillstånd

public bool HasPermission(Permissions _permissionToCheck)
{
	int _permission = Permissions.All;
	return ((_permissionToCheck & _permission) == _permissionToCheck);
}

Nu vet jag inte hur din databas ser ut men rent spontant så hade jag nog inte gjort im klassen så där.
För mig hade den nog sett ut typ:

public class IM
{
     private int id;
     private string name;
     private string username;

     public ...

och sedan lagt alla mess. programmen i en egen tabell i databasen med relation till users för att på så sätt slippa hårdkodningar av programmen.
det blir enkelt då att lägga till ett mess. program då om du skulle vilja! :)

Jag kommer i dagsläget inte ha någon användning av IM alls, det är ett framtida projekt och ingenting som jag bryr mig så himla mycket om men jag håller med dig om att det blir lite bättre maintainability då.

Däremot tror jag inte det kommer sådär himla många nya chatprogram som folk kommer använda sig av.

Själv har jag testat det mesta men lagt ner allt utom Live Messenger.