add click on ASP.net

Question

I am working on a college project and have a gridview that I have to add more rows to. I have two text fields and a button. Everything seems to be fine except the button part of the code in the .cs file What am I doing wring?

Its to make a list of Wines with an ID and a Title and a Year:

Wine.aspx:

<%@ Page Title="" Language="C#" MasterPageFile="~/Site.Master" AutoEventWireup="true" CodeBehind="Wine.aspx.cs" Inherits="WineNotProject2.AdminPages.Wine" %>
<asp:Content ID="Content1" ContentPlaceHolderID="HeadContent" runat="server">
</asp:Content>
<asp:Content ID="Content2" ContentPlaceHolderID="MainContent" runat="server">
   <br /><br />
    <asp:GridView ID="GridView2" runat="server" AutoGenerateColumns="False" DataKeyNames="id" DataSourceID="SqlDataSource1">

        <Columns>
            <asp:CommandField ShowEditButton="True" />
            <asp:BoundField DataField="id" HeaderText="id" InsertVisible="False" ReadOnly="True" SortExpression="id" />
            <asp:BoundField DataField="Title" HeaderText="Title" SortExpression="Title" />
            <asp:BoundField DataField="Year" HeaderText="Year" SortExpression="Year" />

        </Columns>
    </asp:GridView>


<p><asp:Label ID="lblTitle" runat="server" Text="Wine Title"></asp:Label><br />
<asp:TextBox ID="txtTitle" runat="server" Width="176px"></asp:TextBox><br />

<p><asp:Label ID="lblYear" runat="server" Text="Wine Year"></asp:Label><br />
<asp:TextBox ID="txtYear" runat="server" Width="176px"></asp:TextBox><br />
</p>


<asp:Button ID="btnAdd" runat="server" Text="Add Wine" OnClick="btnAdd_Click" />

    <asp:SqlDataSource ID="SqlDataSource1" runat="server" 
        ConnectionString="<%$ ConnectionStrings:ConnectionString %>" 
        SelectCommand="SELECT [id], [Title], [Year] FROM [Wine]">
    </asp:SqlDataSource>


</asp:Content>

Wine.aspx.cs:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.Web.UI;
using System.Web.UI.WebControls;

namespace WineNotProject2.AdminPages
{
    public partial class Wine : System.Web.UI.Page
    {
        private WineEntities10 ent2 = new WineEntities10();

        protected void Page_Load(object sender, EventArgs e)
        {
            if (!IsPostBack)
            {
                RefreshGrid();
            }
        }
            private void RefreshGrid()
            {
                GridView2.DataSource = ent2.Wines.ToList();
                GridView2.DataBind();
            }
            protected void btnAdd_Click(object sender, EventArgs e)
            {
                Wine w2 = new Wine();
                w2.Title = txtTitle.Text;
                w2.Year = txtYear.Text;
                ent2.Wines.AddObject(w2);
                ent2.SaveChanges();

            }

        }
    }

UPDATE:

The error message is:

Error 3 'WineNotProject2.AdminPages.Wine' does not contain a definition for 'Year' and no extension method 'Year' accepting a first argument of type 'WineNotProject2.AdminPages.Wine' could be found (are you missing a using directive or an assembly reference?) C:\Visual Studio 2010\Projects\WineNotProject7\WineNotProject2\AdminPages\Wine.aspx.cs 30 20 WineNotProject2

No I dont have to. But that way I can see if the button click added the info? Do you think it will work if I leave it out? — user2110655
– user2110655, Commented Mar 6, 2013 at 21:14
My problem seems to be at: w2.Year = txtYear.Text; ent2.Wines.AddObject(w2); — user2110655
– user2110655, Commented Mar 6, 2013 at 21:15
Why don't you need to tell the grid that the source has changed? Why do you think that the problem(what problem at all?) is at ent2.Wines.AddObject? You have to clarify your issue. — Tim Schmelter
– Tim Schmelter, Commented Mar 6, 2013 at 21:17

Tim Schmelter · Accepted Answer · 2013-03-06 21:21:22Z

1

I think you need to call RefreshGrid after you have added the new Wine:

protected void btnAdd_Click(object sender, EventArgs e)
{
    Wine w2 = new Wine();
    w2.Title = txtTitle.Text;
    w2.Year = txtYear.Text;
    ent2.Wines.AddObject(w2);
    ent2.SaveChanges();
    RefreshGrid(); // <-------
}

Maybe you also need to parse the year to int:

w2.Year = int.Parse(txtYear.Text);

edited Mar 6, 2013 at 21:21

answered Mar 6, 2013 at 21:15

Tim Schmelter

462k79 gold badges719 silver badges980 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

user2110655 Over a year ago

I will try but the problem is not with the refresh, but with the fact that the btnAdd doesn't even see the w2.Year from the wine.aspx page Sorry if I wasn't clear.

user2110655 Over a year ago

Visual Studio gives me an error: Does not contain a definition for Year. Sorry for not being clear enough. New at this.

Win · Accepted Answer · 2013-03-07 04:43:50Z

0

The problem is you have two classes with same name Wine.

public partial class **Wine** : System.Web.UI.Page

**Wine** w2 = new **Wine**();

Rename ASP.Net page's name to WineList so that

its class name should become public partial class **WineList**

enter image description here

answered Mar 7, 2013 at 4:43

Win

62.4k13 gold badges106 silver badges190 bronze badges

Comments

user1193035 · Accepted Answer · 2013-03-06 21:17:25Z

0

Your code was looking good ,But you miss call the RefreshGrid() end of add button

for

protected void btnAdd_Click(object sender, EventArgs e)
            {
                Wine w2 = new Wine();
                w2.Title = txtTitle.Text;
                w2.Year = txtYear.Text;
                ent2.Wines.AddObject(w2);
                ent2.SaveChanges();
                **RefreshGrid**

            }

If ent2.SaveChanges(); is not working ,then use ent2.SubmitChanges();

answered Mar 6, 2013 at 21:17

user1193035

Collectives™ on Stack Overflow

add click on ASP.net

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related