Better way of writing this piece of code in vb.net

Hello,
i have code :

    If NumberOFPictures <= 47 Then
                            Slidenumber = 2
                        ElseIf NumberOFPictures > 47 And NumberOFPictures <= 94 Then
                            Slidenumber = 3
                        ElseIf NumberOFPictures > 94 And NumberOFPictures <= 141 Then
                            Slidenumber = 4
                        ElseIf NumberOFPictures > 141 And NumberOFPictures <= 188 Then
                            Slidenumber = 5
                        ElseIf NumberOFPictures > 188 And NumberOFPictures <= 235 Then
                            Slidenumber = 6

                        End If

Open in new window

Is there any better way of writing this as the NumberOFPictures can be even 2000

Cheers

RIASAsked: 2018-01-16

Who is Participating?

Solutions

LVL 45

AndyAinscowConnect With a Mentor Freelance programmer / ConsultantCommented: 2018-01-16

A more brute force approach would be something like this:
slideno = 2
While picno > 47
slideno = slideno + 1
picno = picno - 47
End While

If you wanted something other than 47 pictures per page then just adjust accordingly

LVL 13

Nitin SontakkeDeveloperCommented: 2018-01-16

Why not just NumberOFPictures / 47? Take the answer, make it an Integer and just add 1. Will that not work?

LVL 13

Nitin SontakkeDeveloperCommented: 2018-01-16

When is the slide number = 1?

RIASAuthor Commented: 2018-01-16

Thanks, The slide 1 is the title one so it starts with 2 .

LVL 55

Rgonzo1971Connect With a Mentor Commented: 2018-01-16

HI,

pls try

Slidenumber =2+(NumberOFPictures \ 48)

Open in new window

RIASAuthor Commented: 2018-01-16

Thanks, will try and be right back

LVL 45

AndyAinscowFreelance programmer / ConsultantCommented: 2018-01-16

You can use the \ operator to perform integer division (take care with the / operator, they are not the same).
so 46\47 is zero, 47\47 is one, 94\47 is two....

RIASAuthor Commented: 2018-01-16

Any suggestion then Andy

LVL 55

Rgonzo1971Commented: 2018-01-16

I supposed you are counting from 0 but 48 units per page like in your previous example
I'm already using the integer division '\' in my suggestion

LVL 45

AndyAinscowFreelance programmer / ConsultantCommented: 2018-01-16

You want something like
slideno = 2 + (picno \ 47)
but I suspect your original code isn't correct. You have more pictures on the first page than on subsequent pages.