Daniel Martos

I tried to create a collection, as suggested.  I received an error stating that Unable to cast object of type 'System.Int32' to type 'System.Windows.Forms.Control'


Private Sub Button1_Click(sender As Object, e As EventArgs) Handles Button1.Click        'list to store numbers        Dim numbers As New List(Of Integer)        [color=yellow]Dim lotteryPicks As New Collection[/color]        'add desired numbers to list        For count As Integer = 1 To 99            numbers.Add(count)        Next        Dim Rnd As New Random        Dim SB As New System.Text.StringBuilder        Dim Temp As Integer        'select a random number from the list, add to listbox and remove it so it can't be selected again        For count As Integer = 1 To 5            Temp = Rnd.Next(0, numbers.Count)            SB.Append(numbers(Temp) & "  ")            ListBox2.Items.Add(numbers(Temp))            [color=yellow]lotteryPicks.Add(numbers(Temp))[/color]            numbers.RemoveAt(Temp)        Next        Dim numIndex As Integer        For numIndex = 0 To LotteryBox.Items.Count - 1            If LotteryBox.GetItemChecked(numIndex) = True Then ListBox1.Items.Add(LotteryBox.Items(numIndex))        Next        [color=yellow]Dim Matched As Integer = 0        If ListBox1.Contains(lotteryPicks.Item(1)) Then Matched += 1        If ListBox1.Contains(lotteryPicks.Item(2)) Then Matched += 1        If ListBox1.Contains(lotteryPicks.Item(3)) Then Matched += 1        If ListBox1.Contains(lotteryPicks.Item(4)) Then Matched += 1        If ListBox1.Contains(lotteryPicks.Item(5)) Then Matched += 1        LabelOutput.Text = "You've matched " & Matched & " Numbers"[/color]    End Sub End Class

Campbell Ritchie wrote:Can you make the selected items in each of the boxes into a collection and do anything like this on it? That will avoid your having to iterate the box 5×.

Duplicating discussion in our VB forum.

I'm not real sure how to to that.  Here's the entire code:

  Private Sub Button1_Click(sender As Object, e As EventArgs) Handles Button1.Click        'list to store numbers        Dim numbers As New List(Of Integer)        'add desired numbers to list        For count As Integer = 1 To 99            numbers.Add(count)        Next        Dim Rnd As New Random        Dim SB As New System.Text.StringBuilder        Dim Temp As Integer        'select a random number from the list, add to listbox and remove it so it can't be selected again        For count As Integer = 1 To 5            Temp = Rnd.Next(0, numbers.Count)            SB.Append(numbers(Temp) & "  ")            ListBox2.Items.Add(numbers(Temp))            numbers.RemoveAt(Temp)        Next        Dim numIndex As Integer        For numIndex = 0 To LotteryBox.Items.Count - 1            If LotteryBox.GetItemChecked(numIndex) = True Then ListBox1.Items.Add(LotteryBox.Items(numIndex))        Next    End Sub End Class

I have 2 ListBoxes.  ListBox1 is populated with 5 selected numbers from a CheckedListBox(LotteryBox) and ListBox2 is populated with random numbers 1-99.  I want to find a way to verify if the numbers in the two ListBoxes match.  For Example, if they matched one number.  It'll say "You've matched one number".  and so on.  I think I'm on the right path with:

  Dim Matched As Integer = 0        If ListBox2.Contains((LotteryBox.SelectedItem)) Then Matched += 1        If ListBox2.Contains((LotteryBox.SelectedItem)) Then Matched += 1        If ListBox2.Contains((LotteryBox.SelectedItem)) Then Matched += 1        If ListBox2.Contains((LotteryBox.SelectedItem)) Then Matched += 1        If ListBox2.Contains((LotteryBox.SelectedItem)) Then Matched += 1

But how do I differentiate between the five numbers selected in the LotteryBox?

I tried it with a switch statement and it works to tell you if it you choose the incorrect section, but if you choose the right section it skips the rest of the validation (because it's returning true), doesn't print the confirmation message, and advances straight to the form processor page.  Any idea of how I can do this without returning true?  

switch (course) {                case "CIS 100":                    if (section == '100001')                        return true;                    if (section == '100gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 100!");                    return false;                    break;                case "CIS 120":                    if (section == '120001')                        return true;                    if (section == '120gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 120!");                    return false;                    break;                case "CIS 220":                    if (section == '220001')                        return true;                    if (section == '220gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 220!");                    return false;                    break;                case "CIS 299":                    if (section == '299001')                        return true;                    if (section == '299gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 299!");                    return false;                    break;                case "CIS 302":                    if (section == '302gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 302!");                    return false;                    break;                case "CIS 304":                    if (section == '304001')                        return true;                    if (section == '304gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 304!");                    return false;                    break;                case "CIS 321":                    if (section == '321001')                        return true;                    if (section == '321gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 321!");                    return false;                    break;                case "CIS 322":                    if (section == '322gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 322!");                    return false;                    break;                case "CIS 325":                    if (section == '325gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 325!");                    return false;                    break;                case "CIS 330":                    if (section == '330001')                        return true;                    if (section == '330gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 330!");                    return false;                    break;                case "CIS 332":                    if (section == '332001')                        return true;                    if (section == '332gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 332!");                    return false;                    break;                case "CIS 341":                    if (section == '341001')                        return true;                    if (section == '341gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 341!");                    return false;                    break;                case "CIS 343":                    if (section == '34301a')                        return true;                    if (section == '34301b')                        return true;                    else                        window.alert("You must select a valid section for CIS 343!");                    return false;                    break;                case "CIS 352":                    if (section == '352gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 352!");                    return false;                    break;                case "CIS 354":                    if (section == '354001')                        return true;                    if (section == '354gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 354!");                    return false;                    break;                case "CIS 401":                    if (section == '401gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 401!");                    return false;                    break;                case "CIS 419":                    if (section == '419x01')                        return true;                    else                        window.alert("You must select a valid section for CIS 419!");                    return false;                    break;                case "CIS 490":                    if (section == '490001')                        return true;                    if (section == '490gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 490!");                    return false;                    break;                case "CIS 492":                    if (section == '492gw1')                        return true;                    else                        window.alert("You must select a valid section for CIS 492!");                    return false;                    break;                case "MAT 195":                    if (section == '195001')                        return true;                    if (section == '195w01')                        return true;                    else                        window.alert("You must select a valid section for MAT 195!");                    return false;                    break;                case "MAT 215":                    if (section == '215001')                        return true;                    if (section == '215w01')                        return true;                    else                        window.alert("You must select a valid section for MAT 215!");                    return false;                    break;                case "MAT 225":                    if (section == '225001')                        return true;                    if (section == '225w01')                        return true;                    else                        window.alert("You must select a valid section for MAT 225!");                    return false;                    break;                case "MAT 281":                    if (section == '281001')                        return true;                    if (section == '281w01')                        return true;                    else                        window.alert("You must select a valid section for MAT 281!");                    return false;                    break;            }

I'm sorry, that bit of code is:

if (course == "CIS 100") {                if (section == "001100")                    return true;                break;                if (section == "100gw1")                    return true;                break;                else {                    window.alert("You must select a valid section for CIS 100!");                    return false;                }            }

Bear Bibeault wrote:Exactly how is it not working? What's happening differently than your expectations?

Have you learned how to set breakpoints to help with debugging?

This bit of code doesn't display the "You must select a valid section for CIS 100!" message, if the person selects an incorrect section.  It also bypasses the confirmation message and goes straight to the form processor on the next page.  I do not know about breakpoints.

if (course == "CIS 100") {                if (section == "001100")                    return true;                break;                if (section == "CIS 100")                    return true;                break;                else {                    window.alert("You must select a valid section for CIS 100!");                    return false;                } }

Jeanne Boyarsky wrote:Daniel,
Do you have a question about that code or just sharing? Is there something in particular that doesn't work?

Yes, I'm sorry.  The bit of code at the top isn't working and I'm looking for advice about why.

I created a course registration page that allows you to choose a course and then a section.  Right now, you can choose a course and a section that doesn't fit the course.
I want to create validation code that ensures that my course section is valid for for my course.  Here's what I'm trying:

if (course == "CIS 100") {                if (section == "001100")                    return true;                break;                if (section == "CIS 100")                    return true;                break;                else {                    window.alert("You must select a valid section for CIS 100!");                    return false;                } }

And here is my entire code:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"          "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> <html xmlns="http://www.w3.org/1999/xhtml"> <!--Document Head--> <head>    <meta http-equiv="content-type" content="text/html; charset=utf-8" />    <!--Title Element-->    <title>Greendale Community College</title>    <!--Style Element-->    <style type="text/css">        body {            background-color: white;        }        h1 {            text-align: center;            font-family: Impact;            color: green;        }        p {            font-size: 72px;            color: green;        }    </style>    <!--Script Element-->    <script type="text/javascript">        /* <![CDATA[ */        // function to validate and create message confirm box        function submitRegistration() {            var fName = document.registration.firstName.value;            var lName = document.registration.lastName.value;            var cwid = document.registration.cwid.value;            var semester = document.registration.semester.value;            var course = document.registration.courses.value;            var section = document.registration.section.value;            var major = document.registration.needForMajor.value;            var semesterDisplay;            // To display semester info in confirm message            if (semester == "fall")                semesterDisplay = "Fall";            if (semester == "spring")                semesterDisplay = "Spring";            if (semester == "summer")                semesterDisplay = "Summer";            //To display major requirement in confirm message            var checkDisplay;            if (document.registration.needForMajor.checked == true) {                checkDisplay = "Course Needed For Major";            }            else {                checkDisplay = "";            }            //Validates first name            if (fName == "") {                window.alert("You must enter your first name!");                return false;            }            //Validates that first name is non-numeric            if (isNaN(fName) == false) {                window.alert("Your First Name must be non-numeric values!");                return false;            }            //Validates last name            if (lName == "") {                window.alert("You must enter your last name!");                return false;            }            //Validates that last name is non-numeric            if (isNaN(lName) == false) {                window.alert("Your Last Name must be non-numeric values!");                return false;            }            //Validates CWID            if (cwid == "") {                window.alert("You must enter your cwid!");                return false;            }            //Validates that CWID is numeric            if (isNaN(cwid) == true) {                window.alert("Your CWID must be numeric values!");                return false;            }            //Validates semester            var validateSemester = false;            for (var i = 0; i < 3; ++i) {                if (document.registration.semester[i].checked == true) {                    validateSemester = true;                    break;                }            }            if (validateSemester != true) {                window.alert("You must select a Semester!");                return false;            }            //Validates course            if (course == "") {                window.alert("You must select a Course!");                return false;            }            //Validates section            if (section == "") {                window.alert("You must select a Section!");                return false;            }              //Confirm message            var confirmation = window.confirm("Student Name: " + fName + " " + lName + "   CWID: " + cwid + "   Semester: " + semesterDisplay + "   Course: " + course + "   Section: " + section + "   " + checkDisplay);            //Ok and Cancel buttons            if (confirmation) {                window.alert("You have been registered for your course!");            }            else {                window.alert("Your registration has been canceled.");            }        }        //Reset function        function resetRegistration() {            var resetForm = window.confirm("Are you sure you want to reset the form?");            if (resetForm == true)                return true;            return false;        }        //Functions for mouseover and mouseout        function mouseOver(target) {            target.src = 'greendale_paper.png'            target.alt = 'paper'        }        function mouseOut(target) {            target.src = 'greendale.jpg'            target.alt = 'greendale'        }        /* ]]> */    </script> </head> <body>    <!--Heading Element-->        <h1>Greendale Community College</h1>            <!--Added a mouseover and mouseout-->            <center><img src="greendale.jpg" alt="greendale"  width="560" height="315"                onmouseover="mouseOver(this)"                onmouseout="mouseOut(this)"            /></center>        <h3 align="center">Greendale Community College Orientation Video</h3>        <center><iframe width="560" height="315" src="https://www.youtube.com/embed/i3z5QO2O3cU" frameborder="0" allowfullscreen=allowfullscreen></iframe></center>        <h2 align="center">Course Registration Page</h2>    <form action="FormProcessor.html" name="registration" method="get"          onsubmit="return submitRegistration()"          onreset="return resetRegistration()">        <h3>Student Information Form</h3>        <!--Student Information-->        First Name:<input type="text" name="firstName"/><br />        Last Name:<input type="text" name="lastName"/><br />        CWID:<input type="text" name="cwid" size="8" /><br />        <h3>Semester</h3>        <h4>(choose a semester)</h4>        <!--Radio Buttons to Choose Semester-->        <input type="radio" name="semester" value="fall" /> Fall 2018 <br />        <input type="radio" name="semester" value="spring" /> Spring 2018 <br />        <input type="radio" name="semester" value="summer" /> Summer 2018 <br />        <h3>Courses</h3>        <h4>(choose one course)</h4>        <table>            <!--Drop Down Box for Courses-->            <tr><td style="background:white;border:0">Courses:</td>                <td>                    <select name="courses" size="1">                        <option value=""></option>                        <option value="CIS 110">CIS 100 Intro to CIS</option>                        <option value="CIS 120">CIS 120 Application Prog I</option>                        <option value="CIS 220">CIS 220 Application Prog II</option>                        <option value="CIS 299">CIS 299 System Analysis I</option>                        <option value="CIS 302">CIS 302 Visual Programming</option>                        <option value="CIS 304">CIS 304 Cobol</option>                        <option value="CIS 321">CIS 321 DB Mgt Sys and Design</option>                        <option value="CIS 322">CIS 322 DB App Development</option>                        <option value="CIS 325">CIS 225 Dec Support Systems</option>                        <option value="CIS 330">CIS 330 Web Programming I</option>                        <option value="CIS 332">CIS 332 Web Programming II</option>                        <option value="CIS 341">CIS 341 CISCO CCNA I</option>                        <option value="CIS 343">CIS 343 CISCO CCNA III</option>                        <option value="CIS 352">CIS 352 Global Ethics in Comp</option>                        <option value="CIS 354">CIS 354 Sys Project Mgt</option>                        <option value="CIS 401">CIS 401 Concepts Enter Res Planning</option>                        <option value="CIS 419">CIS 419 CIS Internship</option>                        <option value="CIS 490">CIS 490 Systems Analysis II</option>                        <option value="CIS 492">CIS 492 Systems Dev and Imp</option>                        <option value="MAT 195">MAT 195 Discrete Math Structures</option>                        <option value="MAT 215">MAT 215 Statistics</option>                        <option value="MAT 225">MAT 225 Business Statistics</option>                        <option value="MAT 281">MAT 281 Calculus I</option>                    </select>                </td>            </tr>        </table>        <h3>Sections</h3>        <h4>(choose one section)</h4>        <table>            <tr><td style="background:white;border:0">Section Numbers:</td>                <td>                    <!--Selection Box-->                    <select name="section" multiple="multiple" size="5">                        <option value="100001">CIS 100 001</option>                        <option value="100gw1">CIS100 GW1</option>                        <option value="120001">CIS 120 001</option>                        <option value="120gw1">CIS 120 GW1</option>                        <option value="220001">CIS 220 001</option>                        <option value="220gw1">CIS 220 GW1</option>                        <option value="299001">CIS 299 001</option>                        <option value="299gw1">CIS 299 GW1</option>                        <option value="302gw1">CIS 302 GW1</option>                        <option value="304001">CIS 304 001</option>                        <option value="304gw1">CIS 304 GW1</option>                        <option value="321001">CIS 321 001</option>                        <option value="321gw1">CIS 321 GW1</option>                        <option value="322gw1">CIS 322 GW1</option>                        <option value="325gw1">CIS 325 GW1</option>                        <option value="330001">CIS 330 001</option>                        <option value="330gw1">CIS 330 GW1</option>                        <option value="332001">CIS 332 001</option>                        <option value="332gw1">CIS 332 GW1</option>                        <option value="341001">CIS 341 001</option>                        <option value="341gw1">CIS 341 GW1</option>                        <option value="34301a">CIS 343 01A</option>                        <option value="34301b">CIS 343 01B</option>                        <option value="352gw1">CIS 352 GW1</option>                        <option value="354001">CIS 354 001</option>                        <option value="354gw1">CIS 354 GW1</option>                        <option value="401gw1">CIS 401 GW1</option>                        <option value="419x01">CIS 419 X01</option>                        <option value="490001">CIS 490 001</option>                        <option value="490gw1">CIS 490 GW1</option>                        <option value="492gw1">CIS 492 GW1</option>                        <option value="195001">MAT 195 001</option>                        <option value="195w01">MAT 195 W01</option>                        <option value="215001">MAT 215 001</option>                        <option value="215w01">MAT 215 W01</option>                        <option value="225001">MAT 225 001</option>                        <option value="225w01">MAT 225 W01</option>                        <option value="281001">MAT 281 001</option>                        <option value="281w01">MAT 281 W01</option>                    </select>                    </td>                </tr>        </table>        <!--Checkbox-->        <input type="checkbox" name="needForMajor" />        Check if the course is required for your major!<br />        <!--Submit and Reset Buttons Created-->        <input type="submit" name="submit" value="Submit"/><br />        <input type="reset" name="reset" value="Reset"/>        </form>        </body> </html>

My question is twofold. First, I can't seem to figure out how to get my confirm message box's ok and cancel to work correctly. The assignment calls for me to run the message, then validate my info, and if it validates, alert for ok or cancel. I have done it the other way around (validate....message....alert), but I'm not sure how to do this.

Second, my reset button clears my info whether I hit ok or cancel??? Any ideas?


<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"          "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> <html xmlns="http://www.w3.org/1999/xhtml"> <!--Document Head--> <head>    <meta http-equiv="content-type" content="text/html; charset=utf-8" />    <!--Title Element-->    <title>Greendale Community College</title>    <!--Style Element-->    <style type="text/css">        body {            background-color: white;        }        h1 {            text-align: center;            font-family: Impact;            color: green;        }        p {            font-size: 36px;            color: green;        }    </style>    <!--Script Element-->    <script type="text/javascript">        /* <![CDATA[ */        function submitRegistration() {            var fName = document.registration.firstName.value;            var lName = document.registration.lastName.value;            var cwid = document.registration.cwid.value;            var semester = document.registration.semester.value;            var course = document.registration.courses.value;            var section = document.registration.section.value;            var major = document.registration.needForMajor.value;            var semesterDisplay;            if (semester == "fall")                semesterDisplay = "Fall";            if (semester == "spring")                semesterDisplay = "Spring";            if (semester == "summer")                semesterDisplay = "Summer";            var checkDisplay;            if (document.registration.needForMajor.checked == true) {                checkDisplay = "Course Needed For Major";            }            else {                checkDisplay = "";            }            window.confirm("Student Name: " + fName + " " + lName + "   CWID: " + cwid + "   Semester: " + semesterDisplay + "   Course: " + course + "   Section: " + section + "   " + checkDisplay);            if (fName == "") {                window.alert("You must enter your first name!");                return false;            }            if (isNaN(fName) == false) {                window.alert("Your First Name must be non-numeric values!");                return false;            }            if (lName == "") {                window.alert("You must enter your last name!");                return false;            }            if (isNaN(lName) == false) {                window.alert("Your Last Name must be non-numeric values!");                return false;            }            if (cwid == "") {                window.alert("You must enter your cwid!");                return false;            }            if (isNaN(cwid) == true) {                window.alert("Your CWID must be numeric values!");                return false;            }            var validateSemester = false;            for (var i = 0; i < 3; ++i) {                if (document.registration.semester[i].checked == true) {                    validateSemester = true;                    break;                }            }            if (validateSemester != true) {                window.alert("You must select a Semester!");                return false;            }            if (course == "") {                window.alert("You must select a Course!");                return false;            }            if (section == "") {                window.alert("You must select a Section!");                return false;            }              if (true) {                window.alert("You have been registered for your course!");            }            else {                window.alert("Your registration has been canceled.");            }        }        function resetRegistration() {            var resetForm = window.confirm("Are you sure you want to reset the form?");            if (resetForm == true)                return true;            return false;        }        /* ]]> */    </script> </head> <body>    <!--Heading Element-->        <h1>Greendale Community College</h1>            <center><img src="greendale.jpg" alt="greendale" width="512" height="256" /></center>        <h2 align="center">Course Registration Page</h2>    <form action="FormProcessor.html" name="registration" method="get"          onsubmit="submitRegistration();"          onreset="resetRegistration()">        <h3>Student Information Form</h3>        <!--Student Information-->        First Name:<input type="text" name="firstName"><br>        Last Name:<input type="text" name="lastName"><br>        CWID:<input type="text" name="cwid"><br>        <h3>Semester</h3>        <h4>(choose a semester)</h4>        <!--Radio Buttons to Choose Semester-->        <input type="radio" name="semester" value="fall" /> Fall 2018 <br />        <input type="radio" name="semester" value="spring" /> Spring 2018 <br />        <input type="radio" name="semester" value="summer" /> Summer 2018 <br />        <h3>Courses</h3>        <h4>(choose one course)</h4>        <table>            <!--Drop Down Box for Courses-->            <tr><td style="background:white;border:0">Courses:</td>            <tr>                <td>                    <select name="courses" size="1">                        <option value=""></option>                        <option value="CIS 110">Intro to CIS</option>                        <option value="CIS 120">Application Prog I</option>                        <option value="CIS 299">System Analysis I</option>                        <option value="CIS 330">Web Programming I</option>                        <option value="CIS 304">Cobol</option>                    </select>                </td>            </tr>        </table>        <h3>Sections</h3>        <h4>(choose one section)</h4>        <table>            <tr><td style="background:white;border:0">Section Numbers:</td>            <tr>                <td>                    <select name="section" multiple="multiple" size="5">                        <option value="001">CIS 110 001</option>                        <option value="gw1">CIS110 GW1</option>                        <option value="001">CIS 120 001</option>                        <option value="gw1">CIS 120 GW1</option>                        <option value="gw1">CIS 302 GW1</option>                        <option value="001">CIS 304 001</option>                        <option value="gw1">CIS 303 GW1</option>                        <option value="001">CIS 321 001</option>                        <option value="gw1">CIS 321 GW1</option>                        <option value="gw1">CIS 322 GW1</option>                    </select>                    </td>                </tr>        </table>        <input type="checkbox" name="needForMajor" />        Check if the course is required for your major!<br />        <!--Submit and Reset Buttons Created-->        <input type="submit" name="submit" onsubmit="submitRegistration();" value="Submit"><br />        <input type="reset" name="reset" onreset="resetRegistration();" value="Reset">        </form>        </body> </html>

I'm trying to figure out how to calculate tuition using two arrays.  This is incomplete code.  I just want someone to look at my function to even tell me if I'm close with this thing.

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"          "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> <html xmlns="http://www.w3.org/1999/xhtml"> <!--Document Head--> <head>    <meta http-equiv="content-type" content="text/html; charset=utf-8" />        <!--Title Element-->        <title>Greendale Community College</title>        <!--Style Element-->        <style type="text/css">        body {            background-color: white;        }        h1 {            text-align: center;            font-family: Impact;            color: green;        }        p {            font-size: 36px;            color: green;        }        </style>    <!--Script Element-->    <script type="text/javascript">        /* <![CDATA[ */        /* ]]> */        // Created function to calculate tuition cost        var creditCostArray = new Array(3);        creditCost[0] = 302.00;        creditCost[1] = 452.00;        creditCost[2] = 588.00;                var tuitionCostArray = new Array(3);        tuitionCost[0] = 3619.00;        tuitionCost[1] = 5429.00;        tuitionCost[2] = 7057.00;        function calcTuition(total) {            var creditTotal = document.getElementsByName("numCredits")[0].value;            if (document.tuitionCalculator.residency.value == "resI")                if (creditTotal < 12);                    total = creditCost[0] * creditTotal;                        return total;                if (creditTotal > 18);                    total = tuitionCost[0] + (creditTotal * creditCost[0]);                        return total;                else;                    total = tuitionCost[0];                        return total;        }    </script>    </head>    <!--Body Element-->    <body>        <!--Heading Element-->        <h1>Greendale Community College</h1>        <center><img src="greendale.jpg" alt="greendale" width="512" height="256"/></center>        <h2 align="center">Tuition Calculator</h2>        <!--Form-->        <form name="tuitionCalculator" action="" method="get" onsubmit="return false;">        <h2>Semester</h2>        <h3>(choose a semester)</h3>            <!--Radio Buttons to Choose Semester-->            <input type="radio" name="semesterFall" /> Fall 2018 <br />            <input type="radio" name="semesterSpring" /> Spring 2018 <br />            <input type="radio" name="semesterSummer" /> Summer 2018 <br />        <h2>Residency</h2>        <h3>(choose your residency)</h3>            <!--Radio Buttons to Choose Residency-->            <input type="radio" name="residency" value="resI" onclick="calcTuition()"/> In-State <br />            <input type="radio" name="residency" value="resO" onclick="calcTuition()" /> Out-of-State <br />            <input type="radio" name="residency" value="resX" onclick="calcTuition()"/> International <br />        <h2>Credits</h2>        <h3>(enter your number of credits)</h3>            <!--Text Box-->            <input type="text" size="2" maxlength="2" name="numCredits" /><br />            <!--Button to Calculate Tuition-->            <input type="button" onclick="window.alert(calcTuition(i))" value="Calculate Your Tuition" />            </form>    </body> </html>

Bear Bibeault wrote:In order to use getElementById, the element must have an id. The name is not the same as an id. (name is only used for form submission, so you don't really even need one here.)

There is also getElementsByName if you want to continue to use names.

<h3>(enter your number of credits)</h3>            <input type="text" name="credits" id="credits" size="2" onchange="calcTuition(creditCost, numCredits)"/> Credits <br />            <input type="button" name="tuition" onclick="window.alert(calcTuition(creditCost))" value="Calculate your Tuition" />            </form>

Yes, I remember that now, but nothing is happening when I hit the button.

Bear Bibeault wrote:No, those variables are undefined in the onchange handler.

It seems you are just guessing at this point; you need to ask yourself what values you want to actually pass into the function. When onchange triggers, what is it you actually want to have happen?

Trying getElementByID.....still nothing...ugh

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"          "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> <html xmlns="http://www.w3.org/1999/xhtml"> <head>    <meta charset="utf-8" />        <title>Greendale Community College</title>    <script type="text/javascript">        /* <![CDATA[ */        /* ]]> */        var creditCost = 302;        function calcTuition(creditCost,numCredits) {            var numCredits = document.getElementById(credits).value            var tuitionTotal = 0;            var tuitionTotal = creditCost * numCredits;            document.write("Your Tuition Totals $ " + tuitionTotal);        }            </script>    </head>    <body>        <p><center><font face="impact" font size="200" color="green">Greendale Community College</font></center></p>        <center><img    /></center>        <h1><center>Tuition Calculator</center></h1>        <form name="calculator" action="" method="get">        <h2>Semester</h2>        <h3>(choose a semester)</h3>            <input type="radio" name="semesterFall"/> Fall 2018 <br />            <input type="radio" name="semesterSpring"/> Spring 2018 <br />            <input type="radio" name="semesterSummer"/> Summer 2018 <br />        <h2>Residency</h2>        <h3>(choose your residency)</h3>            <input type="radio" name="instate" /> In-State <br />            <input type="radio" name="outstate" /> Out-of-State <br />            <input type="radio" name="international" /> International <br />        <h2>Credits</h2>        <h3>(enter your number of credits)</h3>            <input type="text" name="credits" size="2" onchange="calcTuition(creditCost, numCredits)"/> Credits <br />            <input type="button" name="tuition" onclick="window.alert(calcTuition(creditCost))" value="Calculate your Tuition" />            </form>    </body> </html>

Bear Bibeault wrote:No, those variables are undefined in the onchange handler.

It seems you are just guessing at this point; you need to ask yourself what values you want to actually pass into the function. When onchange triggers, what is it you actually want to have happen?

Not guessing just trying to problem solve.  I did some more research and made just changes, but am still at an I'm at an impasse.

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"          "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> <html xmlns="http://www.w3.org/1999/xhtml"> <head>    <meta charset="utf-8" />        <title>Greendale Community College</title>    <script type="text/javascript">        /* <![CDATA[ */        /* ]]> */        var creditCost = 302;        function calcTuition() {            var tuitionTotal = 0;            var numCredits = number(calculator.credits.value);            tuitionTotal = creditCost * numCredits;            return tuitionTotal;        }        document.calculator.total.value = tuitionTotal;    </script>    </head>    <body>        <p><center><font face="impact" font size="200" color="green">Greendale Community College</font></center></p>        <center><img    /></center>        <h1><center>Tuition Calculator</center></h1>        <form name="calculator" action="" method="get">        <h2>Semester</h2>        <h3>(choose a semester)</h3>            <input type="radio" name="semesterFall"/> Fall 2018 <br />            <input type="radio" name="semesterSpring"/> Spring 2018 <br />            <input type="radio" name="semesterSummer"/> Summer 2018 <br />        <h2>Residency</h2>        <h3>(choose your residency)</h3>            <input type="radio" name="instate" /> In-State <br />            <input type="radio" name="outstate" /> Out-of-State <br />            <input type="radio" name="international" /> International <br />        <h2>Credits</h2>        <h3>(enter your number of credits)</h3>            <input type="text" name="credits" size="2" /> Credits <br />            <input type="button" name="tuition" onclick="window.alert(calcTuition())" value="Calculate your Tuition" />            </form>    </body> </html>

Still the same issue....

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"          "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> <html xmlns="http://www.w3.org/1999/xhtml"> <head>    <meta charset="utf-8" />        <title>Greendale Community College</title>    <script type="text/javascript">        /* <![CDATA[ */        /* ]]> */        var numCredits = 0;        var creditCost = 302;        var instate = 0;        var outstate = 0;        var international = 0;        var tuitionCost = number;        function calcTuition(numCredits, creditCost) {            var tuitionCost = numCredits * creditCost;            document.write("Your tuition cost is $" + tuitionCost);        }    </script>    </head>    <body>        <p><center><font face="impact" font size="200" color="green">Greendale Community College</font></center></p>        <center><img src="greendale.jpg" alt="greendale" width="512" height="256"/></center>        <h1><center>Tuition Calculator</center></h1>        <form name="calculator" action="" method="get">        <h2>Semester</h2>        <h3>(choose a semester)</h3>            <input type="radio" name="semesterFall"/> Fall 2018 <br />            <input type="radio" name="semesterSpring"/> Spring 2018 <br />            <input type="radio" name="semesterSummer"/> Summer 2018 <br />        <h2>Residency</h2>        <h3>(choose your residency)</h3>            <input type="radio" name="instate" /> In-State <br />            <input type="radio" name="outstate" /> Out-of-State <br />            <input type="radio" name="international" /> International <br />        <h2>Credits</h2>        <h3>(enter your number of credits)</h3>            <input type="text" name="numCredits" size="2" onchange="calcTuition(numCredits, creditCost)"/> Credits <br />            <input type="button" name="tuition" onclick="window.alert (calcTuition(numCredits, creditCost))" value="Calculate your Tuition" />            </form>    </body> </html>

Bear Bibeault wrote:In the following code, where do you pass two numbers?
onchange="calcTuition()"
You pass nothing.

What is it you want to be multiplying together? Those values need to be passed to the function.

Oh, I didn't realize you meant down there.  So if I put the variables in the onchange, I will be alright?