My page did not redirect url.

and i use this code in vf page.
<apex:commandButton value="Next" action="{!checkCurrencyAllProductService}"/>

and this in controller.

public void checkCurrencyAllProductService(){
        if(mCountCurrency!=0){
            for(integer i = 0 ; i < addCurrency.size() ; i++){
                //-- Check Currency in Land Hotel --\\
                if(addCurrency[i].CurrencyIsoCode == landHotel.CurrencyIsoCode){
                    landHotel.Rate__c = addCurrency[i].Rate__c;
                }
                //-- Check Currency in Land Hotel --\\
                
                //-- Check Currency in List Land Hotel --\\
                if(addCurrency[i].CurrencyIsoCode == lstLandHotel[i].CurrencyIsoCode){
                    lstLandHotel[i].Rate__c = addCurrency[i].Rate__c;
                }
                //-- Check Currency in List Hotel --\\
                
                //-- Check Currency in Land Restaurant --\\
                if(addCurrency[i].CurrencyIsoCode == landRestaurant.CurrencyIsoCode){
                    landRestaurant.Rate__c = addCurrency[i].Rate__c;
                }
                //-- Check Currency in Land Restaurant --\\
                
                //-- Check Currency in List Land Restaurant --\\
                if(addCurrency[i].CurrencyIsoCode == lstRestaurant[i].CurrencyIsoCode){
                    lstRestaurant[i].Rate__c = addCurrency[i].Rate__c;
                }
                //-- Check Currency in List Restaurant --\\
                
                //-- Check Currency in Land Attraction --\\
                if(addCurrency[i].CurrencyIsoCode == landAttraction.CurrencyIsoCode){
                    landAttraction.Rate__c = addCurrency[i].Rate__c;
                }
                //-- Check Currency in Land Attraction --\\
                
                //-- Check Currency in List Land Attraction --\\
                if(addCurrency[i].CurrencyIsoCode == lstAttraction[i].CurrencyIsoCode){
                    lstAttraction[i].Rate__c = addCurrency[i].Rate__c;
                }
                //-- Check Currency in List Attraction --\\
                
                //-- Check Currency in Land Activities --\\
                if(addCurrency[i].CurrencyIsoCode == landActivities.CurrencyIsoCode){
                    landActivities.Rate__c = addCurrency[i].Rate__c;
                }
                //-- Check Currency in Land Activities --\\
                
                //-- Check Currency in List Land Activies --\\
                if(addCurrency[i].CurrencyIsoCode == lstAttraction[i].CurrencyIsoCode){
                    lstAttraction[i].Rate__c = addCurrency[i].Rate__c;
                }
                //-- Check Currency in List Land Activies --\\
                
                //-- Check Currency in List Coach --\\
                if(mCountCoach >= 0){
                    if(addCurrency[i].CurrencyIsoCode == addCoach[i].CurrencyIsoCode){
                        addCoach[i].Rate__c = addCurrency[i].Rate__c;
                    }
                }
                //-- Check Currency in List Coach --\\
                
                //-- Check Currency in List Option --\\
                if(mCountOption >= 0){
                    if(addCurrency[i].CurrencyIsoCode == addOption[i].CurrencyIsoCode){
                        addOption[i].Rate__c = addCurrency[i].Rate__c;
                    }
                }
                //-- Check Currency in List Option --\\
                
                //-- Check Currency in List Tour Leader --\\
                if(mCountTourLeader >= 0){
                    if(addCurrency[i].CurrencyIsoCode == addTourLeader[i].CurrencyIsoCode){
                        addTourLeader[i].Rate__c = addCurrency[i].Rate__c;
                    }
                }
                //-- Check Currency in List Tour Leader --\\
                
                //-- Check Currency in List Local Guide --\\
                if(mCountLocalGuide >= 0){
                    if(addCurrency[i].CurrencyIsoCode == addLocalGuide[i].CurrencyIsoCode){
                        addLocalGuide[i].Rate__c = addCurrency[i].Rate__c;
                    }
                }
                //-- Check Currency in List Local Guide --\\
                
                //-- Check Currency in List Other --\\
                if(mCountOther >= 0){
                    if(addCurrency[i].CurrencyIsoCode == addOther[i].CurrencyIsoCode){
                        addOther[i].Rate__c = addCurrency[i].Rate__c;
                    }
                }
                //-- Check Currency in List Other --\\
                
                //-- Check Currency in List Commission --\\
                if(mCountCommission >= 0){
                    if(addCurrency[i].CurrencyIsoCode == addCommission[i].CurrencyIsoCode){
                        addCommission[i].Rate__c = addCurrency[i].Rate__c;
                    }
                }
                //-- Check Currency in List Commission --\\
            }
        }
        onSave();
    }

//-- Save --\\
    public PageReference onSave(){
        system.debug('On Save');
        upsert(landHotel);
        upsert(lstLandHotel);
        upsert(landRestaurant);
        upsert(lstRestaurant);
        upsert(landAttraction);
        upsert(lstAttraction);
        upsert(landActivities);
        upsert(lstActivities);
        upsert(handlingFee);
        if(mCountCoach >= 0){
            insert addCoach;
        }
        if(mCountOption >= 0){
            insert addOption;
        }
        if(mCountTourLeader >= 0){
            insert addTourLeader;
        }
        if(mCountLocalGuide >= 0){
            insert addLocalGuide;
        }
        if(mCountOther >= 0){
            insert addOther;
        }
        if(mCountCommission >= 0){
            insert addCommission;
        }
        //return new PageReference('/' + ApexPages.currentPage().getParameters().get('CF00NO0000001EtuG_lkid'));
        String url = '/apex/Cost_Calculator_2?&CF00NO0000001EtuG_lkid='+theProduct.Id+'&CF00NO0000001EtuG='+theProduct.Name;
        system.debug('URL : ' + url);
        PageReference pref=new PageReference(url);
        return pref;
    }
    //-- Save --\\

thank you in advance