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,631 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

Secure Your Active Directory - April 20, 2017

Active Directory plays a critical role in your company’s IT infrastructure and keeping it secure in today’s hacker-infested world is a must.
Microsoft published 300+ pages of guidance, but who has the time, money, and resources to implement? Register now to find an easier way.

Question has a verified solution.

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

Summary: Persistence is the capability of an application to store the state of objects and recover it when necessary. This article compares the two common types of serialization in aspects of data access, readability, and runtime cost. A ready-to…
Performance in games development is paramount: every microsecond counts to be able to do everything in less than 33ms (aiming at 16ms). C# foreach statement is one of the worst performance killers, and here I explain why.
In an interesting question (https://www.experts-exchange.com/questions/29008360/) here at Experts Exchange, a member asked how to split a single image into multiple images. The primary usage for this is to place many photographs on a flatbed scanner…
How to Install VMware Tools in Red Hat Enterprise Linux 6.4 (RHEL 6.4) Step-by-Step Tutorial

735 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