<?xml version="1.0"?>

<ownershipDocument>

    <schemaVersion>X0201</schemaVersion>

    <documentType>4</documentType>

    <periodOfReport>2004-01-19</periodOfReport>


    <notSubjectToSection16>0</notSubjectToSection16>

    <issuer>
        <issuerCik>0000067347</issuerCik>
        <issuerName>MODINE MANUFACTURING CO</issuerName>
        <issuerTradingSymbol>MODI</issuerTradingSymbol>
    </issuer>

    <reportingOwner>
        <reportingOwnerId>
            <rptOwnerCik>0001172676</rptOwnerCik>
            <rptOwnerName>RULSEH JAMES R</rptOwnerName>
        </reportingOwnerId>
        <reportingOwnerAddress>
            <rptOwnerStreet1>1500 DEKOVEN AVENUE</rptOwnerStreet1>
            <rptOwnerStreet2></rptOwnerStreet2>
            <rptOwnerCity>RACINE</rptOwnerCity>
            <rptOwnerState>WI</rptOwnerState>
            <rptOwnerZipCode>53403</rptOwnerZipCode>
            <rptOwnerStateDescription></rptOwnerStateDescription>
        </reportingOwnerAddress>
        <reportingOwnerRelationship>
            <isDirector>0</isDirector>
            <isOfficer>1</isOfficer>
            <isTenPercentOwner>0</isTenPercentOwner>
            <isOther>0</isOther>
            <officerTitle>Group Vice President</officerTitle>
        </reportingOwnerRelationship>
    </reportingOwner>

    <nonDerivativeTable>

        <nonDerivativeTransaction>
            <securityTitle>
                <value>Modine common stock</value>
            </securityTitle>
            <transactionDate>
                <value>2004-01-19</value>
            </transactionDate>
            <transactionCoding>
                <transactionFormType>4</transactionFormType>
                <transactionCode>F</transactionCode>
                <equitySwapInvolved>0</equitySwapInvolved>
            </transactionCoding>
            <transactionTimeliness>
                <value></value>
            </transactionTimeliness>
            <transactionAmounts>
                <transactionShares>
                    <value>740</value>
                    <footnoteId id="F1"/>
                </transactionShares>
                <transactionPricePerShare>
                    <value>27.79</value>
                </transactionPricePerShare>
                <transactionAcquiredDisposedCode>
                    <value>D</value>
                </transactionAcquiredDisposedCode>
            </transactionAmounts>
            <postTransactionAmounts>
                <sharesOwnedFollowingTransaction>
                    <value>21772.0102</value>
                    <footnoteId id="F2"/>
                </sharesOwnedFollowingTransaction>
            </postTransactionAmounts>
            <ownershipNature>
                <directOrIndirectOwnership>
                    <value>D</value>
                </directOrIndirectOwnership>
            </ownershipNature>
        </nonDerivativeTransaction>

    </nonDerivativeTable>

    <footnotes>
        <footnote id="F1">Shares disposed of in private transaction to issuer to satisfy tax withholding obligation.</footnote>
        <footnote id="F2">This total includes 1,186.1142 units of Modine common stock indirectly held in the Registrant's Modine 401(k) Retirement Plan account (7.4631 units were purchased on 1/14/04), 41.5998 units of Modine common stock indirectly held in the Registrant's Modine Deferred Compensation Plan account and 4,423.0334 shares of Modine common stock indirectly held in the Registrant's Modine Employee Stock Ownership Plan account.</footnote>
    </footnotes>


    <ownerSignature>
        <signatureName>/s/D. R. Zakos, Attorney-in-Fact for J. R. Rulseh</signatureName>
        <signatureDate>2004-01-21</signatureDate>
    </ownerSignature>

</ownershipDocument>

