Best way of getting SPWeb in Event Receiver [Best Practices]

Question

what would be the best way of getting SPWeb and List in context below, specially when list title can be different for different cultural websites,

   private bool doesItHaveIt(SPItemEventProperties properties)
            {
                using (SPWeb web = properties.OpenWeb())
                {
                    SPList DocLib= web.Lists["Doc Library"]; //its list where item is being added

                    if (DocLib.ItemCount < 1)
                    {

and also what would be best way of getting resources from 12 hive,

Culture ci = null;
string listName;

try
{
   using (SPWeb web = properties.OpenWeb())
   {
       ci = web.Locale;
       listName = Resources.ListNames.ResourceManager.GetString("MyList", ci);

Answer 1

first of all, SPItemEventProperties has Web method, that you can use. If I'm not wrong, you do not need to dispose this object. Second, use SPUtility.GetLocalizedString.

Answer 2

why dont you follow msdn as its the first place i look for best practices. This is for spweb in an event reciver ;) also your doesItHaveIt(SPItemEventProperties properties) already handels a web object in properties so there is no need to dispose of the object.

http://msdn.microsoft.com/en-us/library/ee724407(v=office.14).aspx

good

// Retrieve SPWeb and SPListItem from SPItemEventProperties instead of
// from a new instance of SPSite.
SPWeb web = properties.OpenWeb();
// Operate on the SPWeb object.
SPListItem item = properties.ListItem;
// Operate on an item.

bad

public override void ItemDeleting(SPItemEventProperties properties)
{
    using (SPSite site = new SPSite(properties.WebUrl))
    {
    using (SPWeb web = site.OpenWeb())
        {
        SPList list = web.Lists[properties.ListId];
        SPListItem item = list.GetItemByUniqueId(properties.ListItemId);
        // Operate on an item.
        }
    }
}

for your resource string:

SPUtility.GetLocalizedString("$Resources: GetAllitems", "core", 1033);

it translates as, get GetAllitems from core.resx with the language of en-US (english) which makes the file core.en-US.resx within web server extensions\12\Resources

as alexander pointed out from msdn:

sourceType:

System.String An ASP.NET resource expression in the form $Resources:keyname, where keyname is the name half of a name/value pair in a resource file (.resx).

defaultResourceFileType:

System.String The base file name of the language resource file containing a localized string value. For example, if you have a series of resource files named myresources.en-us.resx, myresources.es-es.resx, myresources.de-de.resx, and so on, the value to pass in this parameter is myresources.

languageType:

System.UInt32 The LCID of the target language. For a list, see the overview topic for the CultureInfo class.

hope it helps :)