It looks like it should be working. You need to test to see if this $post_id = get_the_ID(); is actually the current post ID. Right after that add echo '<br>',$post_id,'<br>'; and make sure that you’re trying to get the values from the correct post.
Welcome to the Advanced Custom Fields community forum.
Browse through ideas, snippets of code, questions and answers between fellow ACF users
🤔 Curious about the ACF user experience? So are we! Help guide the evolution of ACF by taking part in our first ever Annual Survey and guarantee you’re represented in the results. https://t.co/0cgr9ZFOJ5