www.webdeveloper.com
Results 1 to 3 of 3

Thread: [RESOLVED] Runaway script with regular expression.

Hybrid View

  1. #1
    Join Date
    Sep 2008
    Location
    Jackson MS
    Posts
    585

    resolved [RESOLVED] Runaway script with regular expression.

    I am trying to globally change a line like:
    |Dynamic|Style:ff|Pos:-8 to
    |Dynamic|Style:ff|Opts:Velocity=127,Volume=108|Pos:-8 wherever it appears in a textarea and I have eight Styles and Volumes to loop through.
    Some sample data:
    Code:
    |Dynamic|Style:fff|Pos:-8
    |Note|Dur:4th|Pos:1
    |Dynamic|Style:ff|Pos:-8
    |Note|Dur:4th|Pos:1
    |Dynamic|Style:f|Pos:-8
    |Note|Dur:4th|Pos:1
    |Dynamic|Style:mf|Pos:-8
    |Note|Dur:4th|Pos:1
    |Bar
    |Dynamic|Style:mp|Pos:-8
    |Note|Dur:4th|Pos:1
    |Dynamic|Style:mf|Pos:-8
    and what I have so far:
    Code:
    <head>
    <title>Custom Dynamics</title>
    <script language="JavaScript">
    var default_volumes = [ 127, 108, 92, 75, 60, 45, 30, 10 ]
    function reload() {
    for (var i = 0; i < default_volumes.length; i++)
      document.nwcform.elements[i].value = default_volumes[i];
    }
    function calculate() {
    var dyn_table = [ "fff", "ff", "f", "mf", "mp", "p", "pp", "ppp" ]
    var dyn_vol1 = new Array();
    for (var i = 0; i < 8; i++)
      dyn_vol1[i] = parseInt(document.nwcform.elements[i].value);
    var str = document.nwcform.OutputField.value;
    
    for (i = 0; i < 8; i++) {
     var rxp, idx = 1, data="", startFrom=0, foundAt=0;
     
     while( data = ( rxp = new RegExp( "(\|Dynamic\|Style:)(" + dyn_table[i] + ")(\|Pos)",'g') ).exec( str ) )
     {
      foundAt = data.index;
      startFrom = rxp.lastIndex;  
      str = str.substring( 0, foundAt ) + data[ 1 ] + data[ 2 ] + "|Opts:Velocity=127,Volume=" + dyn_vol1[i] + data[ 3 ] +  + str.substring( startFrom );
    //      Stuff before found expr     "Dynamic|Style:" ff-pp       insert literal             dyn volume    "Pos"           Stuff after Pos   
     }
     
    }
    document.nwcform.OutputField.value = str;
    document.nwcform.OutputField.select();
    }
    </script>
    </head>
    <body onLoad="reload()">
    <form name="nwcform">
      <table>
        <tr>
          <td><B><I>fff </I></B><input size="4"></td>
          <td><B><I>ff </I></B><input size="4"></td>
          <td><B><I>f </I></B><input size="4"></td>
          <td><B><I>mf </I></B><input size="4"></td>
          <td><B><I>mp </I></B><input size="4"></td>
          <td><B><I>p </I></B><input size="4"></td>
          <td><B><I>pp </I></B><input size="4"></td>
          <td><B><I>ppp </I></B><input size="4"></td>
    	  <td><input type="button" name="smbutton" value="Submit" onclick="calculate()"></td>
    	  <td><input type="button" name="rstbutton" value="Reset" onclick="reload()"></td>
    	</tr><tr>
        <td colspan="9" width="1044" height="200"><textarea rows="30" cols="120" name="OutputField" width="2044"></textarea></td></tr>
    </table>
    </form>
    Thanks in advance.
    Last edited by wbport; 02-24-2011 at 04:20 PM. Reason: Remove comments to reduce size

  2. #2
    Join Date
    Sep 2008
    Location
    Jackson MS
    Posts
    585
    Never mind. I split it into lines to work on it poco a poco, building a from and to string for a replace.

  3. #3
    Join Date
    Dec 2010
    Posts
    207
    Code:
    while( data = ( rxp = new RegExp( "(\|Dynamic\|Style:)(" + dyn_table[i] + ")(\|Pos)",'g') ).exec( str ) )
    You should not constantly re-generate a regex that uses the g flag. This has the effect of resetting the lastIndex pointer, thereby causing an infinite loop.

    Edit: Looking closer at this and recognising my own code, I now realise that I told you this previously.
    Last edited by clueful; 02-25-2011 at 07:32 AM.


    Q: I found this code...
    A: Then find its author.

Thread Information

Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)

Tags for this Thread

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  
HTML5 Development Center

"

"

X vBulletin 4.2.2 Debug Information

  • Page Generation 0.10932 seconds
  • Memory Usage 2,949KB
  • Queries Executed 13 (?)
More Information
Template Usage (31):
  • (1)SHOWTHREAD
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_global_above_footer
  • (1)ad_global_below_navbar
  • (1)ad_global_header1
  • (1)ad_global_header2
  • (1)ad_navbar_below
  • (3)bbcode_code
  • (1)bbcode_quote
  • (1)footer
  • (1)forumjump
  • (1)forumrules
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (1)headinclude_bottom
  • (3)memberaction_dropdown
  • (1)navbar
  • (4)navbar_link
  • (1)navbar_moderation
  • (1)navbar_noticebit
  • (1)navbar_tabs
  • (2)option
  • (3)postbit
  • (3)postbit_onlinestatus
  • (3)postbit_wrapper
  • (1)showthread_list
  • (1)spacer_close
  • (1)spacer_open
  • (1)tagbit_wrapper 

Phrase Groups Available (6):
  • global
  • inlinemod
  • postbit
  • posting
  • reputationlevel
  • showthread
Included Files (27):
  • ./showthread.php
  • ./global.php
  • ./includes/class_bootstrap.php
  • ./includes/init.php
  • ./includes/class_core.php
  • ./includes/config.php
  • ./includes/functions.php
  • ./includes/functions_navigation.php
  • ./includes/class_friendly_url.php
  • ./includes/class_hook.php
  • ./includes/class_bootstrap_framework.php
  • ./vb/vb.php
  • ./vb/phrase.php
  • ./includes/functions_facebook.php
  • ./includes/functions_calendar.php
  • ./includes/functions_bigthree.php
  • ./includes/class_postbit.php
  • ./includes/class_bbcode.php
  • ./includes/functions_reputation.php
  • ./includes/functions_notice.php
  • ./includes/functions_threadedmode.php
  • ./packages/vbattach/attach.php
  • ./vb/types.php
  • ./vb/cache.php
  • ./vb/cache/db.php
  • ./vb/cache/observer/db.php
  • ./vb/cache/observer.php 

Hooks Called (72):
  • init_startup
  • friendlyurl_resolve_class
  • init_startup_session_setup_start
  • database_pre_fetch_array
  • database_post_fetch_array
  • init_startup_session_setup_complete
  • global_bootstrap_init_start
  • global_bootstrap_init_complete
  • cache_permissions
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • load_show_variables
  • load_forum_show_variables
  • global_state_check
  • global_bootstrap_complete
  • global_start
  • style_fetch
  • global_setup_complete
  • showthread_start
  • showthread_getinfo
  • strip_bbcode
  • friendlyurl_clean_fragment
  • friendlyurl_geturl
  • forumjump
  • cache_templates
  • cache_templates_process
  • template_register_var
  • template_render_output
  • fetch_template_start
  • fetch_template_complete
  • parse_templates
  • fetch_musername
  • notices_check_start
  • notices_noticebit
  • process_templates_complete
  • friendlyurl_redirect_canonical
  • showthread_post_start
  • showthread_query_postids_threaded
  • showthread_threaded_construct_link
  • showthread_query
  • bbcode_fetch_tags
  • bbcode_create
  • showthread_postbit_create
  • postbit_factory
  • postbit_display_start
  • postbit_imicons
  • bbcode_parse_start
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • postbit_display_complete
  • memberaction_dropdown
  • tag_fetchbit
  • tag_fetchbit_complete
  • forumrules
  • navbits
  • navbits_complete
  • build_navigation_data
  • build_navigation_array
  • check_navigation_permission
  • process_navigation_links_start
  • process_navigation_links_complete
  • set_navigation_menu_element
  • build_navigation_menudata
  • build_navigation_listdata
  • build_navigation_list
  • set_navigation_tab_main
  • set_navigation_tab_fallback
  • navigation_tab_complete
  • fb_like_button
  • showthread_complete
  • page_templates