Create two methods for the same URL pattern with different arguments (with argument and without argument)

Question

I'd like to use one URL mapping /all with request parameters (month, date) or without parameters.

I've tried to create two methods, one without parameters:

@RequestMapping(value = "/all", method = RequestMethod.GET)
public CommonResponse getAll() {

}

And one with parameters:

@RequestMapping(value = "/all", method = RequestMethod.GET)
public CommonResponse getByMonth(@RequestParam int month, @RequestParam(required = false) int year) {

}

But i am getting "Ambiguous mapping found" IllegalStateException. Does Spring have any way to handle this situation?

Note:- Please don't suggest this solution because I have different scenario.

Ahmet Amasyalı · Accepted Answer · 2019-12-10 12:39:17Z

1

Two different mappings with same path is impossible. But maybe you can do something like this :

    @RequestMapping(value = "/all", method = RequestMethod.GET)
    public CommonResponse getByMonth(@RequestParam Integer month, @RequestParam(required = false) Integer year) {
        if(month == null && year == null) {
            return getAll();
        } else {
            return getByMonth(month, year);
        }
    }

Or you can change one variable to a path variable on your second mapping

    @RequestMapping(value = "/all/{month}", method = RequestMethod.GET)
    public CommonResponse getByMonth(@PathVariable("month") Integer month, @RequestParam(required = false) int year) {
    }

answered Dec 10, 2019 at 12:39

Ahmet Amasyalı

1091 silver badge6 bronze badges

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

1 Comment

Victory Over a year ago

This is PathVariable not RequestParam.

b.GHILAS · Accepted Answer · 2019-12-10 12:48:51Z

1

You can't create two methods for the same url, you have to make your month param optional, and check in code if the month is present or not.

@RequestMapping(value = "/all", method = RequestMethod.GET)
public CommonResponse getAllOrByMonth(@RequestParam(required = false) Integer month, 
                                      @RequestParam(required = false) Integer year) {
    if (month != null) {
        // Get by month
    } else {
        // Get all
    }
}

answered Dec 10, 2019 at 12:48

b.GHILAS

2,3431 gold badge11 silver badges19 bronze badges

1 Comment

Victory Over a year ago

You can create two method for the same URL, check this

Victory · Accepted Answer · 2019-12-10 13:20:49Z

1

I got the solution :)

@RequestMapping(value = "/all", method = RequestMethod.GET)
public CommonResponse getAll(@RequestParam(required = false) Optional<Integer> month, 
                             @RequestParam(required = false, defaultValue = "0") int year) {

    if (month.isPresent()) {
       return getByMonth(month.get, year);
    }

    return getAll();
}

edited Dec 10, 2019 at 13:20

answered Dec 10, 2019 at 13:10

Victory

1,2524 gold badges13 silver badges31 bronze badges

Comments

for Responsibility Live · Accepted Answer · 2022-06-14 02:32:03Z

0

You can use like this:

@RequestMapping(value = "/all", method = RequestMethod.GET)
public CommonResponse getAll() {

}

@RequestMapping(value = "/all", params = {"month", "date"}, method = 
RequestMethod.GET)
public CommonResponse getByMonth(@RequestParam("month") int 
month,@RequestParam("date") int date, @RequestParam(required = false) int year) 
{

}

answered Jun 14, 2022 at 2:32

for Responsibility Live

1

Collectives™ on Stack Overflow

Create two methods for the same URL pattern with different arguments (with argument and without argument)

4 Answers 4

1 Comment

1 Comment

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related