﻿<?xml version="1.0" encoding="utf-8"?>
<InstanceReport xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema">
  <Version>2.2.0.7</Version>
  <hasSegments>false</hasSegments>
  <ReportName>Commitments and Contingencies</ReportName>
  <ReportLongName>0217 - Disclosure - Commitments and Contingencies</ReportLongName>
  <DisplayLabelColumn>true</DisplayLabelColumn>
  <ShowElementNames>false</ShowElementNames>
  <RoundingOption />
  <HasEmbeddedReports>false</HasEmbeddedReports>
  <Columns>
    <Column>
      <LabelColumn>false</LabelColumn>
      <Id>1</Id>
      <Labels>
        <Label Id="1" Label="9 Months Ended" />
        <Label Id="2" Label="Sep. 30, 2010" />
      </Labels>
      <CurrencyCode>USD</CurrencyCode>
      <FootnoteIndexer />
      <hasSegments>false</hasSegments>
      <hasScenarios>false</hasScenarios>
      <Segments />
      <Scenarios />
      <Units>
        <Unit>
          <UnitID>USD</UnitID>
          <UnitType>Standard</UnitType>
          <StandardMeasure>
            <MeasureSchema>http://www.xbrl.org/2003/iso4217</MeasureSchema>
            <MeasureValue>USD</MeasureValue>
            <MeasureNamespace>iso4217</MeasureNamespace>
          </StandardMeasure>
          <Scale>0</Scale>
        </Unit>
        <Unit>
          <UnitID>USDEPS</UnitID>
          <UnitType>Divide</UnitType>
          <NumeratorMeasure>
            <MeasureSchema>http://www.xbrl.org/2003/iso4217</MeasureSchema>
            <MeasureValue>USD</MeasureValue>
            <MeasureNamespace>iso4217</MeasureNamespace>
          </NumeratorMeasure>
          <DenominatorMeasure>
            <MeasureSchema>http://www.xbrl.org/2003/instance</MeasureSchema>
            <MeasureValue>shares</MeasureValue>
            <MeasureNamespace>xbrli</MeasureNamespace>
          </DenominatorMeasure>
          <Scale>0</Scale>
        </Unit>
        <Unit>
          <UnitID>Shares</UnitID>
          <UnitType>Standard</UnitType>
          <StandardMeasure>
            <MeasureSchema>http://www.xbrl.org/2003/instance</MeasureSchema>
            <MeasureValue>shares</MeasureValue>
            <MeasureNamespace>xbrli</MeasureNamespace>
          </StandardMeasure>
          <Scale>0</Scale>
        </Unit>
      </Units>
      <CurrencySymbol>$</CurrencySymbol>
    </Column>
  </Columns>
  <Rows>
    <Row>
      <Id>2</Id>
      <Label>Commitments and Contingencies [Abstract]</Label>
      <Level>0</Level>
      <ElementName>nhp_CommitmentsAndContingenciesAbstract</ElementName>
      <ElementPrefix>nhp</ElementPrefix>
      <IsBaseElement>false</IsBaseElement>
      <BalanceType>na</BalanceType>
      <PeriodType>duration</PeriodType>
      <ShortDefinition>Commitments and Contingencies.</ShortDefinition>
      <IsReportTitle>false</IsReportTitle>
      <IsSegmentTitle>false</IsSegmentTitle>
      <IsSubReportEnd>false</IsSubReportEnd>
      <IsCalendarTitle>false</IsCalendarTitle>
      <IsTuple>false</IsTuple>
      <IsAbstractGroupTitle>true</IsAbstractGroupTitle>
      <IsEquityPrevioslyReportedAsRow>false</IsEquityPrevioslyReportedAsRow>
      <IsEquityAdjustmentRow>false</IsEquityAdjustmentRow>
      <IsBeginningBalance>false</IsBeginningBalance>
      <IsEndingBalance>false</IsEndingBalance>
      <IsReverseSign>false</IsReverseSign>
      <PreferredLabelRole />
      <IsEPS>false</IsEPS>
      <FootnoteIndexer />
      <Cells>
        <Cell>
          <Id>1</Id>
          <ShowCurrencySymbol>false</ShowCurrencySymbol>
          <IsNumeric>false</IsNumeric>
          <IsRatio>false</IsRatio>
          <DisplayZeroAsNone>false</DisplayZeroAsNone>
          <NumericAmount>0</NumericAmount>
          <RoundedNumericAmount>0</RoundedNumericAmount>
          <NonNumbericText />
          <NonNumericTextHeader />
          <FootnoteIndexer />
          <hasSegments>false</hasSegments>
          <hasScenarios>false</hasScenarios>
          <DisplayDateInUSFormat>false</DisplayDateInUSFormat>
        </Cell>
      </Cells>
      <OriginalInstanceReportColumns />
      <ElementDataType>xbrli:stringItemType</ElementDataType>
      <SimpleDataType>string</SimpleDataType>
      <ElementDefenition>Commitments and Contingencies.</ElementDefenition>
      <IsTotalLabel>false</IsTotalLabel>
    </Row>
    <Row>
      <Id>3</Id>
      <Label>Commitments and Contingencies</Label>
      <Level>1</Level>
      <ElementName>us-gaap_CommitmentsAndContingenciesDisclosureTextBlock</ElementName>
      <ElementPrefix>us-gaap</ElementPrefix>
      <IsBaseElement>true</IsBaseElement>
      <BalanceType>na</BalanceType>
      <PeriodType>duration</PeriodType>
      <ShortDefinition>No definition available.</ShortDefinition>
      <IsReportTitle>false</IsReportTitle>
      <IsSegmentTitle>false</IsSegmentTitle>
      <IsSubReportEnd>false</IsSubReportEnd>
      <IsCalendarTitle>false</IsCalendarTitle>
      <IsTuple>false</IsTuple>
      <IsAbstractGroupTitle>false</IsAbstractGroupTitle>
      <IsEquityPrevioslyReportedAsRow>false</IsEquityPrevioslyReportedAsRow>
      <IsEquityAdjustmentRow>false</IsEquityAdjustmentRow>
      <IsBeginningBalance>false</IsBeginningBalance>
      <IsEndingBalance>false</IsEndingBalance>
      <IsReverseSign>false</IsReverseSign>
      <PreferredLabelRole>verboselabel</PreferredLabelRole>
      <IsEPS>false</IsEPS>
      <FootnoteIndexer />
      <Cells>
        <Cell>
          <Id>1</Id>
          <ShowCurrencySymbol>false</ShowCurrencySymbol>
          <IsNumeric>false</IsNumeric>
          <IsRatio>false</IsRatio>
          <DisplayZeroAsNone>false</DisplayZeroAsNone>
          <NumericAmount>0</NumericAmount>
          <RoundedNumericAmount>0</RoundedNumericAmount>
          <NonNumbericText>&lt;!--DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd" --&gt;
   &lt;!-- Begin Block Tagged Note 17 - us-gaap:CommitmentsAndContingenciesDisclosureTextBlock--&gt;
   &lt;div style="font-family: 'Times New Roman',Times,serif; margin-left: 0in; "&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 10pt"&gt;&lt;b&gt;17. Commitments and Contingencies&lt;/b&gt;
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 10pt; margin-left: 4%"&gt;&lt;i&gt;Litigation&lt;/i&gt;
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 10pt; text-indent: 8%"&gt;From time to time, we are a party to various legal proceedings, lawsuits and other claims (as
   to some of which we may not be insured) that arise in the normal course of our business. Regardless
   of their merits, these matters may require us to expend significant financial resources. Except as
   described in our Annual Report on Form 10-K for the year ended December&amp;#160;31, 2009 and Quarterly
   Report on Form 10-Q for the period ended March&amp;#160;31, 2010, we are not aware of any other legal
   proceedings or claims that we believe may have, individually or taken together, a material adverse
   effect on our business, results of operations or financial position. However, we are unable to
   predict the ultimate outcome of pending litigation and claims, and if our assessment of our
   liability with respect to these actions and claims is incorrect, such actions and claims could have
   a material adverse effect on our business, results of operations or financial position.
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 10pt; margin-left: 4%"&gt;&lt;i&gt;Development Agreements&lt;/i&gt;
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 10pt; text-indent: 8%"&gt;During the nine months ended September&amp;#160;30, 2010, we entered into Mission Hills JV to develop a
   medical office building (see Note 5) and entered into another agreement to fund the expansion of a
   skilled nursing facility (see Note 4). As of September&amp;#160;30, 2010, we had committed to fund an
   additional $47.4&amp;#160;million under these agreements, of which $36.5&amp;#160;million relates to Mission Hills JV
   and is expected to be funded through a third party construction loan.
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 10pt; margin-left: 4%"&gt;&lt;i&gt;Lines of Credit&lt;/i&gt;
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 10pt; text-indent: 8%"&gt;Under the terms of an agreement with PMB LLC, we agreed to extend to PMB LLC a $10.0&amp;#160;million
   line of credit at an interest rate equal to LIBOR plus 175 basis points to fund certain costs of
   PMB LLC with respect to the proposed development of multi-tenant medical office buildings. During
   the nine months ended September&amp;#160;30, 2010, we funded $0.8&amp;#160;million under the line of credit. As of
   September&amp;#160;30, 2010, $4.0&amp;#160;million was outstanding and is included in the caption &amp;#8220;Other assets&amp;#8221; on
   our consolidated balance sheet.
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 10pt; text-indent: 8%"&gt;We entered into an agreement with PMB LLC, the manager of PMB Pomona LLC, to extend up to $3.0
   million of funding at an interest rate of 7.25%, which was secured by 100% of the membership
   interests in PMB Pomona LLC (see Note 18). During the nine months ended September&amp;#160;30, 2010, we
   funded $0.3&amp;#160;million, and the total $1.9&amp;#160;million outstanding was subsequently repaid. No further
   disbursements will be made under the agreement.
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 10pt; text-indent: 8%"&gt;As of February&amp;#160;1, 2010, in connection with the formation of Gilbert JV, a consolidated joint
   venture, we agreed to loan Gilbert JV up to $8.8&amp;#160;million as project financing at an interest rate
   of 7.00%, including $6.8&amp;#160;million that was disbursed initially and remains outstanding at September
   30, 2010 (see Note 5).
   &lt;/div&gt;
   &lt;!-- Folio --&gt;
   &lt;!-- /Folio --&gt;
   &lt;/div&gt;
   &lt;!-- PAGEBREAK --&gt;
   &lt;div style="font-family: 'Times New Roman',Times,serif; margin-left: 0in; "&gt;
   &lt;div align="center" style="font-size: 10pt; margin-top: 0pt"&gt;
   &lt;b&gt;
   &lt;/b&gt;
   &lt;/div&gt;
   &lt;div align="center" style="font-size: 10pt; margin-top: 0pt"&gt;
   &lt;b&gt;
   &lt;/b&gt;
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 10pt; text-indent: 8%"&gt;As of March&amp;#160;1, 2010, in connection with the formation of Pasadena JV, a consolidated joint
   venture, we agreed to loan Pasadena JV up to $56.5&amp;#160;million as project financing at an initial
   interest rate equal to the greater of 3.50% or LIBOR plus 165 basis points (increasing to the
   greater of 5.125% or LIBOR plus 375 basis points as of April&amp;#160;1, 2010), including $49.8&amp;#160;million that
   was disbursed initially and remains outstanding at September&amp;#160;30, 2010 (see Note 5).
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 10pt; margin-left: 4%"&gt;&lt;i&gt;Indemnities&lt;/i&gt;
   &lt;/div&gt;
   &lt;div align="left" style="font-size: 10pt; margin-top: 10pt; text-indent: 8%"&gt;We have entered into indemnification agreements with those partners who contributed
   appreciated property into NHP/PMB. Under these indemnification agreements, if any of the
   appreciated real estate contributed by the partners is sold by NHP/PMB in a taxable transaction
   within a specified number of years after the property was contributed, we will reimburse the
   affected partners for the federal and state income taxes associated with the pre-contribution gain
   that is specially allocated to the affected partner under the Code. We have no current plans to
   sell any of these properties.
   &lt;/div&gt;
   &lt;/div&gt;
</NonNumbericText>
          <NonNumericTextHeader>&lt;!--DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd" --&gt;
   &lt;!-- Begin Block Tagged Note</NonNumericTextHeader>
          <FootnoteIndexer />
          <hasSegments>false</hasSegments>
          <hasScenarios>false</hasScenarios>
          <DisplayDateInUSFormat>false</DisplayDateInUSFormat>
        </Cell>
      </Cells>
      <OriginalInstanceReportColumns />
      <ElementDataType>us-types:textBlockItemType</ElementDataType>
      <SimpleDataType>textblock</SimpleDataType>
      <ElementDefenition>Includes disclosure of commitments and contingencies. This element may be used as a single block of text to encapsulate the entire disclosure including data and tables.</ElementDefenition>
      <ElementReferences>Reference 1: http://www.xbrl.org/2003/role/presentationRef
 -Publisher FASB
 -Name FASB Interpretation (FIN)
 -Number 14
 -Paragraph 3

Reference 2: http://www.xbrl.org/2003/role/presentationRef
 -Publisher FASB
 -Name Statement of Financial Accounting Standard (FAS)
 -Number 5
 -Paragraph 9, 10, 11, 12

</ElementReferences>
      <IsTotalLabel>false</IsTotalLabel>
    </Row>
  </Rows>
  <Footnotes />
  <NumberOfCols>1</NumberOfCols>
  <NumberOfRows>2</NumberOfRows>
  <HasScenarios>false</HasScenarios>
  <MonetaryRoundingLevel>UnKnown</MonetaryRoundingLevel>
  <SharesRoundingLevel>UnKnown</SharesRoundingLevel>
  <PerShareRoundingLevel>UnKnown</PerShareRoundingLevel>
  <HasPureData>false</HasPureData>
  <SharesShouldBeRounded>true</SharesShouldBeRounded>
</InstanceReport>
