diff --git a/src/generate/index.js b/src/generate/index.js index b54b5217a0..52a92b6b52 100644 --- a/src/generate/index.js +++ b/src/generate/index.js @@ -1,5 +1,6 @@ import MagicString, { Bundle } from 'magic-string'; import { walk } from 'estree-walker'; +import CodeBuilder from '../utils/CodeBuilder.js'; import deindent from '../utils/deindent.js'; import isReference from '../utils/isReference.js'; import counter from './utils/counter.js'; @@ -18,31 +19,32 @@ export default function generate ( parsed, source, options, names ) { addElement ( name, renderStatement, needsIdentifier = false ) { const isToplevel = generator.current.localElementDepth === 0; if ( needsIdentifier || isToplevel ) { - generator.current.initStatements.push( deindent` - var ${name} = ${renderStatement}; - ` ); + generator.current.builders.init.addLine( + `var ${name} = ${renderStatement};` + ); + generator.createMountStatement( name ); } else { - generator.current.initStatements.push( deindent` - ${generator.current.target}.appendChild( ${renderStatement} ); - ` ); + generator.current.builders.init.addLine( + `${generator.current.target}.appendChild( ${renderStatement} );` + ); } + if ( isToplevel ) { - generator.current.detachStatements.push( deindent` - ${name}.parentNode.removeChild( ${name} ); - ` ); + generator.current.builders.detach.addLine( + `${name}.parentNode.removeChild( ${name} );` + ); } }, createMountStatement ( name ) { if ( generator.current.target === 'target' ) { - generator.current.mountStatements.push( deindent` - target.insertBefore( ${name}, anchor ); - ` ); + generator.current.builders.mount.addLine( + `target.insertBefore( ${name}, anchor );` + ); } else { - generator.current.initStatements.push( deindent` - ${generator.current.target}.appendChild( ${name} ); - ` ); + generator.current.builders.init.addLine( + `${generator.current.target}.appendChild( ${name} );` ); } }, @@ -58,13 +60,7 @@ export default function generate ( parsed, source, options, names ) { name, target: 'target', localElementDepth: 0, - - initStatements: [], - mountStatements: [], - updateStatements: [], - detachStatements: [], - teardownStatements: [], - + builders: generator.getBuilders(), getUniqueName: generator.getUniqueNameMaker() }); // walk the children here @@ -77,37 +73,36 @@ export default function generate ( parsed, source, options, names ) { addRenderer ( fragment ) { if ( fragment.autofocus ) { - fragment.initStatements.push( `${fragment.autofocus}.focus();` ); + fragment.builders.init.addLine( `${fragment.autofocus}.focus();` ); } - const detachStatements = fragment.detachStatements.join( '\n\n' ); - const teardownStatements = fragment.teardownStatements.join( '\n\n' ); - - const detachBlock = deindent` - if ( detach ) { - ${detachStatements} - } - `; + // minor hack – we need to ensure that any {{{triples}}} are detached + // first, so we append normal detach statements to detachRaw + fragment.builders.detachRaw.addBlock( fragment.builders.detach ); - const teardownBlock = deindent` - ${teardownStatements}${detachStatements ? `\n\n${detachBlock}` : ``} - `; + if ( !fragment.builders.detachRaw.isEmpty() ) { + fragment.builders.teardown.addBlock( deindent` + if ( detach ) { + ${fragment.builders.detachRaw} + } + ` ); + } renderers.push( deindent` function ${fragment.name} ( ${fragment.params}, component ) { - ${fragment.initStatements.join( '\n\n' )} + ${fragment.builders.init} return { mount: function ( target, anchor ) { - ${fragment.mountStatements.join( '\n\n' )} + ${fragment.builders.mount} }, update: function ( changed, ${fragment.params} ) { - ${fragment.updateStatements.join( '\n\n' )} + ${fragment.builders.update} }, teardown: function ( detach ) { - ${teardownBlock} + ${fragment.builders.teardown} } }; } @@ -174,6 +169,17 @@ export default function generate ( parsed, source, options, names ) { events: {}, + getBuilders () { + return { + init: new CodeBuilder(), + mount: new CodeBuilder(), + update: new CodeBuilder(), + detach: new CodeBuilder(), + detachRaw: new CodeBuilder(), + teardown: new CodeBuilder() + }; + }, + getUniqueName: counter( names ), getUniqueNameMaker () { @@ -293,12 +299,6 @@ export default function generate ( parsed, source, options, names ) { elementDepth: 0, localElementDepth: 0, - initStatements: [], - mountStatements: [], - updateStatements: [], - detachStatements: [], - teardownStatements: [], - contexts: {}, indexes: {}, @@ -306,6 +306,7 @@ export default function generate ( parsed, source, options, names ) { indexNames: {}, listNames: {}, + builders: generator.getBuilders(), getUniqueName: generator.getUniqueNameMaker() }); @@ -313,15 +314,17 @@ export default function generate ( parsed, source, options, names ) { generator.addRenderer( generator.pop() ); - const topLevelStatements = []; + const builders = { + main: new CodeBuilder(), + init: new CodeBuilder(), + set: new CodeBuilder() + }; - const setStatements = [ deindent` - var oldState = state; - state = Object.assign( {}, oldState, newState ); - ` ]; + builders.set.addLine( 'var oldState = state;' ); + builders.set.addLine( 'state = Object.assign( {}, oldState, newState );' ); if ( templateProperties.computed ) { - const statements = []; + const builder = new CodeBuilder(); const dependencies = new Map(); templateProperties.computed.properties.forEach( prop => { @@ -343,7 +346,7 @@ export default function generate ( parsed, source, options, names ) { const deps = dependencies.get( key ); deps.forEach( visit ); - statements.push( deindent` + builder.addBlock( deindent` if ( ${deps.map( dep => `( '${dep}' in newState && typeof state.${dep} === 'object' || state.${dep} !== oldState.${dep} )` ).join( ' || ' )} ) { state.${key} = newState.${key} = template.computed.${key}( ${deps.map( dep => `state.${dep}` ).join( ', ' )} ); } @@ -352,57 +355,49 @@ export default function generate ( parsed, source, options, names ) { templateProperties.computed.properties.forEach( prop => visit( prop.key.name ) ); - topLevelStatements.push( deindent` + builders.main.addBlock( deindent` function applyComputations ( state, newState, oldState ) { - ${statements.join( '\n\n' )} + ${builder} } ` ); - setStatements.push( `applyComputations( state, newState, oldState )` ); + builders.set.addLine( `applyComputations( state, newState, oldState )` ); } - setStatements.push( deindent` + builders.set.addBlock( deindent` dispatchObservers( observers.immediate, newState, oldState ); if ( mainFragment ) mainFragment.update( newState, state ); dispatchObservers( observers.deferred, newState, oldState ); ` ); - const importBlock = imports - .map( ( declaration, i ) => { - if ( format === 'es' ) { - return source.slice( declaration.start, declaration.end ); - } - - const defaultImport = declaration.specifiers.find( x => x.type === 'ImportDefaultSpecifier' || x.type === 'ImportSpecifier' && x.imported.name === 'default' ); - const namespaceImport = declaration.specifiers.find( x => x.type === 'ImportNamespaceSpecifier' ); - const namedImports = declaration.specifiers.filter( x => x.type === 'ImportSpecifier' && x.imported.name !== 'default' ); - - const name = ( defaultImport || namespaceImport ) ? ( defaultImport || namespaceImport ).local.name : `__import${i}`; - declaration.name = name; // hacky but makes life a bit easier later + imports.forEach( ( declaration, i ) => { + if ( format === 'es' ) { + builders.main.addLine( source.slice( declaration.start, declaration.end ) ); + return; + } - const statements = namedImports.map( specifier => { - return `var ${specifier.local.name} = ${name}.${specifier.imported.name}`; - }); + const defaultImport = declaration.specifiers.find( x => x.type === 'ImportDefaultSpecifier' || x.type === 'ImportSpecifier' && x.imported.name === 'default' ); + const namespaceImport = declaration.specifiers.find( x => x.type === 'ImportNamespaceSpecifier' ); + const namedImports = declaration.specifiers.filter( x => x.type === 'ImportSpecifier' && x.imported.name !== 'default' ); - if ( defaultImport ) { - statements.push( `${name} = ( ${name} && ${name}.__esModule ) ? ${name}['default'] : ${name};` ); - } + const name = ( defaultImport || namespaceImport ) ? ( defaultImport || namespaceImport ).local.name : `__import${i}`; + declaration.name = name; // hacky but makes life a bit easier later - return statements.join( '\n' ); - }) - .filter( Boolean ) - .join( '\n' ); + namedImports.forEach( specifier => { + builders.main.addLine( `var ${specifier.local.name} = ${name}.${specifier.imported.name}` ); + }); - if ( parsed.js ) { - if ( imports.length ) { - topLevelStatements.push( importBlock ); + if ( defaultImport ) { + builders.main.addLine( `${name} = ( ${name} && ${name}.__esModule ) ? ${name}['default'] : ${name};` ); } + }); - topLevelStatements.push( `[✂${parsed.js.content.start}-${parsed.js.content.end}✂]` ); + if ( parsed.js ) { + builders.main.addBlock( `[✂${parsed.js.content.start}-${parsed.js.content.end}✂]` ); } if ( parsed.css && options.css !== false ) { - topLevelStatements.push( deindent` + builders.main.addBlock( deindent` let addedCss = false; function addCss () { var style = document.createElement( 'style' ); @@ -414,33 +409,30 @@ export default function generate ( parsed, source, options, names ) { ` ); } - topLevelStatements.push( ...renderers.reverse() ); + let i = renderers.length; + while ( i-- ) builders.main.addBlock( renderers[i] ); const constructorName = options.name || 'SvelteComponent'; - const initStatements = []; - if ( parsed.css && options.css !== false ) { - initStatements.push( `if ( !addedCss ) addCss();` ); + builders.init.addLine( `if ( !addedCss ) addCss();` ); } if ( generator.hasComponents ) { - initStatements.push( deindent` - this.__renderHooks = []; - ` ); + builders.init.addLine( `this.__renderHooks = [];` ); } if ( generator.hasComplexBindings ) { - initStatements.push( deindent` + builders.init.addBlock( deindent` this.__bindings = []; var mainFragment = renderMainFragment( state, this ); if ( options.target ) this._mount( options.target ); while ( this.__bindings.length ) this.__bindings.pop()(); ` ); - setStatements.push( `while ( this.__bindings.length ) this.__bindings.pop()();` ); + builders.set.addLine( `while ( this.__bindings.length ) this.__bindings.pop()();` ); } else { - initStatements.push( deindent` + builders.init.addBlock( deindent` var mainFragment = renderMainFragment( state, this ); if ( options.target ) this._mount( options.target ); ` ); @@ -454,12 +446,12 @@ export default function generate ( parsed, source, options, names ) { } `; - initStatements.push( statement ); - setStatements.push( statement ); + builders.init.addBlock( statement ); + builders.set.addBlock( statement ); } if ( templateProperties.onrender ) { - initStatements.push( deindent` + builders.init.addBlock( deindent` if ( options.root ) { options.root.__renderHooks.push({ fn: template.onrender, context: this }); } else { @@ -470,7 +462,7 @@ export default function generate ( parsed, source, options, names ) { const initialState = templateProperties.data ? `Object.assign( template.data(), options.data )` : `options.data || {}`; - topLevelStatements.push( deindent` + builders.main.addBlock( deindent` function ${constructorName} ( options ) { options = options || {}; @@ -521,7 +513,7 @@ export default function generate ( parsed, source, options, names ) { }; this.set = function set ( newState ) { - ${setStatements.join( '\n\n' )} + ${builders.set} }; this._mount = function mount ( target, anchor ) { @@ -571,15 +563,15 @@ export default function generate ( parsed, source, options, names ) { this.root = options.root; this.yield = options.yield; - ${initStatements.join( '\n\n' )} + ${builders.init} } ` ); if ( templateProperties.methods ) { - topLevelStatements.push( `${constructorName}.prototype = template.methods;` ); + builders.main.addBlock( `${constructorName}.prototype = template.methods;` ); } - const result = topLevelStatements.join( '\n\n' ); + const result = builders.main.toString(); const pattern = /\[✂(\d+)-(\d+)$/; diff --git a/src/generate/visitors/Component.js b/src/generate/visitors/Component.js index 168b4a1e21..c48bf29a42 100644 --- a/src/generate/visitors/Component.js +++ b/src/generate/visitors/Component.js @@ -1,4 +1,5 @@ import deindent from '../../utils/deindent.js'; +import CodeBuilder from '../../utils/CodeBuilder.js'; import addComponentAttributes from './attributes/addComponentAttributes.js'; export default { @@ -13,11 +14,8 @@ export default { allUsedContexts: new Set(), - init: [], - mount: [], - update: [], - detach: [], - teardown: [] + init: new CodeBuilder(), + update: new CodeBuilder() }; const isToplevel = generator.current.localElementDepth === 0; @@ -37,8 +35,13 @@ export default { generator.generateBlock( node, yieldName ); - generator.current.initStatements.push(`var ${name}_yieldFragment = ${yieldName}( root, component );`); - generator.current.updateStatements.push(`${name}_yieldFragment.update ( changed, root );`); + generator.current.builders.init.addLine( + `var ${name}_yieldFragment = ${yieldName}( root, component );` + ); + + generator.current.builders.update.addLine( + `${name}_yieldFragment.update( changed, root );` + ); componentInitProperties.push(`yield: ${name}_yieldFragment`); } @@ -72,7 +75,7 @@ export default { componentInitProperties.push(`data: ${name}_initialData`); } - local.init.unshift( deindent` + local.init.addBlockAtStart( deindent` ${statements.join( '\n\n' )} var ${name} = new template.components.${node.name}({ ${componentInitProperties.join(',\n')} @@ -80,7 +83,7 @@ export default { ` ); if ( isToplevel ) { - local.mount.unshift( `${name}._mount( target, anchor );` ); + generator.current.builders.mount.addLine( `${name}._mount( target, anchor );` ); } if ( local.dynamicAttributes.length ) { @@ -96,7 +99,7 @@ export default { return `${name}_changes.${attribute.name} = ${attribute.value};`; }); - local.update.push( deindent` + local.update.addBlock( deindent` var ${name}_changes = {}; ${updates.join( '\n' )} @@ -105,13 +108,10 @@ export default { ` ); } - local.teardown.push( `${name}.teardown( ${isToplevel ? 'detach' : 'false'} );` ); + generator.current.builders.teardown.addLine( `${name}.teardown( ${isToplevel ? 'detach' : 'false'} );` ); - generator.current.initStatements.push( local.init.join( '\n' ) ); - if ( local.update.length ) generator.current.updateStatements.push( local.update.join( '\n' ) ); - if ( local.mount.length ) generator.current.mountStatements.push( local.mount.join( '\n' ) ); - if ( local.detach.length ) generator.current.detachStatements.push( local.detach.join( '\n' ) ); - generator.current.teardownStatements.push( local.teardown.join( '\n' ) ); + generator.current.builders.init.addBlock( local.init ); + if ( !local.update.isEmpty() ) generator.current.builders.update.addBlock( local.update ); generator.push({ namespace: local.namespace, diff --git a/src/generate/visitors/EachBlock.js b/src/generate/visitors/EachBlock.js index 36c24068c7..0c6a976728 100644 --- a/src/generate/visitors/EachBlock.js +++ b/src/generate/visitors/EachBlock.js @@ -20,7 +20,7 @@ export default { const anchor = generator.createAnchor( name, `#each ${generator.source.slice( node.expression.start, node.expression.end )}` ); - generator.current.initStatements.push( deindent` + generator.current.builders.init.addBlock( deindent` var ${name}_value = ${snippet}; var ${iterations} = []; ${node.else ? `var ${elseName} = null;` : ''} @@ -31,7 +31,7 @@ export default { } ` ); if ( node.else ) { - generator.current.initStatements.push( deindent` + generator.current.builders.init.addBlock( deindent` if ( !${name}_value.length ) { ${elseName} = ${renderElse}( ${params}, component ); ${!isToplevel ? `${elseName}.mount( ${anchor}.parentNode, ${anchor} );` : ''} @@ -40,13 +40,13 @@ export default { } if ( isToplevel ) { - generator.current.mountStatements.push( deindent` + generator.current.builders.mount.addBlock( deindent` for ( var ${i} = 0; ${i} < ${iterations}.length; ${i} += 1 ) { ${iterations}[${i}].mount( ${anchor}.parentNode, ${anchor} ); } ` ); if ( node.else ) { - generator.current.mountStatements.push( deindent` + generator.current.builders.mount.addBlock( deindent` if ( ${elseName} ) { ${elseName}.mount( ${anchor}.parentNode, ${anchor} ); } @@ -54,7 +54,7 @@ export default { } } - generator.current.updateStatements.push( deindent` + generator.current.builders.update.addBlock( deindent` var ${name}_value = ${snippet}; for ( var ${i} = 0; ${i} < ${name}_value.length; ${i} += 1 ) { @@ -74,7 +74,7 @@ export default { ` ); if ( node.else ) { - generator.current.updateStatements.push( deindent` + generator.current.builders.update.addBlock( deindent` if ( !${name}_value.length && ${elseName} ) { ${elseName}.update( changed, ${params} ); } else if ( !${name}_value.length ) { @@ -86,14 +86,14 @@ export default { ` ); } - generator.current.teardownStatements.push( deindent` + generator.current.builders.teardown.addBlock( deindent` for ( var ${i} = 0; ${i} < ${iterations}.length; ${i} += 1 ) { ${iterations}[${i}].teardown( ${isToplevel ? 'detach' : 'false'} ); } ` ); if ( node.else ) { - generator.current.teardownStatements.push( deindent` + generator.current.builders.teardown.addBlock( deindent` if ( ${elseName} ) { ${elseName}.teardown( ${isToplevel ? 'detach' : 'false'} ); } @@ -136,18 +136,17 @@ export default { listNames, params: blockParams, - initStatements: [], - mountStatements: [], - updateStatements: [ Object.keys( contexts ).map( contextName => { - const listName = listNames[ contextName ]; - const indexName = indexNames[ contextName ]; + builders: generator.getBuilders(), + getUniqueName: generator.getUniqueNameMaker() + }); - return `var ${contextName} = ${listName}[${indexName}];`; - }).join( '\n' ) ], - detachStatements: [], - teardownStatements: [], + Object.keys( contexts ).forEach( contextName => { + const listName = listNames[ contextName ]; + const indexName = indexNames[ contextName ]; - getUniqueName: generator.getUniqueNameMaker() + generator.current.builders.update.addLine( + `var ${contextName} = ${listName}[${indexName}];` + ); }); }, diff --git a/src/generate/visitors/Element.js b/src/generate/visitors/Element.js index a428d18006..163378640e 100644 --- a/src/generate/visitors/Element.js +++ b/src/generate/visitors/Element.js @@ -1,3 +1,4 @@ +import CodeBuilder from '../../utils/CodeBuilder.js'; import deindent from '../../utils/deindent.js'; import addElementAttributes from './attributes/addElementAttributes.js'; import Component from './Component.js'; @@ -18,11 +19,8 @@ export default { allUsedContexts: new Set(), - init: [], - mount: [], - update: [], - detach: [], - teardown: [] + init: new CodeBuilder(), + update: new CodeBuilder() }; const isToplevel = generator.current.localElementDepth === 0; @@ -50,13 +48,13 @@ export default { return `${name}.__svelte.${listName} = ${listName};\n${name}.__svelte.${indexName} = ${indexName};`; }).join( '\n' ); - local.init.push( deindent` + local.init.addBlock( deindent` ${name}.__svelte = { ${initialProps} }; ` ); - local.update.push( updates ); + local.update.addBlock( updates ); } let render = local.namespace ? @@ -67,23 +65,22 @@ export default { render += `\n${name}.setAttribute( '${generator.cssId}', '' );`; } - local.init.unshift( render ); + local.init.addLineAtStart( render ); if ( isToplevel ) { - local.detach.push( `${name}.parentNode.removeChild( ${name} );` ); + generator.current.builders.detach.addLine( `${name}.parentNode.removeChild( ${name} );` ); } // special case – bound