When running through IDs for an Image, is this an acceptable practice? (MarketplaceService)

THIS WORKS! I'm not asking for someone to help fix this script!! I just want to know if theres an alternative!!

Anyway, A quick summary of whats happening is, the player gives a text box an ID and then it loops through until it finds the correct ID to use, the ID goes through checks before to make sure its real. I'm just curious whether or not is it proper practice to use MarketplaceService or is there a better alternative? This is my script so you can see whats going on.

local logoID = text
local newID = text
while true do
    newID = tonumber(newID) - 1
    local idName = game.MarketplaceService:GetProductInfo(logoID)
    local newidName = game.MarketplaceService:GetProductInfo(newID)
    if newidName.Creator.Name == idName.Creator.Name then
        StudioImage.Image = "rbxassetid://"..newID
        canContinue = true
        break            
    end    
end
if canContinue then
    local assets = {StudioImage}
    game:GetService("ContentProvider"):PreloadAsync(assets)
    print("ID Successfully Loaded!")
    canContinue = false
end

I used MarketplaceService cause I could check if the two assets are made by the same person. Like I said though, I don't know of another alternative.