Hi,

you need to prepare appropriate collections before accumulating the Amounts - just as you would for a bulk trigger.

Something like the following should do the trick.

Note my comment about what is being returned; you may find that returning a list of amounts indexed by Revenue is more useful to you.

Also, should you need to, you can use the same technique to return amounts by revenue by pricebook - instead of using the first entry in your pricebook list to retrieve Revenue records, use a collection.

You would then return a map of <id, map<id, list<double>>>

Let me know if you need any further assistance!

    public list<Double> getAmount()
    {
        set<id>                    set_RevenueIds                        = new set<id>();
        map<id, list<double>>    map_RevenueIdtoTrainingAmounts        = new map<id, list<double>>();
    
        for (Revenue_Schedule__c [] arrRevenue : [    select    id,
                                                            Date__c
                                                    from    Revenue_Schedule__c
                                                    where    Price_Book_Name__c=:price[0].Id
                                                ])
        {
            for (Revenue_Schedule__c sRevenue : arrRevenue)
            {
                set_RevenueIds.add(sRevenue.Id);
            }
        }     
    
        for (Training_Schedule__c [] arrTraining :    [    select    id,
                                                                Amount__c,
                                                                Revenue_Date__c,
                                                                Revenue_Schedule__c
                                                        from    Training_Schedule__c
                                                        where     Revenue_Schedule__c in :set_RevenueIds
                                                    ])
        {
            for (Training_Schedule__c sTraining : arrTraining)
            {
                if (sTraining.Amount__c != null)
                {
                    list<double> li_Amounts;
    
                    if (!map_RevenueIdtoTrainingAmounts.containsKey(sTraining.Revenue_Schedule__c))
                    {
                        li_Amounts = new list<double>();
                    }
                    else
                    {
                        li_Amounts = map_RevenueIdtoTrainingAmounts.get(sTraining.Revenue_Schedule__c);
                    }

                    li_Amounts.add(sTraining.Amount__c);                                

                    map_RevenueIdtoTrainingAmounts.put(sTraining.Revenue_Schedule__c, li_Amount);
                }                      
            }
        }
    
        // The collection map_RevenueIdtoTrainingAmounts has a list of Amount indexed by Revenue_Schedule id
        // This might be more useful to you than a list of all Amounts
        
        // If not, you can convert this to a plain list as follows
        
        list<double> Amount = new list<double>();
        
        for (list<double> li_Amounts : map_RevenueIdtoTrainingAmounts.values())
        {
            Amount.addAll(li_Amounts);
        }
        
        return Amount;

    }