Solved

Cannot convert method group 'eat' to non-delegate type 'string'. Did you intend to invoke the method?

Posted on 2009-07-07
3
1,612 Views
Last Modified: 2012-08-14
What is wrong?


Error Message:
Error      1      Cannot convert method group 'eat' to non-delegate type 'string'. Did you intend to invoke the method?      C:\Documents and Settings\MFL\Dokumenter\Visual Studio 2008\Projects\Animal_Park\Animal_Park\Animal.cs      342      31      Animal_Park

private void mealButton_Click(object sender, EventArgs e)
        {
            int i = mealComboBox.SelectedIndex;
            MealLabel.Text =  animalManager.AnimalList[i].eat;
        }
 
 
----------------------------------
AnimalManager
----------------------------------
 
public class AnimalManager
    {
        [XmlAttribute]
        private int count = 0;
        private List<Animal> animalList = new List<Animal>();     
 
 
 
--------------------------------------------
Animal
----------------------------------------------
 
 
        /// <summary>
        /// Eat method
        /// </summary>
        /// <param name="food"></param>
        /// <returns></returns>
        public virtual string eat(string food)
        {
            return "Animal eat method";
        }
 
--------------------------------------
PlantEater
------------------------------------
 /// <summary>
        /// Does it like it? Eat method
        /// </summary>
        /// <param name="food"></param>
        /// <returns></returns>
        public override String eat(string food)
        {
            foreach (string eat in likes)
            {
                if (eat.ToLower() == food.ToLower())
                {
                    return "Yummy";
                } //end if
            } //end foreach
 
            return "Uuuusccccccch";
        } //end eat

Open in new window

0
Comment
Question by:Mickeys
  • 2
3 Comments
 
LVL 5

Accepted Solution

by:
Pablissimo earned 500 total points
ID: 24794303
private void mealButton_Click(object sender, EventArgs e)
        {
            int i = mealComboBox.SelectedIndex;
            MealLabel.Text =  animalManager.AnimalList[i].eat;
        }


needs to be

private void mealButton_Click(object sender, EventArgs e)
        {
            int i = mealComboBox.SelectedIndex;
            MealLabel.Text =  animalManager.AnimalList[i].eat();
        }


Parentheses after the eat - it's a method call.
0
 
LVL 5

Assisted Solution

by:Pablissimo
Pablissimo earned 500 total points
ID: 24794309
Actually that's not quite right either - I'm not quite sure what you're trying to achieve but the following:
MealLabel.Text =  animalManager.AnimalList[i].eat( "Some food" );

Would work as an example. Something seems confused in the code though
0
 

Author Closing Comment

by:Mickeys
ID: 31600589
Thx. Been sitting to long today....so I got a bit blind. Of course you are right. I didnt see I should have something in and I forgot the ( ). Here is a working function: :-)

private void mealButton_Click(object sender, EventArgs e)
        {
            Int32 selected = AnimalDataGridView.CurrentRow.Index;
            MealLabel.Text = animalManager.AnimalList[selected].eat(mealComboBox.SelectedItem.ToString());
        }
0

Featured Post

Gigs: Get Your Project Delivered by an Expert

Select from freelancers specializing in everything from database administration to programming, who have proven themselves as experts in their field. Hire the best, collaborate easily, pay securely and get projects done right.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Article by: Najam
Having new technologies does not mean they will completely replace old components.  Recently I had to create WCF that will be called by VB6 component.  Here I will describe what steps one should follow while doing so, please feel free to post any qu…
It was really hard time for me to get the understanding of Delegates in C#. I went through many websites and articles but I found them very clumsy. After going through those sites, I noted down the points in a easy way so here I am sharing that unde…
Windows 10 is mostly good. However the one thing that annoys me is how many clicks you have to do to dial a VPN connection. You have to go to settings from the start menu, (2 clicks), Network and Internet (1 click), Click VPN (another click) then fi…
Email security requires an ever evolving service that stays up to date with counter-evolving threats. The Email Laundry perform Research and Development to ensure their email security service evolves faster than cyber criminals. We apply our Threat…

813 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

13 Experts available now in Live!

Get 1:1 Help Now